From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Date: Mon, 08 Oct 2012 10:42:52 -0600 Subject: [U-Boot] [PATCH v3] tegra: add Colibri T20 board support In-Reply-To: <1349714297.8837.6.camel@tellur> References: <1349645766-21912-1-git-send-email-dev@lynxeye.de> <5072FF12.9070701@wwwdotorg.org> <1349714297.8837.6.camel@tellur> Message-ID: <5073028C.3020005@wwwdotorg.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 10/08/2012 10:38 AM, Lucas Stach wrote: > Am Montag, den 08.10.2012, 10:28 -0600 schrieb Stephen Warren: >> On 10/07/2012 03:36 PM, Lucas Stach wrote: >>> This adds board support for the Toradex Colibri T20 module. >>> >>> Working functions: >>> - SD card boot >>> - USB boot >>> - Network >>> - NAND environment >>> >>> Signed-off-by: Lucas Stach >>> --- >>> v2: >>> - rename .dts file to match board name >>> v3: >>> - make comments more meaningful >>> - split into module and carrier board specific part to ease porting >>> to other carrier boards >> >>> diff --git a/board/toradex/dts/tegra20-colibri_t20_iris.dts b/board/toradex/dts/tegra20-colibri_t20_iris.dts >> >> I suppose we can do this later, but since this patch has the C files >> split into separate board and carrier code, I'd also expect the .dts >> files to be split up in the same way. > > I initially tried exactly that, but since the .dts files are compiled a > bit different in u-boot than in the linux kernel it's not as easy as > doing a simple include. I'm planning to split them up later on, but I > first have to familiarize myself with the build process for dts in > u-boot. Oh right. Perhaps we should start to rely on the latest dtc, which is able to co-exist with the regular C pre-processor, and can support a bunch of include paths or something... > For now I would wish to get this included as is, to get Colibri T20 some > public visibility as a working Tegra platform, so others could take this > board into account when making changes to the Tegra 20 common platform. Sure.