[PATCHv5][ 4/7] board: tbs2910_defconfig: Add support for booting general purpose distributions
smoch at web.de
Sat May 30 19:17:49 CEST 2020
On 30.05.20 05:24, Denis 'GNUtoo' Carikli wrote:
> This is needed to enable distro_boot support later on.
> Adding the necessary configuration to enable booting general
> purpose distributions is typically done by enabling
> However the resulting image size is is already very close to
> the size limit and PXE isn't strictly needed, so instead we
> made sure that all the configuration selected by
> CONFIG_DISTRO_DEFAULTS were selected, with the exception of
> PXE related configuration (CONFIG_BOOTP_PXE and
> CONFIG_CMD_PXE) that are both disabled.
> With arm-linux-gnueabi-gcc 9.3.0-1 from the Parabola
> GNU/Linux distribution, we have the following size
> - text: +7526 bytes
> - data: +28 bytes
> - bss: -12 bytes
> - total: +7542 bytes
> Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo at cyberdimension.org>
It is quite uncommon to use "board: tbs2910_defconfig" instead of
"board: tbs2910" in the patch title.But maybe not that important.
Reviewed-by: Soeren Moch <smoch at web.de>
> Changelog since v4:
> - As requested, I Added in the commit message the reason why the
> change is really needed as the short summary is not self
> explanatory enough.
> - Fixed a typo in the commit message
> (resultimg image instead of resulting image).
> - Switched to the use of CONFIG_CMD_SYSBOOT instead of
> CONFIG_DISTRO_BOOTCMD as suggested
> The only changes between the two resulting .config are the following:
> +# CONFIG_DISTRO_DEFAULTS is not set
> +# CONFIG_CMD_PXE is not set
> configs/tbs2910_defconfig | 2 ++
> 1 file changed, 2 insertions(+)
> diff --git a/configs/tbs2910_defconfig b/configs/tbs2910_defconfig
> index d14ff45615..fc9095c859 100644
> --- a/configs/tbs2910_defconfig
> +++ b/configs/tbs2910_defconfig
> @@ -9,6 +9,7 @@ CONFIG_NR_DRAM_BANKS=1
> CONFIG_BOOTCOMMAND="mmc rescan; if run bootcmd_up1; then run bootcmd_up2; else run bootcmd_mmc; fi"
> @@ -43,6 +44,7 @@ CONFIG_CMD_MII=y
More information about the U-Boot