Update TinyUSB to latest / Xbox One Auth fix #1002
Merged
+140
−139
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…e of race condition (which seemed to be happening)
|
I'm still having issues on this one for Xbox One |
.init, .reset, and .open are called multiple times during USB discovery initial announcement code was moved from .init to .open AFTER the xbox one interface type is declared. Also moved shared buffers to more sane areas and used two instead of one buffer for sharing.
|
This should be fixed and ready to go. The issue was my Xbox One start/announce code was in the .init = function of TinyUSB driver. Because .init, .reset and .open can be opened multiple times on start of a device, I had to move the start/announce code of the controller into .open and ONLY when the device vendor is verified and the interfaces are setup. This matches the XGIP I've seen on other devices, where they send 2 announces and then do auth. |
Extra spaces
arntsonl
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This updates our TinyUSB to the latest library. This removes the get_report hack I added in while tinyusb was updated, and brings us up to 4/30/2024 in the TinyUSB library.