Messages in this thread | | | From | Andy Shevchenko <> | Date | Thu, 1 Mar 2018 12:50:36 +0200 | Subject | Re: [PATCH 1/2] auxdisplay: fix broken menu |
| |
On Thu, Mar 1, 2018 at 4:33 AM, 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 near the end of the file so that the menu > display is continuous. > > Also include ARM_CHARLCD inside of the if AUXDISPLAY/endif block. > Geert says that it should be there. > > Fixes: 39f8ea46724e ("auxdisplay: charlcd: Extract character LCD core from misc/panel") > > Cc: stable@vger.kernel.org # v4.12 > Cc: Geert Uytterhoeven <geert@linux-m68k.org> > Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> Cc: Miguel Ojeda Sandonis <miguel.ojeda.sandonis@gmail.com> > Signed-off-by: Randy Dunlap <rdunlap@infradead.org> > --- > drivers/auxdisplay/Kconfig | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > --- linux-next-20180228.orig/drivers/auxdisplay/Kconfig > +++ linux-next-20180228/drivers/auxdisplay/Kconfig > @@ -14,9 +14,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 > @@ -157,8 +154,6 @@ config HT16K33 > Say yes here to add support for Holtek HT16K33, RAM mapping 16*8 > LED controller driver with keyscan. > > -endif # AUXDISPLAY > - > config ARM_CHARLCD > bool "ARM Ltd. Character LCD Driver" > depends on PLAT_VERSATILE > @@ -169,6 +164,8 @@ config ARM_CHARLCD > line and the Linux version on the second line, but that's > still useful. > > +endif # AUXDISPLAY > + > config PANEL > tristate "Parallel port LCD/Keypad Panel support" > depends on PARPORT > @@ -448,3 +445,6 @@ config PANEL_BOOT_MESSAGE > printf()-formatted message is valid with newline and escape codes. > > endif # PANEL > + > +config CHARLCD > + tristate "Character LCD core support" if COMPILE_TEST > >
-- With Best Regards, Andy Shevchenko
| |