Re: [PATCH 1/3] board: ti: am62x: Kconfig: Add ENV_SOIRCE_FILE for am62x

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.