From patchwork Tue Feb 12 10:41:09 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Thierry Reding X-Patchwork-Id: 219779 X-Patchwork-Delegate: twarren@nvidia.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from theia.denx.de (theia.denx.de [85.214.87.163]) by ozlabs.org (Postfix) with ESMTP id 0FCE82C0293 for ; Tue, 12 Feb 2013 21:41:29 +1100 (EST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 19BB54A01C; Tue, 12 Feb 2013 11:41:26 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at theia.denx.de Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ZzrX9GlVEThf; Tue, 12 Feb 2013 11:41:25 +0100 (CET) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id A33884A025; Tue, 12 Feb 2013 11:41:23 +0100 (CET) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 53B724A025 for ; Tue, 12 Feb 2013 11:41:21 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at theia.denx.de Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id A2k4MwGexrcB for ; Tue, 12 Feb 2013 11:41:19 +0100 (CET) X-policyd-weight: NOT_IN_SBL_XBL_SPAMHAUS=-1.5 NOT_IN_SPAMCOP=-1.5 NOT_IN_BL_NJABL=-1.5 (only DNSBL check requested) Received: from moutng.kundenserver.de (moutng.kundenserver.de [212.227.126.186]) by theia.denx.de (Postfix) with ESMTPS id 9787F4A01C for ; Tue, 12 Feb 2013 11:41:18 +0100 (CET) Received: from mailbox.adnet.avionic-design.de (mailbox.avionic-design.de [109.75.18.3]) by mrelayeu.kundenserver.de (node=mrbap0) with ESMTP (Nemesis) id 0M8pEi-1UCw0S3ViW-00CGAF; Tue, 12 Feb 2013 11:41:13 +0100 Received: from localhost (localhost [127.0.0.1]) by mailbox.adnet.avionic-design.de (Postfix) with ESMTP id 06E342A28141; Tue, 12 Feb 2013 11:41:12 +0100 (CET) X-Virus-Scanned: amavisd-new at avionic-design.de Received: from mailbox.adnet.avionic-design.de ([127.0.0.1]) by localhost (mailbox.avionic-design.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Mo7xrzjQZLTF; Tue, 12 Feb 2013 11:41:09 +0100 (CET) Received: from mailman.adnet.avionic-design.de (mailman.adnet.avionic-design.de [172.20.31.172]) by mailbox.adnet.avionic-design.de (Postfix) with ESMTP id DAC8B2A28102; Tue, 12 Feb 2013 11:41:09 +0100 (CET) Received: from localhost (avionic-0098.adnet.avionic-design.de [172.20.31.233]) by mailman.adnet.avionic-design.de (Postfix) with ESMTP id 23089100481; Tue, 12 Feb 2013 11:41:06 +0100 (CET) Date: Tue, 12 Feb 2013 11:41:09 +0100 From: Thierry Reding To: Tom Warren Message-ID: <20130212104109.GA2824@avionic-0098.mockup.avionic-design.de> References: <1360603051-29478-1-git-send-email-twarren@nvidia.com> <1360603051-29478-2-git-send-email-twarren@nvidia.com> <1360603706.1537.2.camel@tellur> <20130211191107.GB25894@avionic-0098.mockup.avionic-design.de> <20130212065155.GB28850@avionic-0098.mockup.avionic-design.de> MIME-Version: 1.0 In-Reply-To: <20130212065155.GB28850@avionic-0098.mockup.avionic-design.de> User-Agent: Mutt/1.5.21 (2010-09-15) X-Provags-ID: V02:K0:yQ93wttoAfMolr48X3NLjAfJlAvnOAwWzSmB9952smx gDhernDOx39nfzUmIM/l0F+Wn9j3+tCoGeZ1x75wIkgtorVMP8 E/kNIVPK3G/jqm6vo/L6rOfzbzo8koclaZxTu7zaAE8Beq1rwy hWT0PdlscrfunGGAqXMXHNJhmDd7yUX0/88o/h1vN4npPz/sjV 1sMO5jGUBFwqjqsRrNV4pn9BSD8sYsHNK4qlf2+aEcyFWZZsqR 5q/Imv+aoHE/vF+KK2rF3tfg92AQ3a8AGXf6PNYD5XF//A8zEK tB1L5O5884VQs9+2dhTAyBdYU7XGVq0e74SOF/C4t/m2OqLX4S gKj9s8/nAoNZq4aEAuPk1MNcoXHFrcPdym2S0bEYsqkB+uFF91 q0B9HnRvUsIVIwf7HCxg+82Ft2z/BjwtaOZpATPyjkmRaqyKIf HcukM Cc: "u-boot@lists.denx.de" , afleming@freescale.com, Tom Warren , Stephen Warren Subject: Re: [U-Boot] [PATCH v2 1/2] Tegra: fdt: Add/enhance sdhci (mmc) nodes for all T20 DT files X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.11 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: u-boot-bounces@lists.denx.de Errors-To: u-boot-bounces@lists.denx.de On Tue, Feb 12, 2013 at 07:51:55AM +0100, Thierry Reding wrote: > On Mon, Feb 11, 2013 at 12:21:59PM -0700, Tom Warren wrote: > > Thierry/Lucas, > > > > On Mon, Feb 11, 2013 at 12:11 PM, Thierry Reding > > wrote: > > > On Mon, Feb 11, 2013 at 10:56:33AM -0700, Tom Warren wrote: > > >> Lucas, > > >> > > >> On Mon, Feb 11, 2013 at 10:28 AM, Lucas Stach wrote: > > >> > Hi Tom, > > >> > > > >> > Am Montag, den 11.02.2013, 10:17 -0700 schrieb Tom Warren: > > >> >> Linux dts files were used for those boards that didn't already > > >> >> have sdhci info populated. Tamonten has their own dtsi file with > > >> >> common sdhci nodes (sourced from Linux). > > >> >> > > >> >> Signed-off-by: Tom Warren > > >> >> --- > > >> >> v2: > > >> >> - cleanup comments in dts files/match w/kernel files > > >> >> - add sdhci aliases in all dts files > > >> >> - use tegra20-tamonten.dtsi from the kernel for AD boards > > >> >> > > >> >> arch/arm/dts/tegra20-tamonten.dtsi | 489 ++++++++++++++++++++++ > > >> > > > >> > I'm not sure if pushing the whole file in this patch is the right thing > > >> > to do. > > >> > > >> I didn't want to edit it since we seem to be moving towards using the > > >> Linux DTS files in toto in U-Boot (as per Stephen & Simon). Does it do > > >> any harm to have the whole thing here? Saves some work later. Thierry > > >> - what do you think? > > > > > > Given that it isn't used anywhere I don't think we really need it right > > > now. We can always add it later when we can make better use of it. > > > > It actually is used (for SDMMC/sdhci) now, Thierry. That's why it's in > > this patchset. > > Right, I hadn't looked at that patch yet. > > > I had originally put the sdhci node for Avionic Design > > boards in their respective .dts files, but Stephen pointed out that > > the kernel had a tegra20-tamonten.dtsi file with common info, which > > included the sdhci node, and asked that I use it, instead, so we echo > > the kernel layout. So I pulled that file into my MMC DT patchset, and > > used it in all AD board builds (medcom/tec/plutux) - it's pulled in > > via an override of CONFIG_ARCH_DEVICE_TREE in the config files. > > > > So the options seem to be: > > > > a) Don't use the tamonton dtsi file, and put the sdhci nodes in the AD > > dts files, just like all other boards (this was my V1 approach). > > Vetoed by Stephen. > > b) Use tegra20-tamonten.dtsi as is, identical to the kernel file. If > > necessary, I can move it's inclusion to a separate patch, independent > > of the MMC DT patchset, as suggested by Lucas. > > c) Use tegra20-tamonten.dtsi, but just with the sdhci node (is this > > what you're suggesting, Thierry?). I'd still pull it in via a > > CONFIG_ARCH_DEVICE_TREE #define in the AD config files. > > > > Let me know ASAP - I'd like to get V3 upstreamed soon so I can move on > > to work on the T30/T114 MMC patches. > > I think option b) sounds fine given that we want to move to the same DTS > as the kernel eventually anyway. So for the Tamonten (and AD board) > pieces, consider this: > > Acked-by: Thierry Reding > > I can't give you a Tested-by because I have a bunch of other things to > take care of and I probably won't get to testing this for a few days. So it turned out that I need to touch U-Boot anyway, so I decided to give this a spin. I noticed that overriding CONFIG_ARCH_DEVICE_TREE from the board configuration file doesn't work currently. What happens is that the autoconf.mk (which is derived from the board configuration) is included before the CPU config.mk which sets CONFIG_ARCH_DEVICE_TREE to tegra20 (or tegra30, tegra114). I came up with the attached patch to set the variable if not set previously (by the board configuration file). Feel free to squash that in your patch series if you deem it a proper solution. I can also provide a proper separate patch if you prefer. Thierry diff --git a/arch/arm/cpu/armv7/tegra114/config.mk b/arch/arm/cpu/armv7/tegra114/config.mk index cb1a19d..e7c22c0 100644 --- a/arch/arm/cpu/armv7/tegra114/config.mk +++ b/arch/arm/cpu/armv7/tegra114/config.mk @@ -16,4 +16,4 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . # -CONFIG_ARCH_DEVICE_TREE := tegra114 +CONFIG_ARCH_DEVICE_TREE ?= tegra114 diff --git a/arch/arm/cpu/armv7/tegra20/config.mk b/arch/arm/cpu/armv7/tegra20/config.mk index 6432e75..9042664 100644 --- a/arch/arm/cpu/armv7/tegra20/config.mk +++ b/arch/arm/cpu/armv7/tegra20/config.mk @@ -23,4 +23,4 @@ # Foundation, Inc., 59 Temple Place, Suite 330, Boston, # MA 02111-1307 USA # -CONFIG_ARCH_DEVICE_TREE := tegra20 +CONFIG_ARCH_DEVICE_TREE ?= tegra20 diff --git a/arch/arm/cpu/armv7/tegra30/config.mk b/arch/arm/cpu/armv7/tegra30/config.mk index 719ca81..0035bc5 100644 --- a/arch/arm/cpu/armv7/tegra30/config.mk +++ b/arch/arm/cpu/armv7/tegra30/config.mk @@ -16,4 +16,4 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . # -CONFIG_ARCH_DEVICE_TREE := tegra30 +CONFIG_ARCH_DEVICE_TREE ?= tegra30