diff mbox series

[3/4,v3] ARM: gemini: select MIGHT_HAVE_PCI

Message ID 20170227190720.31409-3-linus.walleij@linaro.org
State New
Headers show
Series [1/4,v3] PCI: add DT bindings for Faraday Technology PCI Host Bridge | expand

Commit Message

Linus Walleij Feb. 27, 2017, 7:07 p.m. UTC
As we have a PCI driver for the Gemini, we should select
MIGHT_HAVE_PCI.

Cc: Janos Laube <janos.dev@gmail.com>
Cc: Paulius Zaleckas <paulius.zaleckas@gmail.com>
Cc: Hans Ulli Kroll <ulli.kroll@googlemail.com>
Cc: Florian Fainelli <f.fainelli@gmail.com>
Cc: Feng-Hsin Chiang <john453@faraday-tech.com>
Cc: Greentime Hu <green.hu@gmail.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

---
ChangeLog v1->v2->v3:
- No changes

PCI maintainers: this is FYI only, I will funnel this to the ARM
SoC tree once we are done with the PCI driver.
---
 arch/arm/mach-gemini/Kconfig | 1 +
 1 file changed, 1 insertion(+)

-- 
2.9.3

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Rob Herring (Arm) March 2, 2017, 4:29 p.m. UTC | #1
On Mon, Feb 27, 2017 at 08:07:19PM +0100, Linus Walleij wrote:
> As we have a PCI driver for the Gemini, we should select

> MIGHT_HAVE_PCI.

> 

> Cc: Janos Laube <janos.dev@gmail.com>

> Cc: Paulius Zaleckas <paulius.zaleckas@gmail.com>

> Cc: Hans Ulli Kroll <ulli.kroll@googlemail.com>

> Cc: Florian Fainelli <f.fainelli@gmail.com>

> Cc: Feng-Hsin Chiang <john453@faraday-tech.com>

> Cc: Greentime Hu <green.hu@gmail.com>

> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

> ---

> ChangeLog v1->v2->v3:

> - No changes

> 

> PCI maintainers: this is FYI only, I will funnel this to the ARM

> SoC tree once we are done with the PCI driver.

> ---

>  arch/arm/mach-gemini/Kconfig | 1 +

>  1 file changed, 1 insertion(+)

> 

> diff --git a/arch/arm/mach-gemini/Kconfig b/arch/arm/mach-gemini/Kconfig

> index a5ee5fbab796..21f89a4c16fe 100644

> --- a/arch/arm/mach-gemini/Kconfig

> +++ b/arch/arm/mach-gemini/Kconfig

> @@ -6,6 +6,7 @@ menuconfig ARCH_GEMINI

>  	select FTTMR010_TIMER

>  	select GPIO_GEMINI

>  	select GPIOLIB

> +	select MIGHT_HAVE_PCI


Doesn't Multi-platform do this for you? If not, IMO, we should do that 
or kill this option. We don't do an option like this for I2C, SPI, 
display, anything else (well, except GPIOLIB ;)).

>  	select POWER_RESET

>  	select POWER_RESET_SYSCON

>  	select SERIAL_OF_PLATFORM

> -- 

> 2.9.3

> 

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Linus Walleij March 2, 2017, 11:31 p.m. UTC | #2
On Thu, Mar 2, 2017 at 5:29 PM, Rob Herring <robh@kernel.org> wrote:
> On Mon, Feb 27, 2017 at 08:07:19PM +0100, Linus Walleij wrote:

>> As we have a PCI driver for the Gemini, we should select

>> MIGHT_HAVE_PCI.

>>

>> Cc: Janos Laube <janos.dev@gmail.com>

>> Cc: Paulius Zaleckas <paulius.zaleckas@gmail.com>

>> Cc: Hans Ulli Kroll <ulli.kroll@googlemail.com>

>> Cc: Florian Fainelli <f.fainelli@gmail.com>

>> Cc: Feng-Hsin Chiang <john453@faraday-tech.com>

>> Cc: Greentime Hu <green.hu@gmail.com>

>> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

>> ---

>> ChangeLog v1->v2->v3:

>> - No changes

>>

>> PCI maintainers: this is FYI only, I will funnel this to the ARM

>> SoC tree once we are done with the PCI driver.

>> ---

>>  arch/arm/mach-gemini/Kconfig | 1 +

>>  1 file changed, 1 insertion(+)

>>

>> diff --git a/arch/arm/mach-gemini/Kconfig b/arch/arm/mach-gemini/Kconfig

>> index a5ee5fbab796..21f89a4c16fe 100644

>> --- a/arch/arm/mach-gemini/Kconfig

>> +++ b/arch/arm/mach-gemini/Kconfig

>> @@ -6,6 +6,7 @@ menuconfig ARCH_GEMINI

>>       select FTTMR010_TIMER

>>       select GPIO_GEMINI

>>       select GPIOLIB

>> +     select MIGHT_HAVE_PCI

>

> Doesn't Multi-platform do this for you?


Oh it does, I just missed that.

> If not, IMO, we should do that

> or kill this option. We don't do an option like this for I2C, SPI,

> display, anything else (well, except GPIOLIB ;)).


I killed ARCH_WANT_OPTIONAL_GPIOLIB at least
that thing was really confusing.

Maybe we should have MULTIPLATFORM select
GPIOLIB too.

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox series

Patch

diff --git a/arch/arm/mach-gemini/Kconfig b/arch/arm/mach-gemini/Kconfig
index a5ee5fbab796..21f89a4c16fe 100644
--- a/arch/arm/mach-gemini/Kconfig
+++ b/arch/arm/mach-gemini/Kconfig
@@ -6,6 +6,7 @@  menuconfig ARCH_GEMINI
 	select FTTMR010_TIMER
 	select GPIO_GEMINI
 	select GPIOLIB
+	select MIGHT_HAVE_PCI
 	select POWER_RESET
 	select POWER_RESET_SYSCON
 	select SERIAL_OF_PLATFORM