Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update USB device library to v2.6.0 #774

Merged
merged 3 commits into from
Aug 8, 2022

Conversation

Daft-Freak
Copy link
Collaborator

I think this is the version after the one currently in use. Most important thing from the release notes is "Fix multiple error with SCSI commands handling" (It was entirely broken).

Some things were already fixed in our second copy of usbd_msc_scsi.c, but it wasn't enough and almost any error would make the host think the device was hung.

Had to modify the lib a little as they un-consted some data for some reason. Hopefully this hasn't broken something else...

@Gadgetoid
Copy link
Contributor

Looks like the Emscripten fail was some temporary build failure, but I can't for the life of me seem to re-run the job! Could you give it a kick, please?

This release has signigicantly less broken error handling in the MSC code. Update made a bit harder by the _slightly_ modified copy of usbd_msc_scsi.c we have...
OK/FAIL were inverted... and the caller only treats negative return values as errors
@Daft-Freak
Copy link
Collaborator Author

Kick successful 😄

@Gadgetoid Gadgetoid merged commit 8d09ec5 into 32blit:master Aug 8, 2022
@Gadgetoid
Copy link
Contributor

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants