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

[Sascha Hauer]

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.

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 |