rsi: rsi_91x: misleading debug printk
Subsystems (2)
PATCH 1/1 rsi: rsi_91x: misleading debug printk
There is a missing set of curly braces here so the debug output says "Probe confirm received" unintentionally.
Signed-off-by Dan Carpenter <[email protected]>
drivers/net/wireless/rsi/rsi_91x_mgmt.c +2 -1
@@ -1292,10 +1292,11 @@ int rsi_mgmt_pkt_recv(struct rsi_common *common, u8 *msg)return -EINVAL ; } }else if (msg_type =TX_STATUS_IND ) { -if (msg[15 ] =PROBEREQ_CONFIRM ) +if (msg[15 ] =PROBEREQ_CONFIRM ) { common->mgmt_q_block = false;rsi_dbg (FSM_ZONE ,"%s: Probe confirm received\n" , __func__); + } }else {return rsi_mgmt_pkt_to_core (common, msg, msg_len, msg_type); }
;-)
goto fail;
goto fail;
Thanks, Dan!
Quoted 23 lines · Dan Carpenter
There is a missing set of curly braces here so the debug output says "Probe confirm received" unintentionally.
Signed-off-by Dan Carpenter <[email protected]>drivers/net/wireless/rsi/rsi_91x_mgmt.c +2
-1@@ -1292,10 +1292,11 @@ int rsi_mgmt_pkt_recv(struct rsi_common *common, u8 *msg)return -EINVAL ; } }else if (msg_type =TX_STATUS_IND ) { -if (msg[15 ] =PROBEREQ_CONFIRM ) +if (msg[15 ] =PROBEREQ_CONFIRM ) { common->mgmt_q_block = false;rsi_dbg (FSM_ZONE ,"%s: Probe confirm received\n" , __func__); + } }else {return rsi_mgmt_pkt_to_core (common, msg, msg_len, msg_type); }
Signature · 2 lines
John W. Linville Someday the world will need a hero, and you [email protected] might be all we have. Be ready.