linux-gpio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB
@ 2015-08-02  9:07 Geert Uytterhoeven
  2015-08-02  9:08 ` [PATCH v2 01/11] tpm: " Geert Uytterhoeven
                   ` (11 more replies)
  0 siblings, 12 replies; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-02  9:07 UTC (permalink / raw)
  To: Arnd Bergmann, David S. Miller, David Woodhouse,
	Dmitry Eremin-Solenikov, Dmitry Torokhov, Evgeniy Polyakov,
	Florian Fainelli, Greg Kroah-Hartman,
	Jean-Christophe Plagniol-Villard, Johannes Berg, Marcel Selhorst,
	Noralf Trønnes, Peter Huewe, Sebastian Reichel,
	Thomas Petazzoni, Tomi Valkeinen
  Cc: Linus Walleij, Alexandre Courbot, linux-gpio, linux-kernel,
	Geert Uytterhoeven

	Hi all,

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.
This patch series relaxes the dependencies on GPIOLIB if COMPILE_TEST is
enabled, where appropriate.

There are a few more drivers that depend on GPIOLIB and that may be
eligible for COMPILE_TEST. These I didn't handle yet, as I couldn't
enable them with m68k/all{mod,yes}config due to other dependencies.

This series is the remaining part of ("[PATCH 00/27] gpio: Allow compile
test of GPIO consumers if !GPIOLIB", https://lkml.org/lkml/2015/5/5/779),
updated for new config symbols up to v4.2-rc4.  It applies to
next-20150731.

Thanks for applying!

Geert Uytterhoeven (11):
  tpm: Allow compile test of GPIO consumers if !GPIOLIB
  input: Allow compile test of GPIO consumers if !GPIOLIB
  misc: ti-st: Allow compile test of GPIO consumers if !GPIOLIB
  net: phy: Allow compile test of GPIO consumers if !GPIOLIB
  power: Allow compile test of GPIO consumers if !GPIOLIB
  staging: android: Allow compile test of GPIO consumers if !GPIOLIB
  staging: fbtft: Allow compile test of GPIO consumers if !GPIOLIB
  serial: Allow compile test of GPIO consumers if !GPIOLIB
  fbdev: Allow compile test of GPIO consumers if !GPIOLIB
  w1: Allow compile test of GPIO consumers if !GPIOLIB
  net: rfkill: Allow compile test of GPIO consumers if !GPIOLIB

 drivers/char/tpm/st33zp24/Kconfig |  2 +-
 drivers/input/keyboard/Kconfig    |  4 ++--
 drivers/input/misc/Kconfig        | 11 ++++++-----
 drivers/input/mouse/Kconfig       |  2 +-
 drivers/input/touchscreen/Kconfig |  8 ++++----
 drivers/misc/ti-st/Kconfig        |  3 ++-
 drivers/net/phy/Kconfig           |  3 ++-
 drivers/power/Kconfig             | 17 +++++++++++------
 drivers/staging/android/Kconfig   |  3 ++-
 drivers/staging/fbtft/Kconfig     |  3 ++-
 drivers/tty/serial/Kconfig        |  3 ++-
 drivers/video/fbdev/Kconfig       |  2 +-
 drivers/w1/masters/Kconfig        |  2 +-
 net/rfkill/Kconfig                |  3 ++-
 14 files changed, 39 insertions(+), 27 deletions(-)

-- 
1.9.1

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

^ permalink raw reply	[flat|nested] 23+ messages in thread

* [PATCH v2 01/11] tpm: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:07 [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB Geert Uytterhoeven
@ 2015-08-02  9:08 ` Geert Uytterhoeven
  2015-08-02  9:09 ` [PATCH v2 02/11] input: " Geert Uytterhoeven
                   ` (10 subsequent siblings)
  11 siblings, 0 replies; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-02  9:08 UTC (permalink / raw)
  To: Peter Huewe, Marcel Selhorst
  Cc: tpmdd-devel, Linus Walleij, Alexandre Courbot, linux-gpio,
	Geert Uytterhoeven

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.

Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
appropriate.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
---
v2:
  - Add Acked-by.
---
 drivers/char/tpm/st33zp24/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/tpm/st33zp24/Kconfig b/drivers/char/tpm/st33zp24/Kconfig
index 09cb727864f0950f..19c007461d1cbd7a 100644
--- a/drivers/char/tpm/st33zp24/Kconfig
+++ b/drivers/char/tpm/st33zp24/Kconfig
@@ -1,6 +1,6 @@
 config TCG_TIS_ST33ZP24
 	tristate "STMicroelectronics TPM Interface Specification 1.2 Interface"
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	---help---
 	  STMicroelectronics ST33ZP24 core driver. It implements the core
 	  TPM1.2 logic and hooks into the TPM kernel APIs. Physical layers will
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH v2 02/11] input: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:07 [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB Geert Uytterhoeven
  2015-08-02  9:08 ` [PATCH v2 01/11] tpm: " Geert Uytterhoeven
@ 2015-08-02  9:09 ` Geert Uytterhoeven
  2015-08-04 18:44   ` Dmitry Torokhov
  2015-08-02  9:09 ` [PATCH v2 03/11] misc: ti-st: " Geert Uytterhoeven
                   ` (9 subsequent siblings)
  11 siblings, 1 reply; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-02  9:09 UTC (permalink / raw)
  To: Dmitry Torokhov
  Cc: linux-input, Linus Walleij, Alexandre Courbot, linux-gpio,
	Geert Uytterhoeven

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.

Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
appropriate.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
---
v2:
  - Add Acked-by.
---
 drivers/input/keyboard/Kconfig    |  4 ++--
 drivers/input/misc/Kconfig        | 11 ++++++-----
 drivers/input/mouse/Kconfig       |  2 +-
 drivers/input/touchscreen/Kconfig |  8 ++++----
 4 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/drivers/input/keyboard/Kconfig b/drivers/input/keyboard/Kconfig
index 82a8fb50afacb253..da53ae6a93346c90 100644
--- a/drivers/input/keyboard/Kconfig
+++ b/drivers/input/keyboard/Kconfig
@@ -187,7 +187,7 @@ config KEYBOARD_EP93XX
 
 config KEYBOARD_GPIO
 	tristate "GPIO Buttons"
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	help
 	  This driver implements support for buttons connected
 	  to GPIO pins of various CPUs (and some other chips).
@@ -253,7 +253,7 @@ config KEYBOARD_TCA8418
 
 config KEYBOARD_MATRIX
 	tristate "GPIO driven matrix keypad support"
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	select INPUT_MATRIXKMAP
 	help
 	  Enable support for GPIO driven matrix keypad.
diff --git a/drivers/input/misc/Kconfig b/drivers/input/misc/Kconfig
index d4f0a817e858262b..dade381c7da80eb9 100644
--- a/drivers/input/misc/Kconfig
+++ b/drivers/input/misc/Kconfig
@@ -259,7 +259,7 @@ config INPUT_APANEL
 config INPUT_GP2A
 	tristate "Sharp GP2AP002A00F I2C Proximity/Opto sensor driver"
 	depends on I2C
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	help
 	  Say Y here if you have a Sharp GP2AP002A00F proximity/als combo-chip
 	  hooked to an I2C bus.
@@ -269,7 +269,7 @@ config INPUT_GP2A
 
 config INPUT_GPIO_BEEPER
 	tristate "Generic GPIO Beeper support"
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	help
 	  Say Y here if you have a beeper connected to a GPIO pin.
 
@@ -278,7 +278,7 @@ config INPUT_GPIO_BEEPER
 
 config INPUT_GPIO_TILT_POLLED
 	tristate "Polled GPIO tilt switch"
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	select INPUT_POLLDEV
 	help
 	  This driver implements support for tilt switches connected
@@ -569,7 +569,7 @@ config INPUT_PWM_BEEPER
 
 config INPUT_GPIO_ROTARY_ENCODER
 	tristate "Rotary encoders connected to GPIO pins"
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	help
 	  Say Y here to add support for rotary encoders connected to GPIO lines.
 	  Check file:Documentation/input/rotary-encoder.txt for more
@@ -776,7 +776,8 @@ config INPUT_SOC_BUTTON_ARRAY
 
 config INPUT_DRV260X_HAPTICS
 	tristate "TI DRV260X haptics support"
-	depends on INPUT && I2C && GPIOLIB
+	depends on INPUT && I2C
+	depends on GPIOLIB || COMPILE_TEST
 	select INPUT_FF_MEMLESS
 	select REGMAP_I2C
 	help
diff --git a/drivers/input/mouse/Kconfig b/drivers/input/mouse/Kconfig
index d7820d1152d2ef2d..17f97e5e11e7cd2e 100644
--- a/drivers/input/mouse/Kconfig
+++ b/drivers/input/mouse/Kconfig
@@ -341,7 +341,7 @@ config MOUSE_VSXXXAA
 
 config MOUSE_GPIO
 	tristate "GPIO mouse"
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	select INPUT_POLLDEV
 	help
 	  This driver simulates a mouse on GPIO lines of various CPUs (and some
diff --git a/drivers/input/touchscreen/Kconfig b/drivers/input/touchscreen/Kconfig
index 860d426c480e2088..059edeb7f04a0d40 100644
--- a/drivers/input/touchscreen/Kconfig
+++ b/drivers/input/touchscreen/Kconfig
@@ -118,7 +118,7 @@ config TOUCHSCREEN_ATMEL_MXT
 config TOUCHSCREEN_AUO_PIXCIR
 	tristate "AUO in-cell touchscreen using Pixcir ICs"
 	depends on I2C
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	help
 	  Say Y here if you have a AUO display with in-cell touchscreen
 	  using Pixcir ICs.
@@ -142,7 +142,7 @@ config TOUCHSCREEN_BU21013
 
 config TOUCHSCREEN_CHIPONE_ICN8318
 	tristate "chipone icn8318 touchscreen controller"
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	depends on I2C
 	depends on OF
 	help
@@ -156,7 +156,7 @@ config TOUCHSCREEN_CHIPONE_ICN8318
 config TOUCHSCREEN_CY8CTMG110
 	tristate "cy8ctmg110 touchscreen"
 	depends on I2C
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	help
 	  Say Y here if you have a cy8ctmg110 capacitive touchscreen on
 	  an AAVA device.
@@ -1030,7 +1030,7 @@ config TOUCHSCREEN_TPS6507X
 config TOUCHSCREEN_ZFORCE
 	tristate "Neonode zForce infrared touchscreens"
 	depends on I2C
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	help
 	  Say Y here if you have a touchscreen using the zforce
 	  infraread technology from Neonode.
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH v2 03/11] misc: ti-st: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:07 [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB Geert Uytterhoeven
  2015-08-02  9:08 ` [PATCH v2 01/11] tpm: " Geert Uytterhoeven
  2015-08-02  9:09 ` [PATCH v2 02/11] input: " Geert Uytterhoeven
@ 2015-08-02  9:09 ` Geert Uytterhoeven
  2015-08-02  9:09 ` [PATCH v2 05/11] power: " Geert Uytterhoeven
                   ` (8 subsequent siblings)
  11 siblings, 0 replies; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-02  9:09 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Linus Walleij, Alexandre Courbot, linux-gpio, Geert Uytterhoeven

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.

Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
appropriate.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
---
v2:
  - Add Acked-by.
---
 drivers/misc/ti-st/Kconfig | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/misc/ti-st/Kconfig b/drivers/misc/ti-st/Kconfig
index f34dcc51473052cf..5bb92698bc80f450 100644
--- a/drivers/misc/ti-st/Kconfig
+++ b/drivers/misc/ti-st/Kconfig
@@ -5,7 +5,8 @@
 menu "Texas Instruments shared transport line discipline"
 config TI_ST
 	tristate "Shared transport core driver"
-	depends on NET && GPIOLIB && TTY
+	depends on NET && TTY
+	depends on GPIOLIB || COMPILE_TEST
 	select FW_LOADER
 	help
 	  This enables the shared transport core driver for TI
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH v2 04/11] net: phy: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:07 [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB Geert Uytterhoeven
                   ` (3 preceding siblings ...)
  2015-08-02  9:09 ` [PATCH v2 05/11] power: " Geert Uytterhoeven
@ 2015-08-02  9:09 ` Geert Uytterhoeven
  2015-08-02  9:09 ` [PATCH v2 06/11] staging: android: " Geert Uytterhoeven
                   ` (6 subsequent siblings)
  11 siblings, 0 replies; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-02  9:09 UTC (permalink / raw)
  To: Florian Fainelli
  Cc: netdev, Linus Walleij, Alexandre Courbot, linux-gpio,
	Geert Uytterhoeven

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.

Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
appropriate.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
---
v2:
  - Add Acked-by.
---
 drivers/net/phy/Kconfig | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/net/phy/Kconfig b/drivers/net/phy/Kconfig
index 8ef81914422049ff..b98b4a2b8922a912 100644
--- a/drivers/net/phy/Kconfig
+++ b/drivers/net/phy/Kconfig
@@ -142,7 +142,8 @@ config MDIO_BITBANG
 
 config MDIO_GPIO
 	tristate "Support for GPIO lib-based bitbanged MDIO buses"
-	depends on MDIO_BITBANG && GPIOLIB
+	depends on MDIO_BITBANG
+	depends on GPIOLIB || COMPILE_TEST
 	---help---
 	  Supports GPIO lib-based MDIO busses.
 
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH v2 05/11] power: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:07 [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB Geert Uytterhoeven
                   ` (2 preceding siblings ...)
  2015-08-02  9:09 ` [PATCH v2 03/11] misc: ti-st: " Geert Uytterhoeven
@ 2015-08-02  9:09 ` Geert Uytterhoeven
  2015-08-05  3:49   ` Sebastian Reichel
  2015-08-02  9:09 ` [PATCH v2 04/11] net: phy: " Geert Uytterhoeven
                   ` (7 subsequent siblings)
  11 siblings, 1 reply; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-02  9:09 UTC (permalink / raw)
  To: Sebastian Reichel, Dmitry Eremin-Solenikov, David Woodhouse
  Cc: linux-pm, Linus Walleij, Alexandre Courbot, linux-gpio,
	Geert Uytterhoeven

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.

Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
appropriate.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
---
v2:
  - Add Acked-by,
  - Add recently added CHARGER_BQ24257, CHARGER_BQ25890, and
    CHARGER_RT9455.
---
 drivers/power/Kconfig | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/drivers/power/Kconfig b/drivers/power/Kconfig
index 08beeed5485d3067..f8758d6febf818a7 100644
--- a/drivers/power/Kconfig
+++ b/drivers/power/Kconfig
@@ -333,7 +333,7 @@ config CHARGER_LP8788
 
 config CHARGER_GPIO
 	tristate "GPIO charger"
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	help
 	  Say Y to include support for chargers which report their online status
 	  through a GPIO pin.
@@ -391,26 +391,30 @@ config CHARGER_BQ2415X
 
 config CHARGER_BQ24190
 	tristate "TI BQ24190 battery charger driver"
-	depends on I2C && GPIOLIB
+	depends on I2C
+	depends on GPIOLIB || COMPILE_TEST
 	help
 	  Say Y to enable support for the TI BQ24190 battery charger.
 
 config CHARGER_BQ24257
 	tristate "TI BQ24257 battery charger driver"
-	depends on I2C && GPIOLIB
+	depends on I2C
+	depends on GPIOLIB || COMPILE_TEST
 	depends on REGMAP_I2C
 	help
 	  Say Y to enable support for the TI BQ24257 battery charger.
 
 config CHARGER_BQ24735
 	tristate "TI BQ24735 battery charger support"
-	depends on I2C && GPIOLIB
+	depends on I2C
+	depends on GPIOLIB || COMPILE_TEST
 	help
 	  Say Y to enable support for the TI BQ24735 battery charger.
 
 config CHARGER_BQ25890
 	tristate "TI BQ25890 battery charger driver"
-	depends on I2C && GPIOLIB
+	depends on I2C
+	depends on GPIOLIB || COMPILE_TEST
 	select REGMAP_I2C
 	help
 	  Say Y to enable support for the TI BQ25890 battery charger.
@@ -462,7 +466,8 @@ config BATTERY_RT5033
 
 config CHARGER_RT9455
 	tristate "Richtek RT9455 battery charger driver"
-	depends on I2C && GPIOLIB
+	depends on I2C
+	depends on GPIOLIB || COMPILE_TEST
 	select REGMAP_I2C
 	help
 	  Say Y to enable support for Richtek RT9455 battery charger.
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH v2 06/11] staging: android: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:07 [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB Geert Uytterhoeven
                   ` (4 preceding siblings ...)
  2015-08-02  9:09 ` [PATCH v2 04/11] net: phy: " Geert Uytterhoeven
@ 2015-08-02  9:09 ` Geert Uytterhoeven
  2015-08-02  9:09 ` [PATCH v2 08/11] serial: " Geert Uytterhoeven
                   ` (5 subsequent siblings)
  11 siblings, 0 replies; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-02  9:09 UTC (permalink / raw)
  To: Greg Kroah-Hartman
  Cc: devel, Linus Walleij, Alexandre Courbot, linux-gpio,
	Geert Uytterhoeven

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.

Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
appropriate.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
---
v2:
  - Add Acked-by.
---
 drivers/staging/android/Kconfig | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/android/Kconfig b/drivers/staging/android/Kconfig
index 24d657b3ab9946d0..68307121c9c18830 100644
--- a/drivers/staging/android/Kconfig
+++ b/drivers/staging/android/Kconfig
@@ -20,7 +20,8 @@ config ANDROID_TIMED_OUTPUT
 
 config ANDROID_TIMED_GPIO
 	tristate "Android timed gpio driver"
-	depends on GPIOLIB && ANDROID_TIMED_OUTPUT
+	depends on GPIOLIB || COMPILE_TEST
+	depends on ANDROID_TIMED_OUTPUT
 	default n
         ---help---
 	  Unlike generic gpio is to allow programs to access and manipulate gpio
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH v2 07/11] staging: fbtft: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:07 [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB Geert Uytterhoeven
                   ` (6 preceding siblings ...)
  2015-08-02  9:09 ` [PATCH v2 08/11] serial: " Geert Uytterhoeven
@ 2015-08-02  9:09 ` Geert Uytterhoeven
  2015-08-02  9:09 ` [PATCH v2 09/11] fbdev: " Geert Uytterhoeven
                   ` (3 subsequent siblings)
  11 siblings, 0 replies; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-02  9:09 UTC (permalink / raw)
  To: Thomas Petazzoni, Noralf Trønnes
  Cc: devel, Linus Walleij, Alexandre Courbot, linux-gpio,
	Geert Uytterhoeven

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.

Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
appropriate.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
---
v2:
  - Add Acked-by.
---
 drivers/staging/fbtft/Kconfig | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/fbtft/Kconfig b/drivers/staging/fbtft/Kconfig
index f7e68fdd99047927..d473010fa4748409 100644
--- a/drivers/staging/fbtft/Kconfig
+++ b/drivers/staging/fbtft/Kconfig
@@ -1,6 +1,7 @@
 menuconfig FB_TFT
 	tristate "Support for small TFT LCD display modules"
-	depends on FB && SPI && GPIOLIB
+	depends on FB && SPI
+	depends on GPIOLIB || COMPILE_TEST
 	select FB_SYS_FILLRECT
 	select FB_SYS_COPYAREA
 	select FB_SYS_IMAGEBLIT
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH v2 08/11] serial: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:07 [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB Geert Uytterhoeven
                   ` (5 preceding siblings ...)
  2015-08-02  9:09 ` [PATCH v2 06/11] staging: android: " Geert Uytterhoeven
@ 2015-08-02  9:09 ` Geert Uytterhoeven
  2015-08-02  9:09 ` [PATCH v2 07/11] staging: fbtft: " Geert Uytterhoeven
                   ` (4 subsequent siblings)
  11 siblings, 0 replies; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-02  9:09 UTC (permalink / raw)
  To: Greg Kroah-Hartman
  Cc: linux-serial, Linus Walleij, Alexandre Courbot, linux-gpio,
	Geert Uytterhoeven

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.

Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
appropriate.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
---
v2:
  - Add Acked-by.
---
 drivers/tty/serial/Kconfig | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
index 4c8662ea7bb063ed..d6393e24acb30a2d 100644
--- a/drivers/tty/serial/Kconfig
+++ b/drivers/tty/serial/Kconfig
@@ -1376,7 +1376,8 @@ config SERIAL_ALTERA_UART_CONSOLE
 
 config SERIAL_IFX6X60
         tristate "SPI protocol driver for Infineon 6x60 modem (EXPERIMENTAL)"
-	depends on GPIOLIB && SPI && HAS_DMA
+	depends on GPIOLIB || COMPILE_TEST
+	depends on SPI && HAS_DMA
 	help
 	  Support for the IFX6x60 modem devices on Intel MID platforms.
 
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH v2 09/11] fbdev: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:07 [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB Geert Uytterhoeven
                   ` (7 preceding siblings ...)
  2015-08-02  9:09 ` [PATCH v2 07/11] staging: fbtft: " Geert Uytterhoeven
@ 2015-08-02  9:09 ` Geert Uytterhoeven
  2015-08-11  7:22   ` Tomi Valkeinen
  2015-08-02  9:09 ` [PATCH v2 11/11] net: rfkill: " Geert Uytterhoeven
                   ` (2 subsequent siblings)
  11 siblings, 1 reply; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-02  9:09 UTC (permalink / raw)
  To: Jean-Christophe Plagniol-Villard, Tomi Valkeinen
  Cc: linux-fbdev, Linus Walleij, Alexandre Courbot, linux-gpio,
	Geert Uytterhoeven

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.

Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
appropriate.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
---
v2:
  - Add Acked-by.
---
 drivers/video/fbdev/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/fbdev/Kconfig b/drivers/video/fbdev/Kconfig
index 2d98de535e0f7374..569803f533d1a600 100644
--- a/drivers/video/fbdev/Kconfig
+++ b/drivers/video/fbdev/Kconfig
@@ -2464,7 +2464,7 @@ config FB_SSD1307
 	tristate "Solomon SSD1307 framebuffer support"
 	depends on FB && I2C
 	depends on OF
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	select FB_SYS_FOPS
 	select FB_SYS_FILLRECT
 	select FB_SYS_COPYAREA
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH v2 11/11] net: rfkill: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:07 [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB Geert Uytterhoeven
                   ` (8 preceding siblings ...)
  2015-08-02  9:09 ` [PATCH v2 09/11] fbdev: " Geert Uytterhoeven
@ 2015-08-02  9:09 ` Geert Uytterhoeven
  2015-08-13  8:49   ` Johannes Berg
  2015-08-02  9:09 ` [PATCH v2 10/11] w1: " Geert Uytterhoeven
  2015-08-10 16:01 ` [PATCH v2 00/11] gpio: " Johannes Berg
  11 siblings, 1 reply; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-02  9:09 UTC (permalink / raw)
  To: Johannes Berg, David S. Miller
  Cc: netdev, Linus Walleij, Alexandre Courbot, linux-gpio,
	Geert Uytterhoeven

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.

Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
appropriate.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
---
v2:
  - Add Acked-by.
---
 net/rfkill/Kconfig | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/net/rfkill/Kconfig b/net/rfkill/Kconfig
index 4c10e7e6c9f6ae53..598d374f6a35f714 100644
--- a/net/rfkill/Kconfig
+++ b/net/rfkill/Kconfig
@@ -36,7 +36,8 @@ config RFKILL_REGULATOR
 
 config RFKILL_GPIO
 	tristate "GPIO RFKILL driver"
-	depends on RFKILL && GPIOLIB
+	depends on RFKILL
+	depends on GPIOLIB || COMPILE_TEST
 	default n
 	help
 	  If you say yes here you get support of a generic gpio RFKILL
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH v2 10/11] w1: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:07 [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB Geert Uytterhoeven
                   ` (9 preceding siblings ...)
  2015-08-02  9:09 ` [PATCH v2 11/11] net: rfkill: " Geert Uytterhoeven
@ 2015-08-02  9:09 ` Geert Uytterhoeven
  2015-08-02  9:25   ` Evgeniy Polyakov
  2015-08-10 16:01 ` [PATCH v2 00/11] gpio: " Johannes Berg
  11 siblings, 1 reply; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-02  9:09 UTC (permalink / raw)
  To: Evgeniy Polyakov
  Cc: Linus Walleij, Alexandre Courbot, linux-gpio, Geert Uytterhoeven

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.

Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
appropriate.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
---
v2:
  - Add Acked-by.
---
 drivers/w1/masters/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/w1/masters/Kconfig b/drivers/w1/masters/Kconfig
index 1708b2300c7a9ca2..00827d2897b534bc 100644
--- a/drivers/w1/masters/Kconfig
+++ b/drivers/w1/masters/Kconfig
@@ -49,7 +49,7 @@ config W1_MASTER_DS1WM
 
 config W1_MASTER_GPIO
 	tristate "GPIO 1-wire busmaster"
-	depends on GPIOLIB
+	depends on GPIOLIB || COMPILE_TEST
 	help
 	  Say Y here if you want to communicate with your 1-wire devices using
 	  GPIO pins. This driver uses the GPIO API to control the wire.
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* Re: [PATCH v2 10/11] w1: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:09 ` [PATCH v2 10/11] w1: " Geert Uytterhoeven
@ 2015-08-02  9:25   ` Evgeniy Polyakov
  2015-08-02 12:11     ` Geert Uytterhoeven
  0 siblings, 1 reply; 23+ messages in thread
From: Evgeniy Polyakov @ 2015-08-02  9:25 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Linus Walleij, Alexandre Courbot, linux-gpio@vger.kernel.org

Hi Geert

02.08.2015, 12:09, "Geert Uytterhoeven" <geert@linux-m68k.org>:
> 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.
>
> Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
> appropriate.

> - depends on GPIOLIB
> + depends on GPIOLIB || COMPILE_TEST

Does that mean GPIO subsystem is always available and no additional dependency needed
when COMPILE_TEST is enabled?

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [PATCH v2 10/11] w1: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:25   ` Evgeniy Polyakov
@ 2015-08-02 12:11     ` Geert Uytterhoeven
  2015-08-02 13:55       ` Evgeniy Polyakov
  0 siblings, 1 reply; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-02 12:11 UTC (permalink / raw)
  To: Evgeniy Polyakov
  Cc: Linus Walleij, Alexandre Courbot, linux-gpio@vger.kernel.org

Hi Evgeniy,

On Sun, Aug 2, 2015 at 11:25 AM, Evgeniy Polyakov <zbr@ioremap.net> wrote:
> 02.08.2015, 12:09, "Geert Uytterhoeven" <geert@linux-m68k.org>:
>> 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.
>>
>> Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
>> appropriate.
>
>> - depends on GPIOLIB
>> + depends on GPIOLIB || COMPILE_TEST
>
> Does that mean GPIO subsystem is always available and no additional dependency needed
> when COMPILE_TEST is enabled?

The GPIO subsystem is not always available, but it provides dummy GPIO consumer
functions if GPIOLIB is not enabled.

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

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [PATCH v2 10/11] w1: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02 12:11     ` Geert Uytterhoeven
@ 2015-08-02 13:55       ` Evgeniy Polyakov
  0 siblings, 0 replies; 23+ messages in thread
From: Evgeniy Polyakov @ 2015-08-02 13:55 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Linus Walleij, Alexandre Courbot, linux-gpio@vger.kernel.org

Hi Geert

02.08.2015, 15:11, "Geert Uytterhoeven" <geert@linux-m68k.org>:
>>>  - depends on GPIOLIB
>>>  + depends on GPIOLIB || COMPILE_TEST
>>
>>  Does that mean GPIO subsystem is always available and no additional dependency needed
>>  when COMPILE_TEST is enabled?
>
> The GPIO subsystem is not always available, but it provides dummy GPIO consumer
> functions if GPIOLIB is not enabled.

Ok, I see.
Thanks for clarification

Acked-by: Evgeniy Polyakov <zbr@ioremap.net>
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [PATCH v2 02/11] input: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:09 ` [PATCH v2 02/11] input: " Geert Uytterhoeven
@ 2015-08-04 18:44   ` Dmitry Torokhov
  0 siblings, 0 replies; 23+ messages in thread
From: Dmitry Torokhov @ 2015-08-04 18:44 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: linux-input, Linus Walleij, Alexandre Courbot, linux-gpio

On Sun, Aug 02, 2015 at 11:09:48AM +0200, 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.
> 
> Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
> appropriate.
> 
> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> Acked-by: Linus Walleij <linus.walleij@linaro.org>

Applied, thank you.

> ---
> v2:
>   - Add Acked-by.
> ---
>  drivers/input/keyboard/Kconfig    |  4 ++--
>  drivers/input/misc/Kconfig        | 11 ++++++-----
>  drivers/input/mouse/Kconfig       |  2 +-
>  drivers/input/touchscreen/Kconfig |  8 ++++----
>  4 files changed, 13 insertions(+), 12 deletions(-)
> 
> diff --git a/drivers/input/keyboard/Kconfig b/drivers/input/keyboard/Kconfig
> index 82a8fb50afacb253..da53ae6a93346c90 100644
> --- a/drivers/input/keyboard/Kconfig
> +++ b/drivers/input/keyboard/Kconfig
> @@ -187,7 +187,7 @@ config KEYBOARD_EP93XX
>  
>  config KEYBOARD_GPIO
>  	tristate "GPIO Buttons"
> -	depends on GPIOLIB
> +	depends on GPIOLIB || COMPILE_TEST
>  	help
>  	  This driver implements support for buttons connected
>  	  to GPIO pins of various CPUs (and some other chips).
> @@ -253,7 +253,7 @@ config KEYBOARD_TCA8418
>  
>  config KEYBOARD_MATRIX
>  	tristate "GPIO driven matrix keypad support"
> -	depends on GPIOLIB
> +	depends on GPIOLIB || COMPILE_TEST
>  	select INPUT_MATRIXKMAP
>  	help
>  	  Enable support for GPIO driven matrix keypad.
> diff --git a/drivers/input/misc/Kconfig b/drivers/input/misc/Kconfig
> index d4f0a817e858262b..dade381c7da80eb9 100644
> --- a/drivers/input/misc/Kconfig
> +++ b/drivers/input/misc/Kconfig
> @@ -259,7 +259,7 @@ config INPUT_APANEL
>  config INPUT_GP2A
>  	tristate "Sharp GP2AP002A00F I2C Proximity/Opto sensor driver"
>  	depends on I2C
> -	depends on GPIOLIB
> +	depends on GPIOLIB || COMPILE_TEST
>  	help
>  	  Say Y here if you have a Sharp GP2AP002A00F proximity/als combo-chip
>  	  hooked to an I2C bus.
> @@ -269,7 +269,7 @@ config INPUT_GP2A
>  
>  config INPUT_GPIO_BEEPER
>  	tristate "Generic GPIO Beeper support"
> -	depends on GPIOLIB
> +	depends on GPIOLIB || COMPILE_TEST
>  	help
>  	  Say Y here if you have a beeper connected to a GPIO pin.
>  
> @@ -278,7 +278,7 @@ config INPUT_GPIO_BEEPER
>  
>  config INPUT_GPIO_TILT_POLLED
>  	tristate "Polled GPIO tilt switch"
> -	depends on GPIOLIB
> +	depends on GPIOLIB || COMPILE_TEST
>  	select INPUT_POLLDEV
>  	help
>  	  This driver implements support for tilt switches connected
> @@ -569,7 +569,7 @@ config INPUT_PWM_BEEPER
>  
>  config INPUT_GPIO_ROTARY_ENCODER
>  	tristate "Rotary encoders connected to GPIO pins"
> -	depends on GPIOLIB
> +	depends on GPIOLIB || COMPILE_TEST
>  	help
>  	  Say Y here to add support for rotary encoders connected to GPIO lines.
>  	  Check file:Documentation/input/rotary-encoder.txt for more
> @@ -776,7 +776,8 @@ config INPUT_SOC_BUTTON_ARRAY
>  
>  config INPUT_DRV260X_HAPTICS
>  	tristate "TI DRV260X haptics support"
> -	depends on INPUT && I2C && GPIOLIB
> +	depends on INPUT && I2C
> +	depends on GPIOLIB || COMPILE_TEST
>  	select INPUT_FF_MEMLESS
>  	select REGMAP_I2C
>  	help
> diff --git a/drivers/input/mouse/Kconfig b/drivers/input/mouse/Kconfig
> index d7820d1152d2ef2d..17f97e5e11e7cd2e 100644
> --- a/drivers/input/mouse/Kconfig
> +++ b/drivers/input/mouse/Kconfig
> @@ -341,7 +341,7 @@ config MOUSE_VSXXXAA
>  
>  config MOUSE_GPIO
>  	tristate "GPIO mouse"
> -	depends on GPIOLIB
> +	depends on GPIOLIB || COMPILE_TEST
>  	select INPUT_POLLDEV
>  	help
>  	  This driver simulates a mouse on GPIO lines of various CPUs (and some
> diff --git a/drivers/input/touchscreen/Kconfig b/drivers/input/touchscreen/Kconfig
> index 860d426c480e2088..059edeb7f04a0d40 100644
> --- a/drivers/input/touchscreen/Kconfig
> +++ b/drivers/input/touchscreen/Kconfig
> @@ -118,7 +118,7 @@ config TOUCHSCREEN_ATMEL_MXT
>  config TOUCHSCREEN_AUO_PIXCIR
>  	tristate "AUO in-cell touchscreen using Pixcir ICs"
>  	depends on I2C
> -	depends on GPIOLIB
> +	depends on GPIOLIB || COMPILE_TEST
>  	help
>  	  Say Y here if you have a AUO display with in-cell touchscreen
>  	  using Pixcir ICs.
> @@ -142,7 +142,7 @@ config TOUCHSCREEN_BU21013
>  
>  config TOUCHSCREEN_CHIPONE_ICN8318
>  	tristate "chipone icn8318 touchscreen controller"
> -	depends on GPIOLIB
> +	depends on GPIOLIB || COMPILE_TEST
>  	depends on I2C
>  	depends on OF
>  	help
> @@ -156,7 +156,7 @@ config TOUCHSCREEN_CHIPONE_ICN8318
>  config TOUCHSCREEN_CY8CTMG110
>  	tristate "cy8ctmg110 touchscreen"
>  	depends on I2C
> -	depends on GPIOLIB
> +	depends on GPIOLIB || COMPILE_TEST
>  	help
>  	  Say Y here if you have a cy8ctmg110 capacitive touchscreen on
>  	  an AAVA device.
> @@ -1030,7 +1030,7 @@ config TOUCHSCREEN_TPS6507X
>  config TOUCHSCREEN_ZFORCE
>  	tristate "Neonode zForce infrared touchscreens"
>  	depends on I2C
> -	depends on GPIOLIB
> +	depends on GPIOLIB || COMPILE_TEST
>  	help
>  	  Say Y here if you have a touchscreen using the zforce
>  	  infraread technology from Neonode.
> -- 
> 1.9.1
> 

-- 
Dmitry

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [PATCH v2 05/11] power: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:09 ` [PATCH v2 05/11] power: " Geert Uytterhoeven
@ 2015-08-05  3:49   ` Sebastian Reichel
  0 siblings, 0 replies; 23+ messages in thread
From: Sebastian Reichel @ 2015-08-05  3:49 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Dmitry Eremin-Solenikov, David Woodhouse, linux-pm, Linus Walleij,
	Alexandre Courbot, linux-gpio

[-- Attachment #1: Type: text/plain, Size: 423 bytes --]

Hi,

On Sun, Aug 02, 2015 at 11:09:50AM +0200, 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.
> 
> Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
> appropriate.

Thanks, queued.

-- Sebastian

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:07 [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB Geert Uytterhoeven
                   ` (10 preceding siblings ...)
  2015-08-02  9:09 ` [PATCH v2 10/11] w1: " Geert Uytterhoeven
@ 2015-08-10 16:01 ` Johannes Berg
  2015-08-10 16:23   ` Geert Uytterhoeven
  11 siblings, 1 reply; 23+ messages in thread
From: Johannes Berg @ 2015-08-10 16:01 UTC (permalink / raw)
  To: Geert Uytterhoeven, Arnd Bergmann, David S. Miller,
	David Woodhouse, Dmitry Eremin-Solenikov, Dmitry Torokhov,
	Evgeniy Polyakov, Florian Fainelli, Greg Kroah-Hartman,
	Jean-Christophe Plagniol-Villard, Marcel Selhorst,
	Noralf Trønnes, Peter Huewe, Sebastian Reichel,
	Thomas Petazzoni, Tomi Valkeinen
  Cc: Linus Walleij, Alexandre Courbot, linux-gpio, linux-kernel

On Sun, 2015-08-02 at 11:07 +0200, Geert Uytterhoeven wrote:
> 
> Thanks for applying!
> 

Did you expect the series to get applied somewhere (GPIO tree, if
there's such a thing?), or the single patches to go through the
relevant other trees?

Fine with me either way, but it'd be good to know so we don't have the
patches twice :)

johannes

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-10 16:01 ` [PATCH v2 00/11] gpio: " Johannes Berg
@ 2015-08-10 16:23   ` Geert Uytterhoeven
  2015-08-10 16:29     ` Peter Huewe
  0 siblings, 1 reply; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-10 16:23 UTC (permalink / raw)
  To: Johannes Berg
  Cc: Arnd Bergmann, David S. Miller, David Woodhouse,
	Dmitry Eremin-Solenikov, Dmitry Torokhov, Evgeniy Polyakov,
	Florian Fainelli, Greg Kroah-Hartman,
	Jean-Christophe Plagniol-Villard, Marcel Selhorst,
	Noralf Trønnes, Peter Huewe, Sebastian Reichel,
	Thomas Petazzoni, Tomi Valkeinen, Linus Walleij,
	Alexandre Courbot, linux-gpio@vger.kernel.org,
	linux-kernel@vger.kernel.org

Hi Johannes,

On Mon, Aug 10, 2015 at 6:01 PM, Johannes Berg
<johannes@sipsolutions.net> wrote:
> On Sun, 2015-08-02 at 11:07 +0200, Geert Uytterhoeven wrote:
>> Thanks for applying!
>
> Did you expect the series to get applied somewhere (GPIO tree, if
> there's such a thing?), or the single patches to go through the
> relevant other trees?
>
> Fine with me either way, but it'd be good to know so we don't have the
> patches twice :)

I'd like the individual subsystem maintainers to apply the respective
patches. Sorry for being unclear.

Thanks!

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

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-10 16:23   ` Geert Uytterhoeven
@ 2015-08-10 16:29     ` Peter Huewe
  0 siblings, 0 replies; 23+ messages in thread
From: Peter Huewe @ 2015-08-10 16:29 UTC (permalink / raw)
  To: Geert Uytterhoeven, Johannes Berg
  Cc: Arnd Bergmann, David S. Miller, David Woodhouse,
	Dmitry Eremin-Solenikov, Dmitry Torokhov, Evgeniy Polyakov,
	Florian Fainelli, Greg Kroah-Hartman,
	Jean-Christophe Plagniol-Villard, Marcel Selhorst,
	Noralf Trønnes, Sebastian Reichel, Thomas Petazzoni,
	Tomi Valkeinen, Linus Walleij, Alexandre Courbot,
	linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org



Am 10. August 2015 18:23:52 MESZ, schrieb Geert Uytterhoeven <geert@linux-m68k.org>:
>
>I'd like the individual subsystem maintainers to apply the respective
>patches. Sorry for being unclear.

Okay, thanks for the clarification. 
I'll apply the TPM patch.

Peter
-- 
Sent from my mobile

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [PATCH v2 09/11] fbdev: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:09 ` [PATCH v2 09/11] fbdev: " Geert Uytterhoeven
@ 2015-08-11  7:22   ` Tomi Valkeinen
  2015-08-11  7:28     ` Geert Uytterhoeven
  0 siblings, 1 reply; 23+ messages in thread
From: Tomi Valkeinen @ 2015-08-11  7:22 UTC (permalink / raw)
  To: Geert Uytterhoeven, Jean-Christophe Plagniol-Villard
  Cc: linux-fbdev, Linus Walleij, Alexandre Courbot, linux-gpio

[-- Attachment #1: Type: text/plain, Size: 1430 bytes --]



On 02/08/15 12:09, 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.
> 
> Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
> appropriate.
> 
> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> Acked-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> v2:
>   - Add Acked-by.
> ---
>  drivers/video/fbdev/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/video/fbdev/Kconfig b/drivers/video/fbdev/Kconfig
> index 2d98de535e0f7374..569803f533d1a600 100644
> --- a/drivers/video/fbdev/Kconfig
> +++ b/drivers/video/fbdev/Kconfig
> @@ -2464,7 +2464,7 @@ config FB_SSD1307
>  	tristate "Solomon SSD1307 framebuffer support"
>  	depends on FB && I2C
>  	depends on OF
> -	depends on GPIOLIB
> +	depends on GPIOLIB || COMPILE_TEST
>  	select FB_SYS_FOPS
>  	select FB_SYS_FILLRECT
>  	select FB_SYS_COPYAREA
> 

Applied to 4.3 fbdev.

I do wonder why this is needed, though... I understand COMPILE_TEST if
the dependency in question is difficult/impossible to enable with your
kernel config (say, enabling OMAP SoC support with x86 config), but
isn't GPIOLIB something that can be trivially enabled in any kernel config?

 Tomi


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [PATCH v2 09/11] fbdev: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-11  7:22   ` Tomi Valkeinen
@ 2015-08-11  7:28     ` Geert Uytterhoeven
  0 siblings, 0 replies; 23+ messages in thread
From: Geert Uytterhoeven @ 2015-08-11  7:28 UTC (permalink / raw)
  To: Tomi Valkeinen
  Cc: Jean-Christophe Plagniol-Villard, Linux Fbdev development list,
	Linus Walleij, Alexandre Courbot, linux-gpio@vger.kernel.org

Hi Tomi,

On Tue, Aug 11, 2015 at 9:22 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote:
> On 02/08/15 12:09, 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.
>>
>> Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
>> appropriate.
>>
>> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
>> Acked-by: Linus Walleij <linus.walleij@linaro.org>

> Applied to 4.3 fbdev.

Thanks!

> I do wonder why this is needed, though... I understand COMPILE_TEST if
> the dependency in question is difficult/impossible to enable with your
> kernel config (say, enabling OMAP SoC support with x86 config), but
> isn't GPIOLIB something that can be trivially enabled in any kernel config?

While there are a few mfd and pinctrl drivers that select GPIOLIB, the
presence of GPIOLIB is mostly a platform feature. Not all architectures support
GPIOLIB yet.

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

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [PATCH v2 11/11] net: rfkill: Allow compile test of GPIO consumers if !GPIOLIB
  2015-08-02  9:09 ` [PATCH v2 11/11] net: rfkill: " Geert Uytterhoeven
@ 2015-08-13  8:49   ` Johannes Berg
  0 siblings, 0 replies; 23+ messages in thread
From: Johannes Berg @ 2015-08-13  8:49 UTC (permalink / raw)
  To: Geert Uytterhoeven, David S. Miller
  Cc: netdev, Linus Walleij, Alexandre Courbot, linux-gpio

On Sun, 2015-08-02 at 11:09 +0200, 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.
> 
> Relax the dependency on GPIOLIB if COMPILE_TEST is enabled, where
> appropriate.
> 
Applied.

johannes

^ permalink raw reply	[flat|nested] 23+ messages in thread

end of thread, other threads:[~2015-08-13  8:49 UTC | newest]

Thread overview: 23+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-02  9:07 [PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB Geert Uytterhoeven
2015-08-02  9:08 ` [PATCH v2 01/11] tpm: " Geert Uytterhoeven
2015-08-02  9:09 ` [PATCH v2 02/11] input: " Geert Uytterhoeven
2015-08-04 18:44   ` Dmitry Torokhov
2015-08-02  9:09 ` [PATCH v2 03/11] misc: ti-st: " Geert Uytterhoeven
2015-08-02  9:09 ` [PATCH v2 05/11] power: " Geert Uytterhoeven
2015-08-05  3:49   ` Sebastian Reichel
2015-08-02  9:09 ` [PATCH v2 04/11] net: phy: " Geert Uytterhoeven
2015-08-02  9:09 ` [PATCH v2 06/11] staging: android: " Geert Uytterhoeven
2015-08-02  9:09 ` [PATCH v2 08/11] serial: " Geert Uytterhoeven
2015-08-02  9:09 ` [PATCH v2 07/11] staging: fbtft: " Geert Uytterhoeven
2015-08-02  9:09 ` [PATCH v2 09/11] fbdev: " Geert Uytterhoeven
2015-08-11  7:22   ` Tomi Valkeinen
2015-08-11  7:28     ` Geert Uytterhoeven
2015-08-02  9:09 ` [PATCH v2 11/11] net: rfkill: " Geert Uytterhoeven
2015-08-13  8:49   ` Johannes Berg
2015-08-02  9:09 ` [PATCH v2 10/11] w1: " Geert Uytterhoeven
2015-08-02  9:25   ` Evgeniy Polyakov
2015-08-02 12:11     ` Geert Uytterhoeven
2015-08-02 13:55       ` Evgeniy Polyakov
2015-08-10 16:01 ` [PATCH v2 00/11] gpio: " Johannes Berg
2015-08-10 16:23   ` Geert Uytterhoeven
2015-08-10 16:29     ` Peter Huewe

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).