[PATCH] configs: ls1012a/ls1046a: Define CONFIG_ENV_OVERWRITE

Priyanka Jain (OSS) priyanka.jain at oss.nxp.com
Tue Jul 21 11:19:21 CEST 2020


>-----Original Message-----
>From: U-Boot <u-boot-bounces at lists.denx.de> On Behalf Of Kuldeep Singh
>Sent: Tuesday, June 30, 2020 3:06 PM
>To: u-boot at lists.denx.de
>Cc: Priyanka Jain <priyanka.jain at nxp.com>; Kuldeep Singh
><kuldeep.singh at nxp.com>
>Subject: [PATCH] configs: ls1012a/ls1046a: Define CONFIG_ENV_OVERWRITE
>
>CONFIG_ENV_OVERWRITE disable write protection on various environment
>variables like "ethaddr" and "serial".
>
>Define the config in LS1046A and LS1012A common header as both of them
>require this entry. Also, resolve an error while setting multiple values of
>"ethaddr" variable.
>
>Before the change:
>=> setenv ethaddr 00:E0:0C:00:06:0
>=> setenv ethaddr 00:E0:0C:00:06:1
>Error: Can't overwrite "ethaddr"
>Error inserting "ethaddr" variable, errno=1
>
>After the change:
>=> setenv ethaddr 00:E0:0C:00:06:0
>=> setenv ethaddr 00:E0:0C:00:06:1
>
>Signed-off-by: Kuldeep Singh <kuldeep.singh at nxp.com>
>---
> include/configs/ls1012a_common.h | 2 ++
>include/configs/ls1046a_common.h | 2 ++
> 2 files changed, 4 insertions(+)
>
>diff --git a/include/configs/ls1012a_common.h
>b/include/configs/ls1012a_common.h
>index 3bea9a9..b7ad303 100644
>--- a/include/configs/ls1012a_common.h
>+++ b/include/configs/ls1012a_common.h
>@@ -15,6 +15,8 @@
>
> #define CONFIG_SKIP_LOWLEVEL_INIT
>
>+#define CONFIG_ENV_OVERWRITE
>+
> #ifdef CONFIG_TFABOOT
> #define CONFIG_SYS_INIT_SP_ADDR                CONFIG_SYS_TEXT_BASE
> #else
>diff --git a/include/configs/ls1046a_common.h
>b/include/configs/ls1046a_common.h
>index 8fe6937..2fdc2d3 100644
>--- a/include/configs/ls1046a_common.h
>+++ b/include/configs/ls1046a_common.h
>@@ -42,6 +42,8 @@
>
> #define CONFIG_SKIP_LOWLEVEL_INIT
>
>+#define CONFIG_ENV_OVERWRITE
>+
> #define CONFIG_VERY_BIG_RAM
> #define CONFIG_SYS_DDR_SDRAM_BASE	0x80000000
> #define CONFIG_SYS_FSL_DDR_SDRAM_BASE_PHY	0
>--
>2.7.4
Can you please check if this change can go into defconfig file, instead of adding here?

Regards
Priyanka


More information about the U-Boot mailing list