diff options
| author | Ravi kumar Veeramally <ravikumar.veeramally@linux.intel.com> | 2016-03-02 17:50:15 +0200 |
|---|---|---|
| committer | Jukka Rissanen <jukka.rissanen@linux.intel.com> | 2016-03-03 07:17:56 +0000 |
| commit | bf90337e8720915b3e03c8a33a95a118c3a3d1b1 (patch) | |
| tree | aa00fb025c264c9f5961abccf3aa1fa4885903c2 | |
| parent | fe67b69cd0162abcfd561e909805143a8015463a (diff) | |
net: 802.15.4: Fix the dummy driver
Do not depend on uart_pipe_send return value. Just assume
uart_pipe_send call to uart_poll_out sent data through uart.
Change-Id: Ifc7119f94b625d4f2895d92d44d660013bf50145
Signed-off-by: Ravi kumar Veeramally <ravikumar.veeramally@linux.intel.com>
| -rw-r--r-- | net/ip/dummy_15_4_radio.c | 20 |
1 files changed, 5 insertions, 15 deletions
diff --git a/net/ip/dummy_15_4_radio.c b/net/ip/dummy_15_4_radio.c index 3ac1412a0..3f9c7ec96 100644 --- a/net/ip/dummy_15_4_radio.c +++ b/net/ip/dummy_15_4_radio.c @@ -141,7 +141,7 @@ done: #endif #if defined CONFIG_NETWORKING_WITH_15_4_LOOPBACK_UART -static int uart_send(unsigned char c) +static void uart_send(unsigned char c) { uint8_t buf[1] = { c }; @@ -150,7 +150,7 @@ static int uart_send(unsigned char c) __FUNCTION__, buf[0]); #endif - return uart_pipe_send(&buf[0], 1); + uart_pipe_send(&buf[0], 1); } #endif @@ -235,21 +235,11 @@ send(struct net_buf *buf, const void *payload, unsigned short payload_len) PRINTF("dummy154radio: sending %d bytes\n", len); } - if(!uart_send(DUMMY_RADIO_15_4_FRAME_TYPE)) { /* Type */ - PRINTF("uart_send failed (frame type)\n"); - return RADIO_TX_ERR; - } - - if(!uart_send(len)) { /* Length */ - PRINTF("uart_send failed (length)\n"); - return RADIO_TX_ERR; - } + uart_send(DUMMY_RADIO_15_4_FRAME_TYPE); /* Type */ + uart_send(len); /* Length */ for (i = 0; i < len; i++) { - if (!uart_send(output[i])) { - PRINTF("uart_send failed\n"); - return RADIO_TX_ERR; - } + uart_send(output[i]); } return RADIO_TX_OK; |