You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your enhancement proposal related to a problem? Please describe.
When wanting to route the shell through USB CDC ACM there is a problem with the shell getting possibly initialised before the CDC ACM driver.
Describe the solution you'd like
I managed to successfully accomplish what I need by adding a call to usb_enable() in the enable_shell_uart() function and changing the literal 0 shell initialisation priority to CONFIG_APPLICATION_INIT_PRIORITY (so that it happens after the USB device's CONFIG_KERNEL_INIT_PRIORITY_DEVICE.
However, I'm not sure if this is the correct way to do it with me being possibly unaware of the reason why shell's initialisation needs to be 0, an the best place to usb_enable().
The text was updated successfully, but these errors were encountered:
@carlescufi Oh ok. I always kinda assumed it's annoying to bother everyone with PRs which have not been discussed to be the right way forward, but if that's the preferred workflow I'll go for it!
@carlescufi Oh ok. I always kinda assumed it's annoying to bother everyone with PRs which have not been discussed to be the right way forward, but if that's the preferred workflow I'll go for it!
No actually, many discussions can take place directly in the PR. Thanks for your contribution!
The solution in current Zephyr tree is to configure the shell (CONFIG_UART_SHELL_ON_DEV_NAME="CDC_ACM_0") and enable USB device support in user application.
See also #26122 (comment)
Is your enhancement proposal related to a problem? Please describe.
When wanting to route the shell through USB CDC ACM there is a problem with the shell getting possibly initialised before the CDC ACM driver.
Describe the solution you'd like
I managed to successfully accomplish what I need by adding a call to
usb_enable()
in theenable_shell_uart()
function and changing the literal0
shell initialisation priority toCONFIG_APPLICATION_INIT_PRIORITY
(so that it happens after the USB device'sCONFIG_KERNEL_INIT_PRIORITY_DEVICE
.However, I'm not sure if this is the correct way to do it with me being possibly unaware of the reason why shell's initialisation needs to be
0
, an the best place tousb_enable()
.The text was updated successfully, but these errors were encountered: