From patchwork Fri Jun 7 16:51:07 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jonathan Austin X-Patchwork-Id: 2688781 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from casper.infradead.org (casper.infradead.org [85.118.1.10]) by patchwork2.kernel.org (Postfix) with ESMTP id A1CE2DFB78 for ; Fri, 7 Jun 2013 18:08:10 +0000 (UTC) Received: from merlin.infradead.org ([2001:4978:20e::2]) by casper.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1Ukzwf-0003Ik-Mk; Fri, 07 Jun 2013 16:55:27 +0000 Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1UkztJ-0001ol-Hw; Fri, 07 Jun 2013 16:51:53 +0000 Received: from service87.mimecast.com ([91.220.42.44]) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1UkztH-0001nE-Gb for linux-arm-kernel@lists.infradead.org; Fri, 07 Jun 2013 16:51:52 +0000 Received: from cam-owa2.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.21]) by service87.mimecast.com; Fri, 07 Jun 2013 17:51:23 +0100 Received: from [10.1.199.23] ([10.1.255.212]) by cam-owa2.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.3959); Fri, 7 Jun 2013 17:51:22 +0100 Message-ID: <51B20F7B.1080909@arm.com> Date: Fri, 07 Jun 2013 17:51:07 +0100 From: Jonathan Austin User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130308 Thunderbird/17.0.4 MIME-Version: 1.0 To: Arnd Bergmann Subject: Re: [PATCH v3 0/3] Separate generic header usage from ARCH_MULTIPLATFORM References: <1370602111-28737-1-git-send-email-jonathan.austin@arm.com> <51B1DC74.9090203@arm.com> <3354049.TGIUmJXV0H@wuerfel> <29798869.9FjkuBXpzS@wuerfel> In-Reply-To: <29798869.9FjkuBXpzS@wuerfel> X-OriginalArrivalTime: 07 Jun 2013 16:51:22.0786 (UTC) FILETIME=[3DC91020:01CE639F] X-MC-Unique: 113060717512301001 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20130607_125151_894133_81023B57 X-CRM114-Status: GOOD ( 17.26 ) 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: "linux-arm-kernel@lists.infradead.org" , "nico@fluxnic.net" X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org On 07/06/13 15:04, Arnd Bergmann wrote: > On Friday 07 June 2013 16:00:47 Arnd Bergmann wrote: >>> Doesn't the explicit dependency of CONFIG_MMU for ARCH_MULTIPLATFORM >>> mean that randconfig builds still work, even without the changes below? >>> >>> Apologies if I'm missing a use-case here! >> >> I am somewhat annoyed by the fact that I had to disable ARCH_MULTIPLATFORM >> for !MMU kernels in order to get 'make randconfig' to work and I would rather >> allow it and fix the bugs. > > To expand on this a bit more: I want to eventually make ARCH_MULTIPLATFORM > mandatory for all modern platforms, in particular anything ARMv6 and ARMv7 > based. Forcing all !MMU-kernels to be !ARCH_MULTIPLATFORM means we won't > get there unless we disallow !MMU on modern platforms. > Thanks for the expansion. I like that as an aim :). We should remember it'll still only make sense for platforms where PHYS_OFFSET/DRAM_BASE is the same - not sure how many that will be. I've had a play with keeping !MMU and ARCH_MULTIPLATFORM selectable together, and of the fixes in your diff, only the following is required to get R7 booting on top of Vexpress: -----8<------- ----->8-------- It's nice too, because we don't need to add back Makefile.boot for vexpress. The R7 support, nommu-fixes and MPU patches I sent before and which I hope to have merged for 3.11 (pull request for rmk coming shortly) can all work fine using this instead of the NEED_NO_MACH_HEADERS series (IE this series).... It would be cool, then, to have the small patch above in for 3.11 too if we're not too late and if nobody is against going down this route? Arnd: would you like to send it as a patch to the list for consideration, or shall I take this on? Jonny > Arnd > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index a7f066e..ead2b25 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig @@ -289,8 +289,7 @@ choice config ARCH_MULTIPLATFORM bool "Allow multiple platforms to be selected" - depends on MMU - select ARM_PATCH_PHYS_VIRT + select ARM_PATCH_PHYS_VIRT if MMU select AUTO_ZRELADDR select COMMON_CLK select MULTI_IRQ_HANDLER