Hi Andi, Thanks for the v2 patch series. > adapter->bss_prio_tbl list has already been checked in outer loop. The > inner loop works with priv_tmp->wmm.tid_tbl_ptr list. Also the lock taken, > gives hint that this is likely a copy-paste error. > > Signed-off-by: Andreas Fenkart <andreas.fenkart@xxxxxxxxxxxxxxxxxxx> Acked-by: Bing Zhao <bzhao@xxxxxxxxxxx> Thanks, Bing > --- > drivers/net/wireless/mwifiex/wmm.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/mwifiex/wmm.c b/drivers/net/wireless/mwifiex/wmm.c > index 32adc87..b132c42 100644 > --- a/drivers/net/wireless/mwifiex/wmm.c > +++ b/drivers/net/wireless/mwifiex/wmm.c > @@ -933,8 +933,7 @@ mwifiex_wmm_get_highest_priolist_ptr(struct mwifiex_adapter *adapter, > spin_lock_irqsave(&tid_ptr->tid_tbl_lock, > flags); > is_list_empty = > - list_empty(&adapter->bss_prio_tbl[j] > - .bss_prio_head); > + list_empty(&tid_ptr->ra_list); > spin_unlock_irqrestore(&tid_ptr->tid_tbl_lock, > flags); > if (is_list_empty) > -- > 1.7.10.4 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
