Messages in this thread | | | From | Geert Uytterhoeven <> | Date | Fri, 22 Sep 2017 22:17:27 +0200 | Subject | Re: [PATCH] auxdisplay: fix broken menus |
| |
Hi Randy,
On Fri, Sep 22, 2017 at 10:08 PM, Randy Dunlap <rdunlap@infradead.org> wrote: > On 09/22/17 12:46, Geert Uytterhoeven wrote: >> On Fri, Sep 22, 2017 at 8:23 PM, Randy Dunlap <rdunlap@infradead.org> wrote: >>> On 09/22/17 11:15, Geert Uytterhoeven wrote: >>>> On Fri, Sep 22, 2017 at 6:24 PM, Randy Dunlap <rdunlap@infradead.org> wrote: >>>>> From: Randy Dunlap <rdunlap@infradead.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@vger.kernel.org # v4.12 >>>>> Cc: Geert Uytterhoeven <geert@linux-m68k.org> >>>>> Cc: Miguel Ojeda Sandonis <miguel.ojeda.sandonis@gmail.com> >>>>> Signed-off-by: Randy Dunlap <rdunlap@infradead.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.
Technically it's an auxiliary display, AFAIK.
> 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.
Indeed.
> I was just working on a separate patch to change PANEL from config to > menuconfig. That makes the PANEL sub-menu presentation cleaner.
OK. So let's make progress with your patch for now.
Acked-by: Geert Uytterhoeven <geert@linux-m68k.org>
Gr{oetje,eeting}s,
Geert
-- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
| |