On 8/6/20 8:49 AM, Suman Anna wrote: > Hi Lokesh, > > On 8/5/20 12:14 PM, Lokesh Vutla wrote: >> j7200-evm has minor differences with j721e-evm based on the IPs >> available in the SoC. Introduce separate build targets for j7200-evm >> to incorporate the differences. >> >> Signed-off-by: Lokesh Vutla <lokeshvu...@ti.com> > > Reviewed-by: Suman Anna <s-a...@ti.com> > >> --- >> board/ti/j721e/Kconfig | 53 +++++++++++++++++++++++++++++++++++++ >> include/configs/j721e_evm.h | 4 +-- >> 2 files changed, 55 insertions(+), 2 deletions(-) >> >> diff --git a/board/ti/j721e/Kconfig b/board/ti/j721e/Kconfig >> index e56dc53bfa..2cbe2b2481 100644 >> --- a/board/ti/j721e/Kconfig >> +++ b/board/ti/j721e/Kconfig >> @@ -27,6 +27,26 @@ config TARGET_J721E_R5_EVM >> imply SYS_K3_SPL_ATF >> imply TI_I2C_BOARD_DETECT >> >> +config TARGET_J7200_A72_EVM >> + bool "TI K3 based J7200 EVM running on A72" >> + select ARM64 >> + select SOC_K3_J721E >> + select BOARD_LATE_INIT >> + imply TI_I2C_BOARD_DETECT
Btw, do we need to add this also in board/ti/j721e/evm.c in spl_board_init() function for the daughter card probing? regards Suman >> + select SYS_DISABLE_DCACHE_OPS >> + >> +config TARGET_J7200_R5_EVM >> + bool "TI K3 based J7200 EVM running on R5" >> + select CPU_V7R >> + select SYS_THUMB_BUILD >> + select SOC_K3_J721E >> + select K3_LOAD_SYSFW > > I am assuming this is needed only to provide the fallback/legacy-mode > support, which is perfectly fine. > > regards > Suman > >> + select RAM >> + select SPL_RAM >> + select K3_J721E_DDRSS >> + imply SYS_K3_SPL_ATF >> + imply TI_I2C_BOARD_DETECT >> + >> endchoice >> >> if TARGET_J721E_A72_EVM >> @@ -61,3 +81,36 @@ config SPL_LDSCRIPT >> source "board/ti/common/Kconfig" >> >> endif >> + >> +if TARGET_J7200_A72_EVM >> + >> +config SYS_BOARD >> + default "j721e" >> + >> +config SYS_VENDOR >> + default "ti" >> + >> +config SYS_CONFIG_NAME >> + default "j721e_evm" >> + >> +source "board/ti/common/Kconfig" >> + >> +endif >> + >> +if TARGET_J7200_R5_EVM >> + >> +config SYS_BOARD >> + default "j721e" >> + >> +config SYS_VENDOR >> + default "ti" >> + >> +config SYS_CONFIG_NAME >> + default "j721e_evm" >> + >> +config SPL_LDSCRIPT >> + default "arch/arm/mach-omap2/u-boot-spl.lds" >> + >> +source "board/ti/common/Kconfig" >> + >> +endif >> diff --git a/include/configs/j721e_evm.h b/include/configs/j721e_evm.h >> index 7b917c2247..29744e8d9a 100644 >> --- a/include/configs/j721e_evm.h >> +++ b/include/configs/j721e_evm.h >> @@ -20,7 +20,7 @@ >> #define CONFIG_SYS_SDRAM_BASE1 0x880000000 >> >> /* SPL Loader Configuration */ >> -#ifdef CONFIG_TARGET_J721E_A72_EVM >> +#if defined(CONFIG_TARGET_J721E_A72_EVM) || >> defined(CONFIG_TARGET_J7200_A72_EVM) >> #define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SPL_TEXT_BASE + \ >> CONFIG_SYS_K3_NON_SECURE_MSRAM_SIZE) >> /* Image load address in RAM for DFU boot*/ >> @@ -127,7 +127,7 @@ >> DFU_ALT_INFO_RAM \ >> DFU_ALT_INFO_OSPI >> >> -#ifdef CONFIG_TARGET_J721E_A72_EVM >> +#if defined(CONFIG_TARGET_J721E_A72_EVM) || >> defined(CONFIG_TARGET_J7200_A72_EVM) >> #define EXTRA_ENV_J721E_BOARD_SETTINGS_MTD \ >> "mtdids=" CONFIG_MTDIDS_DEFAULT "\0" \ >> "mtdparts=" CONFIG_MTDPARTS_DEFAULT "\0" >> >