netdev_event_thread: add support for event_thread irq handler

Makefile.dep

@@ -86,6 +86,10 @@ ifneq (,$(filter gnrc_netdev_default,$(USEMODULE)))
   USEMODULE += gnrc_netif
 endif
 
+ifneq (,$(filter netdev_event_thread,$(USEMODULE)))
+  USEMODULE += event_thread_highest
+endif
+
 ifneq (,$(filter netdev_ieee802154,$(USEMODULE)))
   USEMODULE += ieee802154
   USEMODULE += random

cpu/native/Makefile

@@ -7,10 +7,6 @@ ifeq ($(OS),Darwin)
   CFLAGS += -D_XOPEN_SOURCE=600 -D_DARWIN_C_SOURCE
 endif
 
-ifneq (,$(filter netdev_tap,$(USEMODULE)))
-  DIRS += netdev_tap
-endif
-
 ifneq (,$(filter socket_zep,$(USEMODULE)))
   DIRS += socket_zep
 endif

drivers/Makefile.dep

@@ -737,3 +737,7 @@ ifneq (,$(filter xbee,$(USEMODULE)))
   USEMODULE += xtimer
   USEMODULE += netif
 endif
+
+ifneq (,$(filter netdev_%,$(USEMODULE)))
+  USEMODULE += netdev
+endif

drivers/include/net/netdev.h

@@ -200,10 +200,17 @@ extern "C" {
 #include "iolist.h"
 #include "net/netopt.h"
 
+#include "kernel_defines.h"
+
 #ifdef MODULE_L2FILTER
 #include "net/l2filter.h"
 #endif
 
+#if IS_USED(MODULE_NETDEV_EVENT_THREAD)
+#include "event.h"
+#include "event/thread.h"
+#endif
+
 /**
  * @name        Network device types
  * @anchor      net_netdev_type
@@ -290,6 +297,9 @@ struct netdev {
 #ifdef MODULE_L2FILTER
     l2filter_t filter[L2FILTER_LISTSIZE];   /**< link layer address filters */
 #endif
+#if IS_USED(MODULE_NETDEV_EVENT_THREAD)
+    event_t irq_handler;                    /**< event that stores the irq handler */
+#endif
 };
 
 /**
@@ -469,6 +479,31 @@ static inline int netdev_set_notsup(netdev_t *dev, netopt_t opt,
     return -ENOTSUP;
 }
 
+#if IS_USED(MODULE_NETDEV_EVENT_THREAD) || DOXYGEN
+
+/**
+ * @brief default event handler for the `netdev_event_thread` module.
+ *
+ * @param event pointer to the irq handler event.
+ */
+extern void netdev_event_thread_handler(event_t *event);
+
+/**
+ * @brief Initialize the event handler for a given device.
+ *
+ *        This function is only available if the `netdev_event_thread` module
+ *        is used.
+ *
+ * @note  This function MUST be called before `dev->driver->init`.
+ *
+ * @param netdev network device descriptor
+ */
+static inline void netdev_event_thread_init(netdev_t *netdev)
+{
+    netdev->irq_handler.handler = netdev_event_thread_handler;
+}
+#endif
+
 /**
  * @brief Informs netdev there was an interrupt request from the network device.
  *
@@ -479,9 +514,13 @@ static inline int netdev_set_notsup(netdev_t *dev, netopt_t opt,
  */
 static inline void netdev_trigger_event_isr(netdev_t *netdev)
 {
+#if IS_USED(MODULE_NETDEV_EVENT_THREAD)
+    event_post(EVENT_PRIO_HIGHEST, &netdev->irq_handler);
+#else
     if (netdev->event_callback) {
         netdev->event_callback(netdev, NETDEV_EVENT_ISR);
     }
+#endif
 }
 #ifdef __cplusplus
 }

drivers/netdev/Makefile

@@ -0,0 +1,4 @@
+SUBMODULES := 1
+SUBMODULES_NOFORCE := 1
+
+include $(RIOTBASE)/Makefile.base

drivers/netdev/event_thread.c

@@ -0,0 +1,22 @@
+/*
+ * Copyright (C) 2020 HAW Hamburg
+ *
+ * This file is subject to the terms and conditions of the GNU Lesser
+ * General Public License v2.1. See the file LICENSE in the top level
+ * directory for more details.
+ */
+
+/**
+ * @ingroup     drivers_netdev
+ * @{
+ *
+ * @file
+ * @author  José I. Alamos <[email protected]>
+ */
+
+#include "net/netdev.h"
+
+void netdev_event_thread_handler(event_t *event) {
+    netdev_t *dev = container_of(event, netdev_t, irq_handler);
+    dev->driver->isr(dev);
+}

makefiles/pseudomodules.inc.mk

@@ -57,7 +57,7 @@ PSEUDOMODULES += log_color
 PSEUDOMODULES += lora
 PSEUDOMODULES += mpu_stack_guard
 PSEUDOMODULES += nanocoap_%
-PSEUDOMODULES += netdev_default
+PSEUDOMODULES += netdev_%
 PSEUDOMODULES += netstats
 PSEUDOMODULES += netstats_l2
 PSEUDOMODULES += netstats_ipv6