Skip to content

Conversation

@jfischer-no
Copy link
Contributor

Follow up on commit bfd45e5
("drivers: remove Kconfig option CONFIG_UART_CONSOLE_ON_DEV_NAME")
Now we can also remove Kconfig option CONFIG_UART_CONSOLE_ON_DEV_NAME
since all UART drivers are converted to devicetree and we can just use
DEVICE_DT_GET(DT_CHOSEN(zephyr_shell_uart)).

@jfischer-no jfischer-no added the area: Shell Shell subsystem label Aug 24, 2021
@jfischer-no jfischer-no added this to the v2.7.0 milestone Aug 24, 2021
@jfischer-no jfischer-no requested a review from nashif as a code owner August 24, 2021 12:42
@github-actions github-actions bot added area: Documentation area: Samples Samples Release Notes To be mentioned in the release notes labels Aug 24, 2021
Follow up on commit bfd45e5
("drivers: remove Kconfig option CONFIG_UART_CONSOLE_ON_DEV_NAME")
Now we can also remove Kconfig option CONFIG_UART_CONSOLE_ON_DEV_NAME
since all UART drivers are converted to devicetree and we can just use
DEVICE_DT_GET(DT_CHOSEN(zephyr_shell_uart)).

Signed-off-by: Johann Fischer <[email protected]>
@jfischer-no jfischer-no force-pushed the pr-remove_shell_on_dev_name branch from 6eacd9c to 599b6a9 Compare August 24, 2021 16:17
@nashif nashif merged commit 741ae93 into zephyrproject-rtos:main Aug 24, 2021
@jfischer-no jfischer-no deleted the pr-remove_shell_on_dev_name branch August 24, 2021 21:55
coreboot-bot pushed a commit to coreboot/chrome-ec that referenced this pull request Sep 7, 2021
In Zephyr 2.7+, CONFIG_UART_SHELL_ON_DEV_NAME is removed in favor of
using the zephyr,shell-uart device chosen by the UART driver.  Since
we choose zephyr,shell-uart on all of our boards, we are clear to make
this change ahead of the v2.7 uprev.

Also see zephyrproject-rtos/zephyr#37902.

BUG=b:198824039
BRANCH=none
TEST=zmake testall
TEST=compile posix-ec with v2.7_rc1

Signed-off-by: Jack Rosenthal <[email protected]>
Change-Id: Ic066ce0617fa900ae7c1c96a2cbece0b698764bc
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3142750
Reviewed-by: Keith Short <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Documentation area: Samples Samples area: Shell Shell subsystem Release Notes To be mentioned in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants