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

Francisco notifications at github.com
Thu Jun 6 13:46:56 CEST 2019


> +FEATURES_PROVIDED += riotboot

This isn't actually needed, if riotboot is not provided you get an "expect errors message" put the application will still compile (and in this case work).

> 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.

Indeed, it could also be defined for the specific boards/cpu (as done in #11642 and #11643). The good thing about this is that since by default it won't work, it will force the developer to look more carefully into `RIOTBOOT_HDR_LEN` value and set the minimum value accordingly for that board.

The thing is that if we are setting a default value, I think the default value should be sow that it works for most cases

>Would it be possible to assert this requirement in the code too

This could be done by the bootloader, prevent a jump if the it is not aligned.

-- 
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-499461471
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190606/02289a90/attachment.html>


More information about the notifications mailing list