From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH 23/27] backlight: Allow compile test of GPIO consumers if !GPIOLIB Date: Thu, 7 May 2015 08:37:08 +0100 Message-ID: <20150507073708.GA12484@x1> References: <1430836404-15513-1-git-send-email-geert@linux-m68k.org> <1430843563-18615-1-git-send-email-geert@linux-m68k.org> <1430843563-18615-23-git-send-email-geert@linux-m68k.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: <1430843563-18615-23-git-send-email-geert@linux-m68k.org> Sender: linux-kernel-owner@vger.kernel.org To: Geert Uytterhoeven Cc: Linus Walleij , Alexandre Courbot , Arnd Bergmann , linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, Jingoo Han , Jean-Christophe Plagniol-Villard , Tomi Valkeinen , linux-fbdev@vger.kernel.org List-Id: linux-gpio@vger.kernel.org On Tue, 05 May 2015, Geert Uytterhoeven wrote: > The GPIO subsystem provides dummy GPIO consumer functions if GPIOLIB = is > not enabled. Hence drivers that depend on GPIOLIB, but use GPIO consu= mer > functionality only, can still be compiled if GPIOLIB is not enabled. >=20 > Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where > appropriate. >=20 > Signed-off-by: Geert Uytterhoeven > Cc: Jingoo Han > Cc: Lee Jones > Cc: Jean-Christophe Plagniol-Villard > Cc: Tomi Valkeinen > Cc: linux-fbdev@vger.kernel.org > --- > drivers/video/backlight/Kconfig | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) Looks good. Applied, thanks. > diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backligh= t/Kconfig > index 2d9923a60076e380..0505b796d743250e 100644 > --- a/drivers/video/backlight/Kconfig > +++ b/drivers/video/backlight/Kconfig > @@ -36,14 +36,16 @@ config LCD_CORGI > =20 > config LCD_L4F00242T03 > tristate "Epson L4F00242T03 LCD" > - depends on SPI_MASTER && GPIOLIB > + depends on SPI_MASTER > + depends on GPIOLIB || COMPILE_TEST > help > SPI driver for Epson L4F00242T03. This provides basic support > for init and powering the LCD up/down through a sysfs interface. > =20 > config LCD_LMS283GF05 > tristate "Samsung LMS283GF05 LCD" > - depends on SPI_MASTER && GPIOLIB > + depends on SPI_MASTER > + depends on GPIOLIB || COMPILE_TEST > help > SPI driver for Samsung LMS283GF05. This provides basic support > for powering the LCD up/down through a sysfs interface. > @@ -434,7 +436,7 @@ config BACKLIGHT_AS3711 > =20 > config BACKLIGHT_GPIO > tristate "Generic GPIO based Backlight Driver" > - depends on GPIOLIB > + depends on GPIOLIB || COMPILE_TEST > help > If you have a LCD backlight adjustable by GPIO, say Y to enable > this driver. --=20 Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Date: Thu, 07 May 2015 07:37:08 +0000 Subject: Re: [PATCH 23/27] backlight: Allow compile test of GPIO consumers if !GPIOLIB Message-Id: <20150507073708.GA12484@x1> List-Id: References: <1430836404-15513-1-git-send-email-geert@linux-m68k.org> <1430843563-18615-1-git-send-email-geert@linux-m68k.org> <1430843563-18615-23-git-send-email-geert@linux-m68k.org> In-Reply-To: <1430843563-18615-23-git-send-email-geert@linux-m68k.org> MIME-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: quoted-printable To: Geert Uytterhoeven Cc: Linus Walleij , Alexandre Courbot , Arnd Bergmann , linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, Jingoo Han , Jean-Christophe Plagniol-Villard , Tomi Valkeinen , linux-fbdev@vger.kernel.org On Tue, 05 May 2015, Geert Uytterhoeven wrote: > The GPIO subsystem provides dummy GPIO consumer functions if GPIOLIB is > not enabled. Hence drivers that depend on GPIOLIB, but use GPIO consumer > functionality only, can still be compiled if GPIOLIB is not enabled. >=20 > Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where > appropriate. >=20 > Signed-off-by: Geert Uytterhoeven > Cc: Jingoo Han > Cc: Lee Jones > Cc: Jean-Christophe Plagniol-Villard > Cc: Tomi Valkeinen > Cc: linux-fbdev@vger.kernel.org > --- > drivers/video/backlight/Kconfig | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) Looks good. Applied, thanks. > diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kc= onfig > index 2d9923a60076e380..0505b796d743250e 100644 > --- a/drivers/video/backlight/Kconfig > +++ b/drivers/video/backlight/Kconfig > @@ -36,14 +36,16 @@ config LCD_CORGI > =20 > config LCD_L4F00242T03 > tristate "Epson L4F00242T03 LCD" > - depends on SPI_MASTER && GPIOLIB > + depends on SPI_MASTER > + depends on GPIOLIB || COMPILE_TEST > help > SPI driver for Epson L4F00242T03. This provides basic support > for init and powering the LCD up/down through a sysfs interface. > =20 > config LCD_LMS283GF05 > tristate "Samsung LMS283GF05 LCD" > - depends on SPI_MASTER && GPIOLIB > + depends on SPI_MASTER > + depends on GPIOLIB || COMPILE_TEST > help > SPI driver for Samsung LMS283GF05. This provides basic support > for powering the LCD up/down through a sysfs interface. > @@ -434,7 +436,7 @@ config BACKLIGHT_AS3711 > =20 > config BACKLIGHT_GPIO > tristate "Generic GPIO based Backlight Driver" > - depends on GPIOLIB > + depends on GPIOLIB || COMPILE_TEST > help > If you have a LCD backlight adjustable by GPIO, say Y to enable > this driver. --=20 Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog