[riot-notifications] [RIOT-OS/RIOT] dist/jlink.sh: Allow overwriting the 'reset' commands (#11856)

Gaëtan Harter notifications at github.com
Wed Jul 17 15:32:56 CEST 2019


### Contribution description

Add a variable to configure the reset commands.
This is required for board that need a hardware reset.

### Testing procedure

Flashing an resetting with a JLink board did not change.
For example

```
BOARD=nrf52dk make -C tests/bloom_bytes/ reset flash test
```

And the file can be overwritten, here with a non existing one to show the error:
```
JLINK_RESET_FILE=lala BOARD=nrf52dk make -C tests/bloom_bytes/ clean reset flash test
make: Entering directory '/home/harter/work/git/RIOT/tests/bloom_bytes'
/home/harter/work/git/RIOT/dist/tools/jlink/jlink.sh reset
### Resetting Target ###
SEGGER J-Link Commander V6.42d (Compiled Feb 15 2019 13:56:53)
DLL version V6.42d, compiled Feb 15 2019 13:56:43

J-Link Commander will now exit on Error
Could not open J-Link Command File 'lala'
/home/harter/work/git/RIOT/tests/bloom_bytes/../../Makefile.include:592: recipe for target 'reset' failed
make: *** [reset] Error 1
make: Leaving directory '/home/harter/work/git/RIOT/tests/bloom_bytes'
```


### Issues/PRs references

This is a change required for #11833 
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * dist/jlink.sh: Allow overwriting the 'reset' commands

-- File Changes --

    M dist/tools/jlink/jlink.sh (6)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/11856.patch
https://github.com/RIOT-OS/RIOT/pull/11856.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/11856
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190717/987b1f04/attachment.html>


More information about the notifications mailing list