Hi Thomas, On Sat, 21 Apr 2018 16:18:59 +0200, Thomas Petazzoni <thomas.petazzoni@xxxxxxxxxxx> wrote: > Hello, > > On Sat, 21 Apr 2018 16:03:42 +0200, Miquel Raynal wrote: > > From: Victor Gu <xigu@xxxxxxxxxxx> > > > > 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@xxxxxxxxxxx> > > [miquel.raynal@xxxxxxxxxxx: reword of commit message] > > Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx> > > 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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
