On 11/9/20 1:31 AM, [email protected] wrote: > From: Lars Poeschel <[email protected]> > > When building the hd44780_common driver without a driver that actually > uses it like panel or hd44780 you got a build error, because > hd44780_common uses charlcd, but did not select it. It's users did > select it. > This is fixed now. hd4478_common now selects charlcd in Kconfig and > panel and hd44780 do not. They only select hd44780_common. > > Reported-by: Randy Dunlap <[email protected]> > Signed-off-by: Lars Poeschel <[email protected]>
Acked-by: Randy Dunlap <[email protected]> # build-tested Thanks. > --- > drivers/auxdisplay/Kconfig | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/auxdisplay/Kconfig b/drivers/auxdisplay/Kconfig > index a69623124a26..a2b59b84bb88 100644 > --- a/drivers/auxdisplay/Kconfig > +++ b/drivers/auxdisplay/Kconfig > @@ -27,6 +27,7 @@ config CHARLCD > > config HD44780_COMMON > tristate "Common functions for HD44780 (and compatibles) LCD displays" > if COMPILE_TEST > + select CHARLCD > help > This is a module with the common symbols for HD44780 (and compatibles) > displays. This is the code that multiple other modules use. It is not > @@ -37,7 +38,6 @@ config HD44780_COMMON > config HD44780 > tristate "HD44780 Character LCD support" > depends on GPIOLIB || COMPILE_TEST > - select CHARLCD > select HD44780_COMMON > help > Enable support for Character LCDs using a HD44780 controller. > @@ -196,7 +196,6 @@ config ARM_CHARLCD > menuconfig PARPORT_PANEL > tristate "Parallel port LCD/Keypad Panel support" > depends on PARPORT > - select CHARLCD > select HD44780_COMMON > help > Say Y here if you have an HD44780 or KS-0074 LCD connected to your > -- ~Randy

