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

benpicco notifications at github.com
Mon Sep 6 19:44:34 CEST 2021


@benpicco commented on this pull request.



> +#elif defined(RTC_BKP0R)
+#define RTC_MEM_SIZE    1
+#else
+#define RTC_MEM_SIZE    0
+#endif
+
+extern void rtc_lock(void);
+extern void rtc_unlock(void);
+
+size_t rtc_mem_size(void)
+{
+    return RTC_MEM_SIZE * __SIZEOF_POINTER__;
+}
+
+void rtc_mem_write(unsigned offset, const void *data, size_t len)
+{

> 'data' is of type 'const void *'. When using void pointers in calculations, the behaviour is undefined.

Looks like you need something like

```suggestion
{
    const uint8_t *in = data;
```

-- 
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#pullrequestreview-747376259
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210906/8918aa14/attachment.htm>


More information about the notifications mailing list