Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
In-Reply-To: <1507096094-14953-1-git-send-email-ulfalizer@gmail.com>
References: <1507096094-14953-1-git-send-email-ulfalizer@gmail.com>
From:   Ulf Magnusson <ulfalizer@gmail.com>
Date:   Thu, 5 Oct 2017 05:14:41 +0200
Message-ID: <CAFkk2KTp4njaTw5xAPC7V1cArui=7jqu3ORMTCzaACz6dngnyA@mail.gmail.com>
Subject: Re: [PATCH] kconfig: Document the 'symbol' struct
To:     yann.morin.1998@free.fr, linux-kbuild@vger.kernel.org
Cc:     linux-kernel@vger.kernel.org, Ulf Magnusson <ulfalizer@gmail.com>,
        Masahiro Yamada <yamada.masahiro@socionext.com>,
        Arnaud Lacombe <lacombar@gmail.com>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Wed, Oct 4, 2017 at 7:48 AM, Ulf Magnusson <ulfalizer@gmail.com> wrote:
> Visibility and choices in particular might be a bit tricky to figure
> out.
>
> Also fix existing comment to point out that P_MENU is also used for
> menus.
>
> Signed-off-by: Ulf Magnusson <ulfalizer@gmail.com>
> ---
>  scripts/kconfig/expr.h | 45 ++++++++++++++++++++++++++++++++++++++++++++-
>  1 file changed, 44 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/kconfig/expr.h b/scripts/kconfig/expr.h
> index a73f762..6344067 100644
> --- a/scripts/kconfig/expr.h
> +++ b/scripts/kconfig/expr.h
> @@ -74,17 +74,60 @@ enum {
>         S_DEF_COUNT
>  };
>
> +/*
> + * Represents a configuration symbol.
> + *
> + * Choices are represented as a special kind of symbol and have the
> + * SYMBOL_CHOICE bit set in 'flags'.
> + */
>  struct symbol {
> +       /* The next symbol in the same bucket in the symbol hash table */
>         struct symbol *next;
> +
> +       /* The name of the symbol, e.g. "FOO" for 'config FOO' */
>         char *name;
> +
> +       /* S_BOOLEAN, S_TRISTATE, ... */
>         enum symbol_type type;
> +
> +       /*
> +        * The calculated value of the symbol. The SYMBOL_VALID bit is set in
> +        * 'flags' when this is up to date. Note that this value might differ
> +        * from the user value set in e.g. a .config file, due to visibility.
> +        */
>         struct symbol_value curr;
> +
> +       /*
> +        * Values for the symbol provided from outside. def[S_DEF_USER] holds
> +        * the .config value.
> +        */
>         struct symbol_value def[S_DEF_COUNT];
> +
> +       /*
> +        * An upper bound on the tristate value the user can set for the symbol
> +        * if it is a boolean or tristate. Calculated from prompt dependencies,
> +        * which also inherit dependencies from enclosing menus, choices, and
> +        * ifs. If 'n', the user value will be ignored.
> +        *
> +        * Symbols lacking prompts always have visibility 'n'.
> +        */
>         tristate visible;
> +
> +       /* SYMBOL_* flags */
>         int flags;
> +
> +       /* List of properties. See prop_type. */
>         struct property *prop;
> +
> +       /* Dependencies from enclosing menus, choices, and ifs */
>         struct expr_value dir_dep;
> +
> +       /* Reverse dependencies through being selected by other symbols */
>         struct expr_value rev_dep;
> +
> +       /*
> +        * "Weak" reverse dependencies through being implied by other symbols
> +        */
>         struct expr_value implied;
>  };
>
> @@ -133,7 +176,7 @@ enum prop_type {
>         P_UNKNOWN,
>         P_PROMPT,   /* prompt "foo prompt" or "BAZ Value" */
>         P_COMMENT,  /* text associated with a comment */
> -       P_MENU,     /* prompt associated with a menuconfig option */
> +       P_MENU,     /* prompt associated with a menu or menuconfig symbol */
>         P_DEFAULT,  /* default y */
>         P_CHOICE,   /* choice value */
>         P_SELECT,   /* select BAR */
> --
> 2.7.4
>

Adding some CCs just because I forgot. No panic.

Cheers,
Ulf

From 1580304871236582169@xxx Wed Oct 04 05:49:27 +0000 2017
X-GM-THRID: 1580304871236582169
X-Gmail-Labels: Inbox,Category Forums