From: Janani Ravichandran Date: Tue, 16 Feb 2016 19:07:00 +0000 (-0500) Subject: staging: wilc1000: Modify tcp_process to return void rather than int X-Git-Tag: Ubuntu-5.13.0-19.19~16182^2~775 X-Git-Url: https://git.proxmox.com/?a=commitdiff_plain;h=aae96201517678c241ea94da42a6756f80215362;p=mirror_ubuntu-jammy-kernel.git staging: wilc1000: Modify tcp_process to return void rather than int 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 Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/wilc1000/wilc_wlan.c b/drivers/staging/wilc1000/wilc_wlan.c index c44b87896d4e..391173b752ef 100644 --- a/drivers/staging/wilc1000/wilc_wlan.c +++ b/drivers/staging/wilc1000/wilc_wlan.c @@ -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)