summaryrefslogtreecommitdiff
path: root/Documentation
diff options
context:
space:
mode:
authorElyes HAOUAS <ehaouas@noos.fr>2020-09-06 11:29:52 +0200
committerNico Huber <nico.h@gmx.de>2020-09-25 08:41:45 +0000
commit3f56b87c5134b9e6af9adea05d1ce4fc456da9b8 (patch)
tree6976511976cd929e8d29a11db24cd1ed1d68b960 /Documentation
parent3bcb6c7319ac5f23d7e5a9e637917cbd027ac371 (diff)
downloadcoreboot-3f56b87c5134b9e6af9adea05d1ce4fc456da9b8.tar.xz
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 <ehaouas@noos.fr> Reviewed-on: https://review.coreboot.org/c/coreboot/+/45127 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'Documentation')
-rw-r--r--Documentation/getting_started/kconfig.md2
1 files changed, 2 insertions, 0 deletions
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 &lt;expr&gt; \[if &lt;expr&gt;\]
- 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.
--------------------------------------------------------------------------------