[PATCH 2/7] smd: add HAL debug messages

Kalle Valo kvalo at qca.qualcomm.com
Thu May 16 08:42:12 EDT 2013


Easier to follow what's wcn36xx is doing.

Signed-off-by: Kalle Valo <kvalo at qca.qualcomm.com>
---
 smd.c     |   60 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++----
 wcn36xx.h |    1 +
 2 files changed, 57 insertions(+), 4 deletions(-)

diff --git a/smd.c b/smd.c
index 9ea9f7b..7a37025 100644
--- a/smd.c
+++ b/smd.c
@@ -161,6 +161,8 @@ int wcn36xx_smd_init_scan(struct wcn36xx *wcn)
 
 	PREPARE_HAL_BUF(wcn->smd_buf, msg_body);
 
+	wcn36xx_dbg(WCN36XX_DBG_HAL, "hal init scan mode %d", msg_body.mode);
+
 	return wcn36xx_smd_send_and_wait(wcn, msg_body.header.len);
 }
 
@@ -174,6 +176,9 @@ int wcn36xx_smd_start_scan(struct wcn36xx *wcn, u8 ch)
 
 	PREPARE_HAL_BUF(wcn->smd_buf, msg_body);
 
+	wcn36xx_dbg(WCN36XX_DBG_HAL, "hal start scan channel %d",
+		    msg_body.scan_channel);
+
 	return wcn36xx_smd_send_and_wait(wcn, msg_body.header.len);
 }
 int wcn36xx_smd_end_scan(struct wcn36xx *wcn, u8 ch)
@@ -186,6 +191,9 @@ int wcn36xx_smd_end_scan(struct wcn36xx *wcn, u8 ch)
 
 	PREPARE_HAL_BUF(wcn->smd_buf, msg_body);
 
+	wcn36xx_dbg(WCN36XX_DBG_HAL, "hal end scan channel %d",
+		    msg_body.scan_channel);
+
 	return wcn36xx_smd_send_and_wait(wcn, msg_body.header.len);
 }
 int wcn36xx_smd_finish_scan(struct wcn36xx *wcn)
@@ -198,6 +206,9 @@ int wcn36xx_smd_finish_scan(struct wcn36xx *wcn)
 
 	PREPARE_HAL_BUF(wcn->smd_buf, msg_body);
 
+	wcn36xx_dbg(WCN36XX_DBG_HAL, "hal finish scan mode %d",
+		    msg_body.mode);
+
 	return wcn36xx_smd_send_and_wait(wcn, msg_body.header.len);
 }
 int wcn36xx_smd_update_scan_params(struct wcn36xx *wcn){
@@ -217,6 +228,10 @@ int wcn36xx_smd_update_scan_params(struct wcn36xx *wcn){
 
 	PREPARE_HAL_BUF(wcn->smd_buf, msg_body);
 
+	wcn36xx_dbg(WCN36XX_DBG_HAL,
+		    "hal update scan params channel_count %d",
+		    msg_body.channel_count);
+
 	return wcn36xx_smd_send_and_wait(wcn, msg_body.header.len);
 }
 static int wcn36xx_smd_update_scan_params_rsp(void *buf, size_t len)
@@ -225,7 +240,10 @@ static int wcn36xx_smd_update_scan_params_rsp(void *buf, size_t len)
 
 	rsp = (struct wcn36xx_hal_update_scan_params_resp *)buf;
 
-	wcn36xx_info("Scan params stattus=0x%x",rsp->status);
+	wcn36xx_dbg(WCN36XX_DBG_HAL,
+		    "hal rsp update scan params status %d",
+		    rsp->status);
+
 	return 0;
 }
 
@@ -240,6 +258,10 @@ int wcn36xx_smd_add_sta_self(struct wcn36xx *wcn, struct mac_address addr, u32 s
 
 	PREPARE_HAL_BUF(wcn->smd_buf, msg_body);
 
+	wcn36xx_dbg(WCN36XX_DBG_HAL,
+		    "hal add sta self self_addr %pM status %d",
+		    msg_body.self_addr, msg_body.status);
+
 	return wcn36xx_smd_send_and_wait(wcn, msg_body.header.len);
 }
 int wcn36xx_smd_delete_sta(struct wcn36xx *wcn)
@@ -252,6 +274,10 @@ int wcn36xx_smd_delete_sta(struct wcn36xx *wcn)
 
 	PREPARE_HAL_BUF(wcn->smd_buf, msg_body);
 
+	wcn36xx_dbg(WCN36XX_DBG_HAL,
+		    "hal delete sta sta_index %d",
+		    msg_body.sta_index);
+
 	return wcn36xx_smd_send_and_wait(wcn, msg_body.header.len);
 
 }
@@ -270,6 +296,11 @@ int wcn36xx_smd_join(struct wcn36xx *wcn, u8 *bssid, u8 *vif, u8 ch)
 	msg_body.max_tx_power = 0xbf;
 	PREPARE_HAL_BUF(wcn->smd_buf, msg_body);
 
+	wcn36xx_dbg(WCN36XX_DBG_HAL,
+		    "hal join req bssid %pM self_sta_mac_addr %pM channel %d link_state %d",
+		    msg_body.bssid, msg_body.self_sta_mac_addr,
+		    msg_body.channel, msg_body.link_state);
+
 	return wcn36xx_smd_send_and_wait(wcn, msg_body.header.len);
 }
 int wcn36xx_smd_set_link_st(struct wcn36xx *wcn, u8 *bssid, u8 *sta_mac, enum wcn36xx_hal_link_state state)
@@ -284,6 +315,10 @@ int wcn36xx_smd_set_link_st(struct wcn36xx *wcn, u8 *bssid, u8 *sta_mac, enum wc
 
 	PREPARE_HAL_BUF(wcn->smd_buf, msg_body);
 
+	wcn36xx_dbg(WCN36XX_DBG_HAL,
+		    "hal set link state bssid %pM self_mac_addr %pM state %d",
+		    msg_body.bssid, msg_body.self_mac_addr, msg_body.state);
+
 	return wcn36xx_smd_send_and_wait(wcn, msg_body.header.len);
 }
 
@@ -311,6 +346,10 @@ int wcn36xx_smd_config_sta(struct wcn36xx *wcn, u8 *bssid, u16 ass_id, u8 *sta_m
 
 	PREPARE_HAL_BUF(wcn->smd_buf, msg_body);
 
+	wcn36xx_dbg(WCN36XX_DBG_HAL,
+		    "hal config sta bssid %pM mac %pM",
+		    msg_body.u.sta_params.bssid, msg_body.u.sta_params.mac);
+
 	return wcn36xx_smd_send_and_wait(wcn, msg_body.header.len);
 }
 static int wcn36xx_smd_join_rsp(void *buf, size_t len)
@@ -322,8 +361,10 @@ static int wcn36xx_smd_join_rsp(void *buf, size_t len)
 
 	rsp = (struct wcn36xx_hal_join_rsp_msg *)buf;
 
-	wcn36xx_info("Join stattus=%d, Power ver=%d",
-		rsp->status, rsp->tx_mgmt_power);
+	wcn36xx_dbg(WCN36XX_DBG_HAL,
+		    "hal rsp join status %d tx_mgmt_power %d",
+		    rsp->status, rsp->tx_mgmt_power);
+
 	return 0;
 }
 
@@ -398,6 +439,10 @@ int wcn36xx_smd_config_bss(struct wcn36xx *wcn, bool sta_mode, u8 *bssid, u8 upd
 
 	PREPARE_HAL_BUF(wcn->smd_buf, msg_body);
 
+	wcn36xx_dbg(WCN36XX_DBG_HAL,
+		    "hal config bss bss_type %d",
+		    msg_body.u.bss_params.bss_type);
+
 	return wcn36xx_smd_send_and_wait(wcn, msg_body.header.len);
 }
 int wcn36xx_smd_delete_bss(struct wcn36xx *wcn)
@@ -410,6 +455,8 @@ int wcn36xx_smd_delete_bss(struct wcn36xx *wcn)
 
 	PREPARE_HAL_BUF(wcn->smd_buf, msg_body);
 
+	wcn36xx_dbg(WCN36XX_DBG_HAL, "hal delete bss %d", msg_body.bss_index);
+
 	return wcn36xx_smd_send_and_wait(wcn, msg_body.header.len);
 }
 int wcn36xx_smd_send_beacon(struct wcn36xx *wcn, struct sk_buff *skb_beacon, u16 tim_off, u16 p2p_off){
@@ -434,6 +481,10 @@ int wcn36xx_smd_send_beacon(struct wcn36xx *wcn, struct sk_buff *skb_beacon, u16
 	msg_body.p2p_ie_offset = p2p_off;
 	PREPARE_HAL_BUF(wcn->smd_buf, msg_body);
 
+	wcn36xx_dbg(WCN36XX_DBG_HAL,
+		    "hal send beacon beacon_length %d",
+		    msg_body.beacon_length);
+
 	return wcn36xx_smd_send_and_wait(wcn, msg_body.header.len);
 };
 
@@ -491,7 +542,8 @@ static void wcn36xx_smd_rsp_process (void *buf, size_t len)
 		wcn36xx_smd_update_scan_params_rsp(buf, len);
 		break;
 	default:
-		wcn36xx_error("SMD_EVENT not supported");
+		wcn36xx_error("SMD_EVENT %d not supported",
+			      msg_header->msg_type);
 	}
 }
 
diff --git a/wcn36xx.h b/wcn36xx.h
index 82d73b2..e0302d9 100644
--- a/wcn36xx.h
+++ b/wcn36xx.h
@@ -43,6 +43,7 @@ enum wcn36xx_debug_mask {
 	WCN36XX_DBG_RX_DUMP	= 0x00000020,
 	WCN36XX_DBG_TX		= 0x00000040,
 	WCN36XX_DBG_TX_DUMP	= 0x00000080,
+	WCN36XX_DBG_HAL		= 0x00000100,
 	WCN36XX_DBG_ANY		= 0xffffffff,
 };
 




More information about the wcn36xx mailing list