mirror of
https://github.com/torvalds/linux.git
synced 2026-06-04 12:35:52 +02:00
rtlwifi: Fix Smatch warnings
Smatch reports the following: CHECK drivers/net/wireless/realtek/rtlwifi/pci.c drivers/net/wireless/realtek/rtlwifi/pci.c:366 rtl_pci_check_buddy_priv() error: we previously assumed 'tpriv' could be null (see line 368) drivers/net/wireless/realtek/rtlwifi/pci.c:1216 _rtl_pci_init_struct() warn: inconsistent indenting Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
This commit is contained in:
parent
baa1702290
commit
37c52934c6
|
|
@ -359,30 +359,28 @@ static bool rtl_pci_check_buddy_priv(struct ieee80211_hw *hw,
|
|||
struct rtl_priv *rtlpriv = rtl_priv(hw);
|
||||
struct rtl_pci_priv *pcipriv = rtl_pcipriv(hw);
|
||||
bool find_buddy_priv = false;
|
||||
struct rtl_priv *tpriv = NULL;
|
||||
struct rtl_priv *tpriv;
|
||||
struct rtl_pci_priv *tpcipriv = NULL;
|
||||
|
||||
if (!list_empty(&rtlpriv->glb_var->glb_priv_list)) {
|
||||
list_for_each_entry(tpriv, &rtlpriv->glb_var->glb_priv_list,
|
||||
list) {
|
||||
if (tpriv) {
|
||||
tpcipriv = (struct rtl_pci_priv *)tpriv->priv;
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD,
|
||||
"pcipriv->ndis_adapter.funcnumber %x\n",
|
||||
pcipriv->ndis_adapter.funcnumber);
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD,
|
||||
"tpcipriv->ndis_adapter.funcnumber %x\n",
|
||||
tpcipriv->ndis_adapter.funcnumber);
|
||||
tpcipriv = (struct rtl_pci_priv *)tpriv->priv;
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD,
|
||||
"pcipriv->ndis_adapter.funcnumber %x\n",
|
||||
pcipriv->ndis_adapter.funcnumber);
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD,
|
||||
"tpcipriv->ndis_adapter.funcnumber %x\n",
|
||||
tpcipriv->ndis_adapter.funcnumber);
|
||||
|
||||
if ((pcipriv->ndis_adapter.busnumber ==
|
||||
tpcipriv->ndis_adapter.busnumber) &&
|
||||
(pcipriv->ndis_adapter.devnumber ==
|
||||
tpcipriv->ndis_adapter.devnumber) &&
|
||||
(pcipriv->ndis_adapter.funcnumber !=
|
||||
tpcipriv->ndis_adapter.funcnumber)) {
|
||||
find_buddy_priv = true;
|
||||
break;
|
||||
}
|
||||
if ((pcipriv->ndis_adapter.busnumber ==
|
||||
tpcipriv->ndis_adapter.busnumber) &&
|
||||
(pcipriv->ndis_adapter.devnumber ==
|
||||
tpcipriv->ndis_adapter.devnumber) &&
|
||||
(pcipriv->ndis_adapter.funcnumber !=
|
||||
tpcipriv->ndis_adapter.funcnumber)) {
|
||||
find_buddy_priv = true;
|
||||
break;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
@ -1213,7 +1211,8 @@ static void _rtl_pci_init_struct(struct ieee80211_hw *hw,
|
|||
/*Tx/Rx related var */
|
||||
_rtl_pci_init_trx_var(hw);
|
||||
|
||||
/*IBSS*/ mac->beacon_interval = 100;
|
||||
/*IBSS*/
|
||||
mac->beacon_interval = 100;
|
||||
|
||||
/*AMPDU*/
|
||||
mac->min_space_cfg = 0;
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user