* [PATCH] ARM: imx: Drop CONFIG_USE_BOOTCOMMAND=n on i.MX6 DHSOM
@ 2023-07-25 15:24 Marek Vasut
2023-07-25 17:40 ` Tom Rini
2023-09-04 15:18 ` sbabic
0 siblings, 2 replies; 3+ messages in thread
From: Marek Vasut @ 2023-07-25 15:24 UTC (permalink / raw)
To: u-boot
Cc: Marek Vasut, Andreas Geisreiter, Christoph Niedermaier,
Fabio Estevam, Peng Fan, Stefano Babic, u-boot
This board certainly does use default 'run distro_bootcmd' boot command,
make sure this is set in 'bootcmd' variable.
Fixes: 970bf8603b8 ("Convert CONFIG_USE_BOOTCOMMAND et al to Kconfig")
Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Andreas Geisreiter <ageisreiter@dh-electronics.de>
Cc: Christoph Niedermaier <cniedermaier@dh-electronics.com>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Stefano Babic <sbabic@denx.de>
Cc: u-boot@dh-electronics.com
---
configs/dh_imx6_defconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/configs/dh_imx6_defconfig b/configs/dh_imx6_defconfig
index c3f2782f748..79e86330002 100644
--- a/configs/dh_imx6_defconfig
+++ b/configs/dh_imx6_defconfig
@@ -37,7 +37,6 @@ CONFIG_FIT_VERBOSE=y
CONFIG_SPL_FIT=y
CONFIG_DISTRO_DEFAULTS=y
CONFIG_BOOTDELAY=3
-# CONFIG_USE_BOOTCOMMAND is not set
CONFIG_SYS_CONSOLE_OVERWRITE_ROUTINE=y
CONFIG_SYS_SPL_MALLOC=y
CONFIG_SPL_SPI_LOAD=y
--
2.40.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ARM: imx: Drop CONFIG_USE_BOOTCOMMAND=n on i.MX6 DHSOM
2023-07-25 15:24 [PATCH] ARM: imx: Drop CONFIG_USE_BOOTCOMMAND=n on i.MX6 DHSOM Marek Vasut
@ 2023-07-25 17:40 ` Tom Rini
2023-09-04 15:18 ` sbabic
1 sibling, 0 replies; 3+ messages in thread
From: Tom Rini @ 2023-07-25 17:40 UTC (permalink / raw)
To: Marek Vasut
Cc: u-boot, Andreas Geisreiter, Christoph Niedermaier, Fabio Estevam,
Peng Fan, Stefano Babic, u-boot
[-- Attachment #1: Type: text/plain, Size: 801 bytes --]
On Tue, Jul 25, 2023 at 05:24:43PM +0200, Marek Vasut wrote:
> This board certainly does use default 'run distro_bootcmd' boot command,
> make sure this is set in 'bootcmd' variable.
>
> Fixes: 970bf8603b8 ("Convert CONFIG_USE_BOOTCOMMAND et al to Kconfig")
> Signed-off-by: Marek Vasut <marex@denx.de>
This is also:
Fixes: fa2c14676c7c ("configs: Re-sync with CONFIG_DISTRO_DEFAULTS")
As that's where we set CONFIG_USE_BOOTCOMMAND=n but missed that
include/configs/dh_imx6.h was defining CONFIG_BOOTCOMMAND to the default
distro bootcmd. Doing a step back in time to that commit I see that
this was the only case where a platform was in fact setting
CONFIG_BOOTCOMMAND to "run distro_boot" instead of something custom.
Reviewed-by: Tom Rini <trini@konsulko.com>
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] ARM: imx: Drop CONFIG_USE_BOOTCOMMAND=n on i.MX6 DHSOM
2023-07-25 15:24 [PATCH] ARM: imx: Drop CONFIG_USE_BOOTCOMMAND=n on i.MX6 DHSOM Marek Vasut
2023-07-25 17:40 ` Tom Rini
@ 2023-09-04 15:18 ` sbabic
1 sibling, 0 replies; 3+ messages in thread
From: sbabic @ 2023-09-04 15:18 UTC (permalink / raw)
To: Marek Vasut, u-boot
> This board certainly does use default 'run distro_bootcmd' boot command,
> make sure this is set in 'bootcmd' variable.
> Fixes: 970bf8603b8 ("Convert CONFIG_USE_BOOTCOMMAND et al to Kconfig")
> Signed-off-by: Marek Vasut <marex@denx.de>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Applied to u-boot-imx, master, thanks !
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, 82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de
=====================================================================
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-09-04 15:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-07-25 15:24 [PATCH] ARM: imx: Drop CONFIG_USE_BOOTCOMMAND=n on i.MX6 DHSOM Marek Vasut
2023-07-25 17:40 ` Tom Rini
2023-09-04 15:18 ` sbabic
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox