[riot-notifications] [RIOT-OS/RIOT] drivers/sx126x: Add support for Nucleo -WL55JC (#16579)

José Alamos notifications at github.com
Fri Jun 25 10:05:45 CEST 2021


@jia200x commented on this pull request.



> @@ -92,6 +92,23 @@ uint32_t sx126x_get_channel(const sx126x_t *dev);
  */
 void sx126x_set_channel(sx126x_t *dev, uint32_t freq);
 
+/**
+ * @brief   Converts symbol value to milliseconds.
+ *
+ * @param[in] dev                      Device descriptor of the driver
+ * @param[in] symbols                  Symbols
+ *
+ * @return Time for symbols in milliseconds
+ */
+static inline int sx126x_symbol_to_msec(sx126x_t *dev, uint16_t symbols)
+{
+    assert((dev->mod_params.bw <= 0x06) && dev->mod_params.bw >= 0x04);

```suggestion
    assert((dev->mod_params.bw <= SX126X_LORA_BW_125) && dev->mod_params.bw >= SX126X_LORA_BW_500);
```

-- 
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/16579#pullrequestreview-692558655
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210625/97632f82/attachment-0001.htm>


More information about the notifications mailing list