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 lokeshvutla@ti.com
Reviewed-by: Suman Anna s-anna@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"