Skip to content

Conversation

pennam
Copy link

@pennam pennam commented Aug 29, 2025

Do not merge before testing

@KurtE
Copy link

KurtE commented Sep 22, 2025

Quick question: I have synced up to the current code. And have noticed that building C33 there are a lot of Config messages.
Many of them associated with Bluetooth, which I am assuming that this PR may address:

Note: on my zephyrproject - zephyr fork which I synced yesterday to the most recent. And many of my zephyr test apps, would fail to build, saying that the USB stuff has been marked obsolete. So I am trying to move it to USB Next. Which the C33 code is currently configured here to use.

In both cases (ArduinoCore-zephyr and my zephyr) I am seeing the config message:

warning: CDC_ACM_DTE_RATE_CALLBACK_SUPPORT (defined at subsys/usb/device/class/Kconfig.cdc:55) was
assigned the value 'y' but got the value 'n'. Check these unsatisfied dependencies: USB_CDC_ACM                                              (=n), USB_DEVICE_STACK (=n). See
http://docs.zephyrproject.org/latest/kconfig.html#CONFIG_CDC_ACM_DTE_RATE_CALLBACK_SUPPORT and/or
look up CDC_ACM_DTE_RATE_CALLBACK_SUPPORT in the menuconfig/guiconfig interface. The Application
Development Primer, Setting Configuration Values, and Kconfig - Tips and Best Practices sections of  the manual might 
be helpful too.                                                                    

My guess is that setting is a hold over from the non NEXT version? Can we just remove or is there a replacement setting
that should be set?

@pennam
Copy link
Author

pennam commented Sep 22, 2025

My guess is that setting is a hold over from the non NEXT version?

I agree, and i think it can be removed. I'll spin a test as soon as possible and provide a PR! Thanks.

@pennam
Copy link
Author

pennam commented Oct 9, 2025

Tested on

GIGA ✔️
Nano 33 BLE ✔️
Portenta H7 ✔️
Portenta C33 ✔️
OPTA ✔️

@pennam pennam requested a review from pillo79 October 9, 2025 15:03
@pillo79 pillo79 merged commit 9517b3a into arduino:main Oct 10, 2025
18 checks passed
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