Skip to content

Conversation

jfischer-no
Copy link

The new stack comes with a higher demand for flash. We need to adapt the overlay file for the platform used for testing. The legacy stack is in the process of being deprecated. The NRF52840 dongle should not enable the legacy stack by default. The Zephyr's new stack provides helper code to use CDC ACM as the serial backend. The only required change to the serial adapter code is to get the USB device instance and enable it.

@nordicjm
Copy link
Collaborator

Needs submitting to https://github.com/mcu-tools/mcuboot we do not accept PRs on this repo

Copy link
Collaborator

@d3zd3z d3zd3z left a comment

Choose a reason for hiding this comment

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

Is there a reference to this change on the mcuboot repository?

The legacy USB device stack is in the process of being deprecated. In
order to pass CI in the Zephyr Project, we need to adjust the test case
settings in sample.yaml. Clean up both test cases and add a Kconfig
option that allows them to pass Twitter runs despite deprecation
warnings and flash overflows.

Signed-off-by: Johann Fischer <[email protected]>
@jfischer-no jfischer-no force-pushed the pr-usb-device-legacy-deprecation branch from e0cb51e to 2d24473 Compare July 22, 2025 11:24
@jfischer-no jfischer-no marked this pull request as draft July 22, 2025 11:48
@carlescufi carlescufi changed the title boot: zephyr: move CDC ACM serial to the new device stack boot: zephyr: adjust sample.yaml to pass USB deprecation process Jul 23, 2025
@henrikbrixandersen henrikbrixandersen self-requested a review July 25, 2025 17:03
@jfischer-no
Copy link
Author

Is there a reference to this change on the mcuboot repository?

mcu-tools#2413

@nordicjm
Copy link
Collaborator

nordicjm commented Aug 6, 2025

Merged upstream and brought in

@nordicjm nordicjm closed this Aug 6, 2025
@jfischer-no jfischer-no deleted the pr-usb-device-legacy-deprecation branch August 6, 2025 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants