[riot-notifications] [RIOT-OS/RIOT] cpu/cc2538: add riotboot feature (#15392)

Francisco notifications at github.com
Fri Nov 6 17:54:49 CET 2020


@fjmolinas commented on this pull request.



> +    # If using riotboot loose 2 pages to not right over CCA section and
+    # keep page parity
+    ROM_LEN ?= 252K

On a non riotboot application there is no issue with overriding the CCA section.

> Shouldn't we always prevent overwriting the last page then? So unconditionally subtracting 2k?
> And the 2k in the start, aren't they taken into account if you set ROM_OFFSET?

cc2538 pages are 2k long, the bootloader takes 4k so two pages which keeps page parity. But here if we remove only the cca we would loose parity, so I remove an extra page.

-- 
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/15392#discussion_r518878847
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20201106/e36d1bb9/attachment.htm>


More information about the notifications mailing list