[riot-notifications] [RIOT-OS/RIOT] gnrc/pktdump : Expose Configurations to Kconfig (#14071)

Leandro Lanzieri notifications at github.com
Thu May 14 16:45:58 CEST 2020


@leandrolanzieri commented on this pull request.



>  /**
  * @brief   Message queue size for the pktdump thread
  */
-#ifndef GNRC_PKTDUMP_MSG_QUEUE_SIZE
-#define GNRC_PKTDUMP_MSG_QUEUE_SIZE     (8U)
+#ifdef CONFIG_GNRC_PKTDUMP_MSG_QUEUE_SIZE_EXP
+#define CONFIG_GNRC_PKTDUMP_MSG_QUEUE_SIZE  (1<<CONFIG_GNRC_PKTDUMP_MSG_QUEUE_SIZE_EXP)

Well, if this macro is not configurable then the prefix is not needed. But also, the `ifndef` can be removed as the user would configure the queue size via `CONFIG_GNRC_PKTDUMP_MSG_QUEUE_SIZE_EXP`

-- 
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/14071#discussion_r425193961
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200514/6c98cd6b/attachment.htm>


More information about the notifications mailing list