On 10/14/20 1:29 PM, Biju Das wrote:

Hi,

[...]

>>>> Check with the linux maintainers please, surely there should be some
>>>> way to separate the extras in a way that's not too hard to maintain,
>>>> and thus reduce the resulting binary size. For U-Boot, that is quite
>>>> important already, I think the TFA can only load 1 MiB binary in total.
>>>
>>> I agree for bootloader size is important. So I will add macros as per your
>> suggestion (we don't need to look into linux for this).
>>
>> The PFC tables and clock tables are the same between U-Boot and Linux, so if
>> you only change them in U-Boot, it will make it hard to synchronize the 
>> tables
>> later with Linux again. Please fix this in Linux and synchronize to U-Boot.
> 
> I have posted a patch for optimizing pin control size for RZ/G2N in Linux [1]
> 
> [1] 
> https://patchwork.kernel.org/project/linux-renesas-soc/patch/20201014110238.9600-1-biju.das...@bp.renesas.com/
> 
> This approach will save ~ 6KB=(3x 2KB/SoC) of memory on RZ/G2[HMN] u-boot 
> with multi dtb support. 
> 
> 1) By compiling out Automotive parts
> $ size drivers/pinctrl/renesas/pfc-r8a77965.o
>    text          data     bss     dec     hex filename
>   46141             0       0   46141    b43d 
> drivers/pinctrl/renesas/pfc-r8a77965.o
> 
> 2) without patch
> $ size drivers/pinctrl/renesas/pfc-r8a77965.o
>    text          data     bss     dec     hex filename
>   48191             0       0   48191    bc3f 
> drivers/pinctrl/renesas/pfc-r8a77965.o

Have a look at the size of the image of rcar3_salvator-x_defconfig , it
is just a few kiB short of 1MiB , which is the hard limit. Any size
reduction helps.

Reply via email to