| Message ID | 20230428075336.135516-2-n-jain1@ti.com |
|---|---|
| State | Changes Requested |
| Delegated to: | Tom Rini |
| Headers | show |
| Series | Update env support for Am62x | expand |
On Fri, Apr 28, 2023 at 01:23:34PM +0530, Nikhil M Jain wrote: > Set the base name for am62x to use am62x.env file, for setting > environment variable. > > Signed-off-by: Nikhil M Jain <n-jain1@ti.com> > --- > board/ti/am62x/Kconfig | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/board/ti/am62x/Kconfig b/board/ti/am62x/Kconfig > index 5e8dfa3cc4..b681d8e589 100644 > --- a/board/ti/am62x/Kconfig > +++ b/board/ti/am62x/Kconfig > @@ -34,6 +34,9 @@ config SYS_VENDOR > config SYS_CONFIG_NAME > default "am62x_evm" > > +config ENV_SOURCE_FILE > + default "am62x" > + > source "board/ti/common/Kconfig" > > endif Ugh, I see I didn't catch this on j721* in time. Please set this in the defconfig when needed. If we can't use CONFIG_SYS_BOARD.env for the common one, we should probably think a bit harder on what's named what.
diff --git a/board/ti/am62x/Kconfig b/board/ti/am62x/Kconfig index 5e8dfa3cc4..b681d8e589 100644 --- a/board/ti/am62x/Kconfig +++ b/board/ti/am62x/Kconfig @@ -34,6 +34,9 @@ config SYS_VENDOR config SYS_CONFIG_NAME default "am62x_evm" +config ENV_SOURCE_FILE + default "am62x" + source "board/ti/common/Kconfig" endif @@ -52,6 +55,9 @@ config SYS_CONFIG_NAME config SPL_LDSCRIPT default "arch/arm/mach-omap2/u-boot-spl.lds" +config ENV_SOURCE_FILE + default "am62x" + source "board/ti/common/Kconfig" endif
Set the base name for am62x to use am62x.env file, for setting environment variable. Signed-off-by: Nikhil M Jain <n-jain1@ti.com> --- board/ti/am62x/Kconfig | 6 ++++++ 1 file changed, 6 insertions(+)