[U-Boot] [PATCH v2 0/5] Patches to reduce TPL code size

Simon Glass
2017-04-09T19:27:19Z

Hi Heiko,

On 4 April 2017 at 04:43, Heiko Stuebner <heiko@sntech.de> wrote:
> Am Dienstag, 4. April 2017, 11:34:52 CEST schrieb Heiko StÃ¼bner:
>> Am Sonntag, 2. April 2017, 09:50:27 CEST schrieb Simon Glass:
>> > With the rockchip 'rock' board some build and code size problems have come
>> > to light with TPL. This series provides a few ideas to improve things.
>>
>> great stuff!
>>
>> With these patches applied, rk3188-rock still boots and the TPL has
>> come down to 616byte on gcc-4.9 and 592bytes on gcc-6.3, so
>
> Actually, after finding out that I should add a
> # CONFIG_TPL_DM_SERIAL is not set
> to my defconfig, the size goes down even more - to 488 bytes on both
> gcc-4.9 and gcc-6.3.
>
> Still works and all.

OK great thanks for the report.

Regards,
Simon

>
>
>> Tested-by: Heiko Stuebner <heiko@sntech.de>
>>
>> We have like vast amounts of free space in tpl now ;-)
>>
>> I guess I should fold your TINY_MEMSET option into my rock board,
>> once you've applied the core patch?
>>
>>
>> Heiko
>>
>>
>> >
>> > Changes in v2:
>> > - Adjust the option to be SPL-only
>> > - Change the option to default to off (name it CONFIG_SPL_TINY_MEMSET)
>> > - Add a new patch to enable CONFIG_SPL_TINY_MEMSET
>> > - Add new patch to allow driver model to be disabled for TPL
>> > - Add new patch to allow driver-model serial to be disabled for TPL
>> >
>> > Simon Glass (5):
>> > string: Provide a slimmed-down memset()
>> > rockchip: rock: Enable CONFIG_SPL_TINY_MEMSET
>> > Makefile: Provide an option to select SPL or TPL
>> > dm: core: Allow driver model to be disabled for TPL
>> > dm: serial: Allow driver-model serial to be disabled for TPL
>> >
>> > configs/rock_defconfig | 1 +
>> > drivers/Makefile | 2 +-
>> > drivers/core/Kconfig | 14 ++++++++++++++
>> > drivers/serial/Kconfig | 20 ++++++++++++++++++++
>> > drivers/serial/Makefile | 2 +-
>> > lib/Kconfig | 8 ++++++++
>> > lib/string.c | 6 ++++--
>> > scripts/Kbuild.include | 6 ++++++
>> > scripts/Makefile.spl | 6 ++++++
>> > 9 files changed, 61 insertions(+), 4 deletions(-)
>>
>>
>
>