rtlwifi: Convert RT_TRACE macro to use ##__VA_ARGS__
Consolidate printks to avoid possible message interleaving and reduce the object size. Remove unnecessary RT_TRACE parentheses. Miscellaneous typo and grammar fixes. Add missing newlines to formats. Remove duplicate KERN_DEBUG prefixes. Coalesce formats. Align arguments. $ size drivers/net/wireless/rtlwifi/built-in.o* text data bss dec hex filename 594841 55333 129680 779854 be64e drivers/net/wireless/rtlwifi/built-in.o.new 607022 55333 138720 801075 c3933 drivers/net/wireless/rtlwifi/built-in.o.old Signed-off-by: Joe Perches <joe@perches.com> Acked-by: Larry Finger <Larry.Finger@lwfinger.net> Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:

committed by
John W. Linville

parent
af08687b4e
commit
f30d7507a8
@@ -66,7 +66,7 @@ static bool _rtl92s_firmware_enable_cpu(struct ieee80211_hw *hw)
|
||||
cpustatus = rtl_read_byte(rtlpriv, TCR);
|
||||
if (cpustatus & IMEM_RDY) {
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD,
|
||||
("IMEM Ready after CPU has refilled.\n"));
|
||||
"IMEM Ready after CPU has refilled\n");
|
||||
break;
|
||||
}
|
||||
|
||||
@@ -120,9 +120,8 @@ static u8 _rtl92s_firmware_header_map_rftype(struct ieee80211_hw *hw)
|
||||
return 0x22;
|
||||
break;
|
||||
default:
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_EMERG,
|
||||
("Unknown RF type(%x)\n",
|
||||
rtlphy->rf_type));
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_EMERG, "Unknown RF type(%x)\n",
|
||||
rtlphy->rf_type);
|
||||
break;
|
||||
}
|
||||
return 0x22;
|
||||
@@ -177,7 +176,7 @@ static bool _rtl92s_firmware_downloadcode(struct ieee80211_hw *hw,
|
||||
|
||||
if (buffer_len >= MAX_FIRMWARE_CODE_SIZE) {
|
||||
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG,
|
||||
("Size over FIRMWARE_CODE_SIZE!\n"));
|
||||
"Size over FIRMWARE_CODE_SIZE!\n");
|
||||
|
||||
return false;
|
||||
}
|
||||
@@ -231,8 +230,8 @@ static bool _rtl92s_firmware_checkready(struct ieee80211_hw *hw,
|
||||
short pollingcnt = 1000;
|
||||
bool rtstatus = true;
|
||||
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD, ("LoadStaus(%d)\n",
|
||||
loadfw_status));
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD,
|
||||
"LoadStaus(%d)\n", loadfw_status);
|
||||
|
||||
firmware->fwstatus = (enum fw_status)loadfw_status;
|
||||
|
||||
@@ -248,8 +247,8 @@ static bool _rtl92s_firmware_checkready(struct ieee80211_hw *hw,
|
||||
|
||||
if (!(cpustatus & IMEM_CHK_RPT) || (pollingcnt <= 0)) {
|
||||
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG,
|
||||
("FW_STATUS_LOAD_IMEM"
|
||||
" FAIL CPU, Status=%x\r\n", cpustatus));
|
||||
"FW_STATUS_LOAD_IMEM FAIL CPU, Status=%x\n",
|
||||
cpustatus);
|
||||
goto status_check_fail;
|
||||
}
|
||||
break;
|
||||
@@ -266,8 +265,8 @@ static bool _rtl92s_firmware_checkready(struct ieee80211_hw *hw,
|
||||
|
||||
if (!(cpustatus & EMEM_CHK_RPT) || (pollingcnt <= 0)) {
|
||||
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG,
|
||||
("FW_STATUS_LOAD_EMEM"
|
||||
" FAIL CPU, Status=%x\r\n", cpustatus));
|
||||
"FW_STATUS_LOAD_EMEM FAIL CPU, Status=%x\n",
|
||||
cpustatus);
|
||||
goto status_check_fail;
|
||||
}
|
||||
|
||||
@@ -275,7 +274,7 @@ static bool _rtl92s_firmware_checkready(struct ieee80211_hw *hw,
|
||||
rtstatus = _rtl92s_firmware_enable_cpu(hw);
|
||||
if (rtstatus != true) {
|
||||
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG,
|
||||
("Enable CPU fail!\n"));
|
||||
"Enable CPU fail!\n");
|
||||
goto status_check_fail;
|
||||
}
|
||||
break;
|
||||
@@ -291,14 +290,14 @@ static bool _rtl92s_firmware_checkready(struct ieee80211_hw *hw,
|
||||
|
||||
if (!(cpustatus & DMEM_CODE_DONE) || (pollingcnt <= 0)) {
|
||||
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG,
|
||||
("Polling DMEM code done"
|
||||
" fail ! cpustatus(%#x)\n", cpustatus));
|
||||
"Polling DMEM code done fail ! cpustatus(%#x)\n",
|
||||
cpustatus);
|
||||
goto status_check_fail;
|
||||
}
|
||||
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD,
|
||||
("DMEM code download success,"
|
||||
" cpustatus(%#x)\n", cpustatus));
|
||||
"DMEM code download success, cpustatus(%#x)\n",
|
||||
cpustatus);
|
||||
|
||||
/* Prevent Delay too much and being scheduled out */
|
||||
/* Polling Load Firmware ready */
|
||||
@@ -311,14 +310,14 @@ static bool _rtl92s_firmware_checkready(struct ieee80211_hw *hw,
|
||||
} while (pollingcnt--);
|
||||
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD,
|
||||
("Polling Load Firmware ready,"
|
||||
" cpustatus(%x)\n", cpustatus));
|
||||
"Polling Load Firmware ready, cpustatus(%x)\n",
|
||||
cpustatus);
|
||||
|
||||
if (((cpustatus & LOAD_FW_READY) != LOAD_FW_READY) ||
|
||||
(pollingcnt <= 0)) {
|
||||
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG,
|
||||
("Polling Load Firmware"
|
||||
" ready fail ! cpustatus(%x)\n", cpustatus));
|
||||
"Polling Load Firmware ready fail ! cpustatus(%x)\n",
|
||||
cpustatus);
|
||||
goto status_check_fail;
|
||||
}
|
||||
|
||||
@@ -332,7 +331,7 @@ static bool _rtl92s_firmware_checkready(struct ieee80211_hw *hw,
|
||||
RCR_APP_ICV | RCR_APP_MIC));
|
||||
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD,
|
||||
("Current RCR settings(%#x)\n", tmpu4b));
|
||||
"Current RCR settings(%#x)\n", tmpu4b);
|
||||
|
||||
/* Set to normal mode. */
|
||||
rtl_write_byte(rtlpriv, LBKMD_SEL, LBK_NORMAL);
|
||||
@@ -340,14 +339,15 @@ static bool _rtl92s_firmware_checkready(struct ieee80211_hw *hw,
|
||||
|
||||
default:
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_EMERG,
|
||||
("Unknown status check!\n"));
|
||||
"Unknown status check!\n");
|
||||
rtstatus = false;
|
||||
break;
|
||||
}
|
||||
|
||||
status_check_fail:
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD, ("loadfw_status(%d), "
|
||||
"rtstatus(%x)\n", loadfw_status, rtstatus));
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD,
|
||||
"loadfw_status(%d), rtstatus(%x)\n",
|
||||
loadfw_status, rtstatus);
|
||||
return rtstatus;
|
||||
}
|
||||
|
||||
@@ -378,17 +378,17 @@ int rtl92s_download_fw(struct ieee80211_hw *hw)
|
||||
firmware->firmwareversion = byte(pfwheader->version, 0);
|
||||
firmware->pfwheader->fwpriv.hci_sel = 1;/* pcie */
|
||||
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD, ("signature:%x, version:"
|
||||
"%x, size:%x,"
|
||||
"imemsize:%x, sram size:%x\n", pfwheader->signature,
|
||||
RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD,
|
||||
"signature:%x, version:%x, size:%x, imemsize:%x, sram size:%x\n",
|
||||
pfwheader->signature,
|
||||
pfwheader->version, pfwheader->dmem_size,
|
||||
pfwheader->img_imem_size, pfwheader->img_sram_size));
|
||||
pfwheader->img_imem_size, pfwheader->img_sram_size);
|
||||
|
||||
/* 2. Retrieve IMEM image. */
|
||||
if ((pfwheader->img_imem_size == 0) || (pfwheader->img_imem_size >
|
||||
sizeof(firmware->fw_imem))) {
|
||||
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG,
|
||||
("memory for data image is less than IMEM required\n"));
|
||||
"memory for data image is less than IMEM required\n");
|
||||
goto fail;
|
||||
} else {
|
||||
puc_mappedfile += fwhdr_size;
|
||||
@@ -401,7 +401,7 @@ int rtl92s_download_fw(struct ieee80211_hw *hw)
|
||||
/* 3. Retriecve EMEM image. */
|
||||
if (pfwheader->img_sram_size > sizeof(firmware->fw_emem)) {
|
||||
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG,
|
||||
("memory for data image is less than EMEM required\n"));
|
||||
"memory for data image is less than EMEM required\n");
|
||||
goto fail;
|
||||
} else {
|
||||
puc_mappedfile += firmware->fw_imem_len;
|
||||
@@ -436,7 +436,7 @@ int rtl92s_download_fw(struct ieee80211_hw *hw)
|
||||
break;
|
||||
default:
|
||||
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG,
|
||||
("Unexpected Download step!!\n"));
|
||||
"Unexpected Download step!!\n");
|
||||
goto fail;
|
||||
break;
|
||||
}
|
||||
@@ -446,14 +446,14 @@ int rtl92s_download_fw(struct ieee80211_hw *hw)
|
||||
ul_filelength);
|
||||
|
||||
if (rtstatus != true) {
|
||||
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG, ("fail!\n"));
|
||||
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG, "fail!\n");
|
||||
goto fail;
|
||||
}
|
||||
|
||||
/* <3> Check whether load FW process is ready */
|
||||
rtstatus = _rtl92s_firmware_checkready(hw, fwstatus);
|
||||
if (rtstatus != true) {
|
||||
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG, ("fail!\n"));
|
||||
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG, "fail!\n");
|
||||
goto fail;
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user