[riot-notifications] [RIOT-OS/RIOT] drivers/lm75: driver for the lm75 sensor and derivatives (#16678)

benpicco notifications at github.com
Mon Jul 26 17:04:11 CEST 2021


@benpicco commented on this pull request.



> +        lm75_poweron(dev);
+        /* since the temp is only updated every 100ms this is required to ensure the temp register updates for followup readings */
+        if(interval < 100) {
+            xtimer_msleep(100);
+        }
+        else xtimer_msleep(interval);
+        lm75_poweroff(dev);
+    #endif
+
+    return LM75_SUCCESS;
+}
+
+int main(void) {
+
+    /* Enable OS interrupt */
+    gpio_init_int(LM75_PARAM_INT, GPIO_IN_PU, GPIO_FALLING, __irq_os_cb, NULL);

better move this into the driver 

-- 
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/16678#pullrequestreview-714959063
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210726/b070309f/attachment.htm>


More information about the notifications mailing list