On Sat, Jul 08, 2023 at 10:30:47AM +0200, Pali Rohár wrote: > On Friday 07 July 2023 19:43:34 Tom Rini wrote: > > On Sat, Jul 08, 2023 at 12:54:38AM +0200, Pali Rohár wrote: > > > > > To make eMMC partition choosing in default_spl_mmc_emmc_boot_partition() > > > function better understandable, rewrite it via explicit switch-case code > > > pattern. > > > > > > Also indicate an error when eMMC EXT_CSD[179] register is configured by > > > user to value which is not suitable for eMMC booting and SPL do not know > > > how to interpret it. > > > > > > Signed-off-by: Pali Rohár <p...@kernel.org> > > > > I did some quick local testing to check on the size impact and this is > > indeed quite manageable, thanks for reworking things! > > > > Reviewed-by: Tom Rini <tr...@konsulko.com> > > Unfortunately no, sama5d2_xplained_emmc, sama5d2_xplained_mmc and > sama5d2_xplained_qspiflash are failing with v3; but not with v2. > > Seems that these boards have their SPL at limit and adding any new > puts() increase size over the limit. > > So I think that there is no other way than just adding a new config > option to hide this new puts().
We should enable LTO on those platforms then and buy us some space. -- Tom
signature.asc
Description: PGP signature