2007-08-06 23:30:09

by Vincent Legoll

[permalink] [raw]
Subject: [PATCH] - Add "def_bool" Kconfig format documentation

Hello,

here is a small documentation patch for the KConfig file format
"def_bool" type definition that was missing.

I hope the patch is not mangled...

>From 4742eb7a3169001304524e82a88ec1c94f70aa56 Mon Sep 17 00:00:00 2001
From: Vincent Legoll <[email protected]>
Date: Sat, 4 Aug 2007 16:50:06 +0200
Subject: [PATCH] - Add "def_bool" Kconfig format documentation


Signed-off-by: Vincent Legoll <[email protected]>
---
Documentation/kbuild/kconfig-language.txt | 8 ++++++++
1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/Documentation/kbuild/kconfig-language.txt b/Documentation/kbuild/kconfig-language.txt
index 536d5bf..27b8af9 100644
--- a/Documentation/kbuild/kconfig-language.txt
+++ b/Documentation/kbuild/kconfig-language.txt
@@ -77,6 +77,14 @@ applicable everywhere (see syntax).
Optionally, dependencies only for this default value can be added
with
"if".

+- boolean type definition with a default value: "def_bool" <expr>
+ So these two examples are equivalent:
+
+ def_bool <expr>
+ and
+ bool
+ default <expr>
+
- dependencies: "depends on"/"requires" <expr>
This defines a dependency for this menu entry. If multiple
dependencies are defined, they are connected with '&&'. Dependencies
--
1.5.2.4




2007-08-12 05:03:46

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH] - Add "def_bool" Kconfig format documentation

On Tue, 07 Aug 2007 00:29:40 +0200 Vincent Legoll wrote:

> Hello,
>
> here is a small documentation patch for the KConfig file format
> "def_bool" type definition that was missing.

There are also def_boolean and def_tristate.
I suppose that I should resubmit this patch:
http://marc.info/?l=linux-kernel&m=115471883121076&w=2


> I hope the patch is not mangled...
>
> >From 4742eb7a3169001304524e82a88ec1c94f70aa56 Mon Sep 17 00:00:00 2001
> From: Vincent Legoll <[email protected]>
> Date: Sat, 4 Aug 2007 16:50:06 +0200
> Subject: [PATCH] - Add "def_bool" Kconfig format documentation
>
>
> Signed-off-by: Vincent Legoll <[email protected]>
> ---
> Documentation/kbuild/kconfig-language.txt | 8 ++++++++
> 1 files changed, 8 insertions(+), 0 deletions(-)
>
> diff --git a/Documentation/kbuild/kconfig-language.txt b/Documentation/kbuild/kconfig-language.txt
> index 536d5bf..27b8af9 100644
> --- a/Documentation/kbuild/kconfig-language.txt
> +++ b/Documentation/kbuild/kconfig-language.txt
> @@ -77,6 +77,14 @@ applicable everywhere (see syntax).
> Optionally, dependencies only for this default value can be added
> with
> "if".
>
> +- boolean type definition with a default value: "def_bool" <expr>
> + So these two examples are equivalent:
> +
> + def_bool <expr>
> + and
> + bool
> + default <expr>
> +
> - dependencies: "depends on"/"requires" <expr>
> This defines a dependency for this menu entry. If multiple
> dependencies are defined, they are connected with '&&'. Dependencies
> --


---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

2007-08-12 06:45:38

by Robert P. J. Day

[permalink] [raw]
Subject: Re: [PATCH] - Add "def_bool" Kconfig format documentation

On Sat, 11 Aug 2007, Randy Dunlap wrote:

> On Tue, 07 Aug 2007 00:29:40 +0200 Vincent Legoll wrote:
>
> > Hello,
> >
> > here is a small documentation patch for the KConfig file format
> > "def_bool" type definition that was missing.
>
> There are also def_boolean and def_tristate.

while the parser recognizes "def_boolean", it's not used anywhere in
the tree. perhaps it can simply be dropped in favour of def_bool.

rday
--
========================================================================
Robert P. J. Day
Linux Consulting, Training and Annoying Kernel Pedantry
Waterloo, Ontario, CANADA

http://fsdev.net/wiki/index.php?title=Main_Page
========================================================================

2007-08-13 14:11:36

by Vincent Legoll

[permalink] [raw]
Subject: Re: [PATCH] - Add "def_bool" Kconfig format documentation

On 8/12/07, Randy Dunlap <[email protected]> wrote:
> There are also def_boolean and def_tristate.
> I suppose that I should resubmit this patch:
> http://marc.info/?l=linux-kernel&m=115471883121076&w=2

This one is OK for me, you can add:
Signed-off-by: Vincent Legoll <[email protected]>

--
Vincent Legoll