I don't think that messages like

airo(eth1): cmd= 111

airo(eth1): status= 7f11

airo(eth1): Rsp0= 2

airo(eth1): Rsp1= 0

airo(eth1): Rsp2= 0

are interesting to normal users, so switch them from airo_print_err() to 
airo_print_dbg().
Furthermore, remove implicit linefeed in those messages, as they are handled by 
airo_print().

with kind regards,
Robert Schulze

--- linux-2.6.17.1/drivers/net/wireless/airo.c.orig     2006-07-08 
14:02:12.000000000 +0200
+++ linux-2.6.17.1/drivers/net/wireless/airo.c  2006-07-08 17:36:08.000000000 
+0200
@@ -3897,11 +3897,11 @@ static u16 issuecommand(struct airo_info
        pRsp->rsp1 = IN4500(ai, RESP1);
        pRsp->rsp2 = IN4500(ai, RESP2);
        if ((pRsp->status & 0xff00)!=0 && pCmd->cmd != CMD_SOFTRESET) {
-               airo_print_err(ai->dev->name, "cmd= %x\n", pCmd->cmd);
-               airo_print_err(ai->dev->name, "status= %x\n", pRsp->status);
-               airo_print_err(ai->dev->name, "Rsp0= %x\n", pRsp->rsp0);
-               airo_print_err(ai->dev->name, "Rsp1= %x\n", pRsp->rsp1);
-               airo_print_err(ai->dev->name, "Rsp2= %x\n", pRsp->rsp2);
+               airo_print_dbg(ai->dev->name, "cmd= %x", pCmd->cmd);
+               airo_print_dbg(ai->dev->name, "status= %x", pRsp->status);
+               airo_print_dbg(ai->dev->name, "Rsp0= %x", pRsp->rsp0);
+               airo_print_dbg(ai->dev->name, "Rsp1= %x", pRsp->rsp1);
+               airo_print_dbg(ai->dev->name, "Rsp2= %x", pRsp->rsp2);
        }

        // clear stuck command busy if necessary
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to