[PATCH] arm64: dts: marvell: armada-37xx: reserve memory for ATF

[Miquel Raynal]

Hi Thomas,

On Sat, 21 Apr 2018 16:18:59 +0200, Thomas Petazzoni
<thomas.petazzoni at bootlin.com> wrote:

> Hello,
> 
> On Sat, 21 Apr 2018 16:03:42 +0200, Miquel Raynal wrote:
> > From: Victor Gu <xigu at marvell.com>
> > 
> > The PSCI area should be reserved in Linux for PSCI operations such as
> > suspend/resume.
> > 
> > Reserve 2MiB of memory which matches the area used by ATF (BL1, BL2,
> > BL3x, see [1] in ATF source code). This covers all PSCI code and data
> > area and is 2MiB aligned, which is required by Linux for huge pages
> > handling.
> > 
> > [1] plat/marvell/a3700/common/include/platform_def.h
> > 
> > Signed-off-by: Victor Gu <xigu at marvell.com>
> > [miquel.raynal at bootlin.com: reword of commit message]
> > Signed-off-by: Miquel Raynal <miquel.raynal at bootlin.com>  
> 
> Shouldn't this be done automatically by the bootloader, before passing
> the DTB to the kernel ?

I am working with a 2017.09 mainline U-Boot.

I did not know the bootloader was supposedly in charge of that. But
what if it fails doing it? Fixing the bootloader is one thing, I assume
very few people would update it. Is it worth keeping this in Linux DT?

Thanks,
Miquèl

-- 
Miquel Raynal, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com