adafruit-mirror/zephyr
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 9

console: kconfig: Have CONSOLE_{GETCHAR,GETLINE} depend on UART_CONSOLE

Browse source 
CONSOLE_GETCHAR and CONSOLE_GETLINE select CONSOLE_HANDLER and
UART_CONSOLE_DEBUG_SERVER_HOOKS, which depend on UART_CONSOLE, but
UART_CONSOLE is n for some boards, giving a select with unsatisfied
dependencies and a warning. Triggers in CI.

Looking at the code, CONSOLE_{GETCHAR,GETLINE} also depend on
UART_CONSOLE there. Add a 'depends on UART_CONSOLE' to the 'choice' that
contains them.

Maybe the samples/subsys/console/{echo,getchar}/ samples shouldn't be
run on some boards that they're currently run on, but there's still the
issue that CONSOLE_{GETCHAR,GETLINE} shouldn't be shown in menuconfig
when they don't apply.

Signed-off-by: Ulf Magnusson <Ulf.Magnusson@nordicsemi.no>
This commit is contained in:
Ulf Magnusson 2020-01-22 19:21:48 +01:00 committed by Anas Nashif
parent 9b2fa8a5a7
commit 97de99adca
1 changed files with 1 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
subsys/console/Kconfig
View file

@ -11,6 +11,7 @@ if CONSOLE_SUBSYS
choice choice
prompt "Console 'get' function selection" prompt "Console 'get' function selection"
optional optional
depends on UART_CONSOLE
config CONSOLE_GETCHAR config CONSOLE_GETCHAR
bool "Character by character input and output" bool "Character by character input and output"