[OpenOCD-commit] Main OpenOCD repository branch, master, updated. v0.10.0-1387-g1457a1ab4

[OpenOCD-Gerrit <ope...@us...>]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Main OpenOCD repository".

The branch, master has been updated
       via  1457a1ab425febd3dfceeb8c908b69135ff0b8b6 (commit)
      from  f1d77a485ce4ca81ea791f3d584f7d5d452a10a9 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 1457a1ab425febd3dfceeb8c908b69135ff0b8b6
Author: Antonio Borneo <bor...@gm...>
Date:   Sat Jan 26 16:19:55 2019 +0100

    tcl/target: use command 'jtag newtap' to add a boundary scan TAP
    
    A JTAG TAP for boundary scan should be added in the scan chain
    through the command "jtag newtap".
    In some TCL target script the boundary scan TAP is added through
    the command "swj_newdap", command that is inappropriate in this
    context because specific for arm adi-v5 SWJ-DP.
    This situation was probably created to bypass the error with HLA
    framework, caused by missing command "jtag newtap".
    
    Add the command "jtag newtap" in HLA, by reusing the existing
    code for command "hla newtap".
    Fix the TCL target scripts to use the command "jtag newtap" for
    the boundary scan TAPs.
    
    The TCL script target/psoc6.cfg has no evident reference to HLA,
    so the reason for using "swj_newdap" is less clear. Nevertheless
    it uses the wrong command and, once HLA is fixed, there is no
    reason to avoid fixing it too.
    
    Change-Id: Ia92f8221430cf6f3d2c34294e22e5e18963bb88c
    Signed-off-by: Antonio Borneo <bor...@gm...>
    Reviewed-on: http://openocd.zylin.com/4873
    Tested-by: jenkins
    Reviewed-by: Andreas Fritiofson <and...@gm...>

diff --git a/src/jtag/hla/hla_transport.c b/src/jtag/hla/hla_transport.c
index fbdddfd65..338357748 100644
--- a/src/jtag/hla/hla_transport.c
+++ b/src/jtag/hla/hla_transport.c
@@ -62,6 +62,9 @@ hl_transport_stlink_subcommand_handlers[] = {
 
 static const struct command_registration
 hl_transport_jtag_subcommand_handlers[] = {
+	{
+	 .chain = hl_transport_stlink_subcommand_handlers,
+	 },
 	{
 	 .name = "init",
 	 .mode = COMMAND_ANY,
diff --git a/tcl/target/em357.cfg b/tcl/target/em357.cfg
index 572007134..f39f3f428 100644
--- a/tcl/target/em357.cfg
+++ b/tcl/target/em357.cfg
@@ -52,7 +52,7 @@ if { [info exists FLASHSIZE] } {
 swj_newdap $_CHIPNAME cpu -irlen 4 -expected-id $_CPUTAPID
 dap create $_CHIPNAME.dap -chain-position $_CHIPNAME.cpu
 if { [using_jtag] } {
-    swj_newdap $_CHIPNAME bs -irlen 4 -expected-id $_BSTAPID -ircapture 0xe -irmask 0xf
+    jtag newtap $_CHIPNAME bs -irlen 4 -expected-id $_BSTAPID -ircapture 0xe -irmask 0xf
 }
 
 set _TARGETNAME $_CHIPNAME.cpu
diff --git a/tcl/target/psoc6.cfg b/tcl/target/psoc6.cfg
index 51d032b17..bf63fd5d4 100644
--- a/tcl/target/psoc6.cfg
+++ b/tcl/target/psoc6.cfg
@@ -146,5 +146,5 @@ if { $_ENABLE_CM0 } {
 }
 
 if {[using_jtag]} {
-	swj_newdap $_CHIPNAME bs -irlen 18 -expected-id 0x2e200069
+	jtag newtap $_CHIPNAME bs -irlen 18 -expected-id 0x2e200069
 }
diff --git a/tcl/target/stm32h7x.cfg b/tcl/target/stm32h7x.cfg
index 2d92eca92..43a8b024e 100644
--- a/tcl/target/stm32h7x.cfg
+++ b/tcl/target/stm32h7x.cfg
@@ -70,7 +70,7 @@ swj_newdap $_CHIPNAME cpu -irlen 4 -ircapture 0x1 -irmask 0xf -expected-id $_CPU
 dap create $_CHIPNAME.dap -chain-position $_CHIPNAME.cpu
 
 if {[using_jtag]} {
- swj_newdap $_CHIPNAME bs -irlen 5
+   jtag newtap $_CHIPNAME bs -irlen 5
 }
 
 if {![using_hla]} {
diff --git a/tcl/target/stm32w108xx.cfg b/tcl/target/stm32w108xx.cfg
index 3a83fd19e..0470bf6ce 100644
--- a/tcl/target/stm32w108xx.cfg
+++ b/tcl/target/stm32w108xx.cfg
@@ -42,11 +42,11 @@ dap create $_CHIPNAME.dap -chain-position $_CHIPNAME.cpu
 if {[using_jtag]} {
  if { [info exists BSTAPID] } {
    set _BSTAPID $BSTAPID
-   swj_newdap $_CHIPNAME bs -irlen 4 -ircapture 0xe -irmask 0xf -expected-id _BSTAPID
+   jtag newtap $_CHIPNAME bs -irlen 4 -ircapture 0xe -irmask 0xf -expected-id _BSTAPID
  } else {
    set _BSTAPID_1 0x169a862b
    set _BSTAPID_2 0x269a862b
-   swj_newdap $_CHIPNAME bs -irlen 4 -ircapture 0xe -irmask 0xf \
+   jtag newtap $_CHIPNAME bs -irlen 4 -ircapture 0xe -irmask 0xf \
 		-expected-id $_BSTAPID_1 -expected-id $_BSTAPID_2
  }
 }
diff --git a/tcl/target/stm32wlx.cfg b/tcl/target/stm32wlx.cfg
index 98c9a7ee9..ba16fab6b 100644
--- a/tcl/target/stm32wlx.cfg
+++ b/tcl/target/stm32wlx.cfg
@@ -38,7 +38,7 @@ swj_newdap $_CHIPNAME cpu -irlen 4 -ircapture 0x1 -irmask 0xf -expected-id $_CPU
 dap create $_CHIPNAME.dap -chain-position $_CHIPNAME.cpu
 
 if {[using_jtag]} {
-	swj_newdap $_CHIPNAME bs -irlen 5
+	jtag newtap $_CHIPNAME bs -irlen 5
 }
 
 set _TARGETNAME $_CHIPNAME.cpu

-----------------------------------------------------------------------

Summary of changes:
 src/jtag/hla/hla_transport.c | 3 +++
 tcl/target/em357.cfg         | 2 +-
 tcl/target/psoc6.cfg         | 2 +-
 tcl/target/stm32h7x.cfg      | 2 +-
 tcl/target/stm32w108xx.cfg   | 4 ++--
 tcl/target/stm32wlx.cfg      | 2 +-
 6 files changed, 9 insertions(+), 6 deletions(-)


hooks/post-receive
-- 
Main OpenOCD repository