Commit 324960df authored by Lee Jones's avatar Lee Jones Committed by Greg Kroah-Hartman
Browse files

staging: rtl8723bs: core: rtw_sta_mgt: Return error value directly 



Fixes the following W=1 kernel build warning(s):

 drivers/staging/rtl8723bs/core/rtw_sta_mgt.c: In function ‘rtw_init_bcmc_stainfo’:
 drivers/staging/rtl8723bs/core/rtw_sta_mgt.c:512:6: warning: variable ‘res’ set but not used [-Wunused-but-set-variable]

Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Ross Schmidt <ross.schm.dev@gmail.com>
Cc: linux-staging@lists.linux.dev
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
Link: https://lore.kernel.org/r/20210414181129.1628598-18-lee.jones@linaro.org


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

drivers/staging/rtl8723bs/core/rtw_sta_mgt.c

+2 −6
Original line number Diff line number Diff line
@@ -497,7 +497,6 @@ struct sta_info *rtw_get_stainfo(struct sta_priv *pstapriv, u8 *hwaddr) 
u32 rtw_init_bcmc_stainfo(struct adapter *padapter)

{

	struct sta_info *psta;

	u32 res = _SUCCESS;

	NDIS_802_11_MAC_ADDRESS	bcast_addr = {0xff, 0xff, 0xff, 0xff, 0xff, 0xff};



	struct	sta_priv *pstapriv = &padapter->stapriv;
@@ -505,15 +504,12 @@ u32 rtw_init_bcmc_stainfo(struct adapter *padapter) 


	psta = rtw_alloc_stainfo(pstapriv, bcast_addr);



	if (!psta) {

		res = _FAIL;

		goto exit;

	}

	if (!psta)

		return _FAIL;



	/*  default broadcast & multicast use macid 1 */

	psta->mac_id = 1;



exit:

	return _SUCCESS;

}