[riot-notifications] [RIOT-OS/RIOT] sys/senml: add SenML modules (#16384)

Philipp Blum notifications at github.com
Wed Aug 25 17:50:47 CEST 2021


@Citrullin commented on this pull request.



> +#ifndef SENML_SAUL_USE_FLOATS
+#define SENML_SAUL_USE_FLOATS (0)
+#endif
+
+/**
+ * @brief Encode all sensors from @ref drivers_saul as senml+cbor.
+ *
+ * This functions reads all available @ref drivers_saul sensors and encodes the
+ * values as SenML/CBOR.
+ *
+ * @param buf Buffer to store the CBOR in.
+ * @param len Length of the buffer.
+ *
+ * @return Size of the encoded data.
+ */
+size_t senml_saul_encode_cbor(uint8_t *buf, size_t len);

I would find it very helpful to have an API for individual devices as well. As far as I am aware of, you can also have specific endpoints for specific sensors. Like /temp, /humidity etc. 
So, in that case you want only to encode the temperature sensor etc. for the /temp endpoint.

-- 
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/16384#pullrequestreview-738533218
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210825/32118b53/attachment.htm>


More information about the notifications mailing list