From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Tue, 12 Mar 2019 22:11:21 +0100 Subject: [U-Boot] [PATCH 4/4] ARM: renesas: Configure DRAM size from ATF DT fragment In-Reply-To: References: <20190305032557.19788-1-marek.vasut+renesas@gmail.com> <20190305032557.19788-4-marek.vasut+renesas@gmail.com> Message-ID: <6ab7ba1d-df1d-6636-28f7-20817c3d9491@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: quoted-printable To: u-boot@lists.denx.de On 3/12/19 8:30 PM, Eugeniu Rosca wrote: > Hi Marek cc: Michael Hi, > On Tue, Mar 5, 2019 at 4:37 AM Marek Vasut wrote: >> >> The ATF can pass additional information via the first four registers, >> x0...x3. The R-Car Gen3 with mainline ATF, register x1 contains pointer >> to a device tree with platform information. Parse this device tree and >> extract DRAM size information from it. This is useful on systems where >> the DRAM size can vary between configurations. >=20 > 1. Do the ATF changes supporting this feature already exist in mainline A= TF? Yes, for quite some time, see commit 1d85c4bd5b6291b99feb2409525c96f0c1744328 plat: rcar: Pass DTB with DRAM layout from BL2 to next stages > 2. I see more DRAM-related compile-time knobs in vanilla ATF: > =E2=9E=9C arm-trusted-firmware (c48d02bade88) $ grep "DRAM.*:" > plat/renesas/rcar/platform.mk > RCAR_DRAM_SPLIT :=3D 0 > RCAR_DRAM_LPDDR4_MEMCONF :=3D1 > RCAR_DRAM_DDR3L_MEMCONF :=3D1 > RCAR_DRAM_DDR3L_MEMDUAL :=3D1 > RCAR_DRAM_CHANNEL :=3D15 >=20 > My understanding is that these are local to ATF and U-Boot can stay > agnostic about them? Yes, although the long term goal is to get rid of those and replace them with DT configuration too. So ATF BL2 would then parse supplied DT to figure out the DRAM layout instead of using those knobs. > IOW, DRAM start and size are totally enough for U-Boot? No, U-Boot needs to know start/size of each bank. That's what's in the DT that is passed in, plus a machine compatible string, so technically, you might be able to build one U-Boot for all boards already, but then you hit a size limit where the BL33 must be below 1 MiB. I'm looking into that one too. --=20 Best regards, Marek Vasut