diff options
| author | Elyes HAOUAS <ehaouas@noos.fr> | 2020-09-06 11:29:52 +0200 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2020-09-25 08:41:45 +0000 |
| commit | 3f56b87c5134b9e6af9adea05d1ce4fc456da9b8 (patch) | |
| tree | 6976511976cd929e8d29a11db24cd1ed1d68b960 | |
| parent | 3bcb6c7319ac5f23d7e5a9e637917cbd027ac371 (diff) | |
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>
| -rw-r--r-- | Documentation/getting_started/kconfig.md | 2 |
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 <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. -------------------------------------------------------------------------------- |