[PATCH 1/9] rsi: cleanup and added debug prints in file rsi_91x_core.c

[Prameela Rani Garnepudi]

Signed-off-by: Prameela Rani Garnepudi <prameela.j0...@gmail.com>
---
 drivers/net/wireless/rsi/rsi_91x_core.c | 24 +++++++++++++++++++-----
 1 file changed, 19 insertions(+), 5 deletions(-)

diff --git a/drivers/net/wireless/rsi/rsi_91x_core.c 
b/drivers/net/wireless/rsi/rsi_91x_core.c
index f3d3995..b989ec5 100644
--- a/drivers/net/wireless/rsi/rsi_91x_core.c
+++ b/drivers/net/wireless/rsi/rsi_91x_core.c
@@ -142,8 +142,10 @@ static u8 rsi_core_determine_hal_queue(struct rsi_common 
*common)
                return q_num;
        }
 
-       if (common->hw_data_qs_blocked)
+       if (common->hw_data_qs_blocked) {
+               rsi_dbg(INFO_ZONE, "%s: data queue blocked\n", __func__);
                return q_num;
+       }
 
        if (common->pkt_cnt != 0) {
                --common->pkt_cnt;
@@ -210,6 +212,7 @@ static void rsi_core_queue_pkt(struct rsi_common *common,
                               struct sk_buff *skb)
 {
        u8 q_num = skb->priority;
+
        if (q_num >= NUM_SOFT_QUEUES) {
                rsi_dbg(ERR_ZONE, "%s: Invalid Queue Number: q_num = %d\n",
                        __func__, q_num);
@@ -285,7 +288,7 @@ void rsi_core_qos_processor(struct rsi_common *common)
                }
 
                skb = rsi_core_dequeue_pkt(common, q_num);
-               if (skb == NULL) {
+               if (!skb) {
                        rsi_dbg(ERR_ZONE, "skb null\n");
                        mutex_unlock(&common->tx_rxlock);
                        break;
@@ -331,21 +334,32 @@ void rsi_core_xmit(struct rsi_common *common, struct 
sk_buff *skb)
                        __func__);
                goto xmit_fail;
        }
-       info = IEEE80211_SKB_CB(skb);
-       tx_params = (struct skb_info *)info->driver_data;
-       tmp_hdr = (struct ieee80211_hdr *)&skb->data[0];
 
        if (common->fsm_state != FSM_MAC_INIT_DONE) {
                rsi_dbg(ERR_ZONE, "%s: FSM state not open\n", __func__);
                goto xmit_fail;
        }
 
+       info = IEEE80211_SKB_CB(skb);
+       tx_params = (struct skb_info *)info->driver_data;
+       tmp_hdr = (struct ieee80211_hdr *)&skb->data[0];
+
        if ((ieee80211_is_mgmt(tmp_hdr->frame_control)) ||
            (ieee80211_is_ctl(tmp_hdr->frame_control)) ||
            (ieee80211_is_qos_nullfunc(tmp_hdr->frame_control))) {
                q_num = MGMT_SOFT_Q;
                skb->priority = q_num;
+               if (ieee80211_is_probe_req(tmp_hdr->frame_control))
+                       rsi_dbg(MGMT_TX_ZONE, "%s: Probe Request\n", __func__);
+               else if (ieee80211_is_auth(tmp_hdr->frame_control))
+                       rsi_dbg(MGMT_TX_ZONE, "%s: Auth Request\n", __func__);
+               else if (ieee80211_is_assoc_req(tmp_hdr->frame_control))
+                       rsi_dbg(MGMT_TX_ZONE, "%s: Assoc Request\n", __func__);
+               else
+                       rsi_dbg(MGMT_TX_ZONE, "%s: pkt_type=%04x\n",
+                               __func__, tmp_hdr->frame_control);
        } else {
+               rsi_dbg(DATA_TX_ZONE, "%s: Data Packet\n", __func__);
                if (ieee80211_is_data_qos(tmp_hdr->frame_control)) {
                        tid = (skb->data[24] & IEEE80211_QOS_TID);
                        skb->priority = TID_TO_WME_AC(tid);
-- 
2.4.11