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-bha...@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

Reply via email to