wireless: fix warning introduced by "mac80211: QoS related cleanups"
authorJohn W. Linville <linville@tuxdriver.com>
Thu, 1 May 2008 21:07:32 +0000 (17:07 -0400)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 7 May 2008 19:02:27 +0000 (15:02 -0400)
net/mac80211/wme.c: In function ‘wme_qdiscop_enqueue’:
net/mac80211/wme.c:219: warning: comparison is always false due to limited range of data type

drivers/net/wireless/p54/p54common.c: In function ‘p54_conf_tx’:
drivers/net/wireless/p54/p54common.c:947: warning: comparison is always false due to limited range of data type

Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/p54/p54common.c
net/mac80211/wme.c

index 34b91cc..33d608a 100644 (file)
@@ -944,7 +944,7 @@ static int p54_conf_tx(struct ieee80211_hw *dev, u16 queue,
        vdcf = (struct p54_tx_control_vdcf *)(((struct p54_control_hdr *)
                ((void *)priv->cached_vdcf + priv->tx_hdr_len))->data);
 
-       if ((params) && !((queue < 0) || (queue > 4))) {
+       if ((params) && !(queue > 4)) {
                P54_SET_QUEUE(vdcf->queue[queue], params->aifs,
                        params->cw_min, params->cw_max, params->txop);
        } else
index 5eddf1f..b1e20ca 100644 (file)
@@ -216,20 +216,15 @@ static int wme_qdiscop_enqueue(struct sk_buff *skb, struct Qdisc* qd)
                rcu_read_unlock();
        }
 
-       if (unlikely(queue < 0)) {
-                       kfree_skb(skb);
-                       err = NET_XMIT_DROP;
-       } else {
-               tid = skb->priority & QOS_CONTROL_TAG1D_MASK;
-               pkt_data->queue = (unsigned int) queue;
-               qdisc = q->queues[queue];
-               err = qdisc->enqueue(skb, qdisc);
-               if (err == NET_XMIT_SUCCESS) {
-                       qd->q.qlen++;
-                       qd->bstats.bytes += skb->len;
-                       qd->bstats.packets++;
-                       return NET_XMIT_SUCCESS;
-               }
+       tid = skb->priority & QOS_CONTROL_TAG1D_MASK;
+       pkt_data->queue = (unsigned int) queue;
+       qdisc = q->queues[queue];
+       err = qdisc->enqueue(skb, qdisc);
+       if (err == NET_XMIT_SUCCESS) {
+               qd->q.qlen++;
+               qd->bstats.bytes += skb->len;
+               qd->bstats.packets++;
+               return NET_XMIT_SUCCESS;
        }
        qd->qstats.drops++;
        return err;