gnrc/[gomach,lwmac]: Remove unused queue size option#14316
Conversation
fjmolinas
left a comment
There was a problem hiding this comment.
Git grep shows no occurrence other than the ones removed in this pr. ACK
git grep GNRC_GOMACH_IPC_MSG_QUEUE_SIZE
sys/include/net/gnrc/gomach/gomach.h:#ifndef CONFIG_GNRC_GOMACH_IPC_MSG_QUEUE_SIZE_EXP
sys/include/net/gnrc/gomach/gomach.h:#define CONFIG_GNRC_GOMACH_IPC_MSG_QUEUE_SIZE_EXP (3U)
sys/include/net/gnrc/gomach/gomach.h:#ifndef GNRC_GOMACH_IPC_MSG_QUEUE_SIZE
sys/include/net/gnrc/gomach/gomach.h:#define GNRC_GOMACH_IPC_MSG_QUEUE_SIZE (1 << CONFIG_GNRC_GOMACH_IPC_MSG_QUEUE_SIZE_EXP)
Seems like they where unused in the original PR as well, might we be missing something?
No, these are macros for the IPC message queue. The modules where implemented right in the transition period of |
miri64
left a comment
There was a problem hiding this comment.
ACK from my side. Both protocols still build when used with examples/gnrc_networking_mac for samr21-xpro
|
Thanks for the review! |
Contribution description
This removes the IPC message queue size configuration options from GoMacH and LWMAC modules as they are not used:
CONFIG_GNRC_GOMACH_IPC_MSG_QUEUE_SIZE_EXPGNRC_GOMACH_IPC_MSG_QUEUE_SIZECONFIG_GNRC_LWMAC_IPC_MSG_QUEUE_SIZE_EXPGNRC_LWMAC_IPC_MSG_QUEUE_SIZETesting procedure
Issues/PRs references
Spotted while reviewing #14104