sys/stdio_uart: do not cast function pointer.

drivers/at/at.c

@@ -22,17 +22,12 @@
 #define AT_PRINT_INCOMING (0)
 #endif
 
-static void _isrpipe_write_one_wrapper(void *_isrpipe, uint8_t data)
-{
-    isrpipe_write_one(_isrpipe, (char)data);
-}
-
 int at_dev_init(at_dev_t *dev, uart_t uart, uint32_t baudrate, char *buf, size_t bufsize)
 {
     dev->uart = uart;
     isrpipe_init(&dev->isrpipe, buf, bufsize);
 
-    return uart_init(uart, baudrate, _isrpipe_write_one_wrapper,
+    return uart_init(uart, baudrate, isrpipe_write_uartcb,
                      &dev->isrpipe);
 }
 

sys/include/isrpipe.h

@@ -64,6 +64,13 @@ void isrpipe_init(isrpipe_t *isrpipe, char *buf, size_t bufsize);
  */
 int isrpipe_write_one(isrpipe_t *isrpipe, char c);
 
+/**
+ * @brief   Wrapper around isrpipe_write_one() to make it compatible with uart_rx_cb_t.
+ *
+ * The parameters are the same as isrpipe_write_one().
+ */
+void isrpipe_write_uartcb(void *_isrpipe, uint8_t data);
+
 /**
  * @brief   Read data from isrpipe (blocking)
  *

sys/isrpipe/isrpipe.c

@@ -17,6 +17,7 @@
  * @}
  */
 
+#include <stdint.h>
 #include "isrpipe.h"
 
 void isrpipe_init(isrpipe_t *isrpipe, char *buf, size_t bufsize)
@@ -37,6 +38,11 @@ int isrpipe_write_one(isrpipe_t *isrpipe, char c)
     return res;
 }
 
+void isrpipe_write_uartcb(void *_isrpipe, uint8_t data)
+{
+    isrpipe_write_one(_isrpipe, (char)data);
+}
+
 int isrpipe_read(isrpipe_t *isrpipe, char *buffer, size_t count)
 {
     int res;

sys/stdio_uart/Makefile

@@ -1,4 +1 @@
-ifeq (gnu, $(TOOLCHAIN))
-  CFLAGS += -Wno-cast-function-type
-endif
 include $(RIOTBASE)/Makefile.base

sys/stdio_uart/stdio_uart.c

@@ -58,7 +58,7 @@ void stdio_init(void)
     void *arg;
 
 #ifdef MODULE_STDIO_UART_RX
-    cb = (uart_rx_cb_t) isrpipe_write_one;
+    cb = isrpipe_write_uartcb;
     arg = &stdio_uart_isrpipe;
 #else
 #ifdef USE_ETHOS_FOR_STDIO