Skip to content

samples: video: capture: remove stray USB legacy stack configuration #94227

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

The legacy USB device stack configuration used in the board configuration overlay files has been deprecated. It is not causing CI failures because the board lacks a test feature 'video' and is filtered out in CI.

Besides the deprecated configuration, the sample code does not enable the USB device stack, so it is not used anyway. Even if it were needed, the correct way to enable logging via USB CDC ACM would be to use the cdc-acm-console snippet.

The legacy USB device stack configuration used in the board
configuration overlay files has been deprecated. It is not causing CI
failures because the board lacks a test feature 'video' and is filtered
out in CI.

Besides the deprecated configuration, the sample code does not enable
the USB device stack, so it is not used anyway. Even if it were needed,
the correct way to enable logging via USB CDC ACM would be to use the
cdc-acm-console snippet.

Signed-off-by: Johann Fischer <johann.fischer@nordicsemi.no>
Copy link

sonarqubecloud bot commented Aug 7, 2025

Copy link
Contributor

@josuah josuah left a comment

Choose a reason for hiding this comment

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

I probably assumed it was valid device_next and copy-pasted it without looking close enough...

I just tested this branch in-hardware with west build -b arduino_nicla_vision/stm32h747xx/m7 samples/subsys/usb/uvc and it works

Image

Thank you for the correction!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Samples Samples area: Video Video subsystem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants