Skip to content

Conversation

@mathieuchopstm
Copy link
Contributor

When setting endpoint capabilities, the driver uses the MPS stored in each instance's configuration for OUT endpoints, but used a hardcoded 1023 for IN endpoints.

Use the instance MPS when preparing IN endpoints' capabilities too.

@mathieuchopstm mathieuchopstm added this to the v4.3.0 milestone Oct 29, 2025
@mathieuchopstm mathieuchopstm added bug The issue is a bug, or the PR is fixing a bug area: USB Universal Serial Bus platform: STM32 ST Micro STM32 Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. labels Oct 29, 2025
@zephyrbot zephyrbot added the size: XS A PR changing only a single line of code label Oct 29, 2025
@jfischer-no jfischer-no removed Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. size: XS A PR changing only a single line of code labels Oct 29, 2025
When setting endpoint capabilities, the driver uses the MPS stored in each
instance's configuration for OUT endpoints, but used a hardcoded 1023 for
IN endpoints.

Use the instance MPS when preparing IN endpoints' capabilities too.

Signed-off-by: Mathieu Choplain <mathieu.choplain-ext@st.com>
@mathieuchopstm mathieuchopstm force-pushed the stm32_usb_fix_ep_in_mps branch from 7dacb0f to 3cde2c7 Compare October 30, 2025 09:13
@mathieuchopstm
Copy link
Contributor Author

rebased to fix CI failure

@zephyrbot zephyrbot added the size: XS A PR changing only a single line of code label Oct 30, 2025
@sonarqubecloud
Copy link

@mathieuchopstm
Copy link
Contributor Author

Added @etienne-lms as assignee as @erwango is out of office and can't approve

@cfriedt cfriedt merged commit 2e998cf into zephyrproject-rtos:main Oct 31, 2025
28 checks passed
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 bug The issue is a bug, or the PR is fixing a bug platform: STM32 ST Micro STM32 size: XS A PR changing only a single line of code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants