Skip to content

Commit

Permalink
tcl/target/ti_k3: Set _CHIPNAME in one place
Browse files Browse the repository at this point in the history
$_soc is set per platform, no point in duplicating _CHIPNAME to
explicitly set the information provided by $_CHIPNAME itself.

So move it out after the check for CHIP_NAME

Signed-off-by: Nishanth Menon <nm@ti.com>
Change-Id: I60d30d6a9a2ce352f66c5bc03075e4ba638e3062
Reviewed-on: https://review.openocd.org/c/openocd/+/7889
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
  • Loading branch information
nmenon authored and borneoa committed Sep 23, 2023
1 parent c20cb28 commit db1bbbf
Showing 1 changed file with 2 additions and 7 deletions.
9 changes: 2 additions & 7 deletions tcl/target/ti_k3.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,6 @@ set _gp_mcu_ap_unlock_offsets {0xf0 0x60}
# Set configuration overrides for each SOC
switch $_soc {
am654 {
set _CHIPNAME am654
set _K3_DAP_TAPID 0x0bb5a02f

# AM654 has 2 clusters of 2 A53 cores each.
Expand All @@ -78,7 +77,6 @@ switch $_soc {
set _sysctrl_ap_unlock_offsets {0xf0 0x50}
}
am642 {
set _CHIPNAME am642
set _K3_DAP_TAPID 0x0bb3802f

# AM642 has 1 clusters of 2 A53 cores each.
Expand All @@ -97,7 +95,6 @@ switch $_soc {
set _gp_mcu_cores 1
}
am625 {
set _CHIPNAME am625
set _K3_DAP_TAPID 0x0bb7e02f

# AM625 has 1 clusters of 4 A53 cores.
Expand Down Expand Up @@ -132,7 +129,6 @@ switch $_soc {
set _dmem_emu_ap_list 1
}
am62a7 {
set _CHIPNAME am62a7
set _K3_DAP_TAPID 0x0bb8d02f

# AM62a7 has 1 clusters of 4 A53 cores.
Expand All @@ -153,7 +149,6 @@ switch $_soc {
set _sysctrl_ap_unlock_offsets {0xf0 0x78}
}
j721e {
set _CHIPNAME j721e
set _K3_DAP_TAPID 0x0bb6402f
# J721E has 1 cluster of 2 A72 cores.
set _armv8_cpu_name a72
Expand All @@ -173,7 +168,6 @@ switch $_soc {
set _dmem_emu_ap_list 1
}
j7200 {
set _CHIPNAME j7200
set _K3_DAP_TAPID 0x0bb6d02f

# J7200 has 1 cluster of 2 A72 cores.
Expand All @@ -189,7 +183,6 @@ switch $_soc {
set CM3_CTIBASE {0x20001000}
}
j721s2 {
set _CHIPNAME j721s2
set _K3_DAP_TAPID 0x0bb7502f

# J721s2 has 1 cluster of 2 A72 cores.
Expand All @@ -213,6 +206,8 @@ switch $_soc {
}
}

set _CHIPNAME $_soc

swj_newdap $_CHIPNAME cpu -irlen 4 -expected-id $_K3_DAP_TAPID -ignore-version

dap create $_CHIPNAME.dap -chain-position $_CHIPNAME.cpu
Expand Down

0 comments on commit db1bbbf

Please sign in to comment.