mirror of
https://github.com/torvalds/linux.git
synced 2026-09-29 12:24:02 +02:00
staging: rtl8723bs: remove redundant return variable in rtw_make_wlanhdr()
The temporary return variable is no longer needed because the only error path returns directly. Remove the redundant variable and exit label, and return _SUCCESS directly on the success path. No functional change intended. Signed-off-by: Hungyu Lin <dennylin0707@gmail.com> Reviewed-by: Dan Carpenter <error27@gmail.com> Link: https://patch.msgid.link/20260713070537.15903-2-dennylin0707@gmail.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
3536a7affc
commit
bc7493a390
|
|
@ -911,7 +911,6 @@ s32 rtw_make_wlanhdr(struct adapter *padapter, u8 *hdr, struct pkt_attrib *pattr
|
|||
struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
|
||||
struct qos_priv *pqospriv = &pmlmepriv->qospriv;
|
||||
u8 qos_option = false;
|
||||
signed int res = _SUCCESS;
|
||||
__le16 *fctrl = &pwlanhdr->frame_control;
|
||||
|
||||
memset(hdr, 0, WLANHDR_OFFSET);
|
||||
|
|
@ -951,8 +950,7 @@ s32 rtw_make_wlanhdr(struct adapter *padapter, u8 *hdr, struct pkt_attrib *pattr
|
|||
if (pattrib->qos_en)
|
||||
qos_option = true;
|
||||
} else {
|
||||
res = _FAIL;
|
||||
goto exit;
|
||||
return _FAIL;
|
||||
}
|
||||
|
||||
if (pattrib->mdata)
|
||||
|
|
@ -1026,8 +1024,7 @@ s32 rtw_make_wlanhdr(struct adapter *padapter, u8 *hdr, struct pkt_attrib *pattr
|
|||
} else {
|
||||
}
|
||||
|
||||
exit:
|
||||
return res;
|
||||
return _SUCCESS;
|
||||
}
|
||||
|
||||
s32 rtw_txframes_pending(struct adapter *padapter)
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user