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

benpicco notifications at github.com
Tue Jul 27 12:34:52 CEST 2021


@benpicco commented on this pull request.



> + */
+static const lm75_temp_resolutions lm75a_res = {
+    .os_res = 5,        /**< resolution in 0.5ºC */
+    .os_mult = 10,      /**< Must multiply by 10 to get temp in ºC */
+    .os_shift = 7,      /**< Only the 9 most significant bits are needed */
+    .temp_res = 125,    /**< resolution in 0.125ºC */
+    .temp_mult = 1000,  /**< Must multiply by 1000 to get temp in ºC */
+    .temp_shift = 5,    /**< Only the 11 most significant bits are needed */
+};
+
+/**
+ * @brief params required for initialization
+ */
+typedef struct lm75_params {
+    i2c_t   i2c_bus;            /**< I2C Bus used */
+    lm75_temp_resolutions res;  /**< Temperature resolutions */

```suggestion
    const lm75_temp_resolutions *res;  /**< Temperature resolutions */
```

-- 
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-715752656
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210727/0ca78c51/attachment.htm>


More information about the notifications mailing list