[PATCH] configs: ls1012a/ls1046a: Define CONFIG_ENV_OVERWRITE

[Kuldeep Singh]

> -----Original Message-----
> From: Priyanka Jain (OSS) <priyanka.jain at oss.nxp.com>
> Sent: Tuesday, July 21, 2020 2:49 PM
> To: Kuldeep Singh <kuldeep.singh at nxp.com>; u-boot at lists.denx.de
> Cc: Kuldeep Singh <kuldeep.singh at nxp.com>
> Subject: RE: [PATCH] configs: ls1012a/ls1046a: Define
> CONFIG_ENV_OVERWRITE
> 
> >-----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?

This will require conversion of CONFIG_ENV_OVERWRITE to Kconfig option to add and entry in defconfig.

-Kuldeep