[PATCH 13/14] spl: Create a function to init spl_load_info

Simon Glass sjg at chromium.org
Sun Jul 21 12:08:36 CEST 2024


Hi Sean,

On Sat, 20 Jul 2024 at 17:11, Sean Anderson <seanga2 at gmail.com> wrote:
>
> On 7/20/24 02:17, Simon Glass wrote:
> > Rather than having every caller set this up individually, create a
> > common init function. This allows new fields to be added without the
> > risk of them being left uninited.
> >
>
> What is the code size impact of this?

I don't see any, e.g. for socfpga_n5x_atf (which enabled
SPL_FS_FAT)and firefly-rk3288

>
> IIRC the reason why I didn't do this is to avoid growing the code in
> cases where priv is unused.

>
> Although in some places there are memsets which should be removed anyway.

Yes, that might be helping.

Regards,
SImon


More information about the U-Boot mailing list