<borneoa___>
PaulFertser: hey! You are right. I've missed the default enable of port 0. That patch way aimed at programming ITM when there is no specific cortex-m TPIU. Let me try a fix...
<borneoa___>
frankplow: do you have an account on Gerrit for reviewing the patch?
<borneoa___>
review.openocd.org/7900
w00die has joined #openocd
damex has quit [Ping timeout: 255 seconds]
damex has joined #openocd
w00die has quit [Quit: Client closed]
slobodan has joined #openocd
<PaulFertser>
borneoa___: thank you for the prompt fix!
<frankplow>
borneoa___: Patch does the trick for me, thanks a lot!
slobodan has quit [Read error: Connection reset by peer]
slobodan has joined #openocd
nerozero has quit [Remote host closed the connection]
nerozero has joined #openocd
bvernoux has joined #openocd
<frankplow>
Is there any way from a TPIU pre-enable procedure to hook code to run on a microcontroller reset? The TRACECONFIG.TRACEMUX register you need to set to enable SWV on nRF52 MCUs resets every time the MCU resets.
<frankplow>
I think this may actually be an error in the specification, as it says there that the register is retained however I have tested with bare-bones firmware and that register definitely resets
<borneoa___>
frankplow: yes. You can define the events {pre,post}-{enable,disable}, where you can add all the soc specific code to handle the TPIU/SWO
<borneoa___>
frankplow: check the code in tcl/target/stm32f4x.cfg, or other stm32
<PaulFertser>
Is it getting re-run on chip reset?
<karlp>
antto: what do you mean by "earlier" ?
<karlp>
it's a two year old, single developer project?
<karlp>
strong NIH feelings :)
<karlp>
somehwat smells like someone with potentially more inside information as well...
<frankplow>
PaulFertser: I have made a patch which works for me, I suspect this is not the best way to do it but probably easier to see what I am trying to say there: review.openocd.org/7900
<frankplow>
PaulFertser: Oops, meant to be review.openocd.org/7901
<PaulFertser>
frankplow: thank you, that looks nice!
gzlb has quit [Ping timeout: 246 seconds]
gzlb has joined #openocd
crabbedhaloablut has quit []
<borneoa___>
frankplow: looks ok. Do you think the list of devices could grow? Would it make sense to replace the if/then/else with a switch/case?
slobodan has quit [Ping timeout: 255 seconds]
<frankplow>
borneoa___: Does TCL's switch have fallthrough? I couldn't find an example of it used anywhere and thought it would be overly verbose to duplicate the inner block for each case.
bvernoux has quit [Quit: Leaving]
<borneoa___>
frankplow: yes, it has. Check tcl/target/ti_k3.cfg
<borneoa___>
It uses the minus sign
<borneoa___>
Also src/target/startup.tcl
dliviu has quit [Remote host closed the connection]