From patchwork Sat Aug 18 23:06:50 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Linus Walleij X-Patchwork-Id: 1341741 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 B3793DF266 for ; Sat, 18 Aug 2012 23:09:53 +0000 (UTC) Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.76 #1 (Red Hat Linux)) id 1T2s6Z-0007fP-3S; Sat, 18 Aug 2012 23:06:55 +0000 Received: from mail-vc0-f177.google.com ([209.85.220.177]) by merlin.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1T2s6W-0007f7-B1 for linux-arm-kernel@lists.infradead.org; Sat, 18 Aug 2012 23:06:52 +0000 Received: by vcbfl13 with SMTP id fl13so4215986vcb.36 for ; Sat, 18 Aug 2012 16:06:51 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:x-gm-message-state; bh=4dl8DRhNogrSNZXXvQEMF1IA35161KU07NuGELcAaHE=; b=omTSYcrFZ8+ZuNJKpZheVQYE3+44O1zyMuzCWFYqwOWnm9IrA0HVNoIzl5VXiMtRwe Tt4kkm1HaKXsSWu702Wx+pumNLUl/ATAmGK3NAcNmK6UVAwOjvVdXr1TZ6TSV/CcG3yk uOkk9cSBsG1+NLmRt3Txw4vHUrzOcJGXbMVA/PFO8S13wy94s61KOfVZTiMOsTYNvG0I UhCNlvirzCCgy8OH7H4wTrZId0WE1hldGxO1qO0clyAIJXKx6Ya6/3SZQ9hJrGDdaTfQ Fm6jX4ualm1gpnWiEBTRzEPZlxApPVXUf/Row89Ndtjs28B8YnAwVjrjT+3Dk00M0kmZ 1xJw== MIME-Version: 1.0 Received: by 10.58.144.138 with SMTP id sm10mr6880662veb.31.1345331210994; Sat, 18 Aug 2012 16:06:50 -0700 (PDT) Received: by 10.52.180.6 with HTTP; Sat, 18 Aug 2012 16:06:50 -0700 (PDT) In-Reply-To: <201208161344.55723.arnd@arndb.de> References: <1345119317-22600-1-git-send-email-linus.walleij@linaro.org> <201208161344.55723.arnd@arndb.de> Date: Sun, 19 Aug 2012 01:06:50 +0200 Message-ID: Subject: Re: [PATCH 0/6] Integrator devicetree support From: Linus Walleij To: Arnd Bergmann , Nicolas Pitre X-Gm-Message-State: ALoCoQkTNSDwNGLW6+ZMhBbpgChk6abkzsqJbPzJxwyYY4prtUl00NGdJ2H0JEklhbc2I0SmoU6r X-Spam-Note: CRM114 invocation failed 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 [209.85.220.177 listed in list.dnswl.org] -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] Cc: Will Deacon , devicetree-discuss@lists.ozlabs.org, arm@kernel.org, Russell King , 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 Thu, Aug 16, 2012 at 3:44 PM, Arnd Bergmann wrote: > On Thursday 16 August 2012, Linus Walleij wrote: >> This patch set moves all the non-DT platform code into >> #ifndef CONFIG_OF sections for clarity. The plan is to >> delete them after deprecation. > > I'm fine with your playing around with this in any way you > like, but I think in general we should have a new CONFIG_ATAG > symbol, so you can build a kernel that supports booting > both with and without DT for the same platform, depending > on what the boot loader supports. > > Then you can enclose all the non-DT sections in #ifdef > CONFIG_ATAG and we can remove them at some later stage. Since we already have CONFIG_USE_OF I think CONFIG_USE_ATAGS is more apropriate, but I get the idea. So it begins like this: But then we hit #ifdeffing and compiling out a lot of dangerous stuff in arch/arm/kernel and then I get the creeps. I can try, but I think this is typical Nico territory for example... Yours, Linus Walleij diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index e91c7cd..0764ef7 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig @@ -1907,6 +1907,12 @@ endmenu menu "Boot options" +config USE_ATAGS + bool "ATAGS support" + default y + help + Include support for ATAGS-based machine descriptions. + config USE_OF bool "Flattened Device Tree support" select OF @@ -2110,6 +2116,7 @@ config KEXEC config ATAGS_PROC bool "Export atags in procfs" + depends on USE_ATAGS depends on KEXEC default y help