[riot-notifications] [RIOT-OS/RIOT] riotboot/Makefile.include: increase RIOTBOOT_HRD_LEN for ARMv7*-M (#11641)

Ga√ętan Harter notifications at github.com
Thu Jun 6 12:35:26 CEST 2019


One good thing for the variable, we put the start address of a slot in the slot header and so it is not set at from compile time configuration in the type/bootloader code. (was a good choice @kYc0o :))

Which means that a bootloader compiled now would work in the future even if changing the `RIOTBOOT_HDR_LEN`.

So it is not required to always put the highest possible value as it is really board+firmware specific. Having it by default to this value could still be a sane default.

Local test to verify this:

```
BOARD=pba-d-01-kw2x make -C tests/riotboot clean flash test
2019-06-06 12:29:59,739 - INFO # slot 0: metadata: 0x1000 image: 0x00001200 # 0x200
2019-06-06 12:29:59,743 - INFO # slot 1: metadata: 0x20800 image: 0x00000000

RIOTBOOT_HDR_LEN=0x400 BOARD=pba-d-01-kw2x make -C tests/riotboot clean riotboot/flash-slot1 test
2019-06-06 12:30:26,540 - INFO # slot 0: metadata: 0x1000 image: 0x00001200
2019-06-06 12:30:26,544 - INFO # slot 1: metadata: 0x20800 image: 0x00020c00 # 0x400

RIOTBOOT_HDR_LEN=0x800 BOARD=pba-d-01-kw2x make -C tests/riotboot clean riotboot/flash-slot1 test
2019-06-06 12:30:46,332 - INFO # slot 0: metadata: 0x1000 image: 0x00001200
2019-06-06 12:30:46,336 - INFO # slot 1: metadata: 0x20800 image: 0x00021000 # 0x800

RIOTBOOT_HDR_LEN=0x1000 BOARD=pba-d-01-kw2x make -C tests/riotboot clean riotboot/flash-slot0 test
2019-06-06 12:31:53,536 - INFO # slot 0: metadata: 0x1000 image: 0x00002000 # 0x1000
2019-06-06 12:31:53,540 - INFO # slot 1: metadata: 0x20800 image: 0x00021000
```

-- 
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/11641#issuecomment-499442044
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190606/ca86afb1/attachment-0001.html>


More information about the notifications mailing list