rsi: Fixed warnings reported by static code analyzers.
authorJahnavi Meher <jahnavi.meher@gmail.com>
Wed, 25 Jun 2014 10:18:15 +0000 (15:48 +0530)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 25 Jun 2014 19:40:41 +0000 (15:40 -0400)
Fixed a warning related to incorrect return type and removed an
unnecessary semi colon.

Signed-off-by: Jahnavi Meher <jahnavi.meher@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/rsi/rsi_91x_core.c

index 1cd0914db270773723da67d476a229c6ec788ce5..f3d3995d8f6b4c8ada23a5f25c6ad8035b108970 100644 (file)
@@ -92,6 +92,7 @@ static u32 rsi_get_num_pkts_dequeue(struct rsi_common *common, u8 q_num)
        struct sk_buff *skb;
        u32 pkt_cnt = 0;
        s16 txop = common->tx_qinfo[q_num].txop * 32;
+       __le16 r_txop;
        struct ieee80211_rate rate;
 
        rate.bitrate = RSI_RATE_MCS0 * 5 * 10; /* Convert to Kbps */
@@ -104,10 +105,11 @@ static u32 rsi_get_num_pkts_dequeue(struct rsi_common *common, u8 q_num)
                return 0;
 
        do {
-               txop -= ieee80211_generic_frame_duration(adapter->hw,
-                                                        adapter->vifs[0],
-                                                        common->band,
-                                                        skb->len, &rate);
+               r_txop = ieee80211_generic_frame_duration(adapter->hw,
+                                                         adapter->vifs[0],
+                                                         common->band,
+                                                         skb->len, &rate);
+               txop -= le16_to_cpu(r_txop);
                pkt_cnt += 1;
                /*checking if pkts are still there*/
                if (skb_queue_len(&common->tx_queue[q_num]) - pkt_cnt)
@@ -191,7 +193,7 @@ get_queue_num:
        if (q_num == VO_Q || q_num == VI_Q) {
                common->pkt_cnt = rsi_get_num_pkts_dequeue(common, q_num);
                common->pkt_cnt -= 1;
-       };
+       }
 
        return q_num;
 }