panic handler: Print backtraces with calling address not return address

This seems to clean up some of the more wrong addr2line output results (not
sure why, something to do with optimisations I think - perhaps the return
address may also be a jump target from some earlier line of the code.)

diff --git a/components/esp32/panic.c b/components/esp32/panic.c
index 39d05608707442ed9fb339b573fadef5e4b84115..4d8386446776a59ded92bafa7d60d422c5f6a022 100644 (file)
--- a/components/esp32/panic.c
+++ b/components/esp32/panic.c
@@ -375,7 +375,7 @@ static void doBacktrace(XtExcFrame *frame)
             break;
         }
         sp = *((uint32_t *) (sp - 0x10 + 4));
-        putEntry(pc, sp);
+        putEntry(pc - 3, sp); // stack frame addresses are return addresses, so subtract 3 to get the CALL address
         pc = *((uint32_t *) (psp - 0x10));
         if (pc < 0x40000000) {
             break;