Skip to content

drivers: udc_dwc2: Add nRF54LM20A vendor quirks #92842

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jfischer-no
Copy link
Contributor

Initial implementation of nRF54L quirks necessary for nRF54LM20A.

Initial implementation of nRF54L quirks necessary for nRF54LM20A.

Signed-off-by: Tomasz Moń <[email protected]>
Signed-off-by: Johann Fischer <[email protected]>
@jfischer-no jfischer-no self-assigned this Jul 8, 2025
@jfischer-no jfischer-no added area: Drivers area: USB Universal Serial Bus Experimental Experimental features not enabled by default DNM This PR should not be merged (Do Not Merge) labels Jul 8, 2025
Copy link

sonarqubecloud bot commented Jul 8, 2025

#include <zephyr/drivers/clock_control.h>
#include <zephyr/drivers/clock_control/nrf_clock_control.h>

#define NRF_DEFAULT_IRQ_PRIORITY 1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
#define NRF_DEFAULT_IRQ_PRIORITY 1

Comment on lines +358 to +359
IRQ_CONNECT(VREGUSB_IRQn, NRF_DEFAULT_IRQ_PRIORITY,
vregusb_isr, DEVICE_DT_INST_GET(0), 0);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why use NRF_DEFAULT_IRQ_PRIORITY? Why not from devicetree (node) ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no devicetree node for VREGUSB peripheral (there is devicetree for USBHS peripheral, but it is VREGUSB that is used here in vendor quirk).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Drivers area: USB Universal Serial Bus DNM This PR should not be merged (Do Not Merge) Experimental Experimental features not enabled by default
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants