[riot-notifications] [RIOT-OS/RIOT] cpu/stm32: add rtc_mem (#16802)

benpicco notifications at github.com
Mon Sep 6 19:42:24 CEST 2021


@benpicco commented on this pull request.



> @@ -209,7 +209,7 @@ static int bcd2val(uint32_t val, int shift, uint32_t mask)
     return (((tmp >> 4) * 10) + (tmp & 0x0f));
 }
 
-static inline void rtc_unlock(void)
+extern inline void rtc_unlock(void)

The function does quite a few things, I think there is no reason to force it to be inlined.
We should even save some ROM this way. 

-- 
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/16802#discussion_r703025126
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210906/264eec6e/attachment-0001.htm>


More information about the notifications mailing list