<p><b>@cladmi</b> commented on this pull request.</p>

<p>Some wording remarks, plus would like to have the link to the datasheet.<br>
It would allow updating it later for new architecture and have all the context for justifying the values.</p>
<p>As it is CPU¬†specific, its good to put the maximum possible required value by default.</p><hr>

<p>In <a href="https://github.com/RIOT-OS/RIOT/pull/11641#discussion_r294265978">sys/riotboot/Makefile.include</a>:</p>
<pre style='color:#555'>> @@ -1,7 +1,15 @@
-# Indicate the reserved space for a header, 256B by default
-# Notice that it must be 256B aligned. This is restricted by
-# the Cortex-M0+/3/4/7 architecture
-RIOTBOOT_HDR_LEN ?= 0x100
+# Indicate the reserved space for a header.
+# Notice that it must be at least 256B aligned. This is restricted by
</pre>
<p>Could you update to remove the previous <code>Notice that it must be at least 256 aligned</code> and include this part in the following part. Currently it has two different informations mixed.</p>

<hr>

<p>In <a href="https://github.com/RIOT-OS/RIOT/pull/11641#discussion_r294266346">sys/riotboot/Makefile.include</a>:</p>
<pre style='color:#555'>> @@ -1,7 +1,15 @@
-# Indicate the reserved space for a header, 256B by default
-# Notice that it must be 256B aligned. This is restricted by
-# the Cortex-M0+/3/4/7 architecture
-RIOTBOOT_HDR_LEN ?= 0x100
+# Indicate the reserved space for a header.
+# Notice that it must be at least 256B aligned. This is restricted by
+# the Cortex-M0+/3/4/7 architecture. "The Vector table must be naturally
+# aligned to a power of two whose alignment value is greater than or equal
+# to number of Exceptions supported x 4"
</pre>
<p>It would be good if you have a link for this information, it would be useful for porting future boards and justifying the values.</p>

<hr>

<p>In <a href="https://github.com/RIOT-OS/RIOT/pull/11641#discussion_r294266793">sys/riotboot/Makefile.include</a>:</p>
<pre style='color:#555'>> @@ -1,7 +1,15 @@
-# Indicate the reserved space for a header, 256B by default
-# Notice that it must be 256B aligned. This is restricted by
-# the Cortex-M0+/3/4/7 architecture
-RIOTBOOT_HDR_LEN ?= 0x100
+# Indicate the reserved space for a header.
+# Notice that it must be at least 256B aligned. This is restricted by
+# the Cortex-M0+/3/4/7 architecture. "The Vector table must be naturally
+# aligned to a power of two whose alignment value is greater than or equal
+# to number of Exceptions supported x 4"
+# For ARMv7-M/ARMv7E-M (M4, M3, M7). Maximum of 256 exceptions (256*4 bytes == 0x400).
</pre>
<p>You can put it as markdown style items with <code># * lalala</code></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=ABE7WYDP3T2NR3EXJB4P32TP2555TA5CNFSM4HU7OOV2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB3W2IJI#pullrequestreview-250455077">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABE7WYAVA4BPD5J2G5CBZYTP2555TANCNFSM4HU7OOVQ">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABE7WYEZXGSJCQ4BIUJQ6CLP2555TA5CNFSM4HU7OOV2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB3W2IJI.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=ABE7WYDP3T2NR3EXJB4P32TP2555TA5CNFSM4HU7OOV2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB3W2IJI#pullrequestreview-250455077",
"url": "https://github.com/RIOT-OS/RIOT/pull/11641?email_source=notifications\u0026email_token=ABE7WYDP3T2NR3EXJB4P32TP2555TA5CNFSM4HU7OOV2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB3W2IJI#pullrequestreview-250455077",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>