Skip to content

Lock sleep when USB is initialized #9879

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

Merged
merged 1 commit into from
Mar 1, 2019
Merged

Conversation

c1728p9
Copy link
Contributor

@c1728p9 c1728p9 commented Feb 27, 2019

Description

Lock sleep when USB is initialized to ensure USB always has a clock lock deep sleep while it is in use. Thanks @rajkan01 for finding this.

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

@rajkan01

@ciarmcom ciarmcom requested review from rajkan01 and a team February 27, 2019 22:00
@ciarmcom
Copy link
Member

@c1728p9, thank you for your changes.
@rajkan01 @ARMmbed/mbed-os-maintainers please review.

None of the USB drivers currently support entering deep sleep mode
while USB is active. To protect USB from malfunctioning lock deep
sleep in USBPhyHw::init.
@c1728p9
Copy link
Contributor Author

c1728p9 commented Feb 28, 2019

Updated this PR to move the locking into vendor specific code. If their hardware supports automatic wake from deep sleep upon reception of USB traffic and software support for this is added then these locks can be removed.

@cmonr
Copy link
Contributor

cmonr commented Feb 28, 2019

CI started

@cmonr cmonr added the risk: G label Mar 1, 2019
@mbed-ci
Copy link

mbed-ci commented Mar 1, 2019

Test run: SUCCESS

Summary: 13 of 13 test jobs passed
Build number : 1
Build artifacts

@cmonr cmonr merged commit 3d1b55d into ARMmbed:master Mar 1, 2019
AGlass0fMilk added a commit to AGlass0fMilk/mbed-os that referenced this pull request Mar 15, 2019
Added logic to lock deep sleep when USB is initialized to prevent malfunctioning. This is consistent with the other drivers. See PR ARMmbed#9879
AGlass0fMilk added a commit to AGlass0fMilk/mbed-os that referenced this pull request Apr 29, 2019
Added logic to lock deep sleep when USB is initialized to prevent malfunctioning. This is consistent with the other drivers. See PR ARMmbed#9879
AGlass0fMilk added a commit to AGlass0fMilk/mbed-os that referenced this pull request May 28, 2019
Added logic to lock deep sleep when USB is initialized to prevent malfunctioning. This is consistent with the other drivers. See PR ARMmbed#9879
hugueskamba pushed a commit to hugueskamba/mbed-os that referenced this pull request Jun 26, 2019
Added logic to lock deep sleep when USB is initialized to prevent malfunctioning. This is consistent with the other drivers. See PR ARMmbed#9879
artokin pushed a commit that referenced this pull request Jul 3, 2019
Added logic to lock deep sleep when USB is initialized to prevent malfunctioning. This is consistent with the other drivers. See PR #9879
evedon pushed a commit that referenced this pull request Jul 11, 2019
Added logic to lock deep sleep when USB is initialized to prevent malfunctioning. This is consistent with the other drivers. See PR #9879
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants