[riot-notifications] [RIOT-OS/RIOT] drivers/rn2xx3: initial implementation (#7505)

Alexandre Abadie notifications at github.com
Fri Dec 22 15:08:28 CET 2017


aabadie commented on this pull request.



> +#ifndef RN2XX3_DEFAULT_SLEEP
+#define RN2XX3_DEFAULT_SLEEP            (5000U)
+#endif
+
+#if defined(MODULE_RN2903)
+#define RN2XX3_FREQ_BAND                (915U)
+#endif
+
+#if defined(MODULE_RN2483)
+#define RN2XX3_FREQ_BAND                (868U)
+#endif
+
+/**
+ * @brief   Command responses and server replies status codes
+ */
+enum {

Because the same function can return either status codes, error codes and reply codes. This depends on the module behaviour.
Maybe I can just use one enum with only positive values?
I remember I couldn't find a clean solution (at least for me) for correctly handling the different error/status/reply codes.

-- 
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/7505#discussion_r158497799
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20171222/064b34ce/attachment-0001.html>


More information about the notifications mailing list