Skip to content

drivers: usb: udc: stm32: driver cleanup and EP MPS fixes #96926

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.

Sign up for GitHub

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 5 commits into
base: main
Choose a base branch
from
+82 −52
Open

drivers: usb: udc: stm32: driver cleanup and EP MPS fixes #96926

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
d3adb48
drivers: usb: udc: stm32: turn EP0 max packet size into a constant
mathieuchopstm Oct 2, 2025
6c349f0
drivers: usb: udc: stm32: clean up handling of USB buffer table
mathieuchopstm Oct 2, 2025
88aab2b
drivers: usb: udc: stm32: accept non-word-aligned MaxPacketSize or FIFOs
mathieuchopstm Oct 2, 2025
8940344
drivers: usb: udc: stm32: allow EP max packet size up to HW capabilities
mathieuchopstm Oct 2, 2025
00d8ee0
drivers: usb: udc: stm32: configure OTGFS/HS RxFIFO size using Kconfig
mathieuchopstm Oct 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 15 additions & 14 deletions drivers/usb/udc/udc_stm32.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -121,6 +121,17 @@ LOG_MODULE_REGISTER(udc_stm32, CONFIG_UDC_DRIVER_LOG_LEVEL);
(PCD_SPEED_HIGH_IN_FULL), \
(PCD_SPEED_HIGH))))

/*
* Returns max packet size allowed for endpoints of 'usb_node'
*
* Hardware always supports the maximal value allowed
* by the USB Specification at a given operating speed:
* 1024 bytes in High-Speed, 1023 bytes in Full-Speed
*/
#define UDC_STM32_NODE_EP_MPS(node_id) \
((UDC_STM32_NODE_SPEED(node_id) == PCD_SPEED_HIGH) ? 1024U : 1023U)


Copy link
Contributor

Choose a reason for hiding this comment

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

Nitpicking: could you remove this extra empty line?

#if DT_HAS_COMPAT_STATUS_OKAY(st_stm32n6_otghs)
#define USB_USBPHYC_CR_FSEL_24MHZ USB_USBPHYC_CR_FSEL_1
#endif
Expand Down Expand Up @@ -158,11 +169,12 @@ struct udc_stm32_data {
struct udc_stm32_config {
uint32_t num_endpoints;
uint32_t dram_size;
uint16_t ep_mps;
/* PHY selected for use by instance */
uint32_t selected_phy;
/* Speed selected for use by instance */
uint32_t selected_speed;
/* Maximal packet size allowed for endpoints */
uint16_t ep_mps;
};

enum udc_stm32_msg_type {
Expand Down Expand Up @@ -1070,18 +1082,7 @@ static const struct udc_api udc_stm32_api = {
* Kconfig system.
*/
Copy link
Contributor

Choose a reason for hiding this comment

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

Are these inline comment still useful? (non-blocking, liekly a bit outside the scope of this P-R)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

IMO the comment still applies because there is some usage remaining of the Cube macros (see L1115 in new file). But eventually it should be removed yes.

#define USB_NUM_BIDIR_ENDPOINTS DT_INST_PROP(0, num_bidir_endpoints)

#if defined(USB) || defined(USB_DRD_FS)
#define EP_MPS 64U
#define USB_RAM_SIZE DT_INST_PROP(0, ram_size)
#else /* USB_OTG_FS */
#if DT_HAS_COMPAT_STATUS_OKAY(st_stm32_otghs)
#define EP_MPS USB_OTG_HS_MAX_PACKET_SIZE
#elif DT_HAS_COMPAT_STATUS_OKAY(st_stm32_otgfs) || DT_HAS_COMPAT_STATUS_OKAY(st_stm32_usb)
#define EP_MPS USB_OTG_FS_MAX_PACKET_SIZE
#endif
#define USB_RAM_SIZE DT_INST_PROP(0, ram_size)
#endif /* USB */
#define USB_RAM_SIZE DT_INST_PROP(0, ram_size)

static struct udc_stm32_data udc0_priv;

Expand All @@ -1093,7 +1094,7 @@ static struct udc_data udc0_data = {
static const struct udc_stm32_config udc0_cfg = {
.num_endpoints = USB_NUM_BIDIR_ENDPOINTS,
.dram_size = USB_RAM_SIZE,
.ep_mps = EP_MPS,
.ep_mps = UDC_STM32_NODE_EP_MPS(DT_DRV_INST(0)),
.selected_phy = UDC_STM32_NODE_PHY_ITFACE(DT_DRV_INST(0)),
.selected_speed = UDC_STM32_NODE_SPEED(DT_DRV_INST(0)),
};
Expand Down
Loading