From 3f56b87c5134b9e6af9adea05d1ce4fc456da9b8 Mon Sep 17 00:00:00 2001 From: Elyes HAOUAS Date: Sun, 6 Sep 2020 11:29:52 +0200 Subject: Documentation/getting_started/kconfig.md: Add a note about Kconfig defaults When the declaration is done after the default, menuconfig will see that symbol defined at the first place where kconfig tool will find it. For example, if we run menuconfig and search for 'MAINBOARD_VENDOR', we will see it defined at ""src/mainboard/51nb/Kconfig" which is odd. Change-Id: I215a1817e60e6deb6931679f139d110ba762d3c8 Signed-off-by: Elyes HAOUAS Reviewed-on: https://review.coreboot.org/c/coreboot/+/45127 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Nico Huber --- Documentation/getting_started/kconfig.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/getting_started/kconfig.md b/Documentation/getting_started/kconfig.md index ff8d3dcf3b..037371bd34 100644 --- a/Documentation/getting_started/kconfig.md +++ b/Documentation/getting_started/kconfig.md @@ -398,6 +398,8 @@ default <expr> \[if <expr>\] - If there is no 'default' entry for a symbol, it gets set to 'n', 0, 0x0, or “” depending on the type, however the 'bool' type is the only type that should be left without a default value. +- If possible, the declaration should happen before all default entries to make + it visible in Kconfig tools like menuconfig. -------------------------------------------------------------------------------- -- cgit v1.2.3