Remove spurious semicolon resulting in ieee80211_sta_set_buffered() being always called independent from the result of the expression evaluated by the preceding if-clause.
Spotted by gcc-6 while attempting to build the driver outside of the buildroot. Signed-off-by: Daniel Golle <dan...@makrotopia.org> --- tx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tx.c b/tx.c index 05313b6..f13afa2 100644 --- a/tx.c +++ b/tx.c @@ -198,7 +198,7 @@ mt76_txq_dequeue(struct mt76_dev *dev, struct mt76_txq *mtxq, bool ps) if (skb) { u8 tid = skb->priority & IEEE80211_QOS_CTL_TID_MASK; - if (ps && skb_queue_empty(&mtxq->retry_q)); + if (ps && skb_queue_empty(&mtxq->retry_q)) ieee80211_sta_set_buffered(txq->sta, tid, false); return skb; -- 2.9.0 _______________________________________________ Lede-dev mailing list Lede-dev@lists.infradead.org http://lists.infradead.org/mailman/listinfo/lede-dev