-
Notifications
You must be signed in to change notification settings - Fork 3k
Flatten USB driver directory structure #11095
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
Conversation
@gpsimenos There is a CI test failure. |
The failure has nothing to do with the changes in this PR. I didn't touch events at all. |
The events Makefile tests pass on my machine. It might just be an issue with Travis. Try pushing a dummy commit to re-trigger the CI tests. |
@gpsimenos, thank you for your changes. |
Test run: FAILEDSummary: 4 of 4 test jobs failed Failed test jobs:
|
Add missing header file inclusion.
The "bare-metal" profile which is defined in mbed-os/platform/bare_metal/mbed_lib.json only includes only the `platform` and `drivers` dirs in the build. The problem is due to the fact that USB related modules are now included in the build as part of the `drivers` library. They previously were not included given they were outside the `drivers` directory (in the `usb` dir). The solution is to stick an mbed_lib.json file under `mbed-os/drivers/source/usb` in order to make it a library so it does not get automatically included given the `requires` statement in mbed-os/platform/bare_metal/mbed_lib.json
@hugueskamba Please check the unittest failures. Thanks.
|
The Cmake file described hardcoded the relative path to the source file being tested. Changed it to be the new location of the source file.
@SeppoTakalo Thank you. I just pushed a commit to fix to this. 7efaedd |
Test run: FAILEDSummary: 4 of 4 test jobs failed Failed test jobs:
|
This PR is superseded by #11093 |
Test run: FAILEDSummary: 3 of 4 test jobs failed Failed test jobs:
|
This is now obsolete |
Description
Moved all USB source files directly under drivers/usb/
Pull request type
Reviewers
Release Notes