[PATCH 14/26] ARM: lds: add support for a 2nd CSF area

[Marco Felsch]

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 at pengutronix.de>
> > ---
> >  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 |
>