[PATCH v2 1/2] cmd: source: Use script from default config

[Sven Schwermer]

Hi Tom,

Thanks for the pointers. I understand your concerns about the growth in 
size. I'll look into it.

Best regards,
Sven

On 1/24/22 19:08, Tom Rini wrote:
> On Sat, Jan 22, 2022 at 09:36:58PM +0100, Sven Schwermer wrote:
> 
>> Hi Tom,
>>
>> I didn't think this would result in a size increase. Could you elaborate?
> 
> Alright.  So for background, if you build with tools/buildman/buildman
> rather than just "make" directly there's some handy tools like "bloat"
> detection, which shows function size changes from build A to build B.
> Doing something like this will show the changes for pine64_plus for
> example
> export SOURCE_DATE_EPOCH=`date +%s`
> ./tools/buildman/buildman -o /tmp/pine64_plus -devl -SBC pine64_plus
> ./tools/buildman/buildman -o /tmp/pine64_plus -devl -SsB pine64_plus
> 
> That shows in the end:
>     aarch64: (for 1/1 boards) all +551.0 rodata +107.0 text +444.0
>              pine64_plus    : all +551 rodata +107 text +444
>                 u-boot: add: 2/0, grow: 0/-1 bytes: 452/-8 (444)
>                   function                                   old     new   delta
>                   fit_find_config_node                         -     264    +264
>                   board_fit_config_name_match                  -     188    +188
>                   image_source_script                        520     512      -8
> 
> And it's that growth, over 722 configs, that I'm concerned with.
>