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: enable/disable a class driver at runtime #25604

Closed
jli157 opened this issue May 26, 2020 · 5 comments
Closed

USB: enable/disable a class driver at runtime #25604

jli157 opened this issue May 26, 2020 · 5 comments
Assignees
Labels
area: USB Universal Serial Bus Enhancement Changes/Updates/Additions to existing features

Comments

@jli157
Copy link
Contributor

jli157 commented May 26, 2020

Is your feature request related to a problem? Please describe.
In the current USB device stack, class drivers are statically selected during project configuration and can’t be changed at runtime. However, sometimes some class drivers could be required to be enabled or disabled at runtime. For example, the DFU driver could be disabled by default. And it will be helpful if it can be enabled by the application at runtime when the application wants to update the firmware.

Describe the solution you'd like
All selected usb class drivers shall have a ‘enable(bool)’ api to enable / disable it from being discovered by the host at runtime when needed. And a setting value can be used to control if the class driver shall be initialized or disabled.

Describe alternatives you've considered
N/A

Additional context
Some system maintenance USB class drivers, such as CDC-ACM, DFU are not always needed but it is hoped to be included in the application firmware.

@jfischer-no jfischer-no added area: USB Universal Serial Bus Feature Request A request for a new feature labels May 26, 2020
@jfischer-no
Copy link
Collaborator

@jli157 In fact, we have been discussing it for some time. Only more extensive than you described.

@jli157
Copy link
Contributor Author

jli157 commented May 26, 2020

@jli157 In fact, we have been discussing it for some time. Only more extensive than you described.

Is there an existing issue covering the request? And what conclusions have you made?

@jfischer-no
Copy link
Collaborator

jfischer-no commented May 26, 2020

Is there an existing issue covering the request? And what conclusions have you made?

No, still discussing. This is a major change, it affects more than class descriptor initialization and has to be implemented for all classes and applications in the tree. So that will take time.

@carlescufi carlescufi added Enhancement Changes/Updates/Additions to existing features and removed Feature Request A request for a new feature labels May 26, 2020
@eobalski
Copy link
Collaborator

There is no particular issue for that however there are few we would like to address during the rework
#20236 #19713 #24200

@jfischer-no
Copy link
Collaborator

Closing in favor of issue #29135

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: USB Universal Serial Bus Enhancement Changes/Updates/Additions to existing features
Projects
None yet
Development

No branches or pull requests

4 participants