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@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