[PATCH] mvebu: x530: Reduce SPL image size
sr at denx.de
Wed May 12 10:06:44 CEST 2021
On 12.05.21 09:59, Chris Packham wrote:
> 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
> Currently? It's working for me on master. Are there upcoming mvebu
> changes that push us over the limit?
Yes, I'm working on a pull request right now. Some enhancements to the
SPI driver are most likely responsible for the code increase.
>> This patch reduces the SPL size by changing the following Kconfig
>> Enable CONFIG_SPL_TINY_MEMSET
>> Disable CONFIG_SPI_FLASH_BAR
>> By disabling CONFIG_SPI_FLASH_BAR, the tiny SPI NOR framework can be
> 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
Please note that there are better ways to handle the > 16MiB SPI
NOR issue. E.g. by using 3 vs 4 byte commands and Serial Flash
Discoverable Parameters (SFDP) parsing, which is supported in
U-Boot since quite some time. I did not use CONFIG_SPI_FLASH_BAR
for a longer time now IIRC.
>> 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).
This is okay for me.
Thanks for the quick review and test.
>> 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
>> @@ -84,3 +83,4 @@ CONFIG_USB_ETHER_ASIX=y
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr at denx.de
More information about the U-Boot