[riot-notifications] [RIOT-OS/RIOT] Improve shell_commands (#8564)

Loïc Dauphin notifications at github.com
Wed Mar 6 14:20:10 CET 2019


astralien3000 commented on this pull request.



> @@ -0,0 +1,9 @@
+SECTIONS
+{
+  .data :
+  {
+    _shell_command_list = .; /* begining of the command list */
+    KEEP(*(._shell_command_list))
+    KEEP(*(._shell_command_list_end))
+  } > ram

Hello, I don't remember exactly why, but it was something about the main cortexm linker script. Basically, I have tested the code on native and cortexm(3?) targets. Native target puts everything in RAM, so that it is not needed to specify it, while it would make sense to do so for microcontrollers in general, where we need to choose between RAM, EEPROM, Flash.

-- 
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/8564#discussion_r262936577
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190306/451d02b5/attachment.html>


More information about the notifications mailing list