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

drivers/usbdev_synopsys_dwc2: small fixes #18759

Merged
merged 5 commits into from
Oct 18, 2022

Conversation

gschorcht
Copy link
Contributor

Contribution description

This PR is a spit-off from PR #18756 and provides some fixes for the usbdev_synopsys_dwc2:

  • adds the dependency of usbdev_synopsys_dwc2 driver on ztimer_msec
  • removes therefore the dependency on ztimer_msec in ESP32 make files
  • remove the redundant dependency of MODULE_USBDEV_SYNOPSYS_DWC2 on HAS_PERIPH_DEV
  • makes DWC2_USB_OTG_FS_TOTAL_FIFO_SIZE not overridable for ESP32 SoCs
  • uses DWC2_USB_OTG_FS_TOTAL_FIFO_SIZE instead of USB_OTG_FS_TOTAL_FIFO_SIZE

Testing procedure

There are no functional changes. Green CI should be sufficient.

Issues/PRs references

Split-off from PR #18756

The `usbdev_synopsys_dwc2 driver` requires the `ztimer_msec` module and is therefore responsible for pulling it in.
The `usbdev_synopsys_dwc2 driver` requires the `ztimer_msec` module and is therefore responsible for pulling it in. Therefore, the dependency on `ztimer_msec` can be removed here.
`MODULE_USBDEV_SYNOPSYS_DWC2` depends already on `HAS_PERIPH_USBDEV`.
Since the total USB OTG FIFO size is a hardware propertie, it must not be overridable.
Use `DWC2_USB_OTG_FS_TOTAL_FIFO_SIZE` instead of `USB_OTG_FS_TOTAL_FIFO_SIZE` since the latter is only defined in the vendor headers for STM32 MCUs. The STM32-specific problem that `USB_OTG_FS_TOTAL_FIFO_SIZE` is not defined in the vendor headers for all STM32 families has therefore been moved from the driver to the STM32-specific USB device header.
@github-actions github-actions bot added Area: cpu Area: CPU/MCU ports Area: drivers Area: Device drivers Platform: ARM Platform: This PR/issue effects ARM-based platforms Platform: ESP Platform: This PR/issue effects ESP-based platforms labels Oct 17, 2022
@gschorcht gschorcht added the Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation label Oct 17, 2022
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Oct 17, 2022
@riot-ci
Copy link

riot-ci commented Oct 17, 2022

Murdock results

✔️ PASSED

1dfe79f drivers/usbdev_synopsys_dwc2: use DWC2_USB_OTG_FS_TOTAL_FIFO_SIZE

Success Failures Total Runtime
1987 0 1987 06m:34s

Artifacts

This only reflects a subset of all builds from https://ci-prod.riot-os.org. Please refer to https://ci.riot-os.org for a complete build for now.

@benpicco benpicco merged commit a98a706 into RIOT-OS:master Oct 18, 2022
@gschorcht gschorcht deleted the drivers/usbdev_synopsys_dwc2_fixes branch October 18, 2022 06:19
@kaspar030 kaspar030 added this to the Release 2023.01 milestone Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: cpu Area: CPU/MCU ports Area: drivers Area: Device drivers CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms Platform: ESP Platform: This PR/issue effects ESP-based platforms Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants