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

[Soeren Moch]

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
before.

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,
Soeren