staging: rtl8723au: Remove uses of MAC_FMT and MAC_ARG
PATCH 1/1 staging: rtl8723au: Remove uses of MAC_FMT and MAC_ARG
Use the standard vsprintf kernel extension to format mac addresses.
This reduces object code size a bit.
Miscellanea:
o Coalesce formats
o Realign arguments
o Remove the now unused MAC_FMT and MAC_ARG #defines
--- drivers/staging/rtl8723au/core/rtw_ap.c | 41 ++++++------- drivers/staging/rtl8723au/core/rtw_mlme.c | 26 ++++---- drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 72 +++++++++++------------ drivers/staging/rtl8723au/core/rtw_recv.c | 42 +++++-------- drivers/staging/rtl8723au/core/rtw_wlan_util.c | 6 +- drivers/staging/rtl8723au/core/rtw_xmit.c | 8 +-- drivers/staging/rtl8723au/include/ieee80211.h | 3 - drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 18 +++--- drivers/staging/rtl8723au/os_dep/os_intfs.c | 7 +-- 9 files changed, 101 insertions(+), 122 deletions(-)
drivers/staging/rtl8723au/core/rtw_ap.c +21 -20
@@ -260,15 +260,17 @@ void expire_timeout_chk23a(struct rtw_adapter *padapter)list_del_init (&psta->asoc_list); pstapriv->asoc_list_cnt--; -DBG_8723A ("asoc expire " MAC_FMT ", state = 0x%x\n" ,MAC_ARG (psta->hwaddr), psta->state); +DBG_8723A ("asoc expire %pM, state = 0x%x\n" , + psta->hwaddr, psta->state); updated =ap_free_sta23a (padapter, psta, false,WLAN_REASON_DEAUTH_LEAVING ); }else {/* TODO: Aging mechanism to digest frames in sleep_q to avoid running out of xmitframe */ if (psta->sleepq_len > (NR_XMITFRAME /pstapriv->asoc_list_cnt) && padapter->xmitpriv.free_xmitframe_cnt < ((NR_XMITFRAME /pstapriv->asoc_list_cnt)/2 ) ) { -DBG_8723A ("%s sta:" MAC_FMT ", sleepq_len:%u, free_xmitframe_cnt:%u, asoc_list_cnt:%u, clear sleep_q\n" , __func__, -MAC_ARG (psta->hwaddr), +DBG_8723A ("%s sta:%pM, sleepq_len:%u, free_xmitframe_cnt:%u, asoc_list_cnt:%u, clear sleep_q\n" , + __func__, + psta->hwaddr, psta->sleepq_len, padapter->xmitpriv.free_xmitframe_cnt, pstapriv->asoc_list_cnt);
@@ -305,7 +307,8 @@ void expire_timeout_chk23a(struct rtw_adapter *padapter) psta->keep_alive_trycnt++;if (ret == _SUCCESS) { -DBG_8723A ("asoc check, sta(" MAC_FMT ") is alive\n" ,MAC_ARG (psta->hwaddr)); +DBG_8723A ("asoc check, sta(%pM) is alive\n" , + psta->hwaddr); psta->expire_to = pstapriv->expire_to; psta->keep_alive_trycnt =0 ;continue ;
@@ -317,7 +320,8 @@ void expire_timeout_chk23a(struct rtw_adapter *padapter) psta->keep_alive_trycnt =0 ; -DBG_8723A ("asoc expire " MAC_FMT ", state = 0x%x\n" ,MAC_ARG (psta->hwaddr), psta->state); +DBG_8723A ("asoc expire %pM, state = 0x%x\n" , + psta->hwaddr, psta->state);spin_lock_bh (&pstapriv->asoc_list_lock);if (!list_empty (&psta->asoc_list)) {list_del_init (&psta->asoc_list);
@@ -1044,7 +1048,7 @@ int rtw_acl_add_sta23a(struct rtw_adapter *padapter, u8 *addr)struct wlan_acl_pool *pacl_list = &pstapriv->acl_list;struct rtw_queue *pacl_node_q = &pacl_list->acl_node_q; -DBG_8723A ("%s(acl_num =%d) =" MAC_FMT "\n" , __func__, pacl_list->num,MAC_ARG (addr)); +DBG_8723A ("%s(acl_num =%d) =%pM\n" , __func__, pacl_list->num, addr);if ((NUM_ACL -1 ) < pacl_list->num)return -1 ;
@@ -1476,8 +1480,8 @@ void bss_cap_update_on_sta_join23a(struct rtw_adapter *padapter, struct sta_infoif (psta->flags &WLAN_STA_HT ) {u16 ht_capab =le16_to_cpu (psta->htpriv.ht_cap.cap_info); -DBG_8723A ("HT: STA " MAC_FMT " HT Capabilities Info: 0x%04x\n" , -MAC_ARG (psta->hwaddr), ht_capab); +DBG_8723A ("HT: STA %pM HT Capabilities Info: 0x%04x\n" , + psta->hwaddr, ht_capab);if (psta->no_ht_set) { psta->no_ht_set =0 ;
@@ -1489,10 +1493,9 @@ void bss_cap_update_on_sta_join23a(struct rtw_adapter *padapter, struct sta_info psta->no_ht_gf_set =1 ; pmlmepriv->num_sta_ht_no_gf++; } -DBG_8723A ("%s STA " MAC_FMT " - no " -"greenfield, num of non-gf stations %d\n" , - __func__,MAC_ARG (psta->hwaddr), - pmlmepriv->num_sta_ht_no_gf); +DBG_8723A ("%s STA %pM - no greenfield, num of non-gf stations %d\n" , + __func__, psta->hwaddr, + pmlmepriv->num_sta_ht_no_gf); }if ((ht_capab &IEEE80211_HT_CAP_SUP_WIDTH_20_40 ) ==0 ) {
@@ -1500,10 +1503,9 @@ void bss_cap_update_on_sta_join23a(struct rtw_adapter *padapter, struct sta_info psta->ht_20mhz_set =1 ; pmlmepriv->num_sta_ht_20mhz++; } -DBG_8723A ("%s STA " MAC_FMT " - 20 MHz HT, " -"num of 20MHz HT STAs %d\n" , - __func__,MAC_ARG (psta->hwaddr), - pmlmepriv->num_sta_ht_20mhz); +DBG_8723A ("%s STA %pM - 20 MHz HT, num of 20MHz HT STAs %d\n" , + __func__, psta->hwaddr, + pmlmepriv->num_sta_ht_20mhz); } }else {
@@ -1512,10 +1514,9 @@ void bss_cap_update_on_sta_join23a(struct rtw_adapter *padapter, struct sta_info pmlmepriv->num_sta_no_ht++; }if (pmlmepriv->htpriv.ht_option) { -DBG_8723A ("%s STA " MAC_FMT -" - no HT, num of non-HT stations %d\n" , - __func__,MAC_ARG (psta->hwaddr), - pmlmepriv->num_sta_no_ht); +DBG_8723A ("%s STA %pM - no HT, num of non-HT stations %d\n" , + __func__, psta->hwaddr, + pmlmepriv->num_sta_no_ht); } }
drivers/staging/rtl8723au/core/rtw_mlme.c +13 -13
@@ -238,9 +238,9 @@ static void _rtw_roaming(struct rtw_adapter *padapter, pnetwork = &pmlmepriv->cur_network;if (padapter->mlmepriv.to_roaming >0 ) { -DBG_8723A ("roaming from %s(" MAC_FMT "), length:%d\n" , +DBG_8723A ("roaming from %s(%pM), length:%d\n" , pnetwork->network.Ssid.ssid, -MAC_ARG (pnetwork->network.MacAddress), + pnetwork->network.MacAddress, pnetwork->network.Ssid.ssid_len);memcpy (&pmlmepriv->assoc_ssid, &pnetwork->network.Ssid,sizeof (struct cfg80211_ssid ));
@@ -754,8 +754,8 @@ void rtw_free_assoc_resources23a(struct rtw_adapter *adapter,RT_TRACE (_module_rtl871x_mlme_c_, _drv_notice_, ("+rtw_free_assoc_resources23a\n" ));RT_TRACE (_module_rtl871x_mlme_c_, _drv_info_, - ("tgt_network->network.MacAddress=" MAC_FMT " ssid=%s\n" , -MAC_ARG (tgt_network->network.MacAddress), + ("tgt_network->network.MacAddress=%pM ssid=%s\n" , + tgt_network->network.MacAddress, tgt_network->network.Ssid.ssid));if (check_fwstate (pmlmepriv,WIFI_STATION_STATE |WIFI_AP_STATE )) {
@@ -998,8 +998,9 @@ rtw_joinbss_update_network23a(struct rtw_adapter *padapter,DBG_8723A ("%s\n" , __func__);RT_TRACE (_module_rtl871x_mlme_c_, _drv_info_, - ("\nfw_state:%x, BSSID:" MAC_FMT "\n" ,get_fwstate (pmlmepriv), -MAC_ARG (pnetwork->network.MacAddress))); + ("\nfw_state:%x, BSSID:%pM\n" , +get_fwstate (pmlmepriv), + pnetwork->network.MacAddress));/* why not use ptarget_wlan?? */ memcpy(&cur_network->network, &pnetwork->network,
@@ -1330,8 +1331,7 @@ void rtw_stadel_event_callback23a(struct rtw_adapter *adapter, const u8 *pbuf)else mac_id = pstadel->mac_id; -DBG_8723A ("%s(mac_id=%d)=" MAC_FMT "\n" , __func__, mac_id, -MAC_ARG (pstadel->macaddr)); +DBG_8723A ("%s(mac_id=%d)=%pM\n" , __func__, mac_id, pstadel->macaddr);if (check_fwstate (pmlmepriv,WIFI_AP_STATE ))return ;
@@ -1582,13 +1582,12 @@ static int rtw_check_join_candidate(struct mlme_priv *pmlmepriv, }if (updated) { -DBG_8723A ("[by_bssid:%u][assoc_ssid:%s][to_roaming:%u] " -"new candidate: %s(" MAC_FMT ") rssi:%d\n" , +DBG_8723A ("[by_bssid:%u][assoc_ssid:%s][to_roaming:%u] new candidate: %s(%pM) rssi:%d\n" , pmlmepriv->assoc_by_bssid, pmlmepriv->assoc_ssid.ssid, adapter->mlmepriv.to_roaming, (*candidate)->network.Ssid.ssid, -MAC_ARG ((*candidate)->network.MacAddress), + (*candidate)->network.MacAddress, (int )(*candidate)->network.Rssi); }
@@ -1739,9 +1738,10 @@ int rtw_select_and_join_from_scanned_queue23a(struct mlme_priv *pmlmepriv) ret = _FAIL; goto exit; }else { -DBG_8723A ("%s: candidate: %s(" MAC_FMT ", ch:%u)\n" , __func__, +DBG_8723A ("%s: candidate: %s(%pM, ch:%u)\n" , + __func__, candidate->network.Ssid.ssid, -MAC_ARG (candidate->network.MacAddress), + candidate->network.MacAddress, candidate->network.DSConfig); }
drivers/staging/rtl8723au/core/rtw_mlme_ext.c +33 -39
164 lines not on this page · raw message
drivers/staging/rtl8723au/core/rtw_recv.c +16 -26
87 lines not on this page · raw message
drivers/staging/rtl8723au/core/rtw_wlan_util.c +3 -3
@@ -876,9 +876,9 @@ int rtw_check_bcn_info23a(struct rtw_adapter *Adapter, }if (!ether_addr_equal (cur_network->network.MacAddress, mgmt->bssid)) { -DBG_8723A ("%s: linked but recv other bssid bcn" -MAC_FMT MAC_FMT "\n" , __func__,MAC_ARG (mgmt->bssid), -MAC_ARG (cur_network->network.MacAddress)); +DBG_8723A ("%s: linked but recv other bssid bcn %pM %pM\n" , + __func__, mgmt->bssid, + cur_network->network.MacAddress);return _FAIL; }
drivers/staging/rtl8723au/core/rtw_xmit.c +4 -4
@@ -493,8 +493,8 @@ static int update_attrib(struct rtw_adapter *padapter, psta =rtw_get_stainfo23a (pstapriv, pattrib->ra);if (psta ==NULL ) {/* if we cannot get psta => drrp the pkt */ RT_TRACE (_module_rtl871x_xmit_c_, _drv_alert_, - ("\nupdate_attrib => get sta_info fail, ra:" -MAC_FMT "\n" ,MAC_ARG (pattrib->ra))); + ("\nupdate_attrib => get sta_info fail, ra:%pM\n" , + pattrib->ra)); res = _FAIL; goto exit; } else if (check_fwstate(pmlmepriv, WIFI_AP_STATE) &&
@@ -511,8 +511,8 @@ static int update_attrib(struct rtw_adapter *padapter, }else {/* if we cannot get psta => drop the pkt */ RT_TRACE (_module_rtl871x_xmit_c_, _drv_alert_, - ("\nupdate_attrib => get sta_info fail, ra:" MAC_FMT -"\n" ,MAC_ARG (pattrib->ra))); + ("\nupdate_attrib => get sta_info fail, ra:%pM\n" , + pattrib->ra)); res = _FAIL; goto exit; }
drivers/staging/rtl8723au/include/ieee80211.h +0 -3
@@ -251,9 +251,6 @@ join_res:> 0: TID */ -#define MAC_FMT "%02x:%02x:%02x:%02x:%02x:%02x" -#define MAC_ARG (x) ((u8*)(x))[0],((u8*)(x))[1],((u8*)(x))[2],((u8*)(x))[3],((u8*)(x))[4],((u8*)(x))[5] -#define MAXTID 16#define WME_OUI_TYPE 2
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c +8 -10
@@ -1076,8 +1076,7 @@ static int cfg80211_rtw_get_station(struct wiphy *wiphy, ret = -ENOENT ; goto exit; } -DBG_8723A ("%s(%s): mac =" MAC_FMT "\n" , __func__, ndev->name, -MAC_ARG (mac)); +DBG_8723A ("%s(%s): mac=%pM\n" , __func__, ndev->name, mac);/* for infra./P2PClient mode */ if (check_fwstate(pmlmepriv, WIFI_STATION_STATE) &&
@@ -1085,8 +1084,8 @@ static int cfg80211_rtw_get_station(struct wiphy *wiphy,struct wlan_network *cur_network = &pmlmepriv->cur_network;if (!ether_addr_equal (mac, cur_network->network.MacAddress)) { -DBG_8723A ("%s, mismatch bssid =" MAC_FMT "\n" , __func__, -MAC_ARG (cur_network->network.MacAddress)); +DBG_8723A ("%s, mismatch bssid=%pM\n" , + __func__, cur_network->network.MacAddress); ret = -ENOENT ; goto exit; }
@@ -1989,7 +1988,7 @@ static int cfg80211_rtw_connect(struct wiphy *wiphy, struct net_device *ndev,DBG_8723A ("ssid =%s, len =%zu\n" , sme->ssid, sme->ssid_len);if (sme->bssid) -DBG_8723A ("bssid =" MAC_FMT "\n" ,MAC_ARG (sme->bssid)); +DBG_8723A ("bssid=%pM\n" , sme->bssid);if (check_fwstate (pmlmepriv, _FW_UNDER_LINKING)) { ret = -EBUSY ;
@@ -2523,8 +2522,8 @@ static int rtw_cfg80211_monitor_if_xmit_entry(struct sk_buff *skb, mgmt = (struct ieee80211_mgmt *)dot11_hdr; -DBG_8723A ("RTW_Tx:da =" MAC_FMT " via %s(%s)\n" , -MAC_ARG (mgmt->da), __func__, ndev->name); +DBG_8723A ("RTW_Tx:da=%pM via %s(%s)\n" , + mgmt->da, __func__, ndev->name); category = mgmt->u.action.category; action = mgmt->u.action.u.wme_action.action_code; DBG_8723A("RTW_Tx:category(%u), action(%u)\n",
@@ -2877,7 +2876,7 @@ static int cfg80211_rtw_del_station(struct wiphy *wiphy,return ret; } -DBG_8723A ("free sta macaddr =" MAC_FMT "\n" ,MAC_ARG (mac)); +DBG_8723A ("free sta macaddr=%pM\n" , mac);if (is_broadcast_ether_addr (mac))return -EINVAL ;
@@ -3053,8 +3052,7 @@ static int cfg80211_rtw_mgmt_tx(struct wiphy *wiphy, struct wireless_dev *wdev,cfg80211_mgmt_tx_status (padapter->rtw_wdev, *cookie, buf, len, ack,GFP_KERNEL ); -DBG_8723A ("RTW_Tx:tx_ch =%d, da =" MAC_FMT "\n" , tx_ch, -MAC_ARG (hdr->da)); +DBG_8723A ("RTW_Tx:tx_ch =%d, da =%pM\n" , tx_ch, hdr->da); category = hdr->u.action.category; action = hdr->u.action.u.wme_action.action_code;DBG_8723A ("RTW_Tx:category(%u), action(%u)\n" , category, action);
drivers/staging/rtl8723au/os_dep/os_intfs.c +3 -4
@@ -594,8 +594,8 @@ static int _rtw_drv_register_netdev(struct rtw_adapter *padapter, char *name) ret = _FAIL; goto error_register_netdev; } -DBG_8723A ("%s, MAC Address (if%d) = " MAC_FMT "\n" , __func__, - (padapter->iface_id +1 ),MAC_ARG (pnetdev->dev_addr)); +DBG_8723A ("%s, MAC Address (if%d) = %pM\n" , + __func__, padapter->iface_id +1 , pnetdev->dev_addr);return ret; error_register_netdev:
@@ -666,8 +666,7 @@ int netdev_open23a(struct net_device *pnetdev) goto netdev_open23a_error; } -DBG_8723A ("MAC Address = " MAC_FMT "\n" , -MAC_ARG (pnetdev->dev_addr)); +DBG_8723A ("MAC Address = %pM\n" , pnetdev->dev_addr);if (init_hw_mlme_ext23a (padapter) == _FAIL) {DBG_8723A ("can't init mlme_ext_priv\n" );
Quoted 12 lines · Joe Perches
Use the standard vsprintf kernel extension to format mac addresses.
This reduces object code size a bit.
Miscellanea:
o Coalesce formats
o Realign arguments
o Remove the now unusedMAC_FMTandMAC_ARG#definesSigned-off-by Joe Perches <[email protected]>
Quoted 11 lines
--- drivers/staging/rtl8723au/core/rtw_ap.c | 41 ++++++------- drivers/staging/rtl8723au/core/rtw_mlme.c | 26 ++++---- drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 72 +++++++++++------------ drivers/staging/rtl8723au/core/rtw_recv.c | 42 +++++-------- drivers/staging/rtl8723au/core/rtw_wlan_util.c | 6 +- drivers/staging/rtl8723au/core/rtw_xmit.c | 8 +-- drivers/staging/rtl8723au/include/ieee80211.h | 3 - drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 18 +++--- drivers/staging/rtl8723au/os_dep/os_intfs.c | 7 +-- 9 files changed, 101 insertions(+), 122 deletions(-)
Looks good to me
PATCH 1/1 staging: rtl8723au: Update RT_TRACE macro and uses
In reply to Joe PerchesCreate an rt_trace function using %pV to reduce overall code size. Update the macro uses to remove unnecessary and now harmful parentheses.
Miscellanea around these changes:
o Coalesce formats
o Realign arguments
o Remove commented-out RT_TRACE uses
o Spelling fixes in formats
o Add missing newlines to formats
o Remove multiple newlines from formats
o Neaten formats where noticed
o Use %pM in one instance
Reduces code size ~20KB
---
Mostly done by various scripts&emacs.
Compiled, untested, no hardware,
drivers/staging/rtl8723au/core/rtw_cmd.c | 55 ++--- drivers/staging/rtl8723au/core/rtw_efuse.c | 3 - drivers/staging/rtl8723au/core/rtw_ieee80211.c | 64 +++--- drivers/staging/rtl8723au/core/rtw_mlme.c | 134 ++++++----- drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 24 +- drivers/staging/rtl8723au/core/rtw_pwrctrl.c | 29 ++- drivers/staging/rtl8723au/core/rtw_recv.c | 244 ++++++++++----------- drivers/staging/rtl8723au/core/rtw_security.c | 58 +++-- drivers/staging/rtl8723au/core/rtw_sta_mgt.c | 16 +- drivers/staging/rtl8723au/core/rtw_wlan_util.c | 29 ++- drivers/staging/rtl8723au/core/rtw_xmit.c | 181 +++++++-------- drivers/staging/rtl8723au/hal/HalPwrSeqCmd.c | 38 ++-- drivers/staging/rtl8723au/hal/hal_com.c | 12 - drivers/staging/rtl8723au/hal/odm_debug.c | 15 ++ .../staging/rtl8723au/hal/rtl8723a_bt-coexist.c | 19 +- drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 124 +++++------ drivers/staging/rtl8723au/hal/rtl8723a_phycfg.c | 93 +------- drivers/staging/rtl8723au/hal/rtl8723au_recv.c | 2 +- drivers/staging/rtl8723au/hal/rtl8723au_xmit.c | 16 +- drivers/staging/rtl8723au/hal/usb_halinit.c | 40 ++-- drivers/staging/rtl8723au/hal/usb_ops_linux.c | 52 ++--- drivers/staging/rtl8723au/include/rtw_debug.h | 11 +- drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 91 ++++---- drivers/staging/rtl8723au/os_dep/os_intfs.c | 44 ++-- drivers/staging/rtl8723au/os_dep/recv_linux.c | 16 +- drivers/staging/rtl8723au/os_dep/usb_intf.c | 30 +-- drivers/staging/rtl8723au/os_dep/usb_ops_linux.c | 31 ++- drivers/staging/rtl8723au/os_dep/xmit_linux.c | 12 +- 28 files changed, 661 insertions(+), 822 deletions(-)
drivers/staging/rtl8723au/core/rtw_cmd.c +23 -32
@@ -299,9 +299,8 @@ post_process: pcmd_callback = rtw_cmd_callback[pcmd->cmdcode].callback;if (!pcmd_callback) {RT_TRACE (_module_rtl871x_cmd_c_, _drv_info_, - ("mlme_cmd_hdl(): pcmd_callback = 0x%p, " -"cmdcode = 0x%x\n" , - pcmd_callback, pcmd->cmdcode)); +"mlme_cmd_hdl(): pcmd_callback = 0x%p, cmdcode = 0x%x\n" , + pcmd_callback, pcmd->cmdcode);rtw_free_cmd_obj23a (pcmd); }else {/* need consider that free cmd_obj in
@@ -310,8 +309,8 @@ post_process: } }else {RT_TRACE (_module_rtl871x_cmd_c_, _drv_err_, - ("%s: cmdcode = 0x%x callback not defined!\n" , - __func__, pcmd->cmdcode)); +"%s: cmdcode = 0x%x callback not defined!\n" , + __func__, pcmd->cmdcode);rtw_free_cmd_obj23a (pcmd); } }
@@ -343,7 +342,7 @@ int rtw_sitesurvey_cmd23a(struct rtw_adapter *padapter,rtw_free_network_queue23a (padapter);RT_TRACE (_module_rtl871x_cmd_c_, _drv_info_, - ("%s: flush network queue\n" , __func__)); +"%s: flush network queue\n" , __func__);init_h2fwcmd_w_parm_no_rsp (ph2c, psurveyPara,GEN_CMD_CODE (_SiteSurvey));
@@ -412,12 +411,12 @@ int rtw_createbss_cmd23a(struct rtw_adapter *padapter)if (pmlmepriv->assoc_ssid.ssid_len ==0 ) {RT_TRACE (_module_rtl871x_cmd_c_, _drv_info_, - (" createbss for Any SSid:%s\n" , - pmlmepriv->assoc_ssid.ssid)); +"createbss for Any SSid:%s\n" , + pmlmepriv->assoc_ssid.ssid); }else {RT_TRACE (_module_rtl871x_cmd_c_, _drv_info_, - (" createbss for SSid:%s\n" , - pmlmepriv->assoc_ssid.ssid)); +"createbss for SSid:%s\n" , + pmlmepriv->assoc_ssid.ssid); } pcmd =kzalloc (sizeof (struct cmd_obj ),GFP_ATOMIC );
@@ -460,19 +459,18 @@ int rtw_joinbss_cmd23a(struct rtw_adapter *padapter,if (pmlmepriv->assoc_ssid.ssid_len ==0 ) {RT_TRACE (_module_rtl871x_cmd_c_, _drv_info_, - ("+Join cmd: Any SSid\n" )); +"+Join cmd: Any SSid\n" ); }else {RT_TRACE (_module_rtl871x_cmd_c_, _drv_notice_, - ("+Join cmd: SSid =[%s]\n" , - pmlmepriv->assoc_ssid.ssid)); +"+Join cmd: SSid =[%s]\n" , + pmlmepriv->assoc_ssid.ssid); } pcmd =kzalloc (sizeof (struct cmd_obj ),GFP_ATOMIC );if (!pcmd) { res = _FAIL;RT_TRACE (_module_rtl871x_cmd_c_, _drv_err_, - ("rtw_joinbss_cmd23a: memory allocate for cmd_obj " -"fail!!!\n" )); +"rtw_joinbss_cmd23a: memory allocate for cmd_obj fail!!!\n" ); goto exit; }
@@ -497,7 +495,7 @@ int rtw_joinbss_cmd23a(struct rtw_adapter *padapter, res = _FAIL;RT_TRACE (_module_rtl871x_cmd_c_, _drv_err_, - ("rtw_joinbss_cmd23a :psecnetwork == NULL!!!\n" )); +"rtw_joinbss_cmd23a :psecnetwork == NULL!!!\n" ); goto exit; }
@@ -599,7 +597,7 @@ int rtw_disassoc_cmd23a(struct rtw_adapter *padapter, u32 deauth_timeout_ms,int res = _SUCCESS;RT_TRACE (_module_rtl871x_cmd_c_, _drv_notice_, - ("+rtw_disassoc_cmd23a\n" )); +"+rtw_disassoc_cmd23a\n" );/* prepare cmd parameter */ param =kzalloc (sizeof (*param),GFP_ATOMIC );
@@ -1291,8 +1289,7 @@ void rtw_survey_cmd_callback23a(struct rtw_adapter *padapter,mod_timer (&pmlmepriv->scan_to_timer, jiffies +msecs_to_jiffies (1 ));RT_TRACE (_module_rtl871x_cmd_c_, _drv_err_, - ("\n ********Error: MgntActrtw_set_802_11_bssid23a_" -"LIST_SCAN Fail ************\n\n." )); +"********Error: MgntActrtw_set_802_11_bssid23a_LIST_SCAN Fail ************\n" ); }/* free cmd */
@@ -1309,7 +1306,7 @@ void rtw_disassoc_cmd23a_callback(struct rtw_adapter *padapter,set_fwstate (pmlmepriv, _FW_LINKED);spin_unlock_bh (&pmlmepriv->lock);RT_TRACE (_module_rtl871x_cmd_c_, _drv_err_, - ("\n ***Error: disconnect_cmd_callback Fail ***\n." )); +"***Error: disconnect_cmd_callback Fail ***\n" );return ; }
@@ -1329,8 +1326,7 @@ void rtw_joinbss_cmd23a_callback(struct rtw_adapter *padapter, jiffies + msecs_to_jiffies(1)); }else if (pcmd->res !=H2C_SUCCESS ) {RT_TRACE (_module_rtl871x_cmd_c_, _drv_err_, - ("********Error:rtw_select_and_join_from_scanned_" -"queue Wait Sema Fail ************\n" )); +"********Error:rtw_select_and_join_from_scanned_queue Wait Sema Fail ************\n" );mod_timer (&pmlmepriv->assoc_timer, jiffies +msecs_to_jiffies (1 )); }
@@ -1349,8 +1345,7 @@ void rtw_createbss_cmd23a_callback(struct rtw_adapter *padapter,if (pcmd->res !=H2C_SUCCESS ) {RT_TRACE (_module_rtl871x_cmd_c_, _drv_err_, - ("\n ********Error: rtw_createbss_cmd23a_callback " -"Fail ************\n\n." )); +"********Error: rtw_createbss_cmd23a_callback Fail ************\n" );mod_timer (&pmlmepriv->assoc_timer, jiffies +msecs_to_jiffies (1 )); }
@@ -1366,8 +1361,7 @@ void rtw_createbss_cmd23a_callback(struct rtw_adapter *padapter, GFP_KERNEL);if (!psta) {RT_TRACE (_module_rtl871x_cmd_c_, _drv_err_, - ("\nCan't alloc sta_info when " -"createbss_cmd_callback\n" )); +"Can't alloc sta_info when createbss_cmd_callback\n" ); goto createbss_cmd_fail; } }
@@ -1382,8 +1376,7 @@ void rtw_createbss_cmd23a_callback(struct rtw_adapter *padapter, pwlan =rtw_get_oldest_wlan_network23a (&pmlmepriv->scanned_queue);if (!pwlan) {RT_TRACE (_module_rtl871x_cmd_c_, _drv_err_, - ("\n Error: can't get pwlan in " -"rtw23a_joinbss_event_cb\n" )); +"Error: can't get pwlan in rtw23a_joinbss_event_cb\n" );spin_unlock_bh (&pmlmepriv->scanned_queue.lock); goto createbss_cmd_fail; }
@@ -1432,8 +1425,7 @@ void rtw_setstaKey_cmdrsp_callback23a(struct rtw_adapter *padapter,if (!psta) {RT_TRACE (_module_rtl871x_cmd_c_, _drv_err_, - ("\nERROR: rtw_setstaKey_cmdrsp_callback23a => " -"can't get sta_info\n\n" )); +"ERROR: rtw_setstaKey_cmdrsp_callback23a => can't get sta_info\n" ); goto exit; }
@@ -1457,8 +1449,7 @@ void rtw_setassocsta_cmdrsp_callback23a(struct rtw_adapter *padapter,if (psta ==NULL ) {RT_TRACE (_module_rtl871x_cmd_c_, _drv_err_, - ("\nERROR: setassocsta_cmdrsp_callbac => can't " -"get sta_info\n\n" )); +"ERROR: setassocsta_cmdrsp_callbac => can't get sta_info\n" ); goto exit; }
drivers/staging/rtl8723au/core/rtw_efuse.c +0 -3
@@ -335,7 +335,6 @@ EFUSE_Write1Byte(struct rtw_adapter *Adapter, u16 Address, u8 Value)u32 k =0 ;u16 contentLen =0 ; -/* RT_TRACE(COMP_EFUSE, DBG_LOUD, ("Addr =%x Data =%x\n", Address, Value)); */ EFUSE_GetEfuseDefinition23a (Adapter,EFUSE_WIFI ,TYPE_EFUSE_REAL_CONTENT_LEN , (void *)&contentLen);
@@ -404,8 +403,6 @@ efuse_OneByteWrite23a(struct rtw_adapter *pAdapter, u16 addr, u8 data)u8 tmpidx =0 ;int bResult; -/* RT_TRACE(COMP_EFUSE, DBG_LOUD, ("Addr = %x Data =%x\n", addr, data)); */ -/* return 0; */ /* -----------------e-fuse reg ctrl --------------------------------- */
drivers/staging/rtl8723au/core/rtw_ieee80211.c +30 -34
153 lines not on this page · raw message
drivers/staging/rtl8723au/core/rtw_mlme.c +61 -73
359 lines not on this page · raw message
drivers/staging/rtl8723au/core/rtw_mlme_ext.c +12 -12
@@ -657,8 +657,8 @@ void mgt_dispatcher23a(struct rtw_adapter *padapter,if (index >13 ) {RT_TRACE (_module_rtl871x_mlme_c_, _drv_err_, - ("Currently we do not support reserved sub-fr-type =" -"%d\n" , index)); +"Currently we do not support reserved sub-fr-type =%d\n" , + index);return ; } ptable += index;
@@ -2685,7 +2685,7 @@ static int _issue_probereq(struct rtw_adapter *padapter,u8 bc_addr[] = {0xff ,0xff ,0xff ,0xff ,0xff ,0xff };RT_TRACE (_module_rtl871x_mlme_c_, _drv_notice_, - ("+%s\n" , __func__)); +"+%s\n" , __func__); pmgntframe =alloc_mgtxmitframe23a (pxmitpriv);if (!pmgntframe)
@@ -2756,7 +2756,7 @@ static int _issue_probereq(struct rtw_adapter *padapter, pattrib->last_txcmdsz = pattrib->pktlen;RT_TRACE (_module_rtl871x_mlme_c_, _drv_notice_, - ("issuing probe_req, tx_len =%d\n" , pattrib->last_txcmdsz)); +"issuing probe_req, tx_len =%d\n" , pattrib->last_txcmdsz);if (wait_ack) { ret =dump_mgntframe23a_and_wait_ack23a (padapter, pmgntframe);
@@ -4341,7 +4341,8 @@ static void start_create_ibss(struct rtw_adapter *padapter)/* issue beacon */ if (send_beacon23a (padapter) == _FAIL) { -RT_TRACE (_module_rtl871x_mlme_c_, _drv_err_, ("issuing beacon frame fail....\n" )); +RT_TRACE (_module_rtl871x_mlme_c_, _drv_err_, +"issuing beacon frame fail....\n" );report_join_res23a (padapter,-1 ); pmlmeinfo->state =MSR_NOLINK ;
@@ -4525,7 +4526,7 @@ static void process_80211d(struct rtw_adapter *padapter, p +=3 ;RT_TRACE (_module_rtl871x_mlme_c_, _drv_notice_, - ("%s: 802.11d country =%s\n" , __func__, country)); +"%s: 802.11d country =%s\n" , __func__, country); i =0 ;while ((ie - p) >=3 ) {
@@ -4702,9 +4703,8 @@ static void process_80211d(struct rtw_adapter *padapter, chplan_new[i].ScanType =SCAN_ACTIVE ;RT_TRACE (_module_rtl871x_mlme_c_, _drv_notice_, - ("%s: change channel %d scan type " -"from passive to active\n" , - __func__, channel)); +"%s: change channel %d scan type from passive to active\n" , + __func__, channel); }break ; }
@@ -6060,7 +6060,7 @@ int mlme_evt_hdl23a(struct rtw_adapter *padapter, const u8 *pbuf)/* checking if event code is valid */ if (evt_code >=MAX_C2HEVT ) {RT_TRACE (_module_rtl871x_cmd_c_, _drv_err_, - ("\nEvent Code(%d) mismatch!\n" , evt_code)); +"Event Code(%d) mismatch!\n" , evt_code); goto _abort_event_; }
@@ -6068,8 +6068,8 @@ int mlme_evt_hdl23a(struct rtw_adapter *padapter, const u8 *pbuf)if (wlanevents[evt_code].parmsize !=0 && wlanevents[evt_code].parmsize != evt_sz) {RT_TRACE (_module_rtl871x_cmd_c_, _drv_err_, - ("\nEvent(%d) Parm Size mismatch (%d vs %d)!\n" , - evt_code, wlanevents[evt_code].parmsize, evt_sz)); +"Event(%d) Parm Size mismatch (%d vs %d)!\n" , + evt_code, wlanevents[evt_code].parmsize, evt_sz); goto _abort_event_; }
drivers/staging/rtl8723au/core/rtw_pwrctrl.c +14 -15
@@ -202,17 +202,17 @@ void rtw_set_rpwm23a(struct rtw_adapter *padapter, u8 pslv)if (pwrpriv->rpwm == pslv) {RT_TRACE (_module_rtl871x_pwrctrl_c_, _drv_err_, - ("%s: Already set rpwm[0x%02X], new = 0x%02X!\n" , - __func__, pwrpriv->rpwm, pslv)); +"%s: Already set rpwm[0x%02X], new = 0x%02X!\n" , + __func__, pwrpriv->rpwm, pslv);return ; }if (padapter->bSurpriseRemoved == true || padapter->hw_init_completed == false) {RT_TRACE (_module_rtl871x_pwrctrl_c_, _drv_err_, - ("%s: SurpriseRemoved(%d) hw_init_completed(%d)\n" , - __func__, padapter->bSurpriseRemoved, - padapter->hw_init_completed)); +"%s: SurpriseRemoved(%d) hw_init_completed(%d)\n" , + __func__, padapter->bSurpriseRemoved, + padapter->hw_init_completed); pwrpriv->cpwm =PS_STATE_S4 ;
@@ -221,22 +221,21 @@ void rtw_set_rpwm23a(struct rtw_adapter *padapter, u8 pslv)if (padapter->bDriverStopped == true) {RT_TRACE (_module_rtl871x_pwrctrl_c_, _drv_err_, - ("%s: change power state(0x%02X) when DriverStopped\n" , - __func__, pslv)); +"%s: change power state(0x%02X) when DriverStopped\n" , + __func__, pslv);if (pslv <PS_STATE_S2 ) {RT_TRACE (_module_rtl871x_pwrctrl_c_, _drv_err_, - ("%s: Reject to enter PS_STATE(0x%02X) lower " -"than S2 when DriverStopped!!\n" , - __func__, pslv)); +"%s: Reject to enter PS_STATE(0x%02X) lower than S2 when DriverStopped!!\n" , + __func__, pslv);return ; } } rpwm = pslv | pwrpriv->tog;RT_TRACE (_module_rtl871x_pwrctrl_c_, _drv_notice_, - ("rtw_set_rpwm23a: rpwm = 0x%02x cpwm = 0x%02x\n" , - rpwm, pwrpriv->cpwm)); +"rtw_set_rpwm23a: rpwm = 0x%02x cpwm = 0x%02x\n" , + rpwm, pwrpriv->cpwm); pwrpriv->rpwm = pslv;
@@ -282,12 +281,12 @@ void rtw_set_ps_mode23a(struct rtw_adapter *padapter, u8 ps_mode,struct pwrctrl_priv *pwrpriv = &padapter->pwrctrlpriv;RT_TRACE (_module_rtl871x_pwrctrl_c_, _drv_notice_, - ("%s: PowerMode =%d Smart_PS =%d\n" , - __func__, ps_mode, smart_ps)); +"%s: PowerMode =%d Smart_PS =%d\n" , + __func__, ps_mode, smart_ps);if (ps_mode > PM_Card_Disable) {RT_TRACE (_module_rtl871x_pwrctrl_c_, _drv_err_, - ("ps_mode:%d error\n" , ps_mode)); +"ps_mode:%d error\n" , ps_mode);return ; }
drivers/staging/rtl8723au/core/rtw_recv.c +112 -132
591 lines not on this page · raw message
drivers/staging/rtl8723au/core/rtw_security.c +34 -24
132 lines not on this page · raw message
drivers/staging/rtl8723au/core/rtw_sta_mgt.c +7 -9
@@ -136,10 +136,10 @@ rtw_alloc_stainfo23a(struct sta_priv *pstapriv, const u8 *hwaddr, gfp_t gfp) index =wifi_mac_hash (hwaddr);RT_TRACE (_module_rtl871x_sta_mgt_c_, _drv_info_, - ("rtw_alloc_stainfo23a: index = %x" , index)); +"rtw_alloc_stainfo23a: index = %x\n" , index);if (index >=NUM_STA ) {RT_TRACE (_module_rtl871x_sta_mgt_c_, _drv_err_, - ("ERROR => rtw_alloc_stainfo23a: index >= NUM_STA" )); +"ERROR => rtw_alloc_stainfo23a: index >= NUM_STA\n" ); psta =NULL ; goto exit; }
@@ -160,8 +160,8 @@ rtw_alloc_stainfo23a(struct sta_priv *pstapriv, const u8 *hwaddr, gfp_t gfp) &wRxSeqInitialValue, 2);RT_TRACE (_module_rtl871x_sta_mgt_c_, _drv_info_, - ("alloc number_%d stainfo with hwaddr = %pM\n" , - pstapriv->asoc_sta_count, hwaddr)); +"alloc number_%d stainfo with hwaddr = %pM\n" , + pstapriv->asoc_sta_count, hwaddr);init_addba_retry_timer23a (psta);
@@ -249,10 +249,8 @@ int rtw_free_stainfo23a(struct rtw_adapter *padapter, struct sta_info *psta)list_del_init (&psta->hash_list);RT_TRACE (_module_rtl871x_sta_mgt_c_, _drv_err_, - ("\n free number_%d stainfo with hwaddr = 0x%.2x 0x%.2x 0x%.2x 0x%.2x 0x%.2x 0x%.2x\n" , - pstapriv->asoc_sta_count, psta->hwaddr[0 ], - psta->hwaddr[1 ], psta->hwaddr[2 ], psta->hwaddr[3 ], - psta->hwaddr[4 ], psta->hwaddr[5 ])); +"free number_%d stainfo with hwaddr = %pM\n" , + pstapriv->asoc_sta_count, psta->hwaddr); pstapriv->asoc_sta_count--;/* re-init sta_info; 20061114 will be init in alloc_stainfo */
@@ -397,7 +395,7 @@ int rtw_init_bcmc_stainfo23a(struct rtw_adapter *padapter)if (psta ==NULL ) { res = _FAIL;RT_TRACE (_module_rtl871x_sta_mgt_c_, _drv_err_, - ("rtw_alloc_stainfo23a fail" )); +"rtw_alloc_stainfo23a fail\n" );return res; }/* default broadcast & multicast use macid 1 */
drivers/staging/rtl8723au/core/rtw_wlan_util.c +13 -16
@@ -926,10 +926,9 @@ int rtw_check_bcn_info23a(struct rtw_adapter *Adapter, }RT_TRACE (_module_rtl871x_mlme_c_, _drv_info_, - ("%s bssid.Ssid.Ssid:%s bssid.Ssid.SsidLength:%d " -"cur_network->network.Ssid.Ssid:%s len:%d\n" , __func__, - ssid, ssid_len, cur_network->network.Ssid.ssid, - cur_network->network.Ssid.ssid_len)); +"%s bssid.Ssid.Ssid:%s bssid.Ssid.SsidLength:%d cur_network->network.Ssid.Ssid:%s len:%d\n" , + __func__, ssid, ssid_len, cur_network->network.Ssid.ssid, + cur_network->network.Ssid.ssid_len); if (ssid_len != cur_network->network.Ssid.ssid_len || ssid_len > 32 || (ssid_len &&
@@ -947,8 +946,8 @@ int rtw_check_bcn_info23a(struct rtw_adapter *Adapter, privacy =0 ;RT_TRACE (_module_rtl871x_mlme_c_, _drv_info_, - ("%s(): cur_network->network.Privacy is %d, bssid.Privacy " -"is %d\n" , __func__, cur_network->network.Privacy, privacy)); +"%s(): cur_network->network.Privacy is %d, bssid.Privacy is %d\n" , + __func__, cur_network->network.Privacy, privacy);if (cur_network->network.Privacy != privacy) {DBG_8723A ("%s(), privacy is not match return FAIL\n" , __func__); goto _mismatch;
@@ -962,10 +961,9 @@ int rtw_check_bcn_info23a(struct rtw_adapter *Adapter, &pairwise_cipher, &is_8021x);if (r == _SUCCESS)RT_TRACE (_module_rtl871x_mlme_c_, _drv_info_, - ("%s pnetwork->pairwise_cipher: %d, " -"pnetwork->group_cipher: %d, is_802x " -": %d\n" , __func__, pairwise_cipher, - group_cipher, is_8021x)); +"%s pnetwork->pairwise_cipher: %d, pnetwork->group_cipher: %d, is_802x : %d\n" , + __func__, pairwise_cipher, + group_cipher, is_8021x); } }else { p = cfg80211_find_vendor_ie(WLAN_OUI_MICROSOFT,
@@ -977,10 +975,9 @@ int rtw_check_bcn_info23a(struct rtw_adapter *Adapter, &pairwise_cipher, &is_8021x);if (r == _SUCCESS)RT_TRACE (_module_rtl871x_mlme_c_, _drv_info_, - ("%s pnetwork->pairwise_cipher: %d, " -"group_cipher is %d, is_8021x is " -"%d\n" , __func__, pairwise_cipher, - group_cipher, is_8021x)); +"%s pnetwork->pairwise_cipher: %d, group_cipher is %d, is_8021x is %d\n" , + __func__, pairwise_cipher, + group_cipher, is_8021x); }else {if (privacy) crypto =ENCRYP_PROTOCOL_WEP ;
@@ -996,8 +993,8 @@ int rtw_check_bcn_info23a(struct rtw_adapter *Adapter,if (crypto ==ENCRYP_PROTOCOL_WPA || crypto ==ENCRYP_PROTOCOL_WPA2 ) {RT_TRACE (_module_rtl871x_mlme_c_, _drv_err_, - ("%s cur_network->group_cipher is %d: %d\n" , __func__, - cur_network->BcnInfo.group_cipher, group_cipher)); +"%s cur_network->group_cipher is %d: %d\n" , __func__, + cur_network->BcnInfo.group_cipher, group_cipher);if (pairwise_cipher != cur_network->BcnInfo.pairwise_cipher || group_cipher != cur_network->BcnInfo.group_cipher) { DBG_8723A("%s pairwise_cipher(%x:%x) or group_cipher "
drivers/staging/rtl8723au/core/rtw_xmit.c +92 -89
393 lines not on this page · raw message
drivers/staging/rtl8723au/hal/HalPwrSeqCmd.c +15 -23
@@ -59,17 +59,15 @@ u8 HalPwrSeqCmdParsing23a(struct rtw_adapter *padapter, u8 CutVersion, PwrCfgCmd = PwrSeqCmd[AryIdx];RT_TRACE (_module_hal_init_c_, _drv_info_, - ("HalPwrSeqCmdParsing23a: offset(%#x) cut_msk(%#x) " -"fab_msk(%#x) interface_msk(%#x) base(%#x) cmd(%#x) " -"msk(%#x) value(%#x)\n" , -GET_PWR_CFG_OFFSET (PwrCfgCmd), -GET_PWR_CFG_CUT_MASK (PwrCfgCmd), -GET_PWR_CFG_FAB_MASK (PwrCfgCmd), -GET_PWR_CFG_INTF_MASK (PwrCfgCmd), -GET_PWR_CFG_BASE (PwrCfgCmd), -GET_PWR_CFG_CMD (PwrCfgCmd), -GET_PWR_CFG_MASK (PwrCfgCmd), -GET_PWR_CFG_VALUE (PwrCfgCmd))); +"HalPwrSeqCmdParsing23a: offset(%#x) cut_msk(%#x) fab_msk(%#x) interface_msk(%#x) base(%#x) cmd(%#x) msk(%#x) value(%#x)\n" , +GET_PWR_CFG_OFFSET (PwrCfgCmd), +GET_PWR_CFG_CUT_MASK (PwrCfgCmd), +GET_PWR_CFG_FAB_MASK (PwrCfgCmd), +GET_PWR_CFG_INTF_MASK (PwrCfgCmd), +GET_PWR_CFG_BASE (PwrCfgCmd), +GET_PWR_CFG_CMD (PwrCfgCmd), +GET_PWR_CFG_MASK (PwrCfgCmd), +GET_PWR_CFG_VALUE (PwrCfgCmd));/* 2 Only Handle the command whose FAB, CUT, and Interface are matched */
@@ -79,14 +77,12 @@ u8 HalPwrSeqCmdParsing23a(struct rtw_adapter *padapter, u8 CutVersion,switch (GET_PWR_CFG_CMD (PwrCfgCmd)) {case PWR_CMD_READ :RT_TRACE (_module_hal_init_c_, _drv_info_, - ("HalPwrSeqCmdParsing23a: " -"PWR_CMD_READ\n" )); +"HalPwrSeqCmdParsing23a: PWR_CMD_READ\n" );break ;case PWR_CMD_WRITE :RT_TRACE (_module_hal_init_c_, _drv_info_, - ("HalPwrSeqCmdParsing23a: " -"PWR_CMD_WRITE\n" )); +"HalPwrSeqCmdParsing23a: PWR_CMD_WRITE\n" ); offset =GET_PWR_CFG_OFFSET (PwrCfgCmd);/* Read the value from system register */
@@ -102,8 +98,7 @@ u8 HalPwrSeqCmdParsing23a(struct rtw_adapter *padapter, u8 CutVersion,case PWR_CMD_POLLING :RT_TRACE (_module_hal_init_c_, _drv_info_, - ("HalPwrSeqCmdParsing23a: " -"PWR_CMD_POLLING\n" )); +"HalPwrSeqCmdParsing23a: PWR_CMD_POLLING\n" ); bPollingBit = false; offset =GET_PWR_CFG_OFFSET (PwrCfgCmd);
@@ -131,8 +126,7 @@ u8 HalPwrSeqCmdParsing23a(struct rtw_adapter *padapter, u8 CutVersion,case PWR_CMD_DELAY :RT_TRACE (_module_hal_init_c_, _drv_info_, - ("HalPwrSeqCmdParsing23a: " -"PWR_CMD_DELAY\n" )); +"HalPwrSeqCmdParsing23a: PWR_CMD_DELAY\n" );if (GET_PWR_CFG_VALUE (PwrCfgCmd) ==PWRSEQ_DELAY_US )udelay (GET_PWR_CFG_OFFSET (PwrCfgCmd));
@@ -145,14 +139,12 @@ u8 HalPwrSeqCmdParsing23a(struct rtw_adapter *padapter, u8 CutVersion,/* When this command is parsed, end the process */ RT_TRACE (_module_hal_init_c_, _drv_info_, - ("HalPwrSeqCmdParsing23a: " -"PWR_CMD_END\n" )); +"HalPwrSeqCmdParsing23a: PWR_CMD_END\n" );return true;default :RT_TRACE (_module_hal_init_c_, _drv_err_, - ("HalPwrSeqCmdParsing23a: " -"Unknown CMD!!\n" )); +"HalPwrSeqCmdParsing23a: Unknown CMD!!\n" );break ; } }
drivers/staging/rtl8723au/hal/hal_com.c +0 -12
@@ -395,9 +395,6 @@ rtl8723a_set_ampdu_min_space(struct rtw_adapter *padapter, u8 MinSpacingToSet)if (MinSpacingToSet < SecMinSpace) MinSpacingToSet = SecMinSpace; -/* RT_TRACE(COMP_MLME, DBG_LOUD, -("Set HW_VAR_AMPDU_MIN_SPACE: %#x\n", -padapter->MgntInfo.MinSpaceCfg)); */ MinSpacingToSet |=rtl8723au_read8 (padapter,REG_AMPDU_MIN_SPACE ) &0xf8 ; rtl8723au_write8(padapter, REG_AMPDU_MIN_SPACE,
@@ -437,9 +434,6 @@ void rtl8723a_set_ampdu_factor(struct rtw_adapter *padapter, u8 FactorToSet)rtl8723au_write8 (padapter,REG_AGGLEN_LMT + index, pRegToSet[index]); } - -/* RT_TRACE(COMP_MLME, DBG_LOUD, -("Set HW_VAR_AMPDU_FACTOR: %#x\n", FactorToSet)); */ } }
@@ -624,14 +618,8 @@ void rtl8723a_cam_empty_entry(struct rtw_adapter *padapter, u8 ucIndex)/* write content 0 is equall to mark invalid */ /* delay_ms(40); */ rtl8723au_write32 (padapter,WCAMI , ulContent); -/* RT_TRACE(COMP_SEC, DBG_LOUD, -("rtl8723a_cam_empty_entry(): WRITE A4: %lx\n", -ulContent));*/ /* delay_ms(40); */ rtl8723au_write32 (padapter,REG_CAMCMD , ulCommand); -/* RT_TRACE(COMP_SEC, DBG_LOUD, -("rtl8723a_cam_empty_entry(): WRITE A0: %lx\n", -ulCommand));*/ } }
drivers/staging/rtl8723au/hal/odm_debug.c +15 -0
@@ -22,3 +22,18 @@ void ODM_InitDebugSetting23a(struct dm_odm_t *pDM_Odm) }u32 GlobalDebugLevel23A; + +void rt_trace (int comp,int level,const char *fmt, ...) +{ +struct va_format vaf; +va_list args; + +va_start (args, fmt); + + vaf.fmt = fmt; + vaf.va = &args; + +pr_info (DRIVER_PREFIX " [0x%08x,%d] %pV" , comp, level, &vaf); + +va_end (args); +}
drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c +11 -8
@@ -1554,7 +1554,8 @@ static void bthci_ResetBtSec(struct rtw_adapter *padapter, struct bt_security *p pBtSec->bUsedHwEncrypt = true; else pBtSec->bUsedHwEncrypt = false; -RT_TRACE (_module_rtl871x_security_c_, _drv_info_, ("%s: bUsedHwEncrypt =%d\n" , __func__, pBtSec->bUsedHwEncrypt)); +RT_TRACE (_module_rtl871x_security_c_, _drv_info_, +"%s: bUsedHwEncrypt =%d\n" , __func__, pBtSec->bUsedHwEncrypt); pBtSec->RSNIE.Octet = pBtSec->RSNIEBuf; }
@@ -4518,8 +4519,8 @@ bthci_StateConnecting(struct rtw_adapter *padapter,RTPRINT (FIOCTL ,IOCTL_STATE , ("STATE_CMD_MAC_CONNECT_COMPLETE\n" ));if (pBTInfo->BtAsocEntry[EntryNum].AMPRole ==AMP_BTAP_JOINER ) { -RT_TRACE (_module_rtl871x_security_c_, - _drv_info_, ("StateConnecting \n" )); +RT_TRACE (_module_rtl871x_security_c_, _drv_info_, +"StateConnecting\n" ); }break ;case STATE_CMD_DISCONNECT_PHY_LINK :
@@ -11196,15 +11197,17 @@ void rtl8723a_BT_init_hal_vars(struct rtw_adapter *padapter) pHalData->bt_coexist.bt_radiosharedtype = pHalData->EEPROMBluetoothRadioShared;RT_TRACE (_module_hal_init_c_, _drv_info_, - ("BT Coexistance = 0x%x\n" ,rtl8723a_BT_coexist (padapter))); +"BT Coexistance = 0x%x\n" ,rtl8723a_BT_coexist (padapter));if (rtl8723a_BT_coexist (padapter)) {if (pHalData->bt_coexist.BT_Ant_Num == Ant_x2) {BTDM_SetBtCoexCurrAntNum (padapter,2 ); -RT_TRACE (_module_hal_init_c_, _drv_info_, ("BlueTooth BT_Ant_Num = Antx2\n" )); +RT_TRACE (_module_hal_init_c_, _drv_info_, +"BlueTooth BT_Ant_Num = Antx2\n" ); }else if (pHalData->bt_coexist.BT_Ant_Num == Ant_x1) {BTDM_SetBtCoexCurrAntNum (padapter,1 ); -RT_TRACE (_module_hal_init_c_, _drv_info_, ("BlueTooth BT_Ant_Num = Antx1\n" )); +RT_TRACE (_module_hal_init_c_, _drv_info_, +"BlueTooth BT_Ant_Num = Antx1\n" ); } pHalData->bt_coexist.bBTBusyTraffic = false; pHalData->bt_coexist.bBTTrafficModeSet = false;
@@ -11213,8 +11216,8 @@ void rtl8723a_BT_init_hal_vars(struct rtw_adapter *padapter) pHalData->bt_coexist.PreviousState =0 ;RT_TRACE (_module_hal_init_c_, _drv_info_, - ("bt_radiosharedType = 0x%x\n" , - pHalData->bt_coexist.bt_radiosharedtype)); +"bt_radiosharedType = 0x%x\n" , + pHalData->bt_coexist.bt_radiosharedtype); } }
drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c +57 -67
306 lines not on this page · raw message
drivers/staging/rtl8723au/hal/rtl8723a_phycfg.c +1 -92
170 lines not on this page · raw message
drivers/staging/rtl8723au/hal/rtl8723au_recv.c +1 -1
@@ -48,7 +48,7 @@ int rtl8723au_init_recv_priv(struct rtw_adapter *padapter)if (!precvpriv->precv_buf) { res = _FAIL;RT_TRACE (_module_rtl871x_recv_c_, _drv_err_, - ("alloc recv_buf fail!\n" )); +"alloc recv_buf fail!\n" ); goto exit; }
drivers/staging/rtl8723au/hal/rtl8723au_xmit.c +9 -7
@@ -272,7 +272,8 @@ static s32 update_txdesc(struct xmit_frame *pxmitframe, u8 *pmem, s32 sz, u8 bagif (bmcst) ptxdesc->txdw0 |=cpu_to_le32 (BIT (24 )); -RT_TRACE (_module_rtl871x_xmit_c_, _drv_info_, ("offset0-txdesc = 0x%x\n" , ptxdesc->txdw0)); +RT_TRACE (_module_rtl871x_xmit_c_, _drv_info_, +"offset0-txdesc = 0x%x\n" , ptxdesc->txdw0);/* offset 4 */ /* pkt_offset, unit:8 bytes padding */
@@ -303,7 +304,7 @@ static int rtw_dump_xframe(struct rtw_adapter *padapter, mem_addr = pxmitframe->buf_addr; -RT_TRACE (_module_rtl871x_xmit_c_, _drv_info_, ("rtw_dump_xframe()\n" )); +RT_TRACE (_module_rtl871x_xmit_c_, _drv_info_,"rtw_dump_xframe()\n" );for (t =0 ; t < pattrib->nr_frags; t++) {if (inner_ret != _SUCCESS && ret == _SUCCESS)
@@ -311,7 +312,7 @@ static int rtw_dump_xframe(struct rtw_adapter *padapter,if (t != (pattrib->nr_frags -1 )) {RT_TRACE (_module_rtl871x_xmit_c_, _drv_err_, - ("pattrib->nr_frags =%d\n" , pattrib->nr_frags)); +"pattrib->nr_frags =%d\n" , pattrib->nr_frags); sz = pxmitpriv->frag_len; sz = sz -4 - pattrib->icv_len;
@@ -338,7 +339,7 @@ static int rtw_dump_xframe(struct rtw_adapter *padapter,rtw_count_tx_stats23a (padapter, pxmitframe, sz);RT_TRACE (_module_rtl871x_xmit_c_, _drv_info_, - ("rtw_write_port, w_sz =%d\n" , w_sz)); +"rtw_write_port, w_sz =%d\n" , w_sz); mem_addr += w_sz;
@@ -365,7 +366,7 @@ bool rtl8723au_xmitframe_complete(struct rtw_adapter *padapter, phwxmits = pxmitpriv->hwxmits; hwentry = pxmitpriv->hwxmit_entry; -RT_TRACE (_module_rtl871x_xmit_c_, _drv_info_, ("xmitframe_complete()\n" )); +RT_TRACE (_module_rtl871x_xmit_c_, _drv_info_,"xmitframe_complete()\n" );if (pxmitbuf ==NULL ) { pxmitbuf =rtw_alloc_xmitbuf23a (pxmitpriv);
@@ -388,7 +389,8 @@ bool rtl8723au_xmitframe_complete(struct rtw_adapter *padapter,rtw_os_xmit_complete23a (padapter, pxmitframe);/* always return ndis_packet after rtw_xmitframe_coalesce23a */ } -RT_TRACE (_module_rtl871x_xmit_c_, _drv_info_, ("xmitframe_complete(): rtw_dump_xframe\n" )); +RT_TRACE (_module_rtl871x_xmit_c_, _drv_info_, +"xmitframe_complete(): rtw_dump_xframe\n" );if (res == _SUCCESS) {rtw_dump_xframe (padapter, pxmitframe);
@@ -481,7 +483,7 @@ enqueue:if (res != _SUCCESS) {RT_TRACE (_module_xmit_osdep_c_, _drv_err_, - ("pre_xmitframe: enqueue xmitframe fail\n" )); +"pre_xmitframe: enqueue xmitframe fail\n" );rtw_free_xmitframe23a (pxmitpriv, pxmitframe);/* Trick, make the statistics correct */
drivers/staging/rtl8723au/hal/usb_halinit.c +19 -21
@@ -516,7 +516,7 @@ int rtl8723au_hal_init(struct rtw_adapter *Adapter)/* Check if MAC has already power on. by tynli. 2011.05.27. */ val8 =rtl8723au_read8 (Adapter,REG_CR );RT_TRACE (_module_hci_hal_init_c_, _drv_info_, - ("%s: REG_CR 0x100 = 0x%02x\n" , __func__, val8)); +"%s: REG_CR 0x100 = 0x%02x\n" , __func__, val8);/* Fix 92DU-VC S3 hang with the reason is that secondary mac is not initialized. */ /* 0x100 value of first mac is 0xEA while 0x100 value of secondary
@@ -526,13 +526,13 @@ int rtl8723au_hal_init(struct rtw_adapter *Adapter) }else { mac_on = true;RT_TRACE (_module_hci_hal_init_c_, _drv_info_, - ("%s: MAC has already power on\n" , __func__)); +"%s: MAC has already power on\n" , __func__); } status =_InitPowerOn (Adapter);if (status == _FAIL) {RT_TRACE (_module_hci_hal_init_c_, _drv_err_, - ("Failed to init power on!\n" )); +"Failed to init power on!\n" ); goto exit; }
@@ -547,7 +547,7 @@ int rtl8723au_hal_init(struct rtw_adapter *Adapter) status =InitLLTTable23a (Adapter, boundary);if (status == _FAIL) {RT_TRACE (_module_hci_hal_init_c_, _drv_err_, - ("Failed to init LLT table\n" )); +"Failed to init LLT table\n" ); goto exit; } }
@@ -623,8 +623,8 @@ int rtl8723au_hal_init(struct rtw_adapter *Adapter)rtl8723au_write32 (Adapter, rFPGA0_XA_RFInterfaceOE,0x66F60210 );RT_TRACE (_module_hci_hal_init_c_, _drv_info_, - ("%s: 0x870 = value 0x%x\n" , __func__, -rtl8723au_read32 (Adapter,0x870 ))); +"%s: 0x870 = value 0x%x\n" , __func__, +rtl8723au_read32 (Adapter,0x870 ));/* */ /* Joseph Note: Keep RfRegChnlVal for later use. */
@@ -744,7 +744,8 @@ int rtl8723au_hal_init(struct rtw_adapter *Adapter)if (((rtl8723au_read32 (Adapter, rFPGA0_RFMOD) &0xFF000000 ) !=0x83000000 )) {PHY_SetBBReg (Adapter, rFPGA0_RFMOD,BIT (24 ),1 ); -RT_TRACE (_module_hci_hal_init_c_, _drv_err_, ("%s: IQK fail recorver\n" , __func__)); +RT_TRACE (_module_hci_hal_init_c_, _drv_err_, +"%s: IQK fail recover\n" , __func__); }/* ack for xmit mgmt frames. */
@@ -817,7 +818,7 @@ static void phy_SsPwrSwitch92CU(struct rtw_adapter *Adapter,else sps0 &= ~(BIT (0 ) |BIT (3 )); -RT_TRACE (_module_hal_init_c_, _drv_err_, ("SS LVL1\n" )); +RT_TRACE (_module_hal_init_c_, _drv_err_,"SS LVL1\n" );/* Disable RF and BB only for SelectSuspend. */ /* 1. Set BB/RF to shutdown. */
@@ -933,14 +934,14 @@ int rtl8723au_inirp_init(struct rtw_adapter *Adapter) status = _SUCCESS; -RT_TRACE (_module_hci_hal_init_c_, _drv_info_, ("===> usb_inirp_init\n" )); +RT_TRACE (_module_hci_hal_init_c_, _drv_info_,"===> usb_inirp_init\n" );/* issue Rx irp to receive data */ precvbuf = (struct recv_buf *)precvpriv->precv_buf;for (i =0 ; i <NR_RECVBUFF ; i++) {if (rtl8723au_read_port (Adapter,0 , precvbuf) == _FAIL) {RT_TRACE (_module_hci_hal_init_c_, _drv_err_, - ("usb_rx_init: usb_read_port error\n" )); +"usb_rx_init: usb_read_port error\n" ); status = _FAIL; goto exit; }
@@ -948,7 +949,7 @@ int rtl8723au_inirp_init(struct rtw_adapter *Adapter) }if (rtl8723au_read_interrupt (Adapter) == _FAIL) {RT_TRACE (_module_hci_hal_init_c_, _drv_err_, - ("%s: usb_read_interrupt error\n" , __func__)); +"%s: usb_read_interrupt error\n" , __func__); status = _FAIL; } pHalData->IntrMask[0 ] =rtl8723au_read32 (Adapter,REG_USB_HIMR );
@@ -957,7 +958,7 @@ int rtl8723au_inirp_init(struct rtw_adapter *Adapter)rtl8723au_write32 (Adapter,REG_USB_HIMR , pHalData->IntrMask[0 ]); exit:RT_TRACE (_module_hci_hal_init_c_, _drv_info_, - ("<=== usb_inirp_init\n" )); +"<=== usb_inirp_init\n" );return status; }
@@ -966,7 +967,7 @@ int rtl8723au_inirp_deinit(struct rtw_adapter *Adapter)struct hal_data_8723a *pHalData =GET_HAL_DATA (Adapter);RT_TRACE (_module_hci_hal_init_c_, _drv_info_, - ("\n ===> usb_rx_deinit\n" )); +"===> usb_rx_deinit\n" );rtl8723au_read_port_cancel (Adapter); pHalData->IntrMask[0 ] =rtl8723au_read32 (Adapter,REG_USB_HIMR ); MSG_8723A("%s pHalData->IntrMask = 0x%04x\n", __func__,
@@ -974,7 +975,7 @@ int rtl8723au_inirp_deinit(struct rtw_adapter *Adapter) pHalData->IntrMask[0 ] =0x0 ;rtl8723au_write32 (Adapter,REG_USB_HIMR , pHalData->IntrMask[0 ]);RT_TRACE (_module_hci_hal_init_c_, _drv_info_, - ("\n <=== usb_rx_deinit\n" )); +"<=== usb_rx_deinit\n" );return _SUCCESS; }
@@ -1021,11 +1022,10 @@ static void Hal_EfuseParseMACAddr_8723AU(struct rtw_adapter *padapter, }RT_TRACE (_module_hci_hal_init_c_, _drv_notice_, - ("Hal_EfuseParseMACAddr_8723AU: Permanent Address =%02x:%02x:" -"%02x:%02x:%02x:%02x\n" , - pEEPROM->mac_addr[0 ], pEEPROM->mac_addr[1 ], - pEEPROM->mac_addr[2 ], pEEPROM->mac_addr[3 ], - pEEPROM->mac_addr[4 ], pEEPROM->mac_addr[5 ])); +"Hal_EfuseParseMACAddr_8723AU: Permanent Address =%02x:%02x:%02x:%02x:%02x:%02x\n" , + pEEPROM->mac_addr[0 ], pEEPROM->mac_addr[1 ], + pEEPROM->mac_addr[2 ], pEEPROM->mac_addr[3 ], + pEEPROM->mac_addr[4 ], pEEPROM->mac_addr[5 ]); }static void readAdapterInfo (struct rtw_adapter *padapter)
@@ -1165,8 +1165,6 @@ int GetHalDefVar8192CUsb(struct rtw_adapter *Adapter, }break ;default : -/* RT_TRACE(COMP_INIT, DBG_WARNING, ("GetHalDefVar8192CUsb(): " -"Unkown variable: %d!\n", eVariable)); */ bResult = _FAIL;break ; }
drivers/staging/rtl8723au/hal/usb_ops_linux.c +22 -30
118 lines not on this page · raw message
drivers/staging/rtl8723au/include/rtw_debug.h +5 -6
Lines of this file are not shown. View the raw message.
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c +43 -48
91 lines not on this page · raw message
drivers/staging/rtl8723au/os_dep/os_intfs.c +22 -22
44 lines not on this page · raw message
drivers/staging/rtl8723au/os_dep/recv_linux.c +8 -8
Lines of this file are not shown. View the raw message.
drivers/staging/rtl8723au/os_dep/usb_intf.c +15 -15
Lines of this file are not shown. View the raw message.
drivers/staging/rtl8723au/os_dep/usb_ops_linux.c +14 -17
Lines of this file are not shown. View the raw message.
drivers/staging/rtl8723au/os_dep/xmit_linux.c +6 -6
Lines of this file are not shown. View the raw message.
The rest of this diff is not shown. View the raw message.
Signature · 1 line
2.1.2
Part of the diff is not shown here. View the raw message.
Re: [PATCH] staging: rtl8723au: Update RT_TRACE macro and uses
Quoted 14 lines · Joe Perches
Create an
rt_tracefunction using %pV to reduce overall code size. Update the macro uses to remove unnecessary and now harmful parentheses.Miscellanea around these changes:
o Coalesce formats
o Realign arguments
o Remove commented-outRT_TRACEuses
o Spelling fixes in formats
o Add missing newlines to formats
o Remove multiple newlines from formats
o Neaten formats where noticed
o Use %pM in one instance
Quoted 6 lines
Reduces code size ~20KB
Signed-off-by Joe Perches <[email protected]>---
Mostly done by various scripts&emacs.
Compiled, untested, no hardware,
This could be further improved by fixing up all the places where the function name is hard coded into the print statement, instead of using __func__. In particular as a lot of it is carried over from old code and has been renamed since.
It's OK with me to do this in a follow-on patch though.
Jes
Re: [PATCH] staging: rtl8723au: Update RT_TRACE macro and uses
Quoted 3 lines · Jes Sorensen
Quoted 2 lines · Joe Perches
Create an
rt_tracefunction using %pV to reduce overall code size. Update the macro uses to remove unnecessary and now harmful parentheses.
[]
Quoted 4 lines
This could be further improved by fixing up all the places where the function name is hard coded into the print statement, instead of using
__func__. In particular as a lot of it is carried over from old code and has been renamed since.
There's a cocci script for that.
http://cocci.systeme.lip6.narkive.com/nKXf6Bmy/finding-embedded-function-names
I expect someone will run it one day on staging.