<blockquote>
<p>+FEATURES_PROVIDED += riotboot</p>
</blockquote>
<p>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).</p>
<blockquote>
<p>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.</p>
</blockquote>
<p>Indeed, it could also be defined for the specific boards/cpu (as done in <a class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="452935700" data-permission-text="Issue title is private" data-url="https://github.com/RIOT-OS/RIOT/issues/11642" data-hovercard-type="pull_request" data-hovercard-url="/RIOT-OS/RIOT/pull/11642/hovercard" href="https://github.com/RIOT-OS/RIOT/pull/11642">#11642</a> and <a class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="452940097" data-permission-text="Issue title is private" data-url="https://github.com/RIOT-OS/RIOT/issues/11643" data-hovercard-type="pull_request" data-hovercard-url="/RIOT-OS/RIOT/pull/11643/hovercard" href="https://github.com/RIOT-OS/RIOT/pull/11643">#11643</a>). The good thing about this is that since by default it won't work, it will force the developer to look more carefully into <code>RIOTBOOT_HDR_LEN</code> value and set the minimum value accordingly for that board.</p>
<p>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</p>
<blockquote>
<p>Would it be possible to assert this requirement in the code too</p>
</blockquote>
<p>This could be done by the bootloader, prevent a jump if the it is not aligned.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/RIOT-OS/RIOT/pull/11641?email_source=notifications&email_token=ABE7WYAMB2AUDHXPIHXZJG3PZD2LBA5CNFSM4HU7OOV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXCS2XY#issuecomment-499461471">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABE7WYCPL3INJAC5ZMUTY6TPZD2LBANCNFSM4HU7OOVQ">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABE7WYCHWKBQ6DWRQYYVIHDPZD2LBA5CNFSM4HU7OOV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXCS2XY.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/RIOT-OS/RIOT/pull/11641?email_source=notifications\u0026email_token=ABE7WYAMB2AUDHXPIHXZJG3PZD2LBA5CNFSM4HU7OOV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXCS2XY#issuecomment-499461471",
"url": "https://github.com/RIOT-OS/RIOT/pull/11641?email_source=notifications\u0026email_token=ABE7WYAMB2AUDHXPIHXZJG3PZD2LBA5CNFSM4HU7OOV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXCS2XY#issuecomment-499461471",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>