On 23-10-13, Sascha Hauer wrote: > On Tue, Oct 10, 2023 at 04:33:02PM +0200, Marco Felsch wrote: > > Extend the CSF area to be able two store two CSF slots. The 2nd CSF slot > > is required in case of FlexSPI HAB boot support. The first slot is used > > for the MMC-IVT + barebox-pbl; the 2nd slot is used for the FlexSPI-IVT > > + barebox-pbl. This is necessary because of the fact that the HAB ROM > > code requires that the IVT is signed as well. > > > > Use the CONFIG_SPI_NXP_FLEXSPI Kconfig symbol to decide if we need to > > add the support for FlexSPI image signatures as well since early SRAM > > space is limited and 4K can make a difference here. > > > > Please check the documentation for a more detailed information about the > > i.MX8M image layout. > > > > Signed-off-by: Marco Felsch <m.felsch@xxxxxxxxxxxxxx> > > --- > > arch/arm/lib/pbl.lds.S | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/arch/arm/lib/pbl.lds.S b/arch/arm/lib/pbl.lds.S > > index 898d6be33b54..9ab6d22eb59d 100644 > > --- a/arch/arm/lib/pbl.lds.S > > +++ b/arch/arm/lib/pbl.lds.S > > @@ -12,7 +12,11 @@ > > #define BASE (TEXT_BASE - SZ_2M) > > #endif > > > > +#ifdef CONFIG_SPI_NXP_FLEXSPI > > +#define HAB_CSF_LEN 0x4000 > > +#else > > #define HAB_CSF_LEN 0x2000 > > +#endif > > No. Enabling/Disabling the FlexSPI driver shouldn't influence the image > layout. > > Having the driver enabled doesn't necessarily mean you want to use it > for booting. Having the driver disabled doesn't necessarily mean you are > *not* booting from it. You're right, it's a bit to simple. I will rework that. Regards, Marco > > Sascha > > -- > Pengutronix e.K. | | > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | >
