[PATCH]: 47447fdf4 tcl/target: Set target to aarch64 for Cortex-R52

[gerrit]

This is an automated email from Gerrit.

"Julien Massot <julien.mas...@iot.bzh>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/6805

-- gerrit

commit 47447fdf4dda90490926eaa1601967a616eb71c4
Author: Julien Massot <julien.mas...@iot.bzh>
Date:   Wed Jan 12 09:53:06 2022 +0100

    tcl/target: Set target to aarch64 for Cortex-R52
    
    Cortex-R52 is an ARMv8-R processor supporting only
    AArch32 Profile.
    
    This is nicely supported by OpenOCD with the aarch64
    target.
    
    Signed-off-by: Julien Massot <julien.mas...@iot.bzh>
    Change-Id: I663ae4bf1d3026d7c9e4c5950a79e7ddf1bd6564

diff --git a/tcl/target/renesas_rcar_gen3.cfg b/tcl/target/renesas_rcar_gen3.cfg
index 334d25568..9d64a1b29 100644
--- a/tcl/target/renesas_rcar_gen3.cfg
+++ b/tcl/target/renesas_rcar_gen3.cfg
@@ -154,15 +154,20 @@ proc setup_a5x {core_name dbgbase ctibase num boot} {
        }
 }
 
-proc setup_cr7 {core_name dbgbase ctibase num boot} {
+proc setup_crx {core_name dbgbase ctibase num boot} {
        global _CHIPNAME
        global _DAPNAME
        for { set _core 0 } { $_core < $num } { incr _core } {
                set _TARGETNAME $_CHIPNAME.$core_name
                set _CTINAME $_TARGETNAME.cti
                cti create $_CTINAME -dap $_DAPNAME -ap-num 1 -baseaddr $ctibase
-               set _command "target create $_TARGETNAME cortex_r4 -dap 
$_DAPNAME \
-                       -ap-num 1 -dbgbase $dbgbase"
+               if { $core_name == "r52" } {
+                       set _command "target create $_TARGETNAME aarch64 -dap 
$_DAPNAME \
+                               -ap-num 1 -dbgbase $dbgbase -cti $_CTINAME"
+               } else {
+                       set _command "target create $_TARGETNAME cortex_r4 -dap 
$_DAPNAME \
+                               -ap-num 1 -dbgbase $dbgbase"
+               }
                if { $boot == 1 } {
                        set _targets "$_TARGETNAME"
                } else {
@@ -175,20 +180,20 @@ proc setup_cr7 {core_name dbgbase ctibase num boot} {
 # Organize target list based on the boot core
 if { [string equal $_boot_core CA76] } {
        setup_a5x a76 $CA76_DBGBASE $CA76_CTIBASE $_num_ca76 1
-       setup_cr7 r52 $CR52_DBGBASE $CR52_CTIBASE $_num_cr52 0
+       setup_crx r52 $CR52_DBGBASE $CR52_CTIBASE $_num_cr52 0
 } elseif { [string equal $_boot_core CA57] } {
        setup_a5x a57 $CA57_DBGBASE $CA57_CTIBASE $_num_ca57 1
        setup_a5x a53 $CA53_DBGBASE $CA53_CTIBASE $_num_ca53 0
-       setup_cr7 r7  $CR7_DBGBASE  $CR7_CTIBASE  $_num_cr7  0
+       setup_crx r7  $CR7_DBGBASE  $CR7_CTIBASE  $_num_cr7  0
 } elseif { [string equal $_boot_core CA53] } {
        setup_a5x a53 $CA53_DBGBASE $CA53_CTIBASE $_num_ca53 1
        setup_a5x a57 $CA57_DBGBASE $CA57_CTIBASE $_num_ca57 0
-       setup_cr7 r7  $CR7_DBGBASE  $CR7_CTIBASE  $_num_cr7  0
+       setup_crx r7  $CR7_DBGBASE  $CR7_CTIBASE  $_num_cr7  0
 } elseif { [string equal $_boot_core CR52] } {
-       setup_cr7 r52 $CR52_DBGBASE $CR52_CTIBASE $_num_cr52 1
+       setup_crx r52 $CR52_DBGBASE $CR52_CTIBASE $_num_cr52 1
        setup_a5x a76 $CA76_DBGBASE $CA76_CTIBASE $_num_ca76 0
 } else {
-       setup_cr7 r7  $CR7_DBGBASE  $CR7_CTIBASE  $_num_cr7  1
+       setup_crx r7  $CR7_DBGBASE  $CR7_CTIBASE  $_num_cr7  1
        setup_a5x a57 $CA57_DBGBASE $CA57_CTIBASE $_num_ca57 0
        setup_a5x a53 $CA53_DBGBASE $CA53_CTIBASE $_num_ca53 0
 }

--