linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/3] ARM mach Kconfig clean-ups
@ 2014-04-22 19:41 Rob Herring
  2014-04-22 19:41 ` [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects Rob Herring
                   ` (2 more replies)
  0 siblings, 3 replies; 12+ messages in thread
From: Rob Herring @ 2014-04-22 19:41 UTC (permalink / raw)
  To: linux-arm-kernel

From: Rob Herring <robh@kernel.org>

This is a follow-up to the multi-platform kconfig clean-ups I did for 
3.15. This is fixing new things which got introduced in 3.15. For 
example, the BCM2835 changes appear to have been lost with its move to 
mach-bcm.

Rob

Rob Herring (3):
  ARM: mvebu: clean-up unneeded kconfig selects
  ARM: bcm: clean-up unneeded kconfig selects
  ARM: qcom: clean-up unneeded kconfig selects

 arch/arm/mach-bcm/Kconfig   | 8 --------
 arch/arm/mach-mvebu/Kconfig | 6 ------
 arch/arm/mach-qcom/Kconfig  | 2 --
 3 files changed, 16 deletions(-)

-- 
1.9.1

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

* [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects
  2014-04-22 19:41 [PATCH 0/3] ARM mach Kconfig clean-ups Rob Herring
@ 2014-04-22 19:41 ` Rob Herring
  2014-04-22 20:58   ` Andrew Lunn
                     ` (2 more replies)
  2014-04-22 19:41 ` [PATCH 2/3] ARM: bcm: " Rob Herring
  2014-04-22 19:41 ` [PATCH 3/3] ARM: qcom: " Rob Herring
  2 siblings, 3 replies; 12+ messages in thread
From: Rob Herring @ 2014-04-22 19:41 UTC (permalink / raw)
  To: linux-arm-kernel

From: Rob Herring <robh@kernel.org>

Multi-platform support implies all these options are already selected and
individual platforms don't need to select them.

Signed-off-by: Rob Herring <robh@kernel.org>
Cc: Jason Cooper <jason@lakedaemon.net>
Cc: Andrew Lunn <andrew@lunn.ch>
Cc: Gregory Clement <gregory.clement@free-electrons.com>
Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
---
 arch/arm/mach-mvebu/Kconfig | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/arch/arm/mach-mvebu/Kconfig b/arch/arm/mach-mvebu/Kconfig
index 3f73eec..bfc5af1 100644
--- a/arch/arm/mach-mvebu/Kconfig
+++ b/arch/arm/mach-mvebu/Kconfig
@@ -3,7 +3,6 @@ config ARCH_MVEBU
 	select ARCH_SUPPORTS_BIG_ENDIAN
 	select CLKSRC_MMIO
 	select GENERIC_IRQ_CHIP
-	select IRQ_DOMAIN
 	select PINCTRL
 	select PLAT_ORION
 	select MVEBU_MBUS
@@ -11,7 +10,6 @@ config ARCH_MVEBU
 	select ARCH_REQUIRE_GPIOLIB
 	select MIGHT_HAVE_PCI
 	select PCI_QUIRKS if PCI
-	select OF_ADDRESS_PCI
 
 if ARCH_MVEBU
 
@@ -38,7 +36,6 @@ config MACH_ARMADA_375
 	select ARM_ERRATA_753970
 	select ARM_GIC
 	select ARMADA_375_CLK
-	select CPU_V7
 	select MACH_MVEBU_V7
 	select PINCTRL_ARMADA_375
 	help
@@ -51,7 +48,6 @@ config MACH_ARMADA_38X
 	select ARM_ERRATA_753970
 	select ARM_GIC
 	select ARMADA_38X_CLK
-	select CPU_V7
 	select MACH_MVEBU_V7
 	select PINCTRL_ARMADA_38X
 	help
@@ -86,13 +82,11 @@ config MACH_KIRKWOOD
 	select ARCH_REQUIRE_GPIOLIB
 	select CPU_FEROCEON
 	select KIRKWOOD_CLK
-	select OF_IRQ
 	select ORION_IRQCHIP
 	select ORION_TIMER
 	select PCI
 	select PCI_QUIRKS
 	select PINCTRL_KIRKWOOD
-	select USE_OF
 	help
 	  Say 'Y' here if you want your kernel to support boards based
 	  on the Marvell Kirkwood device tree.
-- 
1.9.1

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

* [PATCH 2/3] ARM: bcm: clean-up unneeded kconfig selects
  2014-04-22 19:41 [PATCH 0/3] ARM mach Kconfig clean-ups Rob Herring
  2014-04-22 19:41 ` [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects Rob Herring
@ 2014-04-22 19:41 ` Rob Herring
  2014-04-22 19:41 ` [PATCH 3/3] ARM: qcom: " Rob Herring
  2 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2014-04-22 19:41 UTC (permalink / raw)
  To: linux-arm-kernel

From: Rob Herring <robh@kernel.org>

Multi-platform support implies all these options are already selected and
individual platforms don't need to select them.

Signed-off-by: Rob Herring <robh@kernel.org>
Cc: Christian Daudt <bcm@fixthebug.org>
Cc: Matt Porter <mporter@linaro.org>
Cc: bcm-kernel-feedback-list at broadcom.com
---
 arch/arm/mach-bcm/Kconfig | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig
index 49c914c..9f19636 100644
--- a/arch/arm/mach-bcm/Kconfig
+++ b/arch/arm/mach-bcm/Kconfig
@@ -11,7 +11,6 @@ menu "Broadcom SoC Selection"
 
 config ARCH_BCM_MOBILE
 	bool "Broadcom Mobile SoC" if ARCH_MULTI_V7
-	depends on MMU
 	select ARCH_REQUIRE_GPIOLIB
 	select ARM_ERRATA_754322
 	select ARM_ERRATA_764369 if SMP
@@ -33,10 +32,7 @@ config ARCH_BCM2835
 	select ARM_AMBA
 	select ARM_ERRATA_411920
 	select ARM_TIMER_SP804
-	select CLKDEV_LOOKUP
 	select CLKSRC_OF
-	select CPU_V6
-	select GENERIC_CLOCKEVENTS
 	select PINCTRL
 	select PINCTRL_BCM2835
 	help
@@ -45,14 +41,10 @@ config ARCH_BCM2835
 
 config ARCH_BCM_5301X
 	bool "Broadcom BCM470X / BCM5301X ARM SoC" if ARCH_MULTI_V7
-	depends on MMU
 	select ARM_GIC
 	select CACHE_L2X0
 	select HAVE_ARM_SCU if SMP
 	select HAVE_ARM_TWD if SMP
-	select HAVE_SMP
-	select COMMON_CLK
-	select GENERIC_CLOCKEVENTS
 	select ARM_GLOBAL_TIMER
 	select CLKSRC_ARM_GLOBAL_TIMER_SCHED_CLOCK
 	select MIGHT_HAVE_PCI
-- 
1.9.1

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

* [PATCH 3/3] ARM: qcom: clean-up unneeded kconfig selects
  2014-04-22 19:41 [PATCH 0/3] ARM mach Kconfig clean-ups Rob Herring
  2014-04-22 19:41 ` [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects Rob Herring
  2014-04-22 19:41 ` [PATCH 2/3] ARM: bcm: " Rob Herring
@ 2014-04-22 19:41 ` Rob Herring
  2 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2014-04-22 19:41 UTC (permalink / raw)
  To: linux-arm-kernel

From: Rob Herring <robh@kernel.org>

Multi-platform support implies all these options are already selected and
individual platforms don't need to select them.

Signed-off-by: Rob Herring <robh@kernel.org>
Cc: Kumar Gala <galak@codeaurora.org>
Cc: David Brown <davidb@codeaurora.org>
Cc: linux-arm-msm at vger.kernel.org
---
 arch/arm/mach-qcom/Kconfig | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm/mach-qcom/Kconfig b/arch/arm/mach-qcom/Kconfig
index a028be2..6aa2214 100644
--- a/arch/arm/mach-qcom/Kconfig
+++ b/arch/arm/mach-qcom/Kconfig
@@ -3,8 +3,6 @@ config ARCH_QCOM
 	select ARCH_REQUIRE_GPIOLIB
 	select ARM_GIC
 	select CLKSRC_OF
-	select GENERIC_CLOCKEVENTS
-	select HAVE_SMP
 	select QCOM_SCM if SMP
 	help
 	  Support for Qualcomm's devicetree based systems.
-- 
1.9.1

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

* [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects
  2014-04-22 19:41 ` [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects Rob Herring
@ 2014-04-22 20:58   ` Andrew Lunn
  2014-04-22 21:23     ` Rob Herring
  2014-05-05  4:55   ` Olof Johansson
  2014-06-09 18:43   ` Ezequiel Garcia
  2 siblings, 1 reply; 12+ messages in thread
From: Andrew Lunn @ 2014-04-22 20:58 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Apr 22, 2014 at 02:41:16PM -0500, Rob Herring wrote:
> From: Rob Herring <robh@kernel.org>
> 
> Multi-platform support implies all these options are already selected and
> individual platforms don't need to select them.

Hi Rob

Does this break mvebu_v7_defconfig?

     Andrew

> 
> Signed-off-by: Rob Herring <robh@kernel.org>
> Cc: Jason Cooper <jason@lakedaemon.net>
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Gregory Clement <gregory.clement@free-electrons.com>
> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
> ---
>  arch/arm/mach-mvebu/Kconfig | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/arch/arm/mach-mvebu/Kconfig b/arch/arm/mach-mvebu/Kconfig
> index 3f73eec..bfc5af1 100644
> --- a/arch/arm/mach-mvebu/Kconfig
> +++ b/arch/arm/mach-mvebu/Kconfig
> @@ -3,7 +3,6 @@ config ARCH_MVEBU
>  	select ARCH_SUPPORTS_BIG_ENDIAN
>  	select CLKSRC_MMIO
>  	select GENERIC_IRQ_CHIP
> -	select IRQ_DOMAIN
>  	select PINCTRL
>  	select PLAT_ORION
>  	select MVEBU_MBUS
> @@ -11,7 +10,6 @@ config ARCH_MVEBU
>  	select ARCH_REQUIRE_GPIOLIB
>  	select MIGHT_HAVE_PCI
>  	select PCI_QUIRKS if PCI
> -	select OF_ADDRESS_PCI
>  
>  if ARCH_MVEBU
>  
> @@ -38,7 +36,6 @@ config MACH_ARMADA_375
>  	select ARM_ERRATA_753970
>  	select ARM_GIC
>  	select ARMADA_375_CLK
> -	select CPU_V7
>  	select MACH_MVEBU_V7
>  	select PINCTRL_ARMADA_375
>  	help
> @@ -51,7 +48,6 @@ config MACH_ARMADA_38X
>  	select ARM_ERRATA_753970
>  	select ARM_GIC
>  	select ARMADA_38X_CLK
> -	select CPU_V7
>  	select MACH_MVEBU_V7
>  	select PINCTRL_ARMADA_38X
>  	help
> @@ -86,13 +82,11 @@ config MACH_KIRKWOOD
>  	select ARCH_REQUIRE_GPIOLIB
>  	select CPU_FEROCEON
>  	select KIRKWOOD_CLK
> -	select OF_IRQ
>  	select ORION_IRQCHIP
>  	select ORION_TIMER
>  	select PCI
>  	select PCI_QUIRKS
>  	select PINCTRL_KIRKWOOD
> -	select USE_OF
>  	help
>  	  Say 'Y' here if you want your kernel to support boards based
>  	  on the Marvell Kirkwood device tree.
> -- 
> 1.9.1
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects
  2014-04-22 20:58   ` Andrew Lunn
@ 2014-04-22 21:23     ` Rob Herring
  0 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2014-04-22 21:23 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Apr 22, 2014 at 3:58 PM, Andrew Lunn <andrew@lunn.ch> wrote:
> On Tue, Apr 22, 2014 at 02:41:16PM -0500, Rob Herring wrote:
>> From: Rob Herring <robh@kernel.org>
>>
>> Multi-platform support implies all these options are already selected and
>> individual platforms don't need to select them.
>
> Hi Rob
>
> Does this break mvebu_v7_defconfig?

No, it should all be redundant selects.

Rob

>
>      Andrew
>
>>
>> Signed-off-by: Rob Herring <robh@kernel.org>
>> Cc: Jason Cooper <jason@lakedaemon.net>
>> Cc: Andrew Lunn <andrew@lunn.ch>
>> Cc: Gregory Clement <gregory.clement@free-electrons.com>
>> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
>> ---
>>  arch/arm/mach-mvebu/Kconfig | 6 ------
>>  1 file changed, 6 deletions(-)
>>
>> diff --git a/arch/arm/mach-mvebu/Kconfig b/arch/arm/mach-mvebu/Kconfig
>> index 3f73eec..bfc5af1 100644
>> --- a/arch/arm/mach-mvebu/Kconfig
>> +++ b/arch/arm/mach-mvebu/Kconfig
>> @@ -3,7 +3,6 @@ config ARCH_MVEBU
>>       select ARCH_SUPPORTS_BIG_ENDIAN
>>       select CLKSRC_MMIO
>>       select GENERIC_IRQ_CHIP
>> -     select IRQ_DOMAIN
>>       select PINCTRL
>>       select PLAT_ORION
>>       select MVEBU_MBUS
>> @@ -11,7 +10,6 @@ config ARCH_MVEBU
>>       select ARCH_REQUIRE_GPIOLIB
>>       select MIGHT_HAVE_PCI
>>       select PCI_QUIRKS if PCI
>> -     select OF_ADDRESS_PCI
>>
>>  if ARCH_MVEBU
>>
>> @@ -38,7 +36,6 @@ config MACH_ARMADA_375
>>       select ARM_ERRATA_753970
>>       select ARM_GIC
>>       select ARMADA_375_CLK
>> -     select CPU_V7
>>       select MACH_MVEBU_V7
>>       select PINCTRL_ARMADA_375
>>       help
>> @@ -51,7 +48,6 @@ config MACH_ARMADA_38X
>>       select ARM_ERRATA_753970
>>       select ARM_GIC
>>       select ARMADA_38X_CLK
>> -     select CPU_V7
>>       select MACH_MVEBU_V7
>>       select PINCTRL_ARMADA_38X
>>       help
>> @@ -86,13 +82,11 @@ config MACH_KIRKWOOD
>>       select ARCH_REQUIRE_GPIOLIB
>>       select CPU_FEROCEON
>>       select KIRKWOOD_CLK
>> -     select OF_IRQ
>>       select ORION_IRQCHIP
>>       select ORION_TIMER
>>       select PCI
>>       select PCI_QUIRKS
>>       select PINCTRL_KIRKWOOD
>> -     select USE_OF
>>       help
>>         Say 'Y' here if you want your kernel to support boards based
>>         on the Marvell Kirkwood device tree.
>> --
>> 1.9.1
>>
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel at lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects
  2014-04-22 19:41 ` [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects Rob Herring
  2014-04-22 20:58   ` Andrew Lunn
@ 2014-05-05  4:55   ` Olof Johansson
  2014-05-05 14:42     ` Rob Herring
  2014-06-09 18:43   ` Ezequiel Garcia
  2 siblings, 1 reply; 12+ messages in thread
From: Olof Johansson @ 2014-05-05  4:55 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Apr 22, 2014 at 02:41:16PM -0500, Rob Herring wrote:
> From: Rob Herring <robh@kernel.org>
> 
> Multi-platform support implies all these options are already selected and
> individual platforms don't need to select them.
> 
> Signed-off-by: Rob Herring <robh@kernel.org>
> Cc: Jason Cooper <jason@lakedaemon.net>
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Gregory Clement <gregory.clement@free-electrons.com>
> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>

What's this based on? It doesn't apply to 3.15-rc4.


-Olof

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

* [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects
  2014-05-05  4:55   ` Olof Johansson
@ 2014-05-05 14:42     ` Rob Herring
  2014-05-05 21:22       ` Olof Johansson
  0 siblings, 1 reply; 12+ messages in thread
From: Rob Herring @ 2014-05-05 14:42 UTC (permalink / raw)
  To: linux-arm-kernel

On Sun, May 4, 2014 at 11:55 PM, Olof Johansson <olof@lixom.net> wrote:
> On Tue, Apr 22, 2014 at 02:41:16PM -0500, Rob Herring wrote:
>> From: Rob Herring <robh@kernel.org>
>>
>> Multi-platform support implies all these options are already selected and
>> individual platforms don't need to select them.
>>
>> Signed-off-by: Rob Herring <robh@kernel.org>
>> Cc: Jason Cooper <jason@lakedaemon.net>
>> Cc: Andrew Lunn <andrew@lunn.ch>
>> Cc: Gregory Clement <gregory.clement@free-electrons.com>
>> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
>
> What's this based on? It doesn't apply to 3.15-rc4.

v3.15-rc2. That's strange because there are no other changes since
rc2. Here's a pull request instead.

Rob

The following changes since commit a798c10faf62a505d24e5f6213fbaf904a39623f:

  Linux 3.15-rc2 (2014-04-20 11:08:50 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git
tags/kconfig-cleanups

for you to fetch changes up to e7062b101537a0c4f4a8d725321772eb78100074:

  ARM: qcom: clean-up unneeded kconfig selects (2014-04-22 14:33:27 -0500)

----------------------------------------------------------------
Several mach kconfig clean-ups of redundant selects

----------------------------------------------------------------
Rob Herring (3):
      ARM: mvebu: clean-up unneeded kconfig selects
      ARM: bcm: clean-up unneeded kconfig selects
      ARM: qcom: clean-up unneeded kconfig selects

 arch/arm/mach-bcm/Kconfig   | 8 --------
 arch/arm/mach-mvebu/Kconfig | 6 ------
 arch/arm/mach-qcom/Kconfig  | 2 --
 3 files changed, 16 deletions(-)

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

* [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects
  2014-05-05 14:42     ` Rob Herring
@ 2014-05-05 21:22       ` Olof Johansson
  0 siblings, 0 replies; 12+ messages in thread
From: Olof Johansson @ 2014-05-05 21:22 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, May 05, 2014 at 09:42:43AM -0500, Rob Herring wrote:
> On Sun, May 4, 2014 at 11:55 PM, Olof Johansson <olof@lixom.net> wrote:
> > On Tue, Apr 22, 2014 at 02:41:16PM -0500, Rob Herring wrote:
> >> From: Rob Herring <robh@kernel.org>
> >>
> >> Multi-platform support implies all these options are already selected and
> >> individual platforms don't need to select them.
> >>
> >> Signed-off-by: Rob Herring <robh@kernel.org>
> >> Cc: Jason Cooper <jason@lakedaemon.net>
> >> Cc: Andrew Lunn <andrew@lunn.ch>
> >> Cc: Gregory Clement <gregory.clement@free-electrons.com>
> >> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
> >
> > What's this based on? It doesn't apply to 3.15-rc4.
> 
> v3.15-rc2. That's strange because there are no other changes since
> rc2. Here's a pull request instead.

Hm, I wonder if I had typoed v3.15 vs v3.14 on the base branch. Anyway:

> 
> Rob
> 
> The following changes since commit a798c10faf62a505d24e5f6213fbaf904a39623f:
> 
>   Linux 3.15-rc2 (2014-04-20 11:08:50 -0700)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git
> tags/kconfig-cleanups
> 

Merged. Thanks!


-Olof

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

* [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects
  2014-04-22 19:41 ` [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects Rob Herring
  2014-04-22 20:58   ` Andrew Lunn
  2014-05-05  4:55   ` Olof Johansson
@ 2014-06-09 18:43   ` Ezequiel Garcia
  2014-06-09 19:00     ` Rob Herring
  2 siblings, 1 reply; 12+ messages in thread
From: Ezequiel Garcia @ 2014-06-09 18:43 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Rob,

On 22 Apr 02:41 PM, Rob Herring wrote:
> @@ -11,7 +10,6 @@ config ARCH_MVEBU
>  	select ARCH_REQUIRE_GPIOLIB
>  	select MIGHT_HAVE_PCI
>  	select PCI_QUIRKS if PCI
> -	select OF_ADDRESS_PCI
>  

We needed that one, as Gregory introduced it specifically so we could select it
without selecting PCI.

I'll cook a patch right away, sorry for not seeing this earlier!
-- 
Ezequiel Garc?a, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com

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

* [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects
  2014-06-09 18:43   ` Ezequiel Garcia
@ 2014-06-09 19:00     ` Rob Herring
  2014-06-09 19:05       ` Ezequiel Garcia
  0 siblings, 1 reply; 12+ messages in thread
From: Rob Herring @ 2014-06-09 19:00 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jun 9, 2014 at 1:43 PM, Ezequiel Garcia
<ezequiel.garcia@free-electrons.com> wrote:
> Hi Rob,
>
> On 22 Apr 02:41 PM, Rob Herring wrote:
>> @@ -11,7 +10,6 @@ config ARCH_MVEBU
>>       select ARCH_REQUIRE_GPIOLIB
>>       select MIGHT_HAVE_PCI
>>       select PCI_QUIRKS if PCI
>> -     select OF_ADDRESS_PCI
>>
>
> We needed that one, as Gregory introduced it specifically so we could select it
> without selecting PCI.
>
> I'll cook a patch right away, sorry for not seeing this earlier!

What exactly depends on this? It seems strange to have this
dependency. Some !OF compiling changes for PCI functions went in for
3.16 which may help here.

Rob

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

* [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects
  2014-06-09 19:00     ` Rob Herring
@ 2014-06-09 19:05       ` Ezequiel Garcia
  0 siblings, 0 replies; 12+ messages in thread
From: Ezequiel Garcia @ 2014-06-09 19:05 UTC (permalink / raw)
  To: linux-arm-kernel

On 09 Jun 02:00 PM, Rob Herring wrote:
> On Mon, Jun 9, 2014 at 1:43 PM, Ezequiel Garcia
> <ezequiel.garcia@free-electrons.com> wrote:
> > Hi Rob,
> >
> > On 22 Apr 02:41 PM, Rob Herring wrote:
> >> @@ -11,7 +10,6 @@ config ARCH_MVEBU
> >>       select ARCH_REQUIRE_GPIOLIB
> >>       select MIGHT_HAVE_PCI
> >>       select PCI_QUIRKS if PCI
> >> -     select OF_ADDRESS_PCI
> >>
> >
> > We needed that one, as Gregory introduced it specifically so we could select it
> > without selecting PCI.
> >
> > I'll cook a patch right away, sorry for not seeing this earlier!
> 
> What exactly depends on this? It seems strange to have this
> dependency. Some !OF compiling changes for PCI functions went in for
> 3.16 which may help here.
> 

See Gregory's commit introducing the option, I think it's very clear:

https://lkml.org/lkml/2014/2/19/615

-- 
Ezequiel Garc?a, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com

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

end of thread, other threads:[~2014-06-09 19:05 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-22 19:41 [PATCH 0/3] ARM mach Kconfig clean-ups Rob Herring
2014-04-22 19:41 ` [PATCH 1/3] ARM: mvebu: clean-up unneeded kconfig selects Rob Herring
2014-04-22 20:58   ` Andrew Lunn
2014-04-22 21:23     ` Rob Herring
2014-05-05  4:55   ` Olof Johansson
2014-05-05 14:42     ` Rob Herring
2014-05-05 21:22       ` Olof Johansson
2014-06-09 18:43   ` Ezequiel Garcia
2014-06-09 19:00     ` Rob Herring
2014-06-09 19:05       ` Ezequiel Garcia
2014-04-22 19:41 ` [PATCH 2/3] ARM: bcm: " Rob Herring
2014-04-22 19:41 ` [PATCH 3/3] ARM: qcom: " Rob Herring

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