Commit 6b116455 authored by Martin Kaiser's avatar Martin Kaiser Committed by Greg Kroah-Hartman
Browse files

staging: rtl8188eu: clean up rtw_recv_entry 



Change the return type to int, the function returns 0 or 1.
Remove the goto statement, we're not doing any cleanup on exit.
Summarize variable declarations and assignments.

Signed-off-by: default avatarMartin Kaiser <martin@kaiser.cx>
Link: https://lore.kernel.org/r/20210407170531.29356-6-martin@kaiser.cx


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 889ed8b5

drivers/staging/rtl8188eu/core/rtw_recv.c

+7 −16
Original line number Diff line number Diff line
@@ -1962,28 +1962,19 @@ static int recv_func(struct adapter *padapter, struct recv_frame *rframe) 
	return ret;

}



s32 rtw_recv_entry(struct recv_frame *precvframe)

int rtw_recv_entry(struct recv_frame *precvframe)

{

	struct adapter *padapter;

	struct recv_priv *precvpriv;

	s32 ret = _SUCCESS;



	padapter = precvframe->adapter;



	precvpriv = &padapter->recvpriv;

	struct adapter *padapter = precvframe->adapter;

	struct recv_priv *precvpriv = &padapter->recvpriv;

	int ret;



	ret = recv_func(padapter, precvframe);

	if (ret == _FAIL) {

		RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("%s: recv_func return fail!!!\n", __func__));

		goto _recv_entry_drop;

	}



	if (ret == _SUCCESS)

		precvpriv->rx_pkts++;

	else

		RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("%s: recv_func return fail!!!\n", __func__));



	return ret;



_recv_entry_drop:

	return ret;

}



static void rtw_signal_stat_timer_hdl(struct timer_list *t)