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

[usb,aon_wake] Report 'bus not idle' status to usbdev #22019

Merged
merged 1 commit into from
Mar 15, 2024

Conversation

alees24
Copy link
Contributor

@alees24 alees24 commented Mar 13, 2024

The 'bus not idle' condition had to be inferred by software, from the absence of Bus Reset and Disconnect indications; less than ideal from a robustness perspective even when returning from Deep Sleep, but it afforded no opportunity to indicate a non-idle bus state whilst still suspending/entering sleep. Introduce an explicit indication for increased robustness and error recovery.

This has been separated out from PR #19270 because of the more contentious, still-to-be-resolved change to pull up behavior and handover sequencing in that PR.

@alees24 alees24 requested a review from a-will March 13, 2024 19:09
@alees24 alees24 requested review from a team and msfschaffner as code owners March 13, 2024 19:09
@alees24 alees24 requested review from rswarbrick and removed request for a team, msfschaffner and rswarbrick March 13, 2024 19:09
The 'bus not idle' condition had to be inferred by software,
from the absence of Bus Reset and Disconnect indications;
less than ideal from a robustness perspective even when returning
from Deep Sleep, but it afforded no opportunity to indicate a
non-idle bus state whilst still suspending/entering sleep.
Introduce an explicit indication for increased robustness and
error recovery.

Signed-off-by: Adrian Lees <[email protected]>
Copy link
Contributor

@pamaury pamaury left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I concur that having this bit seems more robust than trying to infer something that could turn out to be wrong.

@alees24 alees24 mentioned this pull request Mar 15, 2024
@alees24 alees24 merged commit 53c6c77 into lowRISC:master Mar 15, 2024
32 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