staging: rtl8188eu: remove return variable from rtw_init_bcmc_stainfo
authorMichael Straube <straube.linux@gmail.com>
Sun, 27 Oct 2019 13:06:03 +0000 (14:06 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 28 Oct 2019 15:40:25 +0000 (16:40 +0100)
Remove variable res, that is used to store the return value, from
rtw_init_bcmc_stainfo. Instead return _FAIL or _SUCCESS directly
and remove the now unneeded exit label.

Signed-off-by: Michael Straube <straube.linux@gmail.com>
Link: https://lore.kernel.org/r/20191027130604.68379-3-straube.linux@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8188eu/core/rtw_sta_mgt.c

index 6c03068..8b7adb9 100644 (file)
@@ -451,24 +451,21 @@ 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;
        u8 bc_addr[ETH_ALEN] = {0xff, 0xff, 0xff, 0xff, 0xff, 0xff};
        struct sta_priv *pstapriv = &padapter->stapriv;
 
        psta = rtw_alloc_stainfo(pstapriv, bc_addr);
 
        if (!psta) {
-               res = _FAIL;
                RT_TRACE(_module_rtl871x_sta_mgt_c_, _drv_err_,
                         ("rtw_alloc_stainfo fail"));
-               goto exit;
+               return _FAIL;
        }
 
        /*  default broadcast & multicast use macid 1 */
        psta->mac_id = 1;
 
-exit:
-       return res;
+       return _SUCCESS;
 }
 
 struct sta_info *rtw_get_bcmc_stainfo(struct adapter *padapter)