<PaulFertser>
Yes, currently it does. You think checkpatch warnings should be just ignored? Or some specific warnings disabled probably?
* karlp
shrugs
<karlp>
i mean, I personally think unbraced ifs should be banned by checkpatch,
<Haohmaru>
+1 karlp, in general
<karlp>
PaulFertser: I ugess I mean, it's not a warning, it's actually a hard error, given actual policy, so either a) make it an actual error, because that's the reality, or, decide that perhaps that's not really somethign that should be an error, and therefore, move it out of warning?
<karlp>
more importantly, I updated to latest master, and ctrl-c; "run" isn't ... running anymore...
<karlp>
I can't see anythign in the commit log that even seems relevant.
<karlp>
I guess I can bisect it after lunch
<karlp>
ffs. bisecting is teh suck when you need patches on top of master
<drath42>
PaulFertser: about electricworry's problem with single stepping: I had issues with single stepping basically not working on an R5f, which is ARMv7R, which shares the implementation with ARMv7A, which is what the Pi2 uses. I found that by default maskisr isn't set in cortex_a.c, and after adding a "maskisr on" it worked just fine. The ARMv8 code on the other hand appears to enabling maskisr by default, which could explain why he
<drath42>
didn't have problems on Pi3.
<PaulFertser>
drath42: hi! Thank you so much for sharing! I'll mail your answer to electricworry then.
<drath42>
PaulFertser: heh, I was hoping you'd pass that on when he joins, which appears to be past my online hours, but that's even better of course
<PaulFertser>
Yeah I will. Can't see e-mail address shared anywhere so that's the only way.
<drath42>
Thanks
<karlp>
well, reported my regression on https://review.openocd.org/c/openocd/+/8285 I don't need one of these TI parts, so I've just added the revert to my stack for now :)