This is an automated email from Gerrit. "Tom Hebb <tommyh...@gmail.com>" just uploaded a new patch set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/6959
-- gerrit commit 78d6f0e323cbcac7eb98e76ca27d1a4b39fab48f Author: Thomas Hebb <tommyh...@gmail.com> Date: Fri Apr 29 23:26:00 2022 -0700 tcl/target/gd32vf103: copy a few minor settings from riscv-openocd These changes bring over some lines from the independently-developed gd32vf103.cfg that I contributed[1] to the riscv-openocd fork of OpenOCD. They're all minor, so I'm squashing them into one review. The changes are as follows: - Mark that the chip's reset pin doesn't reset the debug module. - Add boundary scan TAP. - Mention inconsistency of CPU ID between vendor SDK and real hardware. - Specify that there's no MMU so we don't look for one at runtime. Signed-off-by: Thomas Hebb <tommyh...@gmail.com> Change-Id: Ie8033eff436d6dbdc3eab156769a8908ccb547f6 diff --git a/tcl/target/gd32vf103.cfg b/tcl/target/gd32vf103.cfg index 1b94773f6f..247271bbcd 100644 --- a/tcl/target/gd32vf103.cfg +++ b/tcl/target/gd32vf103.cfg @@ -21,11 +21,21 @@ if { [info exists WORKAREASIZE] } { set _WORKAREASIZE 0x1800 } +reset_config srst_nogate + +# The vendor's configuration expects an ID of 0x1e200a6d, but this one is what +# I have on my board (Sipeed Longan Nano, GD32VF103CBT6). jtag newtap $_CHIPNAME cpu -irlen 5 -expected-id 0x1000563d +jtag newtap $_CHIPNAME bs -irlen 5 -expected-id 0x790007a3 set _TARGETNAME $_CHIPNAME.cpu target create $_TARGETNAME riscv -chain-position $_TARGETNAME +# Disable virtual address translation since we don't have an MMU. Nothing will +# break without this line, but OpenOCD will do a few unnecessary register reads +# to figure it out on its own. +$_TARGETNAME riscv set_enable_virt2phys off + proc default_mem_access {} { riscv set_mem_access progbuf } --