[riot-notifications] [RIOT-OS/RIOT] riotboot/Makefile.include: increase RIOTBOOT_HRD_LEN for ARMv7*-M (#11641)
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:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications