Hi Sean, On Sat, 20 Jul 2024 at 17:11, Sean Anderson <sean...@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