Hi Uwe,

On Tue, Aug 4, 2026 at 6:43 AM Uwe Kleine-König
<[email protected]> wrote:
>
> This setting implies various config knobs that are "commonly needed to
> support a good selection of booting methods." Among them is
> CONFIG_SUPPORT_RAW_INITRD which is convenient for booting a Debian
> installer. Enable it for the (non-SPL) sc5xx defconfigs.
>
> The removed lines are only about settings that are (transitively)
> selected by CONFIG_BOOTSTD_DEFAULTS and thus are not needed any more to
> be listed explicitly. CONFIG_ISO_PARTITION is now listed explicitly
> because it defaults to y in the presence of BOOT_DEFAULTS (which is
> selected by BOOTSTD_DEFAULTS).
>
> Signed-off-by: Uwe Kleine-König <[email protected]>
> ---
>  configs/sc573-ezkit_defconfig | 11 ++---------
>  configs/sc584-ezkit_defconfig | 11 ++---------
>  configs/sc589-ezkit_defconfig | 11 ++---------
>  configs/sc589-mini_defconfig  | 11 ++---------
>  4 files changed, 8 insertions(+), 36 deletions(-)
>
> diff --git a/configs/sc573-ezkit_defconfig b/configs/sc573-ezkit_defconfig
> index fc4ce13cc709..086009a23e15 100644
> --- a/configs/sc573-ezkit_defconfig
> +++ b/configs/sc573-ezkit_defconfig
> @@ -18,13 +18,12 @@ CONFIG_CGU0_CLKOUTSEL=7
>  # CONFIG_EFI_LOADER is not set
>  CONFIG_FIT=y
>  CONFIG_FIT_SIGNATURE=y
> +CONFIG_BOOTSTD_DEFAULTS=y
>  CONFIG_LEGACY_IMAGE_FORMAT=y
> -CONFIG_USE_BOOTCOMMAND=y
>  CONFIG_BOOTCOMMAND="run ramboot"
>  CONFIG_SYS_CBSIZE=512
>  # CONFIG_SPL_RAW_IMAGE_SUPPORT is not set
>  CONFIG_SPL_I2C=y
> -CONFIG_CMD_BOOTZ=y
>  # CONFIG_BOOTM_PLAN9 is not set
>  # CONFIG_BOOTM_RTEMS is not set
>  # CONFIG_BOOTM_VXWORKS is not set
> @@ -39,16 +38,11 @@ CONFIG_CMD_MTD=y
>  CONFIG_CMD_SPI=y
>  CONFIG_CMD_USB=y
>  CONFIG_SYS_DISABLE_AUTOLOAD=y
> -CONFIG_CMD_DHCP=y
>  CONFIG_CMD_DNS=y
> -CONFIG_CMD_MII=y
>  # CONFIG_CMD_MDIO is not set
> -CONFIG_CMD_PING=y
>  CONFIG_CMD_WGET=y
> -CONFIG_CMD_EXT4=y
>  CONFIG_CMD_EXT4_WRITE=y
> -CONFIG_CMD_FAT=y
> -CONFIG_CMD_FS_GENERIC=y
> +# CONFIG_ISO_PARTITION is not set
>  CONFIG_SPL_OF_CONTROL=y
>  CONFIG_OF_EMBED=y
>  CONFIG_ENV_OVERWRITE=y
> @@ -87,6 +81,5 @@ CONFIG_USB=y
>  CONFIG_USB_MUSB_HOST=y
>  CONFIG_USB_MUSB_SC5XX=y
>  CONFIG_USB_MUSB_PIO_ONLY=y
> -CONFIG_USB_STORAGE=y
>  CONFIG_FAT_WRITE=y
>  # CONFIG_REGEX is not set
> diff --git a/configs/sc584-ezkit_defconfig b/configs/sc584-ezkit_defconfig
> index a8c6b9f1224b..161c9ecda038 100644
> --- a/configs/sc584-ezkit_defconfig
> +++ b/configs/sc584-ezkit_defconfig
> @@ -23,14 +23,13 @@ CONFIG_CGU0_CLKOUTSEL=7
>  # CONFIG_EFI_LOADER is not set
>  CONFIG_FIT=y
>  CONFIG_FIT_SIGNATURE=y
> +CONFIG_BOOTSTD_DEFAULTS=y
>  CONFIG_LEGACY_IMAGE_FORMAT=y
> -CONFIG_USE_BOOTCOMMAND=y
>  CONFIG_BOOTCOMMAND="run ramboot"
>  CONFIG_SYS_CBSIZE=512
>  CONFIG_CYCLIC_MAX_CPU_TIME_US=1000
>  # CONFIG_SPL_RAW_IMAGE_SUPPORT is not set
>  CONFIG_SPL_I2C=y
> -CONFIG_CMD_BOOTZ=y
>  # CONFIG_BOOTM_PLAN9 is not set
>  # CONFIG_BOOTM_RTEMS is not set
>  # CONFIG_BOOTM_VXWORKS is not set
> @@ -44,16 +43,11 @@ CONFIG_CMD_MTD=y
>  CONFIG_CMD_SPI=y
>  CONFIG_CMD_USB=y
>  CONFIG_SYS_DISABLE_AUTOLOAD=y
> -CONFIG_CMD_DHCP=y
>  CONFIG_CMD_DNS=y
> -CONFIG_CMD_MII=y
>  # CONFIG_CMD_MDIO is not set
> -CONFIG_CMD_PING=y
>  CONFIG_CMD_WGET=y
> -CONFIG_CMD_EXT4=y
>  CONFIG_CMD_EXT4_WRITE=y
> -CONFIG_CMD_FAT=y
> -CONFIG_CMD_FS_GENERIC=y
> +# CONFIG_ISO_PARTITION is not set
>  CONFIG_SPL_OF_CONTROL=y
>  CONFIG_OF_EMBED=y
>  CONFIG_ENV_OVERWRITE=y
> @@ -91,7 +85,6 @@ CONFIG_USB=y
>  CONFIG_USB_MUSB_HOST=y
>  CONFIG_USB_MUSB_SC5XX=y
>  CONFIG_USB_MUSB_PIO_ONLY=y
> -CONFIG_USB_STORAGE=y
>  CONFIG_FAT_WRITE=y
>  # CONFIG_REGEX is not set
>  # CONFIG_SPL_CRC8 is not set
> diff --git a/configs/sc589-ezkit_defconfig b/configs/sc589-ezkit_defconfig
> index 9cd204e40abb..5b56a0bef9be 100644
> --- a/configs/sc589-ezkit_defconfig
> +++ b/configs/sc589-ezkit_defconfig
> @@ -26,14 +26,13 @@ CONFIG_CGU0_CLKOUTSEL=7
>  # CONFIG_EFI_LOADER is not set
>  CONFIG_FIT=y
>  CONFIG_FIT_SIGNATURE=y
> +CONFIG_BOOTSTD_DEFAULTS=y
>  CONFIG_LEGACY_IMAGE_FORMAT=y
> -CONFIG_USE_BOOTCOMMAND=y
>  CONFIG_BOOTCOMMAND="run ramboot"
>  CONFIG_SYS_CBSIZE=512
>  CONFIG_CYCLIC_MAX_CPU_TIME_US=1000
>  # CONFIG_SPL_RAW_IMAGE_SUPPORT is not set
>  CONFIG_SPL_I2C=y
> -CONFIG_CMD_BOOTZ=y
>  # CONFIG_BOOTM_PLAN9 is not set
>  # CONFIG_BOOTM_RTEMS is not set
>  # CONFIG_BOOTM_VXWORKS is not set
> @@ -48,16 +47,11 @@ CONFIG_CMD_MTD=y
>  CONFIG_CMD_SPI=y
>  CONFIG_CMD_USB=y
>  CONFIG_SYS_DISABLE_AUTOLOAD=y
> -CONFIG_CMD_DHCP=y
>  CONFIG_CMD_DNS=y
> -CONFIG_CMD_MII=y
>  # CONFIG_CMD_MDIO is not set
> -CONFIG_CMD_PING=y
>  CONFIG_CMD_WGET=y
> -CONFIG_CMD_EXT4=y
>  CONFIG_CMD_EXT4_WRITE=y
> -CONFIG_CMD_FAT=y
> -CONFIG_CMD_FS_GENERIC=y
> +# CONFIG_ISO_PARTITION is not set
>  CONFIG_SPL_OF_CONTROL=y
>  CONFIG_OF_EMBED=y
>  CONFIG_ENV_OVERWRITE=y
> @@ -97,7 +91,6 @@ CONFIG_USB=y
>  CONFIG_USB_MUSB_HOST=y
>  CONFIG_USB_MUSB_SC5XX=y
>  CONFIG_USB_MUSB_PIO_ONLY=y
> -CONFIG_USB_STORAGE=y
>  CONFIG_FAT_WRITE=y
>  # CONFIG_REGEX is not set
>  # CONFIG_SPL_CRC8 is not set
> diff --git a/configs/sc589-mini_defconfig b/configs/sc589-mini_defconfig
> index 0908ab1708b3..6c6dc805e606 100644
> --- a/configs/sc589-mini_defconfig
> +++ b/configs/sc589-mini_defconfig
> @@ -24,13 +24,12 @@ CONFIG_ADI_USE_DMC1=y
>  # CONFIG_EFI_LOADER is not set
>  CONFIG_FIT=y
>  CONFIG_FIT_SIGNATURE=y
> +CONFIG_BOOTSTD_DEFAULTS=y
>  CONFIG_LEGACY_IMAGE_FORMAT=y
> -CONFIG_USE_BOOTCOMMAND=y
>  CONFIG_BOOTCOMMAND="run ramboot"
>  CONFIG_SYS_CBSIZE=512
>  # CONFIG_SPL_RAW_IMAGE_SUPPORT is not set
>  CONFIG_SPL_I2C=y
> -CONFIG_CMD_BOOTZ=y
>  # CONFIG_BOOTM_PLAN9 is not set
>  # CONFIG_BOOTM_RTEMS is not set
>  # CONFIG_BOOTM_VXWORKS is not set
> @@ -45,16 +44,11 @@ CONFIG_CMD_MTD=y
>  CONFIG_CMD_SPI=y
>  CONFIG_CMD_USB=y
>  CONFIG_SYS_DISABLE_AUTOLOAD=y
> -CONFIG_CMD_DHCP=y
>  CONFIG_CMD_DNS=y
> -CONFIG_CMD_MII=y
>  # CONFIG_CMD_MDIO is not set
> -CONFIG_CMD_PING=y
>  CONFIG_CMD_WGET=y
> -CONFIG_CMD_EXT4=y
>  CONFIG_CMD_EXT4_WRITE=y
> -CONFIG_CMD_FAT=y
> -CONFIG_CMD_FS_GENERIC=y
> +# CONFIG_ISO_PARTITION is not set
>  CONFIG_SPL_OF_CONTROL=y
>  CONFIG_OF_EMBED=y
>  CONFIG_ENV_OVERWRITE=y
> @@ -94,6 +88,5 @@ CONFIG_USB=y
>  CONFIG_USB_MUSB_HOST=y
>  CONFIG_USB_MUSB_SC5XX=y
>  CONFIG_USB_MUSB_PIO_ONLY=y
> -CONFIG_USB_STORAGE=y
>  CONFIG_FAT_WRITE=y
>  # CONFIG_REGEX is not set
>
> base-commit: ece349ade2973e220f524ce59e59711cc919263f
> --
> 2.55.0.11.g153666a7d9bb
>

This patch does not apply to uboot/main or uboot/next, please rebase
it and resubmit. Note that the sc573-ezkit has been renamed to the
sc573-ezlite since the base commit referenced here, and another series
has modified things like the default bootcmd.

I have additional questions however:
1) This enables things like CONFIG_HUSH_PARSER, CONFIG_BOOTP_PXE, and
others as well which increase the size of uboot on an already tightly
packed spi flash. Are these needed or intentional? Why not enable
CONFIG_SUPPORT_RAW_INITRD individually?
2) How do you intend to use a debian image on these boards? The kernel
does not currently have mainline support or drivers, and the SPI flash
memory is likely too small for a debian image, as it required
extensive tweaking in yocto to build root filesystems that would fit.
Is it going to use exclusively NFS as the root filesystem?
3) Why not make the same change to the sc594/sc598 boards as well?

Thanks,
Greg

Reply via email to