Thread (6 messages) flat view 6 messages, 2 authors, 2021-08-05

Re: [PATCH] Documentation/kbuild: Document the kconfig choice default value

From: Masahiro Yamada <masahiroy@kernel.org>
Date: 2021-07-18 09:37:32
Also in: linux-kbuild, lkml

On Thu, Jul 15, 2021 at 6:26 PM Robert Richter [off-list ref] wrote:
quoted hunk ↗ jump to hunk
Document how choice defaults are determined:

Default of a choice is its first visible choice element [1]. Choice
elements do not support default attributes. [2]

[1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/scripts/kconfig/symbol.c?h=v5.14-rc1#n245
[2] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/scripts/kconfig/menu.c?h=v5.14-rc1#n494

Signed-off-by: Robert Richter <redacted>
---
 Documentation/kbuild/kconfig-language.rst | 3 +++
 1 file changed, 3 insertions(+)
diff --git a/Documentation/kbuild/kconfig-language.rst b/Documentation/kbuild/kconfig-language.rst
index 98c24183d8c3..e4d3463594e1 100644
--- a/Documentation/kbuild/kconfig-language.rst
+++ b/Documentation/kbuild/kconfig-language.rst
@@ -417,6 +417,9 @@ definitions of that choice. If a [symbol] is associated to the choice,
 then you may define the same choice (i.e. with the same entries) in another
 place.

+The default value of a choice is set to the first visible choice element.

The default value of a choice is set to the first visible choice element
unless it is explicitly set by the 'default' property.

... is more precise.



+Choice elements do not support the default attribute like menu entries do.
I doubt this info is useful.
Rather, is it even confusing?


choices support 'default' but
choice elements (i.e. choice values) do not.








+
 comment::

        "comment" <prompt>
--
2.29.2

-- 
Best Regards
Masahiro Yamada
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help