Skip to content

Conversation

tmon-nordic
Copy link
Contributor

PR88642 is backported to apply on 2.9.0-nRF54H20 sdk-zephyr without requiring to pull in other changes.

Control transfers can be abruptly terminated by host. The most common
use case when new SETUP token is received before the previous transfer
ends, is when the host timeouts on previous control transfer.

Fix the SETUP data leak by releasing stored setup buffer if new transfer
is started out of sequence.

Upstream PR #: 86925

Signed-off-by: Tomasz Moń <[email protected]>
DWC2 core sets DIEPCTL0 SNAK when SETUP packet is received. The CNAK bit
results in device sending NAK in response to IN token sent to EP0, but
it does not modify the TxFIFO in any way. The stale data in TxFIFO can
then lead to "FIFO space is too low" error. Solve the issue by disabling
and flushing IN endpoint 0 if previous control transfer did not finish.

Upstream PR #: 88642

Signed-off-by: Tomasz Moń <[email protected]>
Copy link

@carlescufi carlescufi merged commit eceea6b into nrfconnect:v3.7.99-ncs3-branch Apr 17, 2025
16 of 17 checks passed
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