Skip to content

usb: dwc2: masquerade split-interrupt transfers #6936

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

Open
wants to merge 2 commits into
base: rpi-6.12.y
Choose a base branch
from

Conversation

P33M
Copy link
Contributor

@P33M P33M commented Jul 4, 2025

Second attempt at porting the dwc_otg hack to avoid dropped split transactions.

Transactions now "look OK" on the bus analyser but this needs more soak testing with different hub implementations.

@P33M P33M marked this pull request as ready for review July 8, 2025 09:21
P33M added 2 commits July 8, 2025 13:16
Masquerading Interrupt split transfers as Control puts the transfer into
the non-periodic handler in the hub. This stops the hub dropping
complete-split data in the microframe after a CSPLIT should have
arrived, improving resilience to host IRQ latency. Devices are none
the wiser - the handshake tokens are the same.

Originally devised by Hans Petter Selasky @ FreeBSD.

(v2: dwc2 needs an un-masquerade prior to channel interrupt handling)

Signed-off-by: Jonathan Bell <[email protected]>
Control Setup phase transfers needs an 8 byte maxpacket override, or
drivers doing a GET with a small buffer as a split transaction would
cause broken transfers.

Signed-off-by: Jonathan Bell <[email protected]>
@P33M P33M force-pushed the dwc2_masquerade branch from 9d4c59b to fbd7bc9 Compare July 8, 2025 12:16
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.

1 participant