[riot-notifications] [RIOT-OS/RIOT] tools/edbg: Use openocd to reset instead of edbg. (#11125)

Juan I Carrano notifications at github.com
Thu Mar 7 12:57:16 CET 2019


### Contribution description

A few commits ago edbg changed its behaviour and now it resets the board twice. This makes some tests fail, as the start on the first reset, but shortly after the board is reset again.

I already reported upstream (https://github.com/ataradov/edbg/issues/77) but in the meanwhile we need a workaround.

This comments out RESET and RESET_FLAGS in the edbg tool setting. Reset will now be done via OpenOCD, which is available and used anyways because it is needed for `make debug`.

### Testing procedure

Before trying this out, verify that edbg is on the latest version:
```
$ cd <RIOT REPO>
$ rm -rf dist/tools/edbg/edbg dist/tools/edbg/bin
$ make -C examples/hello-world $(pwd)/dist/tools/edbg/edbg
```

Compile and flash `hello-world` in any atmel board:
```
BOARD=saml21-xpro make -C examples/hello-world  all flash term
```

Open another terminal and reset the board:
```
BOARD=saml21-xpro make -C examples/hello-world  reset
```

Without this change: two resets
```
Welcome to pyterm!
Type '/exit' to exit.
2019-03-07 12:56:34,816 - INFO # main(): This is RIOT! (Version: 2019.04-devel-365-gda2f47-HEAD)
2019-03-07 12:56:34,817 - INFO # Hello World!
2019-03-07 12:56:34,822 - INFO # You are running RIOT on a(n) saml21-xpro board.
2019-03-07 12:56:34,838 - INFO # main(): This is RIOT! (Version: 2019.04-devel-365-gda2f47-HEAD)
2019-03-07 12:56:34,839 - INFO # Hello World!
2019-03-07 12:56:34,843 - INFO # You are running RIOT on a(n) saml21-xpro board.
2019-03-07 12:56:34,847 - INFO # This board features a(n) saml21 MCU.
```

With this change: only one reset (there's no point on pasting the output.)

### Issues/PRs references

Issue in edbg: https://github.com/ataradov/edbg/issues/77
You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/11125

-- Commit Summary --

  * tools/edbg: Use openocd to reset instead of edbg.

-- File Changes --

    M makefiles/tools/edbg.inc.mk (10)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/11125.patch
https://github.com/RIOT-OS/RIOT/pull/11125.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11125
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190307/3b4ee67b/attachment-0001.html>


More information about the notifications mailing list