linux-kernel - Re: [PATCH] auxdisplay: fix broken menus

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 22 Sep 2017 13:08:25 -0700
From:   Randy Dunlap <rdunlap@...radead.org>
To:     Geert Uytterhoeven <geert@...ux-m68k.org>
Cc:     LKML <linux-kernel@...r.kernel.org>,
        Linus Torvalds <torvalds@...ux-foundation.org>,
        Andrew Morton <akpm@...ux-foundation.org>,
        stable <stable@...r.kernel.org>,
        Miguel Ojeda Sandonis <miguel.ojeda.sandonis@...il.com>
Subject: Re: [PATCH] auxdisplay: fix broken menus

On 09/22/17 12:46, Geert Uytterhoeven wrote:
> Hi Randy,
> 
> On Fri, Sep 22, 2017 at 8:23 PM, Randy Dunlap <rdunlap@...radead.org> wrote:
>> On 09/22/17 11:15, Geert Uytterhoeven wrote:
>>> On Fri, Sep 22, 2017 at 6:24 PM, Randy Dunlap <rdunlap@...radead.org> wrote:
>>>> From: Randy Dunlap <rdunlap@...radead.org>
>>>>
>>>> Having the CHARLCD Kconfig symbol between "menuconfig AUXDISPLAY"
>>>> and "if AUXDISPLAY" breaks the AUXDISPLAY submenus, so move the
>>>> CHARLCD Kconfig symbol above the "menuconfig" so that the menu
>>>> display is continuous.
>>>>
>>>> Fixes: 39f8ea46724e (auxdisplay: charlcd: Extract character LCD core from misc/panel)
>>>
>>> Oops, sorry for that.
>>>
>>>> Cc: stable@...r.kernel.org # v4.12
>>>> Cc: Geert Uytterhoeven <geert@...ux-m68k.org>
>>>> Cc: Miguel Ojeda Sandonis <miguel.ojeda.sandonis@...il.com>
>>>> Signed-off-by: Randy Dunlap <rdunlap@...radead.org>
>>>> ---
>>>>  drivers/auxdisplay/Kconfig |    6 +++---
>>>>  1 file changed, 3 insertions(+), 3 deletions(-)
>>>>
>>>> --- lnx-414-rc1.orig/drivers/auxdisplay/Kconfig
>>>> +++ lnx-414-rc1/drivers/auxdisplay/Kconfig
>>>> @@ -5,6 +5,9 @@
>>>>  # Auxiliary display drivers configuration.
>>>>  #
>>>>
>>>> +config CHARLCD
>>>> +       tristate "Character LCD core support" if COMPILE_TEST
>>>> +
>>>
>>> What about moving it inside (i.e. right below) the "if AUXDISPLAY" instead?
>>
>> That works.
>> But why?  PANEL also selects CHARLCD, so it isn't specific to AUXDISPLAY.
> 
> OK, I thought PANEL was under AUXDISPLAY, but apparently it isn't.
> Perhaps it should be, just like ARM_CHARLCD?
> I.e. move everything inside the big if AUXDISPLAY block?

ARM_CHARLCD does not depend on AUXDISPLAY either.

I wouldn't mind if all of them were inside the AUXDISPLAY if/endif
for menu cleanness, but AFAIK it's not necessary for code dependencies.

I was just working on a separate patch to change PANEL from config to
menuconfig. That makes the PANEL sub-menu presentation cleaner.

> 
>>>>  menuconfig AUXDISPLAY
>>>>         bool "Auxiliary Display support"
>>>>         ---help---
>>>> @@ -13,9 +16,6 @@ menuconfig AUXDISPLAY
>>>>
>>>>           If you say N, all options in this submenu will be skipped and disabled.
>>>>
>>>> -config CHARLCD
>>>> -       tristate "Character LCD core support" if COMPILE_TEST
>>>> -
>>>>  if AUXDISPLAY
>>>>
>>>>  config HD44780


-- 
~Randy