* [PATCH] ARM: mvebu: fix build breaks from multi-platform conversion
@ 2012-09-27 15:10 Rob Herring
2012-09-27 15:26 ` Thomas Petazzoni
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Rob Herring @ 2012-09-27 15:10 UTC (permalink / raw)
To: linux-arm-kernel
From: Rob Herring <rob.herring@calxeda.com>
Moving ARCH_MVEBU for multi-platform support caused several breakages in
recently added addr-map and pinctrl support for mvebu. This adds the
necessary selects and include paths to fix the build.
Signed-off-by: Rob Herring <rob.herring@calxeda.com>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gregory CLEMENT <gregory.clement@free-electrons.com>
Cc: Jason Cooper <jason@lakedaemon.net>
Cc: Andrew Lunn <andrew@lunn.ch>
---
arch/arm/mach-mvebu/Kconfig | 2 ++
arch/arm/mach-mvebu/Makefile | 3 ++-
arch/arm/plat-orion/Makefile | 1 +
3 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mach-mvebu/Kconfig b/arch/arm/mach-mvebu/Kconfig
index 26a817d..416d46e 100644
--- a/arch/arm/mach-mvebu/Kconfig
+++ b/arch/arm/mach-mvebu/Kconfig
@@ -6,6 +6,8 @@ config ARCH_MVEBU
select GENERIC_IRQ_CHIP
select IRQ_DOMAIN
select MULTI_IRQ_HANDLER
+ select PINCTRL
+ select PLAT_ORION
select SPARSE_IRQ
if ARCH_MVEBU
diff --git a/arch/arm/mach-mvebu/Makefile b/arch/arm/mach-mvebu/Makefile
index cbe5664..57f996b 100644
--- a/arch/arm/mach-mvebu/Makefile
+++ b/arch/arm/mach-mvebu/Makefile
@@ -1,4 +1,5 @@
-ccflags-$(ARCH_MULTIPLATFORM) := -I$(srctree)/$(src)/include
+ccflags-$(CONFIG_ARCH_MULTIPLATFORM) := -I$(srctree)/$(src)/include \
+ -I$(srctree)/arch/arm/plat-orion/include
obj-y += system-controller.o
obj-$(CONFIG_MACH_ARMADA_370_XP) += armada-370-xp.o irq-armada-370-xp.o addr-map.o
diff --git a/arch/arm/plat-orion/Makefile b/arch/arm/plat-orion/Makefile
index 1251e5b..a82cecb 100644
--- a/arch/arm/plat-orion/Makefile
+++ b/arch/arm/plat-orion/Makefile
@@ -1,6 +1,7 @@
#
# Makefile for the linux kernel.
#
+ccflags-$(CONFIG_ARCH_MULTIPLATFORM) := -I$(srctree)/$(src)/include
obj-y += addr-map.o
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] ARM: mvebu: fix build breaks from multi-platform conversion
2012-09-27 15:10 [PATCH] ARM: mvebu: fix build breaks from multi-platform conversion Rob Herring
@ 2012-09-27 15:26 ` Thomas Petazzoni
2012-09-27 16:01 ` Arnd Bergmann
2012-09-27 15:30 ` Arnd Bergmann
2012-09-28 20:32 ` Arnd Bergmann
2 siblings, 1 reply; 6+ messages in thread
From: Thomas Petazzoni @ 2012-09-27 15:26 UTC (permalink / raw)
To: linux-arm-kernel
Dear Rob Herring,
On Thu, 27 Sep 2012 10:10:12 -0500, Rob Herring wrote:
> From: Rob Herring <rob.herring@calxeda.com>
>
> Moving ARCH_MVEBU for multi-platform support caused several breakages
> in recently added addr-map and pinctrl support for mvebu. This adds
> the necessary selects and include paths to fix the build.
>
> Signed-off-by: Rob Herring <rob.herring@calxeda.com>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Gregory CLEMENT <gregory.clement@free-electrons.com>
> Cc: Jason Cooper <jason@lakedaemon.net>
> Cc: Andrew Lunn <andrew@lunn.ch>
> ---
> arch/arm/mach-mvebu/Kconfig | 2 ++
> arch/arm/mach-mvebu/Makefile | 3 ++-
> arch/arm/plat-orion/Makefile | 1 +
> 3 files changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-mvebu/Kconfig b/arch/arm/mach-mvebu/Kconfig
> index 26a817d..416d46e 100644
> --- a/arch/arm/mach-mvebu/Kconfig
> +++ b/arch/arm/mach-mvebu/Kconfig
> @@ -6,6 +6,8 @@ config ARCH_MVEBU
> select GENERIC_IRQ_CHIP
> select IRQ_DOMAIN
> select MULTI_IRQ_HANDLER
> + select PINCTRL
This select was already part added as a part of:
commit 7e8d941567c99a03390154a7bb116d1b03db82b3
Author: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Date: Thu Sep 13 17:41:43 2012 +0200
pinctrl: mvebu: pinctrl driver core
This patch adds a pinctrl driver core for Marvell SoCs plus DT
binding documentation. This core driver will be used by SoC family
specific drivers, i.e. Armada XP, Armada 370, Dove, Kirkwood, aso.
Signed-off-by: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Acked-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Tested-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Reviewed-by: Stephen Warren <swarren@wwwdotorg.org>
Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Conflicts:
arch/arm/Kconfig
which was requested for pull from the kirkwood/drivers branch of Jason
Cooper repository. If it is no longer there, then it is a merge
conflict resolution mistake.
> + select PLAT_ORION
This other select was already added as a part of:
commit abcda1dc3e23366c8543b372e9bf4e9086d5cca1
Author: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Date: Tue Sep 11 14:27:27 2012 +0200
arm: plat-orion: introduce PLAT_ORION_LEGACY hidden config option
Until now, the PLAT_ORION configuration option was common to all the
Marvell EBU SoCs, and selecting this option had the effect of enabling
the MPP code, GPIO code, address decoding and PCIe code from
plat-orion, as well as providing access to driver-specific header
files from plat-orion/include.
However, the Armada 370 and XP SoCs will not use the MPP and GPIO code
(instead some proper pinctrl and gpio drivers are in preparation), and
generally, we want to move away from plat-orion and instead have
everything in mach-mvebu.
That said, in the mean time, we want to leverage the driver-specific
headers as well as the address decoding code, so we introduce
PLAT_ORION_LEGACY. The older Marvell SoCs need to select
PLAT_ORION_LEGACY, while the newer Marvell SoCs need to select
PLAT_ORION. Of course, when PLAT_ORION_LEGACY is selected, it
automatically selects PLAT_ORION.
Then, with just PLAT_ORION, you have the address decoding code plus
the driver-specific headers. If you add PLAT_ORION_LEGACY to this, you
gain the old MPP, GPIO and PCIe code.
Again, this is only a temporary solution until we make all Marvell EBU
platforms converge into the mach-mvebu directory. This solution avoids
duplicating the existing address decoding code into mach-mvebu.
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Acked-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Tested-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Coming from the kirkwood/addr_decode branch of Jason Cooper repository.
So it looks strange that you don't have both selects.
Otherwise, for the rest of patch:
Acked-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARM: mvebu: fix build breaks from multi-platform conversion
2012-09-27 15:10 [PATCH] ARM: mvebu: fix build breaks from multi-platform conversion Rob Herring
2012-09-27 15:26 ` Thomas Petazzoni
@ 2012-09-27 15:30 ` Arnd Bergmann
2012-09-28 20:32 ` Arnd Bergmann
2 siblings, 0 replies; 6+ messages in thread
From: Arnd Bergmann @ 2012-09-27 15:30 UTC (permalink / raw)
To: linux-arm-kernel
On Thursday 27 September 2012, Rob Herring wrote:
> From: Rob Herring <rob.herring@calxeda.com>
>
> Moving ARCH_MVEBU for multi-platform support caused several breakages in
> recently added addr-map and pinctrl support for mvebu. This adds the
> necessary selects and include paths to fix the build.
>
> Signed-off-by: Rob Herring <rob.herring@calxeda.com>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Gregory CLEMENT <gregory.clement@free-electrons.com>
> Cc: Jason Cooper <jason@lakedaemon.net>
> Cc: Andrew Lunn <andrew@lunn.ch>
Acked-by: Arnd Bergmann <arnd@arndb.de>
I actually had the same patch since monday but haven't managed to
send out my series.
Olof probably has a better idea of how to apply this than me
to avoid further merge conflicts.
Arnd
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARM: mvebu: fix build breaks from multi-platform conversion
2012-09-27 15:26 ` Thomas Petazzoni
@ 2012-09-27 16:01 ` Arnd Bergmann
0 siblings, 0 replies; 6+ messages in thread
From: Arnd Bergmann @ 2012-09-27 16:01 UTC (permalink / raw)
To: linux-arm-kernel
On Thursday 27 September 2012, Thomas Petazzoni wrote:
> This select was already part added as a part of:
>
> commit 7e8d941567c99a03390154a7bb116d1b03db82b3
> Author: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
> Date: Thu Sep 13 17:41:43 2012 +0200
>
> pinctrl: mvebu: pinctrl driver core
>
> This patch adds a pinctrl driver core for Marvell SoCs plus DT
> binding documentation. This core driver will be used by SoC family
> specific drivers, i.e. Armada XP, Armada 370, Dove, Kirkwood, aso.
>
> Signed-off-by: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Acked-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Tested-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Reviewed-by: Stephen Warren <swarren@wwwdotorg.org>
> Signed-off-by: Jason Cooper <jason@lakedaemon.net>
>
> Conflicts:
>
> arch/arm/Kconfig
>
> which was requested for pull from the kirkwood/drivers branch of Jason
> Cooper repository. If it is no longer there, then it is a merge
> conflict resolution mistake.
I believe it was lost again in the move to the multiplatform configuration,
which moves the ARCH_MVEBU definition to arch/arm/mach-mvebu/Kconfig.
automated merges with git don't notice this. Same for the other one.
Arnd
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARM: mvebu: fix build breaks from multi-platform conversion
2012-09-27 15:10 [PATCH] ARM: mvebu: fix build breaks from multi-platform conversion Rob Herring
2012-09-27 15:26 ` Thomas Petazzoni
2012-09-27 15:30 ` Arnd Bergmann
@ 2012-09-28 20:32 ` Arnd Bergmann
2012-09-29 21:34 ` Olof Johansson
2 siblings, 1 reply; 6+ messages in thread
From: Arnd Bergmann @ 2012-09-28 20:32 UTC (permalink / raw)
To: linux-arm-kernel
On Thursday 27 September 2012, Rob Herring wrote:
> From: Rob Herring <rob.herring@calxeda.com>
>
> Moving ARCH_MVEBU for multi-platform support caused several breakages in
> recently added addr-map and pinctrl support for mvebu. This adds the
> necessary selects and include paths to fix the build.
>
> Signed-off-by: Rob Herring <rob.herring@calxeda.com>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Gregory CLEMENT <gregory.clement@free-electrons.com>
> Cc: Jason Cooper <jason@lakedaemon.net>
> Cc: Andrew Lunn <andrew@lunn.ch>
Applied on top of the late/kirkwood branch that introduced the incorrect
merge.
Arnd
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARM: mvebu: fix build breaks from multi-platform conversion
2012-09-28 20:32 ` Arnd Bergmann
@ 2012-09-29 21:34 ` Olof Johansson
0 siblings, 0 replies; 6+ messages in thread
From: Olof Johansson @ 2012-09-29 21:34 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Sep 28, 2012 at 08:32:02PM +0000, Arnd Bergmann wrote:
> On Thursday 27 September 2012, Rob Herring wrote:
> > From: Rob Herring <rob.herring@calxeda.com>
> >
> > Moving ARCH_MVEBU for multi-platform support caused several breakages in
> > recently added addr-map and pinctrl support for mvebu. This adds the
> > necessary selects and include paths to fix the build.
> >
> > Signed-off-by: Rob Herring <rob.herring@calxeda.com>
> > Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> > Cc: Gregory CLEMENT <gregory.clement@free-electrons.com>
> > Cc: Jason Cooper <jason@lakedaemon.net>
> > Cc: Andrew Lunn <andrew@lunn.ch>
>
> Applied on top of the late/kirkwood branch that introduced the incorrect
> merge.
Thanks Arnd!
-Olfo
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-09-29 21:34 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-27 15:10 [PATCH] ARM: mvebu: fix build breaks from multi-platform conversion Rob Herring
2012-09-27 15:26 ` Thomas Petazzoni
2012-09-27 16:01 ` Arnd Bergmann
2012-09-27 15:30 ` Arnd Bergmann
2012-09-28 20:32 ` Arnd Bergmann
2012-09-29 21:34 ` Olof Johansson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox