Re: [PATCH v2 12/19] board: ti: j7200: Introduce support for j7200 build targets
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 > > Reviewed-by: Suman Anna > >> --- >> 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 0x88000 >> >> /* 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" >> >
Re: [PATCH v2 12/19] board: ti: j7200: Introduce support for j7200 build targets
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 Reviewed-by: Suman Anna > --- > 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 > + 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 0x88000 > > /* 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" >
[PATCH v2 12/19] board: ti: j7200: Introduce support for j7200 build targets
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 --- 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 + 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 + 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 0x88000 /* 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" -- 2.23.0