Re: [OpenOCD-devel] [PATCH] imx8m: add an m4 target to the imx8m

[Angus A. <an...@ak...>]

On 2018-04-19 14:37, Paul Fertser wrote:
> Hello Angus,
> 
> Thank you for the patch.
> 
> Can you please consider sending this to Gerrit as per HACKING
> instructions?
> 

Ok I'll re-submit based on those instructions.

> Quick review inline.
> 
> On Thu, Apr 19, 2018 at 02:05:47PM -0600, Angus Ainslie wrote:
>> +${_CHIPNAME}.m4 configure -event reset-init { m4_init }
>> +
>> +proc m4_init { } {
>> +  echo "m4 reset-init..."
>> +  cortex_m dbginit
> 
> I do not see dbginit command in cortex_m group, have you actually
> runtime-tested this?
> 

I had tested with an earlier version but I see that there isn't that 
command int the cortex_m group.

>> +  # halt the CPU
>> +  halt
> 
> I expected "reset init" to halt all the enabled targets. Are you sure
> that's not happening?

Right now none of the targets are halting. There seems to be a 
disconnect between what openocd thinks is happening and what is going on 
in the HW. This shows that openocd thinks the A53 core is running but it 
is not.

Trying 127.0.0.1...
Connected to localhost.
Escape character is '^]'.
Open On-Chip Debugger
> targets
     TargetName         Type       Endian TapName            State
--  ------------------ ---------- ------ ------------------ ------------
  0  imx8mq.a53.0       aarch64    little imx8mq.cpu         running
  1  imx8mq.a53.1       aarch64    little imx8mq.cpu         examine 
deferred
  2  imx8mq.a53.2       aarch64    little imx8mq.cpu         examine 
deferred
  3  imx8mq.a53.3       aarch64    little imx8mq.cpu         examine 
deferred
  4* imx8mq.m4          cortex_m   little imx8mq.cpu         reset
> targets imx8mq.a53.0
> reset halt
JTAG tap: imx8mq.cpu tap/device found: 0x5ba00477 (mfg: 0x23b (ARM 
Ltd.), part: 0xba00, ver: 0x5)
Debug regions are unpowered, an unexpected reset might have happened
JTAG-DP STICKY ERROR
Deferring arp_examine of imx8mq.a53.1
Use arp_examine command to examine it manually!
Deferring arp_examine of imx8mq.a53.2
Use arp_examine command to examine it manually!
Deferring arp_examine of imx8mq.a53.3
Use arp_examine command to examine it manually!
Invalid ACK (7) in DAP response
JTAG-DP STICKY ERROR
Invalid ACK (7) in DAP response
JTAG-DP STICKY ERROR
Invalid ACK (7) in DAP response
JTAG-DP STICKY ERROR
AP write error, reset will not halt
Invalid ACK (7) in DAP response
Debug regions are unpowered, an unexpected reset might have happened
JTAG-DP STICKY ERROR
in procedure 'reset'
in procedure 'ocd_bouncer'


> targets
     TargetName         Type       Endian TapName            State
--  ------------------ ---------- ------ ------------------ ------------
  0* imx8mq.a53.0       aarch64    little imx8mq.cpu         running
  1  imx8mq.a53.1       aarch64    little imx8mq.cpu         examine 
deferred
  2  imx8mq.a53.2       aarch64    little imx8mq.cpu         examine 
deferred
  3  imx8mq.a53.3       aarch64    little imx8mq.cpu         examine 
deferred
  4  imx8mq.m4          cortex_m   little imx8mq.cpu         reset