diff mbox series

[v1,2/3] xea: config: Use CONFIG_PREBOOT from Kconfig

Message ID 20201226000902.23020-3-lukma@denx.de
State Accepted
Delegated to: Tom Rini
Headers show
Series arm: xea: Regression fixes for i.MX28 based XEA board | expand

Commit Message

Lukasz Majewski Dec. 26, 2020, 12:09 a.m. UTC
The usage of the preboot feature is now controlled via a separate Kconfig
option - namely CONFIG_USE_PREBOOT.
It must be enabled for preboot code executing commands now defined in
CONFIG_PREBOOT (also moved to the Kconfig).

After defining both CONFIG_USE_PREBOOT and CONFIG_PREBOOT in
imx28_xea_defconfig the define of CONFIG_PREBOOT shall be removed from
xea.h as it is redundant.

Signed-off-by: Lukasz Majewski <lukma@denx.de>
---

 configs/imx28_xea_defconfig | 2 ++
 include/configs/xea.h       | 1 -
 2 files changed, 2 insertions(+), 1 deletion(-)

Comments

Tom Rini Jan. 8, 2021, 3:58 p.m. UTC | #1
On Sat, Dec 26, 2020 at 01:09:01AM +0100, Lukasz Majewski wrote:

> The usage of the preboot feature is now controlled via a separate Kconfig
> option - namely CONFIG_USE_PREBOOT.
> It must be enabled for preboot code executing commands now defined in
> CONFIG_PREBOOT (also moved to the Kconfig).
> 
> After defining both CONFIG_USE_PREBOOT and CONFIG_PREBOOT in
> imx28_xea_defconfig the define of CONFIG_PREBOOT shall be removed from
> xea.h as it is redundant.
> 
> Signed-off-by: Lukasz Majewski <lukma@denx.de>

Applied to u-boot/master, thanks!
Tom Rini Jan. 8, 2021, 3:58 p.m. UTC | #2
On Sat, Dec 26, 2020 at 01:09:01AM +0100, Lukasz Majewski wrote:

> The usage of the preboot feature is now controlled via a separate Kconfig
> option - namely CONFIG_USE_PREBOOT.
> It must be enabled for preboot code executing commands now defined in
> CONFIG_PREBOOT (also moved to the Kconfig).
> 
> After defining both CONFIG_USE_PREBOOT and CONFIG_PREBOOT in
> imx28_xea_defconfig the define of CONFIG_PREBOOT shall be removed from
> xea.h as it is redundant.
> 
> Signed-off-by: Lukasz Majewski <lukma@denx.de>

Applied to u-boot/master, thanks!
diff mbox series

Patch

diff --git a/configs/imx28_xea_defconfig b/configs/imx28_xea_defconfig
index dc42e30a1f..e418d8c9e9 100644
--- a/configs/imx28_xea_defconfig
+++ b/configs/imx28_xea_defconfig
@@ -25,6 +25,8 @@  CONFIG_FIT=y
 CONFIG_OF_BOARD_SETUP=y
 CONFIG_USE_BOOTARGS=y
 CONFIG_BOOTARGS="console=ttyAMA0,115200n8"
+CONFIG_USE_PREBOOT=y
+CONFIG_PREBOOT="run prebootcmd"
 CONFIG_BOARD_EARLY_INIT_F=y
 CONFIG_SPL_BOARD_INIT=y
 # CONFIG_SPL_RAW_IMAGE_SUPPORT is not set
diff --git a/include/configs/xea.h b/include/configs/xea.h
index df058193d6..1207f75017 100644
--- a/include/configs/xea.h
+++ b/include/configs/xea.h
@@ -47,7 +47,6 @@ 
 #define CONFIG_SYS_LOAD_ADDR	CONFIG_LOADADDR
 
 /* Extra Environment */
-#define CONFIG_PREBOOT		"run prebootcmd"
 #define CONFIG_HOSTNAME		"xea"
 
 #define CONFIG_EXTRA_ENV_SETTINGS					\