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

Replace STM32 USB driver with DWC #4412

Closed
galak opened this issue Oct 18, 2017 · 3 comments
Closed

Replace STM32 USB driver with DWC #4412

galak opened this issue Oct 18, 2017 · 3 comments
Assignees
Labels
area: USB Universal Serial Bus Enhancement Changes/Updates/Additions to existing features platform: STM32 ST Micro STM32

Comments

@galak
Copy link
Collaborator

galak commented Oct 18, 2017

The STM32 devices use a designware USB block so we should utilize the dwc driver for the device.

@galak galak added the area: USB Universal Serial Bus label Oct 18, 2017
@erwango
Copy link
Member

erwango commented Oct 19, 2017

To be exact: "The STM32 devices use designware USB blocks"
There are actually variety of USB IPs across STM32 families. All of them being DW but different versions.
On the whole family, there are 4 "lines" of USB IPs. In each line having different versions.
As example of "line": USB DEV FS (F1 family), USB OTGS FS, USB OTG HS and one more which is USB with clock recovery (don't know how much one is different to the others). All of these had independant evolution in time, hence different versions.

Cube HAL API is made to handle this "heterogeneity". And it is possible to provide a single HAL based driver for the while family. From the starting point of having it working on one line, it will take minor steps to add support on other lines.

Since this abstraction is not available in DWC driver, I assume this would require more substantial efforts to get it supporting all these lines.

Hence my proposal is to keep Cube based driver to enable quick deployment on whole STM32 families. In a parallel track there could be some work to get dwc driver work on STM32, and user should chose which driver he wants to use.

@nashif nashif added the Enhancement Changes/Updates/Additions to existing features label Oct 19, 2017
@erwango erwango added the platform: STM32 ST Micro STM32 label Oct 19, 2017
@carlescufi
Copy link
Member

@erwango agreed that Cube seems like the better option here.
@galak any additional input, or should we close this?

@erwango
Copy link
Member

erwango commented May 25, 2020

I'm closing that point as there was no further request nor interest on that point.
Please reopen if need be

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 platform: STM32 ST Micro STM32
Projects
None yet
Development

No branches or pull requests

5 participants