From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751930AbbABJux (ORCPT ); Fri, 2 Jan 2015 04:50:53 -0500 Received: from metis.ext.pengutronix.de ([92.198.50.35]:60689 "EHLO metis.ext.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750727AbbABJuu (ORCPT ); Fri, 2 Jan 2015 04:50:50 -0500 Date: Fri, 2 Jan 2015 10:50:23 +0100 From: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= To: Stefan Agner Cc: shawn.guo@linaro.org, kernel@pengutronix.de, linux@arm.linux.org.uk, jason@lakedaemon.net, olof@lixom.net, arnd@arndb.de, daniel.lezcano@linaro.org, tglx@linutronix.de, mark.rutland@arm.com, pawel.moll@arm.com, robh+dt@kernel.org, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, marc.zyngier@arm.com, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 09/12] ARM: efm32: move into multiplatform Message-ID: <20150102095023.GE28003@pengutronix.de> References: <1419895935-22966-1-git-send-email-stefan@agner.ch> <1419895935-22966-10-git-send-email-stefan@agner.ch> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1419895935-22966-10-git-send-email-stefan@agner.ch> User-Agent: Mutt/1.5.21 (2010-09-15) X-SA-Exim-Connect-IP: 2001:67c:670:100:1d::c0 X-SA-Exim-Mail-From: ukl@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello, On Tue, Dec 30, 2014 at 12:32:12AM +0100, Stefan Agner wrote: > From: Arnd Bergmann > > Since the multiplatform configuration can support no-MMU kernels now, > there is nothing stopping us from moving the efm32 platform in there > as well. This introduces a new ARCH_MULTI_V7M CPU architecture selection > option, since v7-M is incompatible with v7-A, and we can have either > of the two enabled for multiplatform, but not both at the same time. > > Signed-off-by: Arnd Bergmann > Signed-off-by: Stefan Agner > --- > arch/arm/Kconfig | 35 ++++++++++++++++------------------- > arch/arm/configs/efm32_defconfig | 2 ++ > 2 files changed, 18 insertions(+), 19 deletions(-) > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig > index 95007b9..8fe035b 100644 > --- a/arch/arm/Kconfig > +++ b/arch/arm/Kconfig > @@ -314,7 +314,7 @@ config ARCH_MULTIPLATFORM > select COMMON_CLK > select GENERIC_CLOCKEVENTS > select MIGHT_HAVE_PCI > - select MULTI_IRQ_HANDLER > + select MULTI_IRQ_HANDLER if !ARCH_MULTI_V7M > select SPARSE_IRQ > select USE_OF > > @@ -400,24 +400,6 @@ config ARCH_EBSA110 > Ethernet interface, two PCMCIA sockets, two serial ports and a > parallel port. > > -config ARCH_EFM32 > - bool "Energy Micro efm32" > - depends on !MMU > - select ARCH_REQUIRE_GPIOLIB > - select ARM_NVIC > - select AUTO_ZRELADDR > - select CLKSRC_OF > - select COMMON_CLK > - select CPU_V7M > - select GENERIC_CLOCKEVENTS > - select NO_DMA > - select NO_IOPORT_MAP > - select SPARSE_IRQ > - select USE_OF > - help > - Support for Energy Micro's (now Silicon Labs) efm32 Giant Gecko > - processors. > - > config ARCH_EP93XX > bool "EP93xx-based" > select ARCH_HAS_HOLES_MEMORYMODEL > @@ -778,6 +760,14 @@ menu "Multiple platform selection" > > comment "CPU Core family selection" > > +config ARCH_MULTI_V7M > + bool "ARMv7-M based platforms (Cortex-M)" > + depends on !MMU && !(ARCH_MULTI_V4_V5 || ARCH_MULTI_V6_V7) As ARCH_MULTI_V7M depends on !MMU already ... > + select CPU_V7M > + select ARM_NVIC > + select NO_DMA # for now > + select NO_IOPORT_MAP # for now > + > config ARCH_MULTI_V4 > bool "ARMv4 based platforms (FA526)" > depends on !ARCH_MULTI_V6_V7 && MMU > @@ -834,6 +824,13 @@ config ARCH_VIRT > select ARM_PSCI > select HAVE_ARM_ARCH_TIMER > > +config ARCH_EFM32 > + bool "Energy Micro efm32" if ARCH_MULTI_V7M > + depends on !MMU ... you don't need to repeat it here. Other than that you can have my Ack. I didn't test it though and must have missed the changes mentioned in the commit log. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ |