Same for the commit subject. On Sat, Sep 14, 2024 at 2:12 AM David Hunter <david.hunter.linux@xxxxxxxxx> wrote: > > Track default options on the second line. On the second line of some > config entries, default and depndency options sometimes appear. In those > instances, the state will be "NEW" and not "DEP". > > Signed-off-by: David Hunter <david.hunter.linux@xxxxxxxxx> > --- > scripts/kconfig/streamline_config.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/kconfig/streamline_config.pl b/scripts/kconfig/streamline_config.pl > index ddc630f2264a..bb1f19a1ab5e 100755 > --- a/scripts/kconfig/streamline_config.pl > +++ b/scripts/kconfig/streamline_config.pl > @@ -220,7 +220,7 @@ sub read_kconfig { > $depends{$config} = $1; > } elsif ($state eq "DEP" && /^\s*depends\s+on\s+(.*)$/) { > $depends{$config} .= " " . $1; > - } elsif ($state eq "DEP" && /^\s*def(_(bool|tristate)|ault)\s+(\S.*)$/) { > + } elsif (($state eq "DEP" || $state eq "NEW") && /^\s*def(_(bool|tristate)|ault)\s+(\S.*)$/) { I agree this is correct, but you can also fix it to $state ne "NONE" This is more consistent with the existing code. A few lines below, we already have this: } elsif ($state ne "NONE" && /^\s*((bool|tristate)\s+\S|prompt\b)/) { This will work unless you introduce a new state, "CHOICE". I am not sure if it is necessary. > my $dep = $3; > if ($dep !~ /^\s*(y|m|n)\s*$/) { > $dep =~ s/.*\sif\s+//; > -- > 2.43.0 > -- Best Regards Masahiro Yamada