makefiles/sam0: fix debugging with edbg#12653
Conversation
makefiles/boards/sam0.inc.mk
Outdated
| endif | ||
|
|
||
| else | ||
| # this board uses openocd for debug and possibly flashing |
OpenOCD should *always* be included for debugging if JLinkExe is not used. fixes RIOT-OS#12652
f758d03 to
756cab9
Compare
miri64
left a comment
There was a problem hiding this comment.
With this change make debug works again for samr21-xpro
[mlenders@sarajevo RIOT]<3 git log --oneline -1
756cab9385 (HEAD) makefiles/sam0: fix debugging with edbg
[mlenders@sarajevo RIOT]<3 BOARD=samr21-xpro make -C examples/gnrc_networking debug
make: Entering directory '/home/mlenders/Repositories/RIOT-OS/RIOT/examples/gnrc_networking'
/home/mlenders/Repositories/RIOT-OS/RIOT/dist/tools/openocd/openocd.sh debug /home/mlenders/Repositories/RIOT-OS/RIOT/examples/gnrc_networking/bin/samr21-xpro/gnrc_networking.elf
### Starting Debugging ###
Open On-Chip Debugger 0.10.0+dev-00910-g4dbcb1e7 (2019-06-17-16:24)
Licensed under GNU GPL v2
For bug reports, read
http://openocd.org/doc/doxygen/bugs.html
Reading symbols from /home/mlenders/Repositories/RIOT-OS/RIOT/examples/gnrc_networking/bin/samr21-xpro/gnrc_networking.elf...
Remote debugging using :3333
__set_PRIMASK (priMask=1) at include/vendor/cmsis_gcc.h:414
warning: Source file is more recent than executable.
414 __ASM volatile ("MSR primask, %0" : : "r" (priMask) : "memory");
(gdb) ^CQuit
(gdb) quit
|
(and |
|
This also slipped into the 2019.10 release 😳 |
|
That's up to @kb2ma. Will there be a 2019.10.1 release (also potentially with other bug fixes)? |
|
I don't recall a guideline on the topic of point releases, so I'll review past instances for some context. Certainly this fix is worth of a backport so that a user at least could use a fresh checkout of the '2019.10-branch' branch, so please do that and I would be happy to merge it. Ahhh, release management -- the gift that keeps on giving. ;-) |
|
Hm, how should I interpret the output of I just did |
|
@benpicco what is the context? |
There were instances of bug-fixing releases in the past see e.g. 2018.10.1 (and it was also discussed for 2019.04, IIRC, maybe @MrKevinWeiss give more info on that). I think all in all it's up to the severity of the bug and if the release manager sees their job done after the major release or not (there were release managers in the past that specifically opted out to not have bug fix releases). |
|
@miri64 oh sorry, that was the |
|
Thanks for the retrospective, @miri64. Yes, I saw 2018.10.1, which looks to have been triggered by a serious security issue, #10739. If I understand correctly, the current issue includes all the sam*-xpro and arduino-zero boards. This is painful, but I don't think it rises to the need for a point release. I think the backport to 2019.10-branch is good enough. If someone doesn't want to mess with git, they can just download it from the GitHub interface. I have added this topic to my release debrief for the upcoming assembly. Based on that, I plan to add a couple of sentences on this subject to the release guidelines for future reference. |
|
Backport provided in #12685 |
|
Went ahead and created the backport to keep things moving. |
Contribution description
OpenOCD should always be included for debugging if
JLinkExeis not used.Testing procedure
run
make BOARD=samr21-xpro debugIssues/PRs references
fixes #12652