[RFC PATCH v2 3/5] arch: mach-k3: Introduce support for AM62D2-EVM

[Paresh Bhagat]

Hi Andrew,


On 09/09/25 19:24, Andrew Davis wrote:
> On 9/9/25 12:34 AM, Paresh Bhagat wrote:
>> Introduce the basic functions and definitions needed to properly
>
> What "functions"?


Thanks i will improve on the description.
>
>> initialize TI's AM62D2-EVM.
>>
>> Signed-off-by: Paresh Bhagat <p-bhagat at ti.com>
>> ---
>>   arch/arm/mach-k3/am62ax/Kconfig | 25 +++++++++++++++++++++++++
>>   1 file changed, 25 insertions(+)
>>
>> diff --git a/arch/arm/mach-k3/am62ax/Kconfig 
>> b/arch/arm/mach-k3/am62ax/Kconfig
>> index f8cdcdca57a..e7ab299fe5d 100644
>> --- a/arch/arm/mach-k3/am62ax/Kconfig
>> +++ b/arch/arm/mach-k3/am62ax/Kconfig
>> @@ -50,9 +50,34 @@ config TARGET_PHYCORE_AM62AX_R5
>>       select BINMAN
>>       imply SYS_K3_SPL_ATF
>>   +config TARGET_AM62D2_A53_EVM
>> +    bool "TI K3 based AM62D2 EVM running on A53"
>> +    select ARM64
>> +    select BINMAN
>> +    imply BOARD
>> +    imply SPL_BOARD
>> +    imply TI_I2C_BOARD_DETECT
>> +
>> +config TARGET_AM62D2_R5_EVM
>> +    bool "TI K3 based AM62D2 EVM running on R5"
>> +    select CPU_V7R
>> +    select SYS_THUMB_BUILD
>> +    select K3_LOAD_SYSFW
>> +    select RAM
>> +    select SPL_RAM
>> +    select K3_DDRSS
>> +    select BINMAN
>> +    imply SYS_K3_SPL_ATF
>> +    imply TI_I2C_BOARD_DETECT
>> +
>>   endchoice
>>   +if TARGET_AM62A7_A53_EVM || TARGET_AM62A7_R5_EVM
>
> The Phycore boards are not our EVMs, you have them guarded by this
> new if block. Why have these if block at all? The Kconfigs below
> have the if checks already inside.
>
> Andrew


Thanks i missed it. will correct this in next revision.
>
>>   source "board/ti/am62ax/Kconfig"
>>   source "board/phytec/phycore_am62ax/Kconfig"
>> +endif
>>   +if TARGET_AM62D2_A53_EVM || TARGET_AM62D2_R5_EVM
>> +source "board/ti/am62dx/Kconfig"
>> +endif
>>   endif
>