Re: [PATCH 1/3] kconfig: link menus to a symbol

From: Nicolas Schier
Date: Mon Mar 04 2024 - 04:30:14 EST


On Sun, Mar 03, 2024 at 01:00:33PM +0900 Masahiro Yamada wrote:
> Currently, there is no direct link from (struct symbol *) to
> (struct menu *).
>
> It is still possible to access associated menus through the P_SYMBOL
> property, because property::menu is the relevant menu entry, but it
> results in complex code, as seen in get_symbol_str().
>
> Use a linked list for simpler traversal of relevant menus.
>
> Signed-off-by: Masahiro Yamada <masahiroy@xxxxxxxxxx>
> ---
>
> scripts/kconfig/expr.h | 5 +++++
> scripts/kconfig/menu.c | 4 +++-
> scripts/kconfig/symbol.c | 4 ++++
> 3 files changed, 12 insertions(+), 1 deletion(-)

Thanks, the whole series looks good to me.

Reviewed-by: Nicolas Schier <nicolas@xxxxxxxxx>