[PATCH] mvebu: x530: Reduce SPL image size

[Chris Packham]

On Wed, May 12, 2021 at 7:18 PM Stefan Roese <sr at denx.de> wrote:
>
> Currently, building U-Boot for x530 fails since the SPL image is too
> big.

Currently? It's working for me on master. Are there upcoming mvebu
changes that push us over the limit?

> This patch reduces the SPL size by changing the following Kconfig
> options:
>
> Enable CONFIG_SPL_TINY_MEMSET
> Disable CONFIG_SPI_FLASH_BAR
>
> By disabling CONFIG_SPI_FLASH_BAR, the tiny SPI NOR framework can be
> used.

I've got a nagging feeling I needed CONFIG_SPI_FLASH_BAR for some
reason. Possibly for some alternate part we were looking at, on the
other hand it could also be from the db88f6820-amc board which I based
the initial defconfig on.

Looking at what CONFIG_SPI_FLASH_BAR does I think we should be safe.
We only fit 16MiB parts and aren't likely to start fitting bigger
parts unless all the spi-nor vendors decide to stop making ones
<16MiB.

>
> Signed-off-by: Stefan Roese <sr at denx.de>
> Cc: Chris Packham <chris.packham at alliedtelesis.co.nz>
> Cc: Pratyush Yadav <p.yadav at ti.com>
> Cc: Tom Rini <trini at konsulko.com>

Tested-by: Chris Packham <chris.packham at alliedtelesis.co.nz>

(yes I'm mixing work and personal email addresses. I can re-send
tomorrow from the office if that's a concern).

> ---
>  configs/x530_defconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/configs/x530_defconfig b/configs/x530_defconfig
> index 890c94b5c1fe..76574c4de4be 100644
> --- a/configs/x530_defconfig
> +++ b/configs/x530_defconfig
> @@ -62,7 +62,6 @@ CONFIG_SYS_NAND_USE_FLASH_BBT=y
>  CONFIG_NAND_PXA3XX=y
>  CONFIG_SF_DEFAULT_BUS=1
>  CONFIG_SF_DEFAULT_SPEED=50000000
> -CONFIG_SPI_FLASH_BAR=y
>  CONFIG_SPI_FLASH_MACRONIX=y
>  CONFIG_SPI_FLASH_STMICRO=y
>  CONFIG_SPI_FLASH_SST=y
> @@ -84,3 +83,4 @@ CONFIG_USB_ETHER_ASIX=y
>  CONFIG_USB_ETHER_ASIX88179=y
>  CONFIG_WDT=y
>  CONFIG_WDT_ORION=y
> +CONFIG_SPL_TINY_MEMSET=y
> --
> 2.31.1
>