linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] pinctrl: sprd: Restrict PINCTRL_SPRD to ARCH_SPRD or COMPILE_TEST
@ 2017-09-06 16:08 Geert Uytterhoeven
  2017-09-06 16:18 ` Randy Dunlap
  2017-09-12  9:21 ` Linus Walleij
  0 siblings, 2 replies; 3+ messages in thread
From: Geert Uytterhoeven @ 2017-09-06 16:08 UTC (permalink / raw)
  To: Linus Walleij, Baolin Wang
  Cc: Arnd Bergmann, linux-gpio, linux-kernel, Geert Uytterhoeven

The Spreadtrum pinctrl drivers are only useful when building for a
Spreadtrum platform.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
On top of Arnd's:
  - pinctrl: sprd: add missing Kconfig dependency
  - pinctrl: sprd: add CONFIG_OF dependency

 drivers/pinctrl/sprd/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/pinctrl/sprd/Kconfig b/drivers/pinctrl/sprd/Kconfig
index e75cde2e1b460d66..bc7f3fab22f1c431 100644
--- a/drivers/pinctrl/sprd/Kconfig
+++ b/drivers/pinctrl/sprd/Kconfig
@@ -5,6 +5,7 @@
 config PINCTRL_SPRD
 	bool "Spreadtrum pinctrl driver"
 	depends on OF
+	depends on ARCH_SPRD || COMPILE_TEST
 	select PINMUX
 	select PINCONF
 	select GENERIC_PINCONF
-- 
2.7.4

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

* Re: [PATCH] pinctrl: sprd: Restrict PINCTRL_SPRD to ARCH_SPRD or COMPILE_TEST
  2017-09-06 16:08 [PATCH] pinctrl: sprd: Restrict PINCTRL_SPRD to ARCH_SPRD or COMPILE_TEST Geert Uytterhoeven
@ 2017-09-06 16:18 ` Randy Dunlap
  2017-09-12  9:21 ` Linus Walleij
  1 sibling, 0 replies; 3+ messages in thread
From: Randy Dunlap @ 2017-09-06 16:18 UTC (permalink / raw)
  To: Geert Uytterhoeven, Linus Walleij, Baolin Wang
  Cc: Arnd Bergmann, linux-gpio, linux-kernel

On 09/06/17 09:08, Geert Uytterhoeven wrote:
> The Spreadtrum pinctrl drivers are only useful when building for a
> Spreadtrum platform.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> On top of Arnd's:
>   - pinctrl: sprd: add missing Kconfig dependency
>   - pinctrl: sprd: add CONFIG_OF dependency

or (probably with fuzz) on top of my previous patch:
https://lkml.org/lkml/2017/9/4/336

>  drivers/pinctrl/sprd/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/pinctrl/sprd/Kconfig b/drivers/pinctrl/sprd/Kconfig
> index e75cde2e1b460d66..bc7f3fab22f1c431 100644
> --- a/drivers/pinctrl/sprd/Kconfig
> +++ b/drivers/pinctrl/sprd/Kconfig
> @@ -5,6 +5,7 @@
>  config PINCTRL_SPRD
>  	bool "Spreadtrum pinctrl driver"
>  	depends on OF
> +	depends on ARCH_SPRD || COMPILE_TEST
>  	select PINMUX
>  	select PINCONF
>  	select GENERIC_PINCONF
> 


-- 
~Randy

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

* Re: [PATCH] pinctrl: sprd: Restrict PINCTRL_SPRD to ARCH_SPRD or COMPILE_TEST
  2017-09-06 16:08 [PATCH] pinctrl: sprd: Restrict PINCTRL_SPRD to ARCH_SPRD or COMPILE_TEST Geert Uytterhoeven
  2017-09-06 16:18 ` Randy Dunlap
@ 2017-09-12  9:21 ` Linus Walleij
  1 sibling, 0 replies; 3+ messages in thread
From: Linus Walleij @ 2017-09-12  9:21 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Baolin Wang, Arnd Bergmann, linux-gpio, linux-kernel

On Wed, Sep 6, 2017 at 6:08 PM, Geert Uytterhoeven
<geert+renesas@glider.be> wrote:

> The Spreadtrum pinctrl drivers are only useful when building for a
> Spreadtrum platform.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> On top of Arnd's:
>   - pinctrl: sprd: add missing Kconfig dependency
>   - pinctrl: sprd: add CONFIG_OF dependency

Fuzzed on top of Randy's patch and applied.

Yours,
Linus Walleij

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

end of thread, other threads:[~2017-09-12  9:21 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-06 16:08 [PATCH] pinctrl: sprd: Restrict PINCTRL_SPRD to ARCH_SPRD or COMPILE_TEST Geert Uytterhoeven
2017-09-06 16:18 ` Randy Dunlap
2017-09-12  9:21 ` Linus Walleij

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).