From mboxrd@z Thu Jan 1 00:00:00 1970 From: Krzysztof Kozlowski Subject: Re: [PATCH v2] ARM: dts: tegra: Add support for 256 MB Colibri-T20 (plus such board on Iris) Date: Fri, 4 May 2018 12:03:36 +0200 Message-ID: References: <1525360112-9893-1-git-send-email-krzk@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Stefan Agner Cc: Rob Herring , Mark Rutland , Thierry Reding , Jonathan Hunter , devicetree@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org, Marcel Ziswiler , Lucas Stach List-Id: linux-tegra@vger.kernel.org On Fri, May 4, 2018 at 11:19 AM, Stefan Agner wrote: > On 03.05.2018 17:08, Krzysztof Kozlowski wrote: >> Colibri-T20 can come in 256 MB RAM (with 512 MB NAND) or 512 MB RAM >> (with 1024 MB NAND) flavors. Add support for the 256 MB version on Iris >> evaluation board. > > To we really need to specify memory size these days? I think all common > boot loaders fill the memory node anyway. > > arch/arm/boot/dts/imx6qdl-apalis.dtsi: > /* Will be filled by the bootloader */ > memory@10000000 { > reg = <0x10000000 0>; > }; > > > I think we should just rename tegra20-colibri-512.dtsi => > tegra20-colibri.dtsi and tegra20-iris-512.dts => tegra20-iris.dts to > avoid confusion and add such an empty node. Having memory node is requirement of DeviceTree specification (at least one cpu and memory node). Of course if bootloader fills it, then we could assume that specification is satisfied... but what if some bootloader skips it? This creates quite specific dependency between kernel's DTS and bootloader. One safe solution would be to specify 256 MB memory node for both boards and assume that bootloader will bump it to 512 MB for specific module. >> Signed-off-by: Krzysztof Kozlowski >> >> --- >> >> Changes since v1: >> 1. Fix memory size in tegra20-colibri-256.dtsi (was working fine because >> my bootloader uses mem= argument). > > What boot loader are you using? You even can omit mem= if your boot > loader fills the memory node properly. I use the one provided by vendor - Toradex. By default it passes both mem= and few other mem-like arguments (for FB and video processor) to command line. I believe it does not adjust the DTB itself because Toradex prepared this bootloader for... 3.1 kernel without DTB. 7 years old kernel. :) Best regards, Krzysztof