staging: wilc1000: Modify tcp_process to return void rather than int
authorJanani Ravichandran <janani.rvchndrn@gmail.com>
Tue, 16 Feb 2016 19:07:00 +0000 (14:07 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 20 Feb 2016 00:58:06 +0000 (16:58 -0800)
tcp_process() returns int, which is not used anywhere. So, change
function header to return void intead of int and remove all uses of
the local variable ret, which is used as a return variable in the
function.

Signed-off-by: Janani Ravichandran <janani.rvchndrn@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wilc1000/wilc_wlan.c

index c44b87896d4eaa1bd48aed322a8854f8aabbd991..391173b752efad4f0d60f0da4d0283146d667f9f 100644 (file)
@@ -207,9 +207,8 @@ static inline int add_tcp_pending_ack(u32 ack, u32 session_index,
        return 0;
 }
 
-static inline int tcp_process(struct net_device *dev, struct txq_entry_t *tqe)
+static inline void tcp_process(struct net_device *dev, struct txq_entry_t *tqe)
 {
-       int ret;
        u8 *eth_hdr_ptr;
        u8 *buffer = tqe->buffer;
        unsigned short h_proto;
@@ -267,14 +266,9 @@ static inline int tcp_process(struct net_device *dev, struct txq_entry_t *tqe)
                                add_tcp_pending_ack(ack_no, i, tqe);
                        }
 
-               } else {
-                       ret = 0;
                }
-       } else {
-               ret = 0;
        }
        spin_unlock_irqrestore(&wilc->txq_spinlock, flags);
-       return ret;
 }
 
 static int wilc_wlan_txq_filter_dup_tcp_ack(struct net_device *dev)