All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm: sc5xx: Enable BOOTSTD_DEFAULTS
@ 2026-08-04 10:43 Uwe Kleine-König
  2026-08-05 22:48 ` Greg Malysa
  0 siblings, 1 reply; 4+ messages in thread
From: Uwe Kleine-König @ 2026-08-04 10:43 UTC (permalink / raw)
  To: Greg Malysa, Vasileios Bimpikas, Arturs Artamonovs
  Cc: Tom Rini, Nathan Barrett-Morrison, Philip Molloy, Utsav Agarwal,
	Ozan Durgut, Caleb Ethridge, linux, u-boot

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 <u.kleine-koenig@baylibre.com>
---
 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


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] arm: sc5xx: Enable BOOTSTD_DEFAULTS
  2026-08-04 10:43 [PATCH] arm: sc5xx: Enable BOOTSTD_DEFAULTS Uwe Kleine-König
@ 2026-08-05 22:48 ` Greg Malysa
  2026-08-06  5:48   ` Uwe Kleine-König
  0 siblings, 1 reply; 4+ messages in thread
From: Greg Malysa @ 2026-08-05 22:48 UTC (permalink / raw)
  To: Uwe Kleine-König
  Cc: Vasileios Bimpikas, Arturs Artamonovs, Tom Rini,
	Nathan Barrett-Morrison, Philip Molloy, Ozan Durgut,
	Caleb Ethridge, linux, u-boot

Hi Uwe,

On Tue, Aug 4, 2026 at 6:43 AM Uwe Kleine-König
<u.kleine-koenig@baylibre.com> 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 <u.kleine-koenig@baylibre.com>
> ---
>  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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] arm: sc5xx: Enable BOOTSTD_DEFAULTS
  2026-08-05 22:48 ` Greg Malysa
@ 2026-08-06  5:48   ` Uwe Kleine-König
  2026-08-06 18:02     ` Greg Malysa
  0 siblings, 1 reply; 4+ messages in thread
From: Uwe Kleine-König @ 2026-08-06  5:48 UTC (permalink / raw)
  To: Greg Malysa
  Cc: Vasileios Bimpikas, Arturs Artamonovs, Tom Rini,
	Nathan Barrett-Morrison, Philip Molloy, Ozan Durgut,
	Caleb Ethridge, linux, u-boot

[-- Attachment #1: Type: text/plain, Size: 2257 bytes --]

Hello,

thanks for your feedback.

On Wed, Aug 05, 2026 at 06:48:56PM -0400, Greg Malysa wrote:
> On Tue, Aug 4, 2026 at 6:43 AM Uwe Kleine-König
> <u.kleine-koenig@baylibre.com> wrote:
> > [...]
> > base-commit: ece349ade2973e220f524ce59e59711cc919263f
> > --
> > 2.55.0.11.g153666a7d9bb
> >
> 
> This patch does not apply to uboot/main or uboot/next, please rebase

Oh, it was based on master, didn't notice there are newer other
branches.

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

I did that because in general I'm annoyed by (vendor) U-Boots that lack
the stuff that is considered good and needed for alternative Linux
installations.

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

My usecase here was to be able to start the installer to create a rootfs
on the eMMC for kernel development on "my" sc598 board.
And I'm using a custom kernel with the Debian rootfs because indeed
mainline doesn't work on these machines yet.

> 3) Why not make the same change to the sc594/sc598 boards as well?

That commit is a port of
https://github.com/analogdevicesinc/u-boot/pull/136 to mainline. I just
modified all non-spl config/sc5* without much thinking.

I will respin this series on a newer base commit and verify the
resulting image fits into flash---at least on the one machine I have on
my desk. I don't know other sc5xx boards, if some of these have a
smaller flash, enabling CONFIG_BOOTSTD_DEFAULTS for these probably
doesn't work.

Best regards
Uwe

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] arm: sc5xx: Enable BOOTSTD_DEFAULTS
  2026-08-06  5:48   ` Uwe Kleine-König
@ 2026-08-06 18:02     ` Greg Malysa
  0 siblings, 0 replies; 4+ messages in thread
From: Greg Malysa @ 2026-08-06 18:02 UTC (permalink / raw)
  To: Uwe Kleine-König
  Cc: Vasileios Bimpikas, Arturs Artamonovs, Tom Rini,
	Nathan Barrett-Morrison, Philip Molloy, Ozan Durgut,
	Caleb Ethridge, linux, u-boot

Hi Uwe,

On Thu, Aug 6, 2026 at 1:48 AM Uwe Kleine-König
<u.kleine-koenig@baylibre.com> wrote:
>
>
> > 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?
>
> My usecase here was to be able to start the installer to create a rootfs
> on the eMMC for kernel development on "my" sc598 board.
> And I'm using a custom kernel with the Debian rootfs because indeed
> mainline doesn't work on these machines yet.

Sounds cool! I've never tried running a non-yocto setup on these boards.

>
> > 3) Why not make the same change to the sc594/sc598 boards as well?
>
> That commit is a port of
> https://github.com/analogdevicesinc/u-boot/pull/136 to mainline. I just
> modified all non-spl config/sc5* without much thinking.
>
> I will respin this series on a newer base commit and verify the
> resulting image fits into flash---at least on the one machine I have on
> my desk. I don't know other sc5xx boards, if some of these have a
> smaller flash, enabling CONFIG_BOOTSTD_DEFAULTS for these probably
> doesn't work.

I think for sc598 and newer this is a good idea. For the older 32-bit
ones I agree with Philip that it is probably not worth it. The 598 som
has much larger flash as well so it isn't a problem, but the old 573
ezkit has 16 MB total that we tried to get uboot, kernel, and rfs
into.

Thanks,
Greg

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2026-08-06 18:03 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-04 10:43 [PATCH] arm: sc5xx: Enable BOOTSTD_DEFAULTS Uwe Kleine-König
2026-08-05 22:48 ` Greg Malysa
2026-08-06  5:48   ` Uwe Kleine-König
2026-08-06 18:02     ` Greg Malysa

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.