Staging: rt2860: Sanitize DBGPRINT_ERR macro

Cleaner implementation. Avoids the need of the double parenthesis to call the
macro.

Signed-off-by: L. Alberto Giménez <agimenez@sysvalve.es>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

diff --git a/drivers/staging/rt2860/chip/rtmp_phy.h b/drivers/staging/rt2860/chip/rtmp_phy.h
index 9f924ea6ca35b127bf7880fa11b375fc44869bae..98454df30a221b8bc6e8069917e2a456497ce516 100644 (file)
--- a/drivers/staging/rt2860/chip/rtmp_phy.h
+++ b/drivers/staging/rt2860/chip/rtmp_phy.h
@@ -247,7 +247,7 @@
                        }                                               \
                }                                                       \
                if (BbpCsr.field.Busy == BUSY) {                        \
-                       DBGPRINT_ERR(("BBP(viaMCU=%d) read R%d fail\n", (_bViaMCU), _bbpID));   \
+                       DBGPRINT_ERR("BBP(viaMCU=%d) read R%d fail\n", (_bViaMCU), _bbpID);     \
                        *(_pV) = (_pAd)->BbpWriteLatch[_bbpID];               \
                        if ((_bViaMCU) == TRUE) {                       \
                                RTMP_IO_READ32(_pAd, _regID, &BbpCsr.word);                             \
@@ -336,11 +336,11 @@
                        }                                               \
                }                                                       \
        } else {                                                        \
-               DBGPRINT_ERR((" , brt30xxBanMcuCmd = %d, Read BBP %d \n", (_A)->brt30xxBanMcuCmd, (_I)));       \
+               DBGPRINT_ERR(" , brt30xxBanMcuCmd = %d, Read BBP %d \n", (_A)->brt30xxBanMcuCmd, (_I)); \
                *(_pV) = (_A)->BbpWriteLatch[_I];                       \
        }                                                               \
        if ((BbpCsr.field.Busy == BUSY) || ((_A)->bPCIclkOff == TRUE)) { \
-               DBGPRINT_ERR(("BBP read R%d=0x%x fail\n", _I, BbpCsr.word)); \
+               DBGPRINT_ERR("BBP read R%d=0x%x fail\n", _I, BbpCsr.word); \
                *(_pV) = (_A)->BbpWriteLatch[_I];                       \
        }                                                               \
 }
@@ -378,7 +378,7 @@
                        break;                                          \
                }                                                       \
                if (_busyCnt == MAX_BUSY_COUNT) {                       \
-                       DBGPRINT_ERR(("BBP write R%d fail\n", _bbpID)); \
+                       DBGPRINT_ERR("BBP write R%d fail\n", _bbpID);   \
                        if ((_bViaMCU) == TRUE) {                       \
                                RTMP_IO_READ32(_pAd, H2M_BBP_AGENT, &BbpCsr.word);      \
                                BbpCsr.field.Busy = 0;                  \
@@ -459,15 +459,15 @@
                                break;                                                                                                                  \
                        }                                                                                                                                       \
                } else {                                                \
-                       DBGPRINT_ERR(("  brt30xxBanMcuCmd = %d. Write BBP %d \n",  (_A)->brt30xxBanMcuCmd, (_I)));      \
+                       DBGPRINT_ERR("  brt30xxBanMcuCmd = %d. Write BBP %d \n",  (_A)->brt30xxBanMcuCmd, (_I));        \
                }                                                                                                                                       \
                if ((BusyCnt == MAX_BUSY_COUNT) || ((_A)->bPCIclkOff == TRUE)) { \
                        if (BusyCnt == MAX_BUSY_COUNT)                          \
                                (_A)->AccessBBPFailCount++;                                     \
-                       DBGPRINT_ERR(("BBP write R%d=0x%x fail. BusyCnt= %d.bPCIclkOff = %d. \n", _I, BbpCsr.word, BusyCnt, (_A)->bPCIclkOff)); \
+                       DBGPRINT_ERR("BBP write R%d=0x%x fail. BusyCnt= %d.bPCIclkOff = %d. \n", _I, BbpCsr.word, BusyCnt, (_A)->bPCIclkOff);   \
                }                                                                                                                                       \
        } else {                                                        \
-               DBGPRINT_ERR(("****** BBP_Write_Latch Buffer exceeds max boundry ****** \n"));  \
+               DBGPRINT_ERR("****** BBP_Write_Latch Buffer exceeds max boundry ****** \n");    \
        }                                                                                                                                               \
 }
 #endif /* RTMP_MAC_PCI // */

diff --git a/drivers/staging/rt2860/common/ba_action.c b/drivers/staging/rt2860/common/ba_action.c
index ed8854b902d18517069b404d8144d92867f6c6bc..b046c2b814c53a017a908170368f81f37016b34b 100644 (file)
--- a/drivers/staging/rt2860/common/ba_action.c
+++ b/drivers/staging/rt2860/common/ba_action.c
@@ -1270,13 +1270,13 @@ BOOLEAN CntlEnqueueForRecv(struct rt_rtmp_adapter *pAd,
 
        /* First check the size, it MUST not exceed the mlme queue size */
        if (MsgLen > MGMT_DMA_BUFFER_SIZE) {
-               DBGPRINT_ERR(("CntlEnqueueForRecv: frame too large, size = %ld \n", MsgLen));
+               DBGPRINT_ERR("CntlEnqueueForRecv: frame too large, size = %ld \n", MsgLen);
                return FALSE;
        } else if (MsgLen != sizeof(struct rt_frame_ba_req)) {
-               DBGPRINT_ERR(("CntlEnqueueForRecv: BlockAck Request frame length size = %ld incorrect\n", MsgLen));
+               DBGPRINT_ERR("CntlEnqueueForRecv: BlockAck Request frame length size = %ld incorrect\n", MsgLen);
                return FALSE;
        } else if (MsgLen != sizeof(struct rt_frame_ba_req)) {
-               DBGPRINT_ERR(("CntlEnqueueForRecv: BlockAck Request frame length size = %ld incorrect\n", MsgLen));
+               DBGPRINT_ERR("CntlEnqueueForRecv: BlockAck Request frame length size = %ld incorrect\n", MsgLen);
                return FALSE;
        }
 

diff --git a/drivers/staging/rt2860/common/cmm_data.c b/drivers/staging/rt2860/common/cmm_data.c
index 93a53479d7664148f55032054bc7fbaade93b49e..2204c2bda386d84b477e4a07f18550a55918b8c9 100644 (file)
--- a/drivers/staging/rt2860/common/cmm_data.c
+++ b/drivers/staging/rt2860/common/cmm_data.c
@@ -1366,7 +1366,7 @@ void RTMPResumeMsduTransmission(struct rt_rtmp_adapter *pAd)
        /* R66 should not be 0 */
        if (pAd->BbpTuning.R66CurrentValue == 0) {
                pAd->BbpTuning.R66CurrentValue = 0x38;
-               DBGPRINT_ERR(("RTMPResumeMsduTransmission, R66CurrentValue=0...\n"));
+               DBGPRINT_ERR("RTMPResumeMsduTransmission, R66CurrentValue=0...\n");
        }
 
        RTMP_BBP_IO_WRITE8_BY_REG_ID(pAd, BBP_R66,

diff --git a/drivers/staging/rt2860/common/cmm_mac_pci.c b/drivers/staging/rt2860/common/cmm_mac_pci.c
index e26ba494287789b341af54c9d97714b2b9f88881..850f0fbc6d902768d92462c468ccc0cb7b890104 100644 (file)
--- a/drivers/staging/rt2860/common/cmm_mac_pci.c
+++ b/drivers/staging/rt2860/common/cmm_mac_pci.c
@@ -89,7 +89,7 @@ int RTMPAllocTxRxRingMemory(struct rt_rtmp_adapter *pAd)
 
                        if (pAd->TxDescRing[num].AllocVa == NULL) {
                                ErrorValue = ERRLOG_OUT_OF_SHARED_MEMORY;
-                               DBGPRINT_ERR(("Failed to allocate a big buffer\n"));
+                               DBGPRINT_ERR("Failed to allocate a big buffer\n");
                                Status = NDIS_STATUS_RESOURCES;
                                break;
                        }
@@ -121,7 +121,7 @@ int RTMPAllocTxRxRingMemory(struct rt_rtmp_adapter *pAd)
 
                        if (pAd->TxBufSpace[num].AllocVa == NULL) {
                                ErrorValue = ERRLOG_OUT_OF_SHARED_MEMORY;
-                               DBGPRINT_ERR(("Failed to allocate a big buffer\n"));
+                               DBGPRINT_ERR("Failed to allocate a big buffer\n");
                                Status = NDIS_STATUS_RESOURCES;
                                break;
                        }
@@ -197,7 +197,7 @@ int RTMPAllocTxRxRingMemory(struct rt_rtmp_adapter *pAd)
 
                if (pAd->MgmtDescRing.AllocVa == NULL) {
                        ErrorValue = ERRLOG_OUT_OF_SHARED_MEMORY;
-                       DBGPRINT_ERR(("Failed to allocate a big buffer\n"));
+                       DBGPRINT_ERR("Failed to allocate a big buffer\n");
                        Status = NDIS_STATUS_RESOURCES;
                        break;
                }
@@ -251,7 +251,7 @@ int RTMPAllocTxRxRingMemory(struct rt_rtmp_adapter *pAd)
 
                if (pAd->RxDescRing.AllocVa == NULL) {
                        ErrorValue = ERRLOG_OUT_OF_SHARED_MEMORY;
-                       DBGPRINT_ERR(("Failed to allocate a big buffer\n"));
+                       DBGPRINT_ERR("Failed to allocate a big buffer\n");
                        Status = NDIS_STATUS_RESOURCES;
                        break;
                }
@@ -304,7 +304,7 @@ int RTMPAllocTxRxRingMemory(struct rt_rtmp_adapter *pAd)
                        /* Error handling */
                        if (pDmaBuf->AllocVa == NULL) {
                                ErrorValue = ERRLOG_OUT_OF_SHARED_MEMORY;
-                               DBGPRINT_ERR(("Failed to allocate RxRing's 1st buffer\n"));
+                               DBGPRINT_ERR("Failed to allocate RxRing's 1st buffer\n");
                                Status = NDIS_STATUS_RESOURCES;
                                break;
                        }

diff --git a/drivers/staging/rt2860/common/cmm_mac_usb.c b/drivers/staging/rt2860/common/cmm_mac_usb.c
index 72731cbb81df0446a6d0b30337739d5fbc42475b..64a65a460c2982bc5996f39aea487637b8465dff 100644 (file)
--- a/drivers/staging/rt2860/common/cmm_mac_usb.c
+++ b/drivers/staging/rt2860/common/cmm_mac_usb.c
@@ -236,7 +236,7 @@ int NICInitTransmit(struct rt_rtmp_adapter *pAd)
                os_alloc_mem(pAd, (u8 **) (&pAd->MgmtDescRing.AllocVa),
                             pAd->MgmtDescRing.AllocSize);
                if (pAd->MgmtDescRing.AllocVa == NULL) {
-                       DBGPRINT_ERR(("Failed to allocate a big buffer for MgmtDescRing!\n"));
+                       DBGPRINT_ERR("Failed to allocate a big buffer for MgmtDescRing!\n");
                        Status = NDIS_STATUS_RESOURCES;
                        goto out1;
                }

diff --git a/drivers/staging/rt2860/common/cmm_wpa.c b/drivers/staging/rt2860/common/cmm_wpa.c
index e37b64b6a608fcf37cfccdbd7b96ce904644f12b..0040f45b629bae30e2446145b90a4c09dfa8243e 100644 (file)
--- a/drivers/staging/rt2860/common/cmm_wpa.c
+++ b/drivers/staging/rt2860/common/cmm_wpa.c
@@ -2794,7 +2794,7 @@ u8 *GetSuiteFromRSNIE(u8 *rsnie,
 
        /* Check length */
        if ((len <= 0) || (pEid->Len != len)) {
-               DBGPRINT_ERR(("%s : The length is invalid\n", __func__));
+               DBGPRINT_ERR("%s : The length is invalid\n", __func__);
                return NULL;
        }
        /* Check WPA or WPA2 */
@@ -2803,14 +2803,13 @@ u8 *GetSuiteFromRSNIE(u8 *rsnie,
                u16 ucount;
 
                if (len < sizeof(struct rt_rsnie)) {
-                       DBGPRINT_ERR(("%s : The length is too short for WPA\n",
-                                     __func__));
+                       DBGPRINT_ERR("%s : The length is too short for WPA\n", __func__);
                        return NULL;
                }
                /* Get the count of pairwise cipher */
                ucount = cpu2le16(pRsnie->ucount);
                if (ucount > 2) {
-                       DBGPRINT_ERR(("%s : The count(%d) of pairwise cipher is invlaid\n", __func__, ucount));
+                       DBGPRINT_ERR("%s : The count(%d) of pairwise cipher is invlaid\n", __func__, ucount);
                        return NULL;
                }
                /* Get the group cipher */
@@ -2836,14 +2835,13 @@ u8 *GetSuiteFromRSNIE(u8 *rsnie,
                isWPA2 = TRUE;
 
                if (len < sizeof(struct rt_rsnie2)) {
-                       DBGPRINT_ERR(("%s : The length is too short for WPA2\n",
-                                     __func__));
+                       DBGPRINT_ERR("%s : The length is too short for WPA2\n", __func__);
                        return NULL;
                }
                /* Get the count of pairwise cipher */
                ucount = cpu2le16(pRsnie->ucount);
                if (ucount > 2) {
-                       DBGPRINT_ERR(("%s : The count(%d) of pairwise cipher is invlaid\n", __func__, ucount));
+                       DBGPRINT_ERR("%s : The count(%d) of pairwise cipher is invlaid\n", __func__, ucount);
                        return NULL;
                }
                /* Get the group cipher */
@@ -2863,7 +2861,7 @@ u8 *GetSuiteFromRSNIE(u8 *rsnie,
                offset = sizeof(struct rt_rsnie2) + (4 * (ucount - 1));
 
        } else {
-               DBGPRINT_ERR(("%s : Unknown IE (%d)\n", __func__, pEid->Eid));
+               DBGPRINT_ERR("%s : Unknown IE (%d)\n", __func__, pEid->Eid);
                return NULL;
        }
 
@@ -2872,8 +2870,7 @@ u8 *GetSuiteFromRSNIE(u8 *rsnie,
        len -= offset;
 
        if (len < sizeof(struct rt_rsnie_auth)) {
-               DBGPRINT_ERR(("%s : The length of RSNIE is too short\n",
-                             __func__));
+               DBGPRINT_ERR("%s : The length of RSNIE is too short\n", __func__);
                return NULL;
        }
        /* pointer to AKM count */
@@ -2882,8 +2879,7 @@ u8 *GetSuiteFromRSNIE(u8 *rsnie,
        /* Get the count of pairwise cipher */
        acount = cpu2le16(pAkm->acount);
        if (acount > 2) {
-               DBGPRINT_ERR(("%s : The count(%d) of AKM is invlaid\n",
-                             __func__, acount));
+               DBGPRINT_ERR("%s : The count(%d) of AKM is invlaid\n", __func__, acount);
                return NULL;
        }
        /* Get the AKM suite */
@@ -2910,7 +2906,7 @@ u8 *GetSuiteFromRSNIE(u8 *rsnie,
                        return pBuf;
                }
        } else {
-               DBGPRINT_ERR(("%s : it can't get any more information beyond AKM \n", __func__));
+               DBGPRINT_ERR("%s : it can't get any more information beyond AKM \n", __func__);
                return NULL;
        }
 

diff --git a/drivers/staging/rt2860/common/mlme.c b/drivers/staging/rt2860/common/mlme.c
index 7300c6e9c175d7814d19314ee60b849609f643df..d9c3fd5c2166c4df407265308ef6ac6faba065f5 100644 (file)
--- a/drivers/staging/rt2860/common/mlme.c
+++ b/drivers/staging/rt2860/common/mlme.c
@@ -550,7 +550,7 @@ void MlmeHandler(struct rt_rtmp_adapter *pAd)
                        Elem->MsgLen = 0;
 
                } else {
-                       DBGPRINT_ERR(("MlmeHandler: MlmeQueue empty\n"));
+                       DBGPRINT_ERR("MlmeHandler: MlmeQueue empty\n");
                }
        }
 
@@ -4698,8 +4698,7 @@ BOOLEAN MlmeEnqueue(struct rt_rtmp_adapter *pAd,
 
        /* First check the size, it MUST not exceed the mlme queue size */
        if (MsgLen > MGMT_DMA_BUFFER_SIZE) {
-               DBGPRINT_ERR(("MlmeEnqueue: msg too large, size = %ld \n",
-                             MsgLen));
+               DBGPRINT_ERR("MlmeEnqueue: msg too large, size = %ld \n", MsgLen);
                return FALSE;
        }
 
@@ -4762,12 +4761,12 @@ BOOLEAN MlmeEnqueueForRecv(struct rt_rtmp_adapter *pAd,
        if (RTMP_TEST_FLAG
            (pAd,
             fRTMP_ADAPTER_HALT_IN_PROGRESS | fRTMP_ADAPTER_NIC_NOT_EXIST)) {
-               DBGPRINT_ERR(("MlmeEnqueueForRecv: fRTMP_ADAPTER_HALT_IN_PROGRESS\n"));
+               DBGPRINT_ERR("MlmeEnqueueForRecv: fRTMP_ADAPTER_HALT_IN_PROGRESS\n");
                return FALSE;
        }
        /* First check the size, it MUST not exceed the mlme queue size */
        if (MsgLen > MGMT_DMA_BUFFER_SIZE) {
-               DBGPRINT_ERR(("MlmeEnqueueForRecv: frame too large, size = %ld \n", MsgLen));
+               DBGPRINT_ERR("MlmeEnqueueForRecv: frame too large, size = %ld \n", MsgLen);
                return FALSE;
        }
 
@@ -4777,7 +4776,7 @@ BOOLEAN MlmeEnqueueForRecv(struct rt_rtmp_adapter *pAd,
 
        {
                if (!MsgTypeSubst(pAd, pFrame, &Machine, &MsgType)) {
-                       DBGPRINT_ERR(("MlmeEnqueueForRecv: un-recongnized mgmt->subtype=%d\n", pFrame->Hdr.FC.SubType));
+                       DBGPRINT_ERR("MlmeEnqueueForRecv: un-recongnized mgmt->subtype=%d\n", pFrame->Hdr.FC.SubType);
                        return FALSE;
                }
        }
@@ -4867,7 +4866,7 @@ void MlmeRestartStateMachine(struct rt_rtmp_adapter *pAd)
                        Elem->MsgLen = 0;
 
                } else {
-                       DBGPRINT_ERR(("MlmeRestartStateMachine: MlmeQueue empty\n"));
+                       DBGPRINT_ERR("MlmeRestartStateMachine: MlmeQueue empty\n");
                }
        }
 #endif /* RTMP_MAC_PCI // */

diff --git a/drivers/staging/rt2860/common/rt_rf.c b/drivers/staging/rt2860/common/rt_rf.c
index 519121d8104007677e7b141c4e9b9c5a2a85d529..2895447ffc4893e612d151db2d255d4610f9e9d1 100644 (file)
--- a/drivers/staging/rt2860/common/rt_rf.c
+++ b/drivers/staging/rt2860/common/rt_rf.c
@@ -131,8 +131,7 @@ int RT30xxReadRFRegister(struct rt_rtmp_adapter *pAd,
                }
        }
        if (rfcsr.field.RF_CSR_KICK == BUSY) {
-               DBGPRINT_ERR(("RF read R%d=0x%x fail, i[%d], k[%d]\n", regID,
-                             rfcsr.word, i, k));
+               DBGPRINT_ERR("RF read R%d=0x%x fail, i[%d], k[%d]\n", regID, rfcsr.word, i, k);
                return STATUS_UNSUCCESSFUL;
        }
 

diff --git a/drivers/staging/rt2860/common/rtmp_init.c b/drivers/staging/rt2860/common/rtmp_init.c
index 3628e85802e3c696d55df69734fff4e5a4f08f24..d359a14f5d4d792a5681ee8d97e8959d7af1a089 100644 (file)
--- a/drivers/staging/rt2860/common/rtmp_init.c
+++ b/drivers/staging/rt2860/common/rtmp_init.c
@@ -169,14 +169,14 @@ int RTMPAllocAdapterBlock(void *handle,
                pBeaconBuf = kmalloc(MAX_BEACON_SIZE, MEM_ALLOC_FLAG);
                if (pBeaconBuf == NULL) {
                        Status = NDIS_STATUS_FAILURE;
-                       DBGPRINT_ERR(("Failed to allocate memory - BeaconBuf!\n"));
+                       DBGPRINT_ERR("Failed to allocate memory - BeaconBuf!\n");
                        break;
                }
                NdisZeroMemory(pBeaconBuf, MAX_BEACON_SIZE);
 
                Status = AdapterBlockAllocateMemory(handle, (void **) & pAd);
                if (Status != NDIS_STATUS_SUCCESS) {
-                       DBGPRINT_ERR(("Failed to allocate memory - ADAPTER\n"));
+                       DBGPRINT_ERR("Failed to allocate memory - ADAPTER\n");
                        break;
                }
                pAd->BeaconBuf = pBeaconBuf;
@@ -785,8 +785,7 @@ void NICReadEEPROMParameters(struct rt_rtmp_adapter *pAd, u8 *mac_addr)
                  Version.field.Version, Version.field.FaeReleaseNumber));
 
        if (Version.field.Version > VALID_EEPROM_VERSION) {
-               DBGPRINT_ERR(("E2PROM: WRONG VERSION 0x%x, should be %d\n",
-                             Version.field.Version, VALID_EEPROM_VERSION));
+               DBGPRINT_ERR("E2PROM: WRONG VERSION 0x%x, should be %d\n", Version.field.Version, VALID_EEPROM_VERSION);
                /*pAd->SystemErrorBitmap |= 0x00000001;
 
                   // hard-code default value when no proper E2PROM installed
@@ -2911,7 +2910,7 @@ void RTMPSetTimer(struct rt_ralink_timer *pTimer, unsigned long Value)
                        RTMP_OS_Add_Timer(&pTimer->TimerObj, Value);
                }
        } else {
-               DBGPRINT_ERR(("RTMPSetTimer failed, Timer hasn't been initialize!\n"));
+               DBGPRINT_ERR("RTMPSetTimer failed, Timer hasn't been initialize!\n");
        }
 }
 
@@ -2947,7 +2946,7 @@ void RTMPModTimer(struct rt_ralink_timer *pTimer, unsigned long Value)
                        RTMP_OS_Mod_Timer(&pTimer->TimerObj, Value);
                }
        } else {
-               DBGPRINT_ERR(("RTMPModTimer failed, Timer hasn't been initialize!\n"));
+               DBGPRINT_ERR("RTMPModTimer failed, Timer hasn't been initialize!\n");
        }
 }
 
@@ -2989,7 +2988,7 @@ void RTMPCancelTimer(struct rt_ralink_timer *pTimer, OUT BOOLEAN * pCancelled)
                RtmpTimerQRemove(pTimer->pAd, pTimer);
 #endif /* RTMP_TIMER_TASK_SUPPORT // */
        } else {
-               DBGPRINT_ERR(("RTMPCancelTimer failed, Timer hasn't been initialize!\n"));
+               DBGPRINT_ERR("RTMPCancelTimer failed, Timer hasn't been initialize!\n");
        }
 }
 
@@ -3251,8 +3250,7 @@ int rt28xx_init(struct rt_rtmp_adapter *pAd,
        /* Load 8051 firmware */
        Status = NICLoadFirmware(pAd);
        if (Status != NDIS_STATUS_SUCCESS) {
-               DBGPRINT_ERR(("NICLoadFirmware failed, Status[=0x%08x]\n",
-                             Status));
+               DBGPRINT_ERR("NICLoadFirmware failed, Status[=0x%08x]\n", Status);
                goto err1;
        }
 
@@ -3268,8 +3266,7 @@ int rt28xx_init(struct rt_rtmp_adapter *pAd,
 
        Status = RTMPAllocTxRxRingMemory(pAd);
        if (Status != NDIS_STATUS_SUCCESS) {
-               DBGPRINT_ERR(("RTMPAllocDMAMemory failed, Status[=0x%08x]\n",
-                             Status));
+               DBGPRINT_ERR("RTMPAllocDMAMemory failed, Status[=0x%08x]\n", Status);
                goto err1;
        }
 
@@ -3284,7 +3281,7 @@ int rt28xx_init(struct rt_rtmp_adapter *pAd,
 
        Status = MlmeInit(pAd);
        if (Status != NDIS_STATUS_SUCCESS) {
-               DBGPRINT_ERR(("MlmeInit failed, Status[=0x%08x]\n", Status));
+               DBGPRINT_ERR("MlmeInit failed, Status[=0x%08x]\n", Status);
                goto err2;
        }
        /* Initialize pAd->StaCfg, pAd->ApCfg, pAd->CommonCfg to manufacture default */
@@ -3309,8 +3306,7 @@ int rt28xx_init(struct rt_rtmp_adapter *pAd,
        /* */
        Status = NICInitializeAdapter(pAd, TRUE);
        if (Status != NDIS_STATUS_SUCCESS) {
-               DBGPRINT_ERR(("NICInitializeAdapter failed, Status[=0x%08x]\n",
-                             Status));
+               DBGPRINT_ERR("NICInitializeAdapter failed, Status[=0x%08x]\n", Status);
                if (Status != NDIS_STATUS_SUCCESS)
                        goto err3;
        }

diff --git a/drivers/staging/rt2860/common/rtmp_mcu.c b/drivers/staging/rt2860/common/rtmp_mcu.c
index 844d4b987b78eef1c0b3df837522fbea55c195e6..80fa4160ed6246ea0f58d13b1e2fd7ba4cce7f66 100644 (file)
--- a/drivers/staging/rt2860/common/rtmp_mcu.c
+++ b/drivers/staging/rt2860/common/rtmp_mcu.c
@@ -267,7 +267,7 @@ int RtmpAsicSendCommandToMcu(struct rt_rtmp_adapter *pAd,
                } while (i++ < 100);
 
                if (i > 100) {
-                       DBGPRINT_ERR(("H2M_MAILBOX still hold by MCU. command fail\n"));
+                       DBGPRINT_ERR("H2M_MAILBOX still hold by MCU. command fail\n");
                        return FALSE;
                }
 
@@ -296,7 +296,7 @@ int RtmpAsicSendCommandToMcu(struct rt_rtmp_adapter *pAd,
 #ifdef RTMP_MAC_PCI
 #endif /* RTMP_MAC_PCI // */
                        {
-                               DBGPRINT_ERR(("H2M_MAILBOX still hold by MCU. command fail\n"));
+                               DBGPRINT_ERR("H2M_MAILBOX still hold by MCU. command fail\n");
                        }
                        return FALSE;
                }

diff --git a/drivers/staging/rt2860/common/spectrum.c b/drivers/staging/rt2860/common/spectrum.c
index 2d5f847e6cc6ce73631379c62432fdbbc619930d..1dfb802aab9a4e93ae402def74a0d63c4d375e1e 100644 (file)
--- a/drivers/staging/rt2860/common/spectrum.c
+++ b/drivers/staging/rt2860/common/spectrum.c
@@ -1837,7 +1837,7 @@ static void PeerChSwAnnAction(struct rt_rtmp_adapter *pAd, struct rt_mlme_queue_
                        }
 
                        if (index >= pAd->ChannelListNum) {
-                               DBGPRINT_ERR(("&&&&&&&&&&&&&&&&&&&&&&&&&&PeerChSwAnnAction(can not find New Channel=%d in ChannelList[%d]\n", pAd->CommonCfg.Channel, pAd->ChannelListNum));
+                               DBGPRINT_ERR("&&&&&&&&&&&&&&&&&&&&&&&&&&PeerChSwAnnAction(can not find New Channel=%d in ChannelList[%d]\n", pAd->CommonCfg.Channel, pAd->ChannelListNum);
                        }
                }
        }

diff --git a/drivers/staging/rt2860/rt_linux.h b/drivers/staging/rt2860/rt_linux.h
index a449b0bb85f11b384d668d6466c69bee8056b154..92ff5438e77739aa47653ce2e7f204565536b773 100644 (file)
--- a/drivers/staging/rt2860/rt_linux.h
+++ b/drivers/staging/rt2860/rt_linux.h
@@ -423,11 +423,7 @@ do{                                   \
 
 #define DBGPRINT(Level, Fmt)    DBGPRINT_RAW(Level, Fmt)
 
-#define DBGPRINT_ERR(Fmt)           \
-{                                   \
-    printk("ERROR! ");          \
-    printk Fmt;                  \
-}
+#define DBGPRINT_ERR(fmt, args...) printk(KERN_ERR fmt, ##args)
 
 #define DBGPRINT_S(Status, Fmt)                \
 {                                                                      \

diff --git a/drivers/staging/rt2860/sta/assoc.c b/drivers/staging/rt2860/sta/assoc.c
index ab0a83bf76efcbf705694d33fda8b4f293cbbfe7..59e931c3190d4a4ff827e0c9b34e9e369f77880d 100644 (file)
--- a/drivers/staging/rt2860/sta/assoc.c
+++ b/drivers/staging/rt2860/sta/assoc.c
@@ -1055,7 +1055,7 @@ void AssocPostProc(struct rt_rtmp_adapter *pAd, u8 *pAddr2, u16 CapabilityInfo,
        /* Set New WPA information */
        Idx = BssTableSearch(&pAd->ScanTab, pAddr2, pAd->MlmeAux.Channel);
        if (Idx == BSS_NOT_FOUND) {
-               DBGPRINT_ERR(("ASSOC - Can't find BSS after receiving Assoc response\n"));
+               DBGPRINT_ERR("ASSOC - Can't find BSS after receiving Assoc response\n");
        } else {
                /* Init variable */
                pAd->MacTab.Content[BSSID_WCID].RSNIE_Len = 0;

diff --git a/drivers/staging/rt2860/sta/auth.c b/drivers/staging/rt2860/sta/auth.c
index a2bfafddb20ea0652f7bc86c92cdc21d7214cab6..23ea00b896b0ddae4ce33882301202d20a0f9852 100644 (file)
--- a/drivers/staging/rt2860/sta/auth.c
+++ b/drivers/staging/rt2860/sta/auth.c
@@ -509,8 +509,7 @@ BOOLEAN AUTH_ReqSend(struct rt_rtmp_adapter *pAd,
                RTMPSetTimer(pAuthTimer, Timeout);
                return TRUE;
        } else {
-               DBGPRINT_ERR(("%s - MlmeAuthReqAction() sanity check failed\n",
-                             pSMName));
+               DBGPRINT_ERR("%s - MlmeAuthReqAction() sanity check failed\n", pSMName);
                return FALSE;
        }
 

diff --git a/drivers/staging/rt2860/sta/connect.c b/drivers/staging/rt2860/sta/connect.c
index 4b2c84e6509eb8b858123cf9ff29bc18f591f6a3..4996258f6ecd7a269de2027faad97b277c2fb5a3 100644 (file)
--- a/drivers/staging/rt2860/sta/connect.c
+++ b/drivers/staging/rt2860/sta/connect.c
@@ -215,8 +215,7 @@ void MlmeCntlMachinePerformAction(struct rt_rtmp_adapter *pAd,
                break;
 #endif /* RTMP_MAC_USB // */
        default:
-               DBGPRINT_ERR(("ERROR! CNTL - Illegal message type(=%ld)",
-                             Elem->MsgType));
+               DBGPRINT_ERR("ERROR! CNTL - Illegal message type(=%ld)", Elem->MsgType);
                break;
        }
 }

diff --git a/drivers/staging/rt2860/sta/rtmp_data.c b/drivers/staging/rt2860/sta/rtmp_data.c
index d7c2012be8a5e913ce2881e3cccc33e59ef9f2b1..e82c6b669eb2f8313dde7fd0bf5c116cfa6a57a9 100644 (file)
--- a/drivers/staging/rt2860/sta/rtmp_data.c
+++ b/drivers/staging/rt2860/sta/rtmp_data.c
@@ -645,7 +645,7 @@ void STAHandleRxMgmtFrame(struct rt_rtmp_adapter *pAd, struct rt_rx_blk *pRxBlk)
 
                /* First check the size, it MUST not exceed the mlme queue size */
                if (pRxWI->MPDUtotalByteCount > MGMT_DMA_BUFFER_SIZE) {
-                       DBGPRINT_ERR(("STAHandleRxMgmtFrame: frame too large, size = %d \n", pRxWI->MPDUtotalByteCount));
+                       DBGPRINT_ERR("STAHandleRxMgmtFrame: frame too large, size = %d \n", pRxWI->MPDUtotalByteCount);
                        break;
                }
 

diff --git a/drivers/staging/rt2860/sta/sync.c b/drivers/staging/rt2860/sta/sync.c
index 05007d9c9b69b167d9c923f1a81414cf528aeecb..7054ba1323d0fd0e525ffe9571af156657ccfc80 100644 (file)
--- a/drivers/staging/rt2860/sta/sync.c
+++ b/drivers/staging/rt2860/sta/sync.c
@@ -284,7 +284,7 @@ void MlmeScanReqAction(struct rt_rtmp_adapter *pAd, struct rt_mlme_queue_elem *E
                DBGPRINT(RT_DEBUG_TRACE, ("SYNC - BBP R4 to 20MHz.l\n"));
                ScanNextChannel(pAd);
        } else {
-               DBGPRINT_ERR(("SYNC - MlmeScanReqAction() sanity check fail\n"));
+               DBGPRINT_ERR("SYNC - MlmeScanReqAction() sanity check fail\n");
                pAd->Mlme.SyncMachine.CurrState = SYNC_IDLE;
                Status = MLME_INVALID_FORMAT;
                MlmeEnqueue(pAd, MLME_CNTL_STATE_MACHINE, MT2_SCAN_CONF, 2,
@@ -536,7 +536,7 @@ void MlmeStartReqAction(struct rt_rtmp_adapter *pAd, struct rt_mlme_queue_elem *
                MlmeEnqueue(pAd, MLME_CNTL_STATE_MACHINE, MT2_START_CONF, 2,
                            &Status);
        } else {
-               DBGPRINT_ERR(("SYNC - MlmeStartReqAction() sanity check fail.\n"));
+               DBGPRINT_ERR("SYNC - MlmeStartReqAction() sanity check fail.\n");
                pAd->Mlme.SyncMachine.CurrState = SYNC_IDLE;
                Status = MLME_INVALID_FORMAT;
                MlmeEnqueue(pAd, MLME_CNTL_STATE_MACHINE, MT2_START_CONF, 2,
@@ -1208,7 +1208,7 @@ void PeerBeacon(struct rt_rtmp_adapter *pAd, struct rt_mlme_queue_elem *Elem)
                        }
 
                        if (index >= pAd->ChannelListNum) {
-                               DBGPRINT_ERR(("PeerBeacon(can not find New Channel=%d in ChannelList[%d]\n", pAd->CommonCfg.Channel, pAd->ChannelListNum));
+                               DBGPRINT_ERR("PeerBeacon(can not find New Channel=%d in ChannelList[%d]\n", pAd->CommonCfg.Channel, pAd->ChannelListNum);
                        }
                }
                /* if the ssid matched & bssid unmatched, we should select the bssid with large value. */