[riot-notifications] [RIOT-OS/RIOT] cpu/saml21: Make Low-Power SRAM available to programs (#11486)

Dylan Laduranty notifications at github.com
Thu Sep 12 20:37:44 CEST 2019


dylad commented on this pull request.

What about introduce a second variable alongside with CPU_HAS_BACKUP_RAM like BKUP_RAM_IS_RETAINED. This way you can set the first one to 1 for SAML21 but the latter to 0 so it cannot be used by your test. This way you may have more flexibility I guess.
Maybe we can also rename these variables to something more generic across SAM families ?

This is just some ideas and we could discuss this :)
What do you think ? 

> @@ -78,7 +88,7 @@ __attribute__((weak)) void post_startup (void)
 void reset_handler_default(void)
 {
     uint32_t *dst;
-    uint32_t *src = &_etext;
+    const uint32_t *src = &_etext;

unrelated ?

-- 
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/11486#pullrequestreview-287620801
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190912/0a54b52f/attachment.htm>


More information about the notifications mailing list