From patchwork Fri Dec 7 14:48:54 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jonathan Austin X-Patchwork-Id: 1851011 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) by patchwork2.kernel.org (Postfix) with ESMTP id 1BCFBDF2EE for ; Fri, 7 Dec 2012 14:51:45 +0000 (UTC) Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.76 #1 (Red Hat Linux)) id 1TgzEt-0005Xs-Bq; Fri, 07 Dec 2012 14:49:19 +0000 Received: from service87.mimecast.com ([91.220.42.44]) by merlin.infradead.org with esmtp (Exim 4.76 #1 (Red Hat Linux)) id 1TgzEp-0005Wn-1Z for linux-arm-kernel@lists.infradead.org; Fri, 07 Dec 2012 14:49:16 +0000 Received: from cam-owa2.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.21]) by service87.mimecast.com; Fri, 07 Dec 2012 14:49:10 +0000 Received: from [10.1.68.58] ([10.1.255.212]) by cam-owa2.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.3959); Fri, 7 Dec 2012 14:49:09 +0000 Message-ID: <50C201D6.7060408@arm.com> Date: Fri, 07 Dec 2012 14:48:54 +0000 From: Jonathan Austin User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0 MIME-Version: 1.0 To: Fabio Estevam Subject: Re: [PATCH v2] ARM: Kconfig: Do not force selection of ARCH_VEXPRESS by ARCH_MULTI_V7 References: <1353431918-9326-1-git-send-email-festevam@gmail.com> In-Reply-To: <1353431918-9326-1-git-send-email-festevam@gmail.com> X-OriginalArrivalTime: 07 Dec 2012 14:49:09.0999 (UTC) FILETIME=[03EC0FF0:01CDD48A] X-MC-Unique: 112120714491004601 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20121207_094915_316104_5AEABF48 X-CRM114-Status: GOOD ( 14.42 ) X-Spam-Score: -2.6 (--) X-Spam-Report: SpamAssassin version 3.3.2 on merlin.infradead.org summary: Content analysis details: (-2.6 points) pts rule name description ---- ---------------------- -------------------------------------------------- -0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at http://www.dnswl.org/, low trust [91.220.42.44 listed in list.dnswl.org] -0.0 SPF_PASS SPF: sender matches SPF record -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] Cc: "thomas.petazzoni@free-electrons.com" , Fabio Estevam , "linux@arm.linux.org.uk" , Pawel Moll , "rob.herring@calxeda.com" , "kernel@pengutronix.de" , "linux-arm-kernel@lists.infradead.org" X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-arm-kernel-bounces@lists.infradead.org Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org On 20/11/12 17:18, Fabio Estevam wrote: > From: Fabio Estevam > > When building a kernel using imx_v6_v7_defconfig, it is not possible to unselect > ARCH_VEXPRESS due to the fact that it is automatically selected by ARCH_MULTI_V7. > > This is not good when someone wants to create a minimal kernel build, so let > multi_v7_defconfig select ARCH_VEXPRESS instead. This patch solves one aspect of a problem previously discussed for !MMU [1], but we still have the dependency of ARCH_MULTI_V7 for ARCH_VEXPRESS. ARCH_MULTI_V7 doesn't make very much sense without an MMU because we need to hardcode things like PHYS_OFFSET and the vectors. Looking back though the log, I can't see quite why we need this dependency. With Fabio's patch that selects ARCH_VEXPRESS in the defconfig, is it still required? How do people feel about something like the patch below, to allow NOMMU things to use Versatile Express without MULTI_V7? Jonny [1]http://lists.infradead.org/pipermail/linux-arm-kernel/2012-November/129976.html -----8<----------- diff --git a/arch/arm/mach-vexpress/Kconfig b/arch/arm/mach-vexpress/Kconfig index c952960..7d3fff4 100644 --- a/arch/arm/mach-vexpress/Kconfig +++ b/arch/arm/mach-vexpress/Kconfig @@ -1,5 +1,5 @@ config ARCH_VEXPRESS - bool "ARM Ltd. Versatile Express family" if ARCH_MULTI_V7 + bool "ARM Ltd. Versatile Express family" if (ARCH_MULTI_V7 || !MMU) select ARCH_WANT_OPTIONAL_GPIOLIB select ARM_AMBA select ARM_GIC