[riot-notifications] [RIOT-OS/RIOT] riotboot, mcuboot: Use canned recipe for flashing. (#11110)

Juan I Carrano notifications at github.com
Tue Mar 5 16:51:30 CET 2019


### Contribution description

#10548 introduced a canned recipe for flashing. The aim was to reduce code duplication, and also to be able to perform more than one command. This is important for boards that need more complex procedure, for example to put them in a special mode or disabling the watchdog (currently done in "preflash")

By having the flashing recipe concentrated in a single place maintenance is simplified. Eventually the flash recipe will be defined with `?=` to allow overriding, and then the benefits of this change will become clearer.

### Testing procedure

Build and flash `tests/riotboot`. I could not test mcuboot because I do not have a supported board.

Flashing continue to work fine.

### Issues/PRs references

See: #10548.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * makefiles/riotboot: use canned recipe for flashing.
  * makefiles/mcuboot: use canned recipe for flashing.

-- File Changes --

    M makefiles/boot/riotboot.mk (8)
    M makefiles/mcuboot.mk (6)

-- Patch Links --

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


More information about the notifications mailing list