[riot-notifications] [RIOT-OS/RIOT] boards/frdm-kw41z-k64f: add riotboot (#11562)

Francisco notifications at github.com
Tue Jun 4 09:27:35 CEST 2019


> My bad, I did not see the difference with elf32-bigarm, even in your comment it's only the line length that showed me "double check it is different".

Looking into the code base I don't think the `-EB -EL` (efl32-bigarm elf32-littlearm) are not being used anyway. So changing `cortexm_base.ld` so `-EB` is `elf32-bigarm` would work. Not of these options are actually used, but if they are to be used it makes more sense that `-EB` be elf32-bigarm.

I read a bunch about this over the weekend and couldn't find any reference to the need or lack off need for declaring ` OUTPUT_FORMAT/OUTPUT_ARCH` at the beginning of a script. Testing everything still works as before, so I think by changing `cortexm_base.ld`  we can remove `OUTPUT_FORMAT/OUTPUT_ARCH`  from kinetis.ld. 

I created a new `cortexm_rom_offset.ld` for the common rom_offset calculation.

Tell me what you think of the changes! If you agree I would probably split the last 5 commits regarding linker script cleanup in another PR

-- 
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/11562#issuecomment-498555141
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190604/90ab6b41/attachment.html>


More information about the notifications mailing list