[PATCH v3][ 4/6] board: tbs2910: enable CONFIG_DISTRO_DEFAULTS

Soeren Moch smoch at web.de
Sat Feb 8 16:47:08 CET 2020

On 08.02.20 06:38, Denis 'GNUtoo' Carikli wrote:
> The side effect is that it increase the size of the
> resultimg image, which is already very close to the
> size limit.
> With arm-linux-gnueabi-gcc 9.2.0-1 from the Parabola
> GNU/Linux distribution, we have the following size
> increase:
> - text: 8744 bytes
> - data: 132 bytes
> - bss: 60 bytes
> - total: 8936 bytes
> Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo at cyberdimension.org>
Thank you very much for splitting up the patches and for your thorough
size analysis. This makes it very easy to understand the size
contributions of the several parts.

It was really surprising for me that CONFIG_DISTRO_DEFAULTS brings so
much additional text. I thought this would only enable
CONFIG_CMD_SYSBOOT, but we also get PXE support. I wasn't aware of this

As you already noticed, we are quite close to the size limit of the
u-boot binary. So I would prefer to omit PXE.

Do you really want to use PXE boot on this board? Or would it be OK to
only enable CMD_SYSBOOT instead of DISTRO_DEFAULTS (and remove DHCP and
PXE boot targets in patch 5/6)?

Sorry for not bringing up this earlier,

More information about the U-Boot mailing list