Re: [PATCH 14/16] staging: rtl8723bs: remove all RT_TRACE logs in core/rtw_wlan_util.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Apr 02, 2021 at 08:20:17AM -0700, Joe Perches wrote:
> On Fri, 2021-04-02 at 14:51 +0200, Fabio Aiuto wrote:
> > On Fri, Apr 02, 2021 at 03:37:57AM -0700, Joe Perches wrote:
> > > On Fri, 2021-04-02 at 12:01 +0200, Fabio Aiuto wrote:
> > > > remove all RT_TRACE logs
> > > > 
> > > > fix patch-related checkpatch issues
> > > > 
> > > > Signed-off-by: Fabio Aiuto <fabioaiuto83@xxxxxxxxx>
> > > > ---
> > > >  .../staging/rtl8723bs/core/rtw_wlan_util.c    | 26 +++++--------------
> > > >  1 file changed, 6 insertions(+), 20 deletions(-)
> > > > 
> > > > diff --git a/drivers/staging/rtl8723bs/core/rtw_wlan_util.c b/drivers/staging/rtl8723bs/core/rtw_wlan_util.c
> > > []
> > > > @@ -1382,25 +1374,19 @@ int rtw_check_bcn_info(struct adapter *Adapter, u8 *pframe, u32 packet_len)
> > > >  	if (encryp_protocol == ENCRYP_PROTOCOL_WPA || encryp_protocol == ENCRYP_PROTOCOL_WPA2) {
> > > >  		pbuf = rtw_get_wpa_ie(&bssid->IEs[12], &wpa_ielen, bssid->IELength-12);
> > > >  		if (pbuf && (wpa_ielen > 0)) {
> > > > -			if (_SUCCESS == rtw_parse_wpa_ie(pbuf, wpa_ielen+2, &group_cipher, &pairwise_cipher, &is_8021x)) {
> > > > -				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));
> > > > -			}
> > > > +			if (rtw_parse_wpa_ie(pbuf, wpa_ielen + 2, &group_cipher,
> > > > +					     &pairwise_cipher, &is_8021x) == _SUCCESS)
> > > > +				;
> > > 
> > > 		This sort of if is a bit silly.
> > > 		Better would be to remove the test and just use:
> > > 
> > > 			rtw_parse_wpa_ie(pbuf, wpa_ielen + 2, &group_cipher,
> > > 					 &pairwise_cipher, &is_8021x);
> > > 
> > > >  		} else {
> > > >  			pbuf = rtw_get_wpa2_ie(&bssid->IEs[12], &wpa_ielen, bssid->IELength-12);
> > > >  
> > > > 
> > > >  			if (pbuf && (wpa_ielen > 0)) {
> > > > -				if (_SUCCESS == rtw_parse_wpa2_ie(pbuf, wpa_ielen+2, &group_cipher, &pairwise_cipher, &is_8021x)) {
> > > > -					RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_,
> > > > -							("%s pnetwork->pairwise_cipher: %d, pnetwork->group_cipher is %d, is_802x is %d\n",
> > > > -							 __func__, pairwise_cipher, group_cipher, is_8021x));
> > > > -				}
> > > > +				if (rtw_parse_wpa2_ie(pbuf, wpa_ielen + 2, &group_cipher,
> > > > +						      &pairwise_cipher, &is_8021x) == _SUCCESS)
> > > > +					;
> > > 
> > > 				rtw_parse_wpa2_ie(pbuf, wpa_ielen + 2, &group_cipher,
> > > 						  &pairwise_cipher, &is_8021x);
> > > 
> > > etc...
> > > 
> > > Lastly, another suggestion would be to just submit a single patch
> > > removing _ALL_ the RT_TRACE uses not intermixing various other cleanups
> > > with the series and then do those other cleanups.
> > > 
> > > Using a coccinelle script like:
> > > 
> > > $ cat RT_TRACE.cocci
> > > @@
> > > expression a, b, c;
> > > @@
> > > 
> > > -	RT_TRACE(a, b, (c));
> > > 
> > > $ spatch -sp-file RT_TRACE.cocci drivers/staging/rtl8723bs/
> > > 
> > > And then clean up the various bits you think are inappropriately done.
> > > 
> > > 
> > 
> > thank you Joe, I tried with (RT_TRACE.cocci in parent folder)
> > 
> > user@host:~/src/git/kernels/staging$ spatch -sp-file ../RT_TRACE.cocci drivers/staging/rtl8723bs/
> > init_defs_builtins: /usr/local/bin/../lib/coccinelle/standard.h
> > 0 files match
> 
> Likely you are running the script on the tree after you have
> applied all your patches.
> 
> Try running the cocci script on a fresh copy of -next.
> 
> Using the script and adding the script in the commit message helps
> others to verify that the changes you make do not have any other effect.
> 
> $ cat RT_TRACE.cocci
> @@
> expression a, b, c;
> @@
> 
> -	RT_TRACE(a, b, (c));
> 
> $ git checkout next-20210401
> $ spatch -sp-file RT_TRACE.cocci --in-place --no-show-diff --very-quiet drivers/staging/rtl8723bs/
> 31 files match
> $ git diff --stat -p
>  drivers/staging/rtl8723bs/core/rtw_cmd.c          |  34 +------
>  drivers/staging/rtl8723bs/core/rtw_eeprom.c       |  11 ---
>  drivers/staging/rtl8723bs/core/rtw_ieee80211.c    |  41 --------
>  drivers/staging/rtl8723bs/core/rtw_ioctl_set.c    |  68 +------------
>  drivers/staging/rtl8723bs/core/rtw_mlme.c         |  72 +-------------
>  drivers/staging/rtl8723bs/core/rtw_mlme_ext.c     |  24 -----
>  drivers/staging/rtl8723bs/core/rtw_pwrctrl.c      |  45 ---------
>  drivers/staging/rtl8723bs/core/rtw_recv.c         | 113 +---------------------
>  drivers/staging/rtl8723bs/core/rtw_security.c     |  35 -------
>  drivers/staging/rtl8723bs/core/rtw_sta_mgt.c      |  25 -----
>  drivers/staging/rtl8723bs/core/rtw_wlan_util.c    |  16 ---
>  drivers/staging/rtl8723bs/core/rtw_xmit.c         |  59 +----------
>  drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c      |  46 ---------
>  drivers/staging/rtl8723bs/hal/hal_intf.c          |   2 -
>  drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c |  93 ------------------
>  drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c   |  32 +-----
>  drivers/staging/rtl8723bs/hal/rtl8723bs_recv.c    |   2 -
>  drivers/staging/rtl8723bs/hal/rtl8723bs_xmit.c    |  32 ------
>  drivers/staging/rtl8723bs/hal/sdio_halinit.c      |  34 -------
>  drivers/staging/rtl8723bs/hal/sdio_ops.c          |  31 ------
>  drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c |   6 --
>  drivers/staging/rtl8723bs/os_dep/ioctl_linux.c    |  53 ----------
>  drivers/staging/rtl8723bs/os_dep/mlme_linux.c     |   4 -
>  drivers/staging/rtl8723bs/os_dep/os_intfs.c       |  46 ---------
>  drivers/staging/rtl8723bs/os_dep/osdep_service.c  |   1 -
>  drivers/staging/rtl8723bs/os_dep/recv_linux.c     |   9 --
>  drivers/staging/rtl8723bs/os_dep/sdio_intf.c      |  15 ---
>  drivers/staging/rtl8723bs/os_dep/xmit_linux.c     |   5 -
>  28 files changed, 19 insertions(+), 935 deletions(-)
> 
> diff --git a/drivers/staging/rtl8723bs/core/rtw_cmd.c b/drivers/staging/rtl8723bs/core/rtw_cmd.c
> index 2aaf25b48f96..ff4d771d8b8a 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_cmd.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_cmd.c
> @@ -221,8 +221,6 @@ int rtw_init_evt_priv(struct evt_priv *pevtpriv)
>  
>  void _rtw_free_evt_priv(struct	evt_priv *pevtpriv)
>  {
> -	RT_TRACE(_module_rtl871x_cmd_c_, _drv_info_, ("+_rtw_free_evt_priv\n"));
> -
>  	_cancel_workitem_sync(&pevtpriv->c2h_wk);
>  	while (pevtpriv->c2h_wk_alive)
>  		msleep(10);
> @@ -233,8 +231,6 @@ void _rtw_free_evt_priv(struct	evt_priv *pevtpriv)
>  			kfree(c2h);
>  	}
>  	kfree(pevtpriv->c2h_queue);
> -
> -	RT_TRACE(_module_rtl871x_cmd_c_, _drv_info_, ("-_rtw_free_evt_priv\n"));
>  }
>  
>  void _rtw_free_cmd_priv(struct	cmd_priv *pcmdpriv)
> @@ -299,13 +295,11 @@ struct	cmd_obj	*_rtw_dequeue_cmd(struct __queue *queue)
>  
>  void rtw_free_evt_priv(struct	evt_priv *pevtpriv)
>  {
> -	RT_TRACE(_module_rtl871x_cmd_c_, _drv_info_, ("rtw_free_evt_priv\n"));
>  	_rtw_free_evt_priv(pevtpriv);
>  }
>  
>  void rtw_free_cmd_priv(struct	cmd_priv *pcmdpriv)
>  {
> -	RT_TRACE(_module_rtl871x_cmd_c_, _drv_info_, ("rtw_free_cmd_priv\n"));
>  	_rtw_free_cmd_priv(pcmdpriv);
>  }
>  
> @@ -415,8 +409,6 @@ int rtw_cmd_thread(void *context)
>  	atomic_set(&(pcmdpriv->cmdthd_running), true);
>  	complete(&pcmdpriv->terminate_cmdthread_comp);
>  
> -	RT_TRACE(_module_rtl871x_cmd_c_, _drv_info_, ("start r871x rtw_cmd_thread !!!!\n"));
> -
>  	while (1) {
>  		if (wait_for_completion_interruptible(&pcmdpriv->cmd_queue_comp)) {
>  			DBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT" wait_for_completion_interruptible(&pcmdpriv->cmd_queue_comp) return != 0, break\n", FUNC_ADPT_ARG(padapter));
> @@ -440,8 +432,6 @@ int rtw_cmd_thread(void *context)
>  		}
>  
>  		if (rtw_register_cmd_alive(padapter) != _SUCCESS) {
> -			RT_TRACE(_module_hal_xmit_c_, _drv_notice_,
> -					 ("%s: wait to leave LPS_LCLK\n", __func__));
>  			continue;
>  		}
>  
> @@ -512,14 +502,12 @@ int rtw_cmd_thread(void *context)
>  		if (pcmd->cmdcode < ARRAY_SIZE(rtw_cmd_callback)) {
>  			pcmd_callback = rtw_cmd_callback[pcmd->cmdcode].callback;
>  			if (pcmd_callback == NULL) {
> -				RT_TRACE(_module_rtl871x_cmd_c_, _drv_info_, ("mlme_cmd_hdl(): pcmd_callback = 0x%p, cmdcode = 0x%x\n", pcmd_callback, pcmd->cmdcode));
>  				rtw_free_cmd_obj(pcmd);
>  			} else {
>  				/* todo: !!! fill rsp_buf to pcmd->rsp if (pcmd->rsp!= NULL) */
>  				pcmd_callback(pcmd->padapter, pcmd);/* need consider that free cmd_obj in rtw_cmd_callback */
>  			}
>  		} else {
> -			RT_TRACE(_module_rtl871x_cmd_c_, _drv_err_, ("%s: cmdcode = 0x%x callback not defined!\n", __func__, pcmd->cmdcode));
>  			rtw_free_cmd_obj(pcmd);
>  		}
>  
> @@ -583,8 +571,6 @@ u8 rtw_sitesurvey_cmd(struct adapter  *padapter, struct ndis_802_11_ssid *ssid,
>  
>  	rtw_free_network_queue(padapter, false);
>  
> -	RT_TRACE(_module_rtl871x_cmd_c_, _drv_info_, ("%s: flush network queue\n", __func__));
> -
>  	init_h2fwcmd_w_parm_no_rsp(ph2c, psurveyPara, GEN_CMD_CODE(_SiteSurvey));
>  
>  	/* psurveyPara->bsslimit = 48; */
> @@ -677,9 +663,9 @@ u8 rtw_createbss_cmd(struct adapter  *padapter)
>  	u8 res = _SUCCESS;
>  
>  	if (pmlmepriv->assoc_ssid.SsidLength == 0)
> -		RT_TRACE(_module_rtl871x_cmd_c_, _drv_info_, (" 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));
> +		{}
>  
>  	pcmd = rtw_zmalloc(sizeof(struct cmd_obj));
>  	if (pcmd == NULL) {
> @@ -767,14 +753,13 @@ u8 rtw_joinbss_cmd(struct adapter  *padapter, struct wlan_network *pnetwork)
>  	u8 *ptmp = NULL;
>  
>  	if (pmlmepriv->assoc_ssid.SsidLength == 0)
> -		RT_TRACE(_module_rtl871x_cmd_c_, _drv_info_, ("+Join cmd: Any SSid\n"));
> +		{}
>  	else
> -		RT_TRACE(_module_rtl871x_cmd_c_, _drv_notice_, ("+Join cmd: SSid =[%s]\n", pmlmepriv->assoc_ssid.Ssid));
> +		{}
>  
>  	pcmd = rtw_zmalloc(sizeof(struct cmd_obj));
>  	if (pcmd == NULL) {
>  		res = _FAIL;
> -		RT_TRACE(_module_rtl871x_cmd_c_, _drv_err_, ("rtw_joinbss_cmd: memory allocate for cmd_obj fail!!!\n"));
>  		goto exit;
>  	}
>  	/* for IEs is fix buf size */
> @@ -883,8 +868,6 @@ u8 rtw_disassoc_cmd(struct adapter *padapter, u32 deauth_timeout_ms, bool enqueu
>  	struct cmd_priv *cmdpriv = &padapter->cmdpriv;
>  	u8 res = _SUCCESS;
>  
> -	RT_TRACE(_module_rtl871x_cmd_c_, _drv_notice_, ("+rtw_disassoc_cmd\n"));
> -
>  	/* prepare cmd parameter */
>  	param = rtw_zmalloc(sizeof(*param));
>  	if (param == NULL) {
> @@ -1209,8 +1192,6 @@ u8 rtw_set_chplan_cmd(struct adapter *padapter, u8 chplan, u8 enqueue, u8 swconf
>  
>  	u8 res = _SUCCESS;
>  
> -	RT_TRACE(_module_rtl871x_cmd_c_, _drv_notice_, ("+rtw_set_chplan_cmd\n"));
> -
>  	/*  check if allow software config */
>  	if (swconfig && rtw_hal_is_disable_sw_channel_plan(padapter) == true) {
>  		res = _FAIL;
> @@ -1981,7 +1962,6 @@ void rtw_survey_cmd_callback(struct adapter *padapter,  struct cmd_obj *pcmd)
>  		_set_timer(&pmlmepriv->scan_to_timer, 1);
>  	} else if (pcmd->res != H2C_SUCCESS) {
>  		_set_timer(&pmlmepriv->scan_to_timer, 1);
> -		RT_TRACE(_module_rtl871x_cmd_c_, _drv_err_, ("\n ********Error: MgntActrtw_set_802_11_bssid_LIST_SCAN Fail ************\n\n."));
>  	}
>  
>  	/*  free cmd */
> @@ -1997,7 +1977,6 @@ void rtw_disassoc_cmd_callback(struct adapter *padapter,  struct cmd_obj *pcmd)
>  		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."));
>  		return;
>  	}
>  	/*  free cmd */
> @@ -2031,7 +2010,6 @@ void rtw_createbss_cmd_callback(struct adapter *padapter, struct cmd_obj *pcmd)
>  		goto exit;
>  
>  	if ((pcmd->res != H2C_SUCCESS)) {
> -		RT_TRACE(_module_rtl871x_cmd_c_, _drv_err_, ("\n ********Error: rtw_createbss_cmd_callback  Fail ************\n\n."));
>  		_set_timer(&pmlmepriv->assoc_timer, 1);
>  	}
>  
> @@ -2045,7 +2023,6 @@ void rtw_createbss_cmd_callback(struct adapter *padapter, struct cmd_obj *pcmd)
>  		if (!psta) {
>  			psta = rtw_alloc_stainfo(&padapter->stapriv, pnetwork->MacAddress);
>  			if (psta == NULL) {
> -				RT_TRACE(_module_rtl871x_cmd_c_, _drv_err_, ("\nCan't alloc sta_info when createbss_cmd_callback\n"));
>  				goto createbss_cmd_fail;
>  			}
>  		}
> @@ -2057,7 +2034,6 @@ void rtw_createbss_cmd_callback(struct adapter *padapter, struct cmd_obj *pcmd)
>  		if (pwlan == NULL) {
>  			pwlan = rtw_get_oldest_wlan_network(&pmlmepriv->scanned_queue);
>  			if (pwlan == NULL) {
> -				RT_TRACE(_module_rtl871x_cmd_c_, _drv_err_, ("\n Error:  can't get pwlan in rtw_joinbss_event_callback\n"));
>  				spin_unlock_bh(&(pmlmepriv->scanned_queue.lock));
>  				goto createbss_cmd_fail;
>  			}
> @@ -2102,7 +2078,6 @@ void rtw_setstaKey_cmdrsp_callback(struct adapter *padapter,  struct cmd_obj *pc
>  	struct sta_info *psta = rtw_get_stainfo(pstapriv, psetstakey_rsp->addr);
>  
>  	if (psta == NULL) {
> -		RT_TRACE(_module_rtl871x_cmd_c_, _drv_err_, ("\nERROR: rtw_setstaKey_cmdrsp_callback => can't get sta_info\n\n"));
>  		goto exit;
>  	}
>  exit:
> @@ -2118,7 +2093,6 @@ void rtw_setassocsta_cmdrsp_callback(struct adapter *padapter,  struct cmd_obj *
>  	struct sta_info *psta = rtw_get_stainfo(pstapriv, passocsta_parm->addr);
>  
>  	if (psta == NULL) {
> -		RT_TRACE(_module_rtl871x_cmd_c_, _drv_err_, ("\nERROR: setassocsta_cmdrsp_callbac => can't get sta_info\n\n"));
>  		goto exit;
>  	}
>  
> diff --git a/drivers/staging/rtl8723bs/core/rtw_eeprom.c b/drivers/staging/rtl8723bs/core/rtw_eeprom.c
> index 3cbd65dee741..796868304359 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_eeprom.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_eeprom.c
> @@ -36,7 +36,6 @@ void shift_out_bits(_adapter *padapter, u16 data, u16 count)
>  _func_enter_;
>  
>  	if (padapter->bSurpriseRemoved == true) {
> -		RT_TRACE(_module_rtl871x_eeprom_c_, _drv_err_, ("padapter->bSurpriseRemoved==true"));
>  		goto out;
>  	}
>  	mask = 0x01 << (count - 1);
> @@ -49,7 +48,6 @@ _func_enter_;
>  		if (data & mask)
>  			x |= _EEDI;
>  		if (padapter->bSurpriseRemoved == true) {
> -		RT_TRACE(_module_rtl871x_eeprom_c_, _drv_err_, ("padapter->bSurpriseRemoved==true"));
>  		goto out;
>  		}
>  		rtw_write8(padapter, EE_9346CR, (u8)x);
> @@ -59,7 +57,6 @@ _func_enter_;
>  		mask = mask >> 1;
>  	} while (mask);
>  	if (padapter->bSurpriseRemoved == true) {
> -		RT_TRACE(_module_rtl871x_eeprom_c_, _drv_err_, ("padapter->bSurpriseRemoved==true"));
>  		goto out;
>  	}
>  	x &= ~_EEDI;
> @@ -73,7 +70,6 @@ u16 shift_in_bits(_adapter *padapter)
>  	u16 x, d = 0, i;
>  _func_enter_;
>  	if (padapter->bSurpriseRemoved == true) {
> -		RT_TRACE(_module_rtl871x_eeprom_c_, _drv_err_, ("padapter->bSurpriseRemoved==true"));
>  		goto out;
>  	}
>  	x = rtw_read8(padapter, EE_9346CR);
> @@ -85,7 +81,6 @@ _func_enter_;
>  		d = d << 1;
>  		up_clk(padapter, &x);
>  	if (padapter->bSurpriseRemoved == true) {
> -		RT_TRACE(_module_rtl871x_eeprom_c_, _drv_err_, ("padapter->bSurpriseRemoved==true"));
>  		goto out;
>  	}
>  		x = rtw_read8(padapter, EE_9346CR);
> @@ -123,23 +118,19 @@ void eeprom_clean(_adapter *padapter)
>  	u16 x;
>  _func_enter_;
>  	if (padapter->bSurpriseRemoved == true) {
> -		RT_TRACE(_module_rtl871x_eeprom_c_, _drv_err_, ("padapter->bSurpriseRemoved==true"));
>  		goto out;
>  	}
>  	x = rtw_read8(padapter, EE_9346CR);
>  	if (padapter->bSurpriseRemoved == true) {
> -		RT_TRACE(_module_rtl871x_eeprom_c_, _drv_err_, ("padapter->bSurpriseRemoved==true"));
>  		goto out;
>  	}
>  	x &= ~(_EECS | _EEDI);
>  	rtw_write8(padapter, EE_9346CR, (u8)x);
>  	if (padapter->bSurpriseRemoved == true) {
> -		RT_TRACE(_module_rtl871x_eeprom_c_, _drv_err_, ("padapter->bSurpriseRemoved==true"));
>  		goto out;
>  	}
>  	up_clk(padapter, &x);
>  		if (padapter->bSurpriseRemoved == true) {
> -		RT_TRACE(_module_rtl871x_eeprom_c_, _drv_err_, ("padapter->bSurpriseRemoved==true"));
>  		goto out;
>  	}
>  	down_clk(padapter, &x);
> @@ -156,14 +147,12 @@ u16 eeprom_read16(_adapter *padapter, u16 reg) /*ReadEEprom*/
>  _func_enter_;
>  
>  	if (padapter->bSurpriseRemoved == true) {
> -		RT_TRACE(_module_rtl871x_eeprom_c_, _drv_err_, ("padapter->bSurpriseRemoved==true"));
>  		goto out;
>  	}
>  	/* select EEPROM, reset bits, set _EECS*/
>  	x = rtw_read8(padapter, EE_9346CR);
>  
>  	if (padapter->bSurpriseRemoved == true) {
> -		RT_TRACE(_module_rtl871x_eeprom_c_, _drv_err_, ("padapter->bSurpriseRemoved==true"));
>  		goto out;
>  	}
>  
> diff --git a/drivers/staging/rtl8723bs/core/rtw_ieee80211.c b/drivers/staging/rtl8723bs/core/rtw_ieee80211.c
> index cccbea555a32..4e447383f8a3 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_ieee80211.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_ieee80211.c
> @@ -489,8 +489,6 @@ int rtw_parse_wpa_ie(u8 *wpa_ie, int wpa_ie_len, int *group_cipher, int *pairwis
>  		left -= WPA_SELECTOR_LEN;
>  
>  	} else if (left > 0) {
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("%s: ie length mismatch, %u too much", __func__, left));
> -
>  		return _FAIL;
>  	}
>  
> @@ -502,8 +500,6 @@ int rtw_parse_wpa_ie(u8 *wpa_ie, int wpa_ie_len, int *group_cipher, int *pairwis
>  		left -= 2;
>  
>  		if (count == 0 || left < count * WPA_SELECTOR_LEN) {
> -			RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("%s: ie count botch (pairwise), "
> -						"count %u left %u", __func__, count, left));
>  			return _FAIL;
>  		}
>  
> @@ -515,7 +511,6 @@ int rtw_parse_wpa_ie(u8 *wpa_ie, int wpa_ie_len, int *group_cipher, int *pairwis
>  		}
>  
>  	} else if (left == 1) {
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("%s: ie too short (for key mgmt)",   __func__));
>  		return _FAIL;
>  	}
>  
> @@ -523,7 +518,6 @@ int rtw_parse_wpa_ie(u8 *wpa_ie, int wpa_ie_len, int *group_cipher, int *pairwis
>  		if (left >= 6) {
>  			pos += 2;
>  			if (!memcmp(pos, SUITE_1X, 4)) {
> -				RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("%s : there has 802.1x auth\n", __func__));
>  				*is_8021x = 1;
>  			}
>  		}
> @@ -560,7 +554,6 @@ int rtw_parse_wpa2_ie(u8 *rsn_ie, int rsn_ie_len, int *group_cipher, int *pairwi
>  		left -= RSN_SELECTOR_LEN;
>  
>  	} else if (left > 0) {
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("%s: ie length mismatch, %u too much", __func__, left));
>  		return _FAIL;
>  	}
>  
> @@ -572,8 +565,6 @@ int rtw_parse_wpa2_ie(u8 *rsn_ie, int rsn_ie_len, int *group_cipher, int *pairwi
>  		left -= 2;
>  
>  		if (count == 0 || left < count * RSN_SELECTOR_LEN) {
> -			RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("%s: ie count botch (pairwise), "
> -						 "count %u left %u", __func__, count, left));
>  			return _FAIL;
>  		}
>  
> @@ -585,8 +576,6 @@ int rtw_parse_wpa2_ie(u8 *rsn_ie, int rsn_ie_len, int *group_cipher, int *pairwi
>  		}
>  
>  	} else if (left == 1) {
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("%s: ie too short (for key mgmt)",  __func__));
> -
>  		return _FAIL;
>  	}
>  
> @@ -594,7 +583,6 @@ int rtw_parse_wpa2_ie(u8 *rsn_ie, int rsn_ie_len, int *group_cipher, int *pairwi
>  		if (left >= 6) {
>  			pos += 2;
>  			if (!memcmp(pos, SUITE_1X, 4)) {
> -				RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("%s (): there has 802.1x auth\n", __func__));
>  				*is_8021x = 1;
>  			}
>  		}
> @@ -630,9 +618,6 @@ int rtw_get_wapi_ie(u8 *in_ie, uint in_len, u8 *wapi_ie, u16 *wapi_len)
>  				memcpy(wapi_ie, &in_ie[cnt], in_ie[cnt+1]+2);
>  
>  				for (i = 0; i < (in_ie[cnt+1]+2); i = i+8) {
> -					RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("\n %2x,%2x,%2x,%2x,%2x,%2x,%2x,%2x\n",
> -								wapi_ie[i], wapi_ie[i+1], wapi_ie[i+2], wapi_ie[i+3], wapi_ie[i+4],
> -								wapi_ie[i+5], wapi_ie[i+6], wapi_ie[i+7]));
>  				}
>  			}
>  
> @@ -668,15 +653,10 @@ void rtw_get_sec_ie(u8 *in_ie, uint in_len, u8 *rsn_ie, u16 *rsn_len, u8 *wpa_ie
>  		authmode = in_ie[cnt];
>  
>  		if ((authmode == WLAN_EID_VENDOR_SPECIFIC) && (!memcmp(&in_ie[cnt+2], &wpa_oui[0], 4))) {
> -				RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("\n rtw_get_wpa_ie: sec_idx =%d in_ie[cnt+1]+2 =%d\n", sec_idx, in_ie[cnt+1]+2));
> -
>  				if (wpa_ie) {
>  				memcpy(wpa_ie, &in_ie[cnt], in_ie[cnt+1]+2);
>  
>  				for (i = 0; i < (in_ie[cnt+1]+2); i = i+8) {
> -						RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("\n %2x,%2x,%2x,%2x,%2x,%2x,%2x,%2x\n",
> -									wpa_ie[i], wpa_ie[i+1], wpa_ie[i+2], wpa_ie[i+3], wpa_ie[i+4],
> -									wpa_ie[i+5], wpa_ie[i+6], wpa_ie[i+7]));
>  					}
>  				}
>  
> @@ -684,15 +664,10 @@ void rtw_get_sec_ie(u8 *in_ie, uint in_len, u8 *rsn_ie, u16 *rsn_len, u8 *wpa_ie
>  				cnt += in_ie[cnt+1]+2;  /* get next */
>  		} else {
>  			if (authmode == WLAN_EID_RSN) {
> -				RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("\n get_rsn_ie: sec_idx =%d in_ie[cnt+1]+2 =%d\n", sec_idx, in_ie[cnt+1]+2));
> -
>  				if (rsn_ie) {
>  				memcpy(rsn_ie, &in_ie[cnt], in_ie[cnt+1]+2);
>  
>  				for (i = 0; i < (in_ie[cnt+1]+2); i = i+8) {
> -						RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("\n %2x,%2x,%2x,%2x,%2x,%2x,%2x,%2x\n",
> -									rsn_ie[i], rsn_ie[i+1], rsn_ie[i+2], rsn_ie[i+3], rsn_ie[i+4],
> -									rsn_ie[i+5], rsn_ie[i+6], rsn_ie[i+7]));
>  					}
>  				}
>  
> @@ -1137,28 +1112,20 @@ static int rtw_get_cipher_info(struct wlan_network *pnetwork)
>  	pbuf = rtw_get_wpa_ie(&pnetwork->network.IEs[12], &wpa_ielen, pnetwork->network.IELength-12);
>  
>  	if (pbuf && (wpa_ielen > 0)) {
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_get_cipher_info: wpa_ielen: %d", wpa_ielen));
>  		if (_SUCCESS == rtw_parse_wpa_ie(pbuf, wpa_ielen+2, &group_cipher, &pairwise_cipher, &is8021x)) {
>  			pnetwork->BcnInfo.pairwise_cipher = pairwise_cipher;
>  			pnetwork->BcnInfo.group_cipher = group_cipher;
>  			pnetwork->BcnInfo.is_8021x = is8021x;
> -			RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("%s: pnetwork->pairwise_cipher: %d, is_8021x is %d",
> -						__func__, pnetwork->BcnInfo.pairwise_cipher, pnetwork->BcnInfo.is_8021x));
>  			ret = _SUCCESS;
>  		}
>  	} else {
>  		pbuf = rtw_get_wpa2_ie(&pnetwork->network.IEs[12], &wpa_ielen, pnetwork->network.IELength-12);
>  
>  		if (pbuf && (wpa_ielen > 0)) {
> -			RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("get RSN IE\n"));
>  			if (_SUCCESS == rtw_parse_wpa2_ie(pbuf, wpa_ielen+2, &group_cipher, &pairwise_cipher, &is8021x)) {
> -				RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("get RSN IE  OK!!!\n"));
>  				pnetwork->BcnInfo.pairwise_cipher = pairwise_cipher;
>  				pnetwork->BcnInfo.group_cipher = group_cipher;
>  				pnetwork->BcnInfo.is_8021x = is8021x;
> -				RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("%s: pnetwork->pairwise_cipher: %d,"
> -							"pnetwork->group_cipher is %d, is_8021x is %d",	__func__, pnetwork->BcnInfo.pairwise_cipher,
> -							pnetwork->BcnInfo.group_cipher, pnetwork->BcnInfo.is_8021x));
>  				ret = _SUCCESS;
>  			}
>  		}
> @@ -1188,10 +1155,6 @@ void rtw_get_bcn_info(struct wlan_network *pnetwork)
>  		pnetwork->BcnInfo.encryp_protocol = ENCRYP_PROTOCOL_OPENSYS;
>  	}
>  	rtw_get_sec_ie(pnetwork->network.IEs, pnetwork->network.IELength, NULL, &rsn_len, NULL, &wpa_len);
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_get_bcn_info: ssid =%s\n", pnetwork->network.Ssid.Ssid));
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_get_bcn_info: wpa_len =%d rsn_len =%d\n", wpa_len, rsn_len));
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_get_bcn_info: ssid =%s\n", pnetwork->network.Ssid.Ssid));
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_get_bcn_info: wpa_len =%d rsn_len =%d\n", wpa_len, rsn_len));
>  
>  	if (rsn_len > 0) {
>  		pnetwork->BcnInfo.encryp_protocol = ENCRYP_PROTOCOL_WPA2;
> @@ -1201,10 +1164,6 @@ void rtw_get_bcn_info(struct wlan_network *pnetwork)
>  		if (bencrypt)
>  			pnetwork->BcnInfo.encryp_protocol = ENCRYP_PROTOCOL_WEP;
>  	}
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_get_bcn_info: pnetwork->encryp_protocol is %x\n",
> -				pnetwork->BcnInfo.encryp_protocol));
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_get_bcn_info: pnetwork->encryp_protocol is %x\n",
> -				pnetwork->BcnInfo.encryp_protocol));
>  	rtw_get_cipher_info(pnetwork);
>  
>  	/* get bwmode and ch_offset */
> diff --git a/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c b/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c
> index cb14855742f7..0eb2e7114f6f 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c
> @@ -28,7 +28,6 @@ u8 rtw_validate_ssid(struct ndis_802_11_ssid *ssid)
>  	u8 ret = true;
>  
>  	if (ssid->SsidLength > 32) {
> -		RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_, ("ssid length >32\n"));
>  		ret = false;
>  		goto exit;
>  	}
> @@ -50,8 +49,6 @@ u8 rtw_do_join(struct adapter *padapter)
>  	phead = get_list_head(queue);
>  	plist = get_next(phead);
>  
> -	RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("\n rtw_do_join: phead = %p; plist = %p\n\n\n", phead, plist));
> -
>  	pmlmepriv->cur_network.join_res = -2;
>  
>  	set_fwstate(pmlmepriv, _FW_UNDER_LINKING);
> @@ -70,12 +67,10 @@ u8 rtw_do_join(struct adapter *padapter)
>  		if (pmlmepriv->LinkDetectInfo.bBusyTraffic == false
>  			|| rtw_to_roam(padapter) > 0
>  		) {
> -			RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("rtw_do_join(): site survey if scanned_queue is empty\n."));
>  			/*  submit site_survey_cmd */
>  			ret = rtw_sitesurvey_cmd(padapter, &pmlmepriv->assoc_ssid, 1, NULL, 0);
>  			if (_SUCCESS != ret) {
>  				pmlmepriv->to_join = false;
> -				RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_, ("rtw_do_join(): site survey return error\n."));
>  			}
>  		} else {
>  			pmlmepriv->to_join = false;
> @@ -108,15 +103,12 @@ u8 rtw_do_join(struct adapter *padapter)
>  				rtw_generate_random_ibss(pibss);
>  
>  				if (rtw_createbss_cmd(padapter) != _SUCCESS) {
> -					RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_, ("***Error =>do_goin: rtw_createbss_cmd status FAIL***\n "));
>  					ret =  false;
>  					goto exit;
>  				}
>  
>  				pmlmepriv->to_join = false;
>  
> -				RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("***Error => rtw_select_and_join_from_scanned_queue FAIL under STA_Mode***\n "));
> -
>  			} else {
>  				/*  can't associate ; reset under-linking */
>  				_clr_fwstate_(pmlmepriv, _FW_UNDER_LINKING);
> @@ -130,7 +122,6 @@ u8 rtw_do_join(struct adapter *padapter)
>  					ret = rtw_sitesurvey_cmd(padapter, &pmlmepriv->assoc_ssid, 1, NULL, 0);
>  					if (_SUCCESS != ret) {
>  						pmlmepriv->to_join = false;
> -						RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_, ("do_join(): site survey return error\n."));
>  					}
>  				} else {
>  					ret = _FAIL;
> @@ -170,16 +161,10 @@ u8 rtw_set_802_11_bssid(struct adapter *padapter, u8 *bssid)
>  		goto release_mlme_lock;
>  
>  	if (check_fwstate(pmlmepriv, _FW_LINKED|WIFI_ADHOC_MASTER_STATE) == true) {
> -		RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("set_bssid: _FW_LINKED||WIFI_ADHOC_MASTER_STATE\n"));
> -
>  		if (!memcmp(&pmlmepriv->cur_network.network.MacAddress, bssid, ETH_ALEN)) {
>  			if (check_fwstate(pmlmepriv, WIFI_STATION_STATE) == false)
>  				goto release_mlme_lock;/* it means driver is in WIFI_ADHOC_MASTER_STATE, we needn't create bss again. */
>  		} else {
> -			RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("Set BSSID not the same bssid\n"));
> -			RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("set_bssid =%pM\n", MAC_ARG(bssid)));
> -			RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("cur_bssid =%pM\n", MAC_ARG(pmlmepriv->cur_network.network.MacAddress)));
> -
>  			rtw_disassoc_cmd(padapter, 0, true);
>  
>  			if (check_fwstate(pmlmepriv, _FW_LINKED) == true)
> @@ -213,8 +198,6 @@ u8 rtw_set_802_11_bssid(struct adapter *padapter, u8 *bssid)
>  	spin_unlock_bh(&pmlmepriv->lock);
>  
>  exit:
> -	RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_,
> -		("rtw_set_802_11_bssid: status =%d\n", status));
>  
>  	return status;
>  }
> @@ -230,8 +213,6 @@ u8 rtw_set_802_11_ssid(struct adapter *padapter, struct ndis_802_11_ssid *ssid)
>  			ssid->Ssid, get_fwstate(pmlmepriv));
>  
>  	if (padapter->hw_init_completed == false) {
> -		RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_,
> -			 ("set_ssid: hw_init_completed ==false =>exit!!!\n"));
>  		status = _FAIL;
>  		goto exit;
>  	}
> @@ -245,16 +226,9 @@ u8 rtw_set_802_11_ssid(struct adapter *padapter, struct ndis_802_11_ssid *ssid)
>  		goto release_mlme_lock;
>  
>  	if (check_fwstate(pmlmepriv, _FW_LINKED|WIFI_ADHOC_MASTER_STATE) == true) {
> -		RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_,
> -			 ("set_ssid: _FW_LINKED||WIFI_ADHOC_MASTER_STATE\n"));
> -
>  		if ((pmlmepriv->assoc_ssid.SsidLength == ssid->SsidLength) &&
>  		    (!memcmp(&pmlmepriv->assoc_ssid.Ssid, ssid->Ssid, ssid->SsidLength))) {
>  			if ((check_fwstate(pmlmepriv, WIFI_STATION_STATE) == false)) {
> -				RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_,
> -					 ("Set SSID is the same ssid, fw_state = 0x%08x\n",
> -					  get_fwstate(pmlmepriv)));
> -
>  				if (rtw_is_same_ibss(padapter, pnetwork) == false) {
>  					/* if in WIFI_ADHOC_MASTER_STATE | WIFI_ADHOC_STATE, create bss or rejoin again */
>  					rtw_disassoc_cmd(padapter, 0, true);
> @@ -275,10 +249,6 @@ u8 rtw_set_802_11_ssid(struct adapter *padapter, struct ndis_802_11_ssid *ssid)
>  				rtw_lps_ctrl_wk_cmd(padapter, LPS_CTRL_JOINBSS, 1);
>  			}
>  		} else {
> -			RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("Set SSID not the same ssid\n"));
> -			RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("set_ssid =[%s] len = 0x%x\n", ssid->Ssid, (unsigned int)ssid->SsidLength));
> -			RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("assoc_ssid =[%s] len = 0x%x\n", pmlmepriv->assoc_ssid.Ssid, (unsigned int)pmlmepriv->assoc_ssid.SsidLength));
> -
>  			rtw_disassoc_cmd(padapter, 0, true);
>  
>  			if (check_fwstate(pmlmepriv, _FW_LINKED) == true)
> @@ -316,8 +286,6 @@ u8 rtw_set_802_11_ssid(struct adapter *padapter, struct ndis_802_11_ssid *ssid)
>  	spin_unlock_bh(&pmlmepriv->lock);
>  
>  exit:
> -	RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_,
> -		("-rtw_set_802_11_ssid: status =%d\n", status));
>  
>  	return status;
>  }
> @@ -343,8 +311,6 @@ u8 rtw_set_802_11_connect(struct adapter *padapter, u8 *bssid, struct ndis_802_1
>  	}
>  
>  	if (padapter->hw_init_completed == false) {
> -		RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_,
> -			 ("set_ssid: hw_init_completed ==false =>exit!!!\n"));
>  		status = _FAIL;
>  		goto exit;
>  	}
> @@ -396,12 +362,7 @@ u8 rtw_set_802_11_infrastructure_mode(struct adapter *padapter,
>  	struct	wlan_network	*cur_network = &pmlmepriv->cur_network;
>  	enum ndis_802_11_network_infrastructure *pold_state = &(cur_network->network.InfrastructureMode);
>  
> -	RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_notice_,
> -		 ("+rtw_set_802_11_infrastructure_mode: old =%d new =%d fw_state = 0x%08x\n",
> -		  *pold_state, networktype, get_fwstate(pmlmepriv)));
> -
>  	if (*pold_state != networktype) {
> -		RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, (" change mode!"));
>  		/* DBG_871X("change mode, old_mode =%d, new_mode =%d, fw_state = 0x%x\n", *pold_state, networktype, get_fwstate(pmlmepriv)); */
>  
>  		if (*pold_state == Ndis802_11APMode) {
> @@ -468,8 +429,6 @@ u8 rtw_set_802_11_disassociate(struct adapter *padapter)
>  	spin_lock_bh(&pmlmepriv->lock);
>  
>  	if (check_fwstate(pmlmepriv, _FW_LINKED) == true) {
> -		RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("MgntActrtw_set_802_11_disassociate: rtw_indicate_disconnect\n"));
> -
>  		rtw_disassoc_cmd(padapter, 0, true);
>  		rtw_indicate_disconnect(padapter);
>  		/* modify for CONFIG_IEEE80211W, none 11w can use it */
> @@ -488,28 +447,24 @@ u8 rtw_set_802_11_bssid_list_scan(struct adapter *padapter, struct ndis_802_11_s
>  	struct	mlme_priv 	*pmlmepriv = &padapter->mlmepriv;
>  	u8 res = true;
>  
> -	RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_, ("+rtw_set_802_11_bssid_list_scan(), fw_state =%x\n", get_fwstate(pmlmepriv)));
> -
>  	if (padapter == NULL) {
>  		res = false;
>  		goto exit;
>  	}
>  	if (padapter->hw_init_completed == false) {
>  		res = false;
> -		RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_, ("\n ===rtw_set_802_11_bssid_list_scan:hw_init_completed ==false ===\n"));
>  		goto exit;
>  	}
>  
>  	if ((check_fwstate(pmlmepriv, _FW_UNDER_SURVEY|_FW_UNDER_LINKING) == true) ||
>  		(pmlmepriv->LinkDetectInfo.bBusyTraffic == true)) {
>  		/*  Scan or linking is in progress, do nothing. */
> -		RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_, ("rtw_set_802_11_bssid_list_scan fail since fw_state = %x\n", get_fwstate(pmlmepriv)));
>  		res = true;
>  
>  		if (check_fwstate(pmlmepriv, (_FW_UNDER_SURVEY|_FW_UNDER_LINKING)) == true)
> -			RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_, ("\n###_FW_UNDER_SURVEY|_FW_UNDER_LINKING\n\n"));
> +			{}
>  		else
> -			RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_, ("\n###pmlmepriv->sitesurveyctrl.traffic_busy ==true\n\n"));
> +			{}
>  	} else {
>  		if (rtw_is_scan_deny(padapter)) {
>  			DBG_871X(FUNC_ADPT_FMT": scan deny\n", FUNC_ADPT_ARG(padapter));
> @@ -534,12 +489,8 @@ u8 rtw_set_802_11_authentication_mode(struct adapter *padapter, enum ndis_802_11
>  	int res;
>  	u8 ret;
>  
> -	RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("set_802_11_auth.mode(): mode =%x\n", authmode));
> -
>  	psecuritypriv->ndisauthtype = authmode;
>  
> -	RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("rtw_set_802_11_authentication_mode:psecuritypriv->ndisauthtype =%d", psecuritypriv->ndisauthtype));
> -
>  	if (psecuritypriv->ndisauthtype > 3)
>  		psecuritypriv->dot11AuthAlgrthm = dot11AuthAlgrthm_8021X;
>  
> @@ -563,7 +514,6 @@ u8 rtw_set_802_11_add_wep(struct adapter *padapter, struct ndis_802_11_wep *wep)
>  	keyid = wep->KeyIndex & 0x3fffffff;
>  
>  	if (keyid >= 4) {
> -		RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_, ("MgntActrtw_set_802_11_add_wep:keyid>4 =>fail\n"));
>  		ret = false;
>  		goto exit;
>  	}
> @@ -571,35 +521,21 @@ u8 rtw_set_802_11_add_wep(struct adapter *padapter, struct ndis_802_11_wep *wep)
>  	switch (wep->KeyLength) {
>  	case 5:
>  		psecuritypriv->dot11PrivacyAlgrthm = _WEP40_;
> -		RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("MgntActrtw_set_802_11_add_wep:wep->KeyLength =5\n"));
>  		break;
>  	case 13:
>  		psecuritypriv->dot11PrivacyAlgrthm = _WEP104_;
> -		RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("MgntActrtw_set_802_11_add_wep:wep->KeyLength = 13\n"));
>  		break;
>  	default:
>  		psecuritypriv->dot11PrivacyAlgrthm = _NO_PRIVACY_;
> -		RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("MgntActrtw_set_802_11_add_wep:wep->KeyLength!=5 or 13\n"));
>  		break;
>  	}
>  
> -	RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_,
> -		 ("rtw_set_802_11_add_wep:before memcpy, wep->KeyLength = 0x%x wep->KeyIndex = 0x%x  keyid =%x\n",
> -		  wep->KeyLength, wep->KeyIndex, keyid));
> -
>  	memcpy(&(psecuritypriv->dot11DefKey[keyid].skey[0]), &(wep->KeyMaterial), wep->KeyLength);
>  
>  	psecuritypriv->dot11DefKeylen[keyid] = wep->KeyLength;
>  
>  	psecuritypriv->dot11PrivacyKeyIndex = keyid;
>  
> -	RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_info_, ("rtw_set_802_11_add_wep:security key material : %x %x %x %x %x %x %x %x %x %x %x %x %x\n",
> -		psecuritypriv->dot11DefKey[keyid].skey[0], psecuritypriv->dot11DefKey[keyid].skey[1], psecuritypriv->dot11DefKey[keyid].skey[2],
> -		psecuritypriv->dot11DefKey[keyid].skey[3], psecuritypriv->dot11DefKey[keyid].skey[4], psecuritypriv->dot11DefKey[keyid].skey[5],
> -		psecuritypriv->dot11DefKey[keyid].skey[6], psecuritypriv->dot11DefKey[keyid].skey[7], psecuritypriv->dot11DefKey[keyid].skey[8],
> -		psecuritypriv->dot11DefKey[keyid].skey[9], psecuritypriv->dot11DefKey[keyid].skey[10], psecuritypriv->dot11DefKey[keyid].skey[11],
> -		psecuritypriv->dot11DefKey[keyid].skey[12]));
> -
>  	res = rtw_set_key(padapter, psecuritypriv, keyid, 1, true);
>  
>  	if (res == _FAIL)
> diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c b/drivers/staging/rtl8723bs/core/rtw_mlme.c
> index 95cfef118a94..95aa7552ff42 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_mlme.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_mlme.c
> @@ -157,8 +157,6 @@ struct	wlan_network *rtw_alloc_network(struct	mlme_priv *pmlmepriv)
>  
>  	list_del_init(&pnetwork->list);
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_,
> -		 ("rtw_alloc_network: ptr =%p\n", plist));
>  	pnetwork->network_type = 0;
>  	pnetwork->fixed = false;
>  	pnetwork->last_scanned = jiffies;
> @@ -301,7 +299,6 @@ signed int rtw_if_up(struct adapter *padapter)
>  
>  	if (padapter->bDriverStopped || padapter->bSurpriseRemoved ||
>  		(check_fwstate(&padapter->mlmepriv, _FW_LINKED) == false)) {
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_if_up:bDriverStopped(%d) OR bSurpriseRemoved(%d)", padapter->bDriverStopped, padapter->bSurpriseRemoved));
>  		res = false;
>  	} else
>  		res =  true;
> @@ -341,7 +338,6 @@ u8 *rtw_get_beacon_interval_from_ie(u8 *ie)
>  
>  void rtw_free_mlme_priv(struct mlme_priv *pmlmepriv)
>  {
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("rtw_free_mlme_priv\n"));
>  	_rtw_free_mlme_priv(pmlmepriv);
>  }
>  
> @@ -615,7 +611,6 @@ void rtw_update_scanned_network(struct adapter *adapter, struct wlan_bssid_ex *t
>  			/* list_del_init(&oldest->list); */
>  			pnetwork = oldest;
>  			if (!pnetwork) {
> -				RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("\n\n\nsomething wrong here\n\n\n"));
>  				goto exit;
>  			}
>  			memcpy(&(pnetwork->network), target,  get_wlan_bssid_ex_sz(target));
> @@ -636,7 +631,6 @@ void rtw_update_scanned_network(struct adapter *adapter, struct wlan_bssid_ex *t
>  			pnetwork = rtw_alloc_network(pmlmepriv); /*  will update scan_time */
>  
>  			if (!pnetwork) {
> -				RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("\n\n\nsomething wrong here\n\n\n"));
>  				goto exit;
>  			}
>  
> @@ -757,7 +751,6 @@ int rtw_is_desired_network(struct adapter *adapter, struct wlan_network *pnetwor
>  /* TODO: Perry : For Power Management */
>  void rtw_atimdone_event_callback(struct adapter	*adapter, u8 *pbuf)
>  {
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("receive atimdone_event\n"));
>  }
>  
>  void rtw_survey_event_callback(struct adapter	*adapter, u8 *pbuf)
> @@ -768,11 +761,8 @@ void rtw_survey_event_callback(struct adapter	*adapter, u8 *pbuf)
>  
>  	pnetwork = (struct wlan_bssid_ex *)pbuf;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_survey_event_callback, ssid =%s\n",  pnetwork->Ssid.Ssid));
> -
>  	len = get_wlan_bssid_ex_sz(pnetwork);
>  	if (len > (sizeof(struct wlan_bssid_ex))) {
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("\n ****rtw_survey_event_callback: return a wrong bss ***\n"));
>  		return;
>  	}
>  
> @@ -819,14 +809,10 @@ void rtw_surveydone_event_callback(struct adapter	*adapter, u8 *pbuf)
>  		pmlmepriv->wps_probe_req_ie = NULL;
>  	}
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_surveydone_event_callback: fw_state:%x\n\n", get_fwstate(pmlmepriv)));
> -
>  	if (check_fwstate(pmlmepriv, _FW_UNDER_SURVEY)) {
>  		del_timer_sync(&pmlmepriv->scan_to_timer);
>  		_clr_fwstate_(pmlmepriv, _FW_UNDER_SURVEY);
>  	} else {
> -
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("nic status =%x, survey done event comes too late!\n", get_fwstate(pmlmepriv)));
>  	}
>  
>  	rtw_set_signal_stat_timer(&adapter->recvpriv);
> @@ -845,8 +831,6 @@ void rtw_surveydone_event_callback(struct adapter	*adapter, u8 *pbuf)
>  					/* pmlmepriv->fw_state ^= _FW_UNDER_SURVEY;because don't set assoc_timer */
>  					_clr_fwstate_(pmlmepriv, _FW_UNDER_SURVEY);
>  
> -					RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("switching to adhoc master\n"));
> -
>  					memcpy(&pdev_network->Ssid, &pmlmepriv->assoc_ssid, sizeof(struct ndis_802_11_ssid));
>  
>  					rtw_update_registrypriv_dev_network(adapter);
> @@ -855,7 +839,7 @@ void rtw_surveydone_event_callback(struct adapter	*adapter, u8 *pbuf)
>  					pmlmepriv->fw_state = WIFI_ADHOC_MASTER_STATE;
>  
>  					if (rtw_createbss_cmd(adapter) != _SUCCESS)
> -						RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("Error =>rtw_createbss_cmd status FAIL\n"));
> +						{}
>  
>  					pmlmepriv->to_join = false;
>  				}
> @@ -927,7 +911,6 @@ static void free_scanqueue(struct	mlme_priv *pmlmepriv)
>  	struct __queue *scan_queue = &pmlmepriv->scanned_queue;
>  	struct list_head	*plist, *phead, *ptemp;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_notice_, ("+free_scanqueue\n"));
>  	spin_lock_bh(&scan_queue->lock);
>  	spin_lock_bh(&free_queue->lock);
>  
> @@ -965,7 +948,7 @@ static void find_network(struct adapter *adapter)
>  	if (pwlan)
>  		pwlan->fixed = false;
>  	else
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("rtw_free_assoc_resources : pwlan == NULL\n\n"));
> +		{}
>  
>  	if (check_fwstate(pmlmepriv, WIFI_ADHOC_MASTER_STATE) &&
>  	    (adapter->stapriv.asoc_sta_count == 1))
> @@ -983,10 +966,6 @@ void rtw_free_assoc_resources(struct adapter *adapter, int lock_scanned_queue)
>  	struct dvobj_priv *psdpriv = adapter->dvobj;
>  	struct debug_priv *pdbgpriv = &psdpriv->drv_dbg;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_notice_, ("+rtw_free_assoc_resources\n"));
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("tgt_network->network.MacAddress =%pM ssid =%s\n",
> -		MAC_ARG(tgt_network->network.MacAddress), tgt_network->network.Ssid.Ssid));
> -
>  	if (check_fwstate(pmlmepriv, WIFI_STATION_STATE|WIFI_AP_STATE)) {
>  		struct sta_info *psta;
>  
> @@ -1024,8 +1003,6 @@ void rtw_indicate_connect(struct adapter *padapter)
>  {
>  	struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("+rtw_indicate_connect\n"));
> -
>  	pmlmepriv->to_join = false;
>  
>  	if (!check_fwstate(&padapter->mlmepriv, _FW_LINKED)) {
> @@ -1037,8 +1014,6 @@ void rtw_indicate_connect(struct adapter *padapter)
>  
>  	rtw_set_to_roam(padapter, 0);
>  	rtw_set_scan_deny(padapter, 3000);
> -
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("-rtw_indicate_connect: fw_state = 0x%08x\n", get_fwstate(pmlmepriv)));
>  }
>  
>  /*
> @@ -1048,8 +1023,6 @@ void rtw_indicate_disconnect(struct adapter *padapter)
>  {
>  	struct	mlme_priv *pmlmepriv = &padapter->mlmepriv;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("+rtw_indicate_disconnect\n"));
> -
>  	_clr_fwstate_(pmlmepriv, _FW_UNDER_LINKING|WIFI_UNDER_WPS);
>  
>  	/* DBG_871X("clear wps when %s\n", __func__); */
> @@ -1215,9 +1188,6 @@ static void rtw_joinbss_update_network(struct adapter *padapter, struct wlan_net
>  	struct mlme_priv *pmlmepriv = &(padapter->mlmepriv);
>  	struct wlan_network  *cur_network = &(pmlmepriv->cur_network);
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("\nfw_state:%x, BSSID:%pM\n"
> -		, get_fwstate(pmlmepriv), MAC_ARG(pnetwork->network.MacAddress)));
> -
>  	/*  why not use ptarget_wlan?? */
>  	memcpy(&cur_network->network, &pnetwork->network, pnetwork->network.Length);
>  	/*  some IEs in pnetwork is wrong, so we should use ptarget_wlan IEs */
> @@ -1259,7 +1229,6 @@ static void rtw_joinbss_update_network(struct adapter *padapter, struct wlan_net
>  			break;
>  	default:
>  			pmlmepriv->fw_state = WIFI_NULL_STATE;
> -			RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("Invalid network_mode\n"));
>  			break;
>  	}
>  
> @@ -1288,20 +1257,17 @@ void rtw_joinbss_event_prehandle(struct adapter *adapter, u8 *pbuf)
>  	struct wlan_network	*pcur_wlan = NULL, *ptarget_wlan = NULL;
>  	unsigned int		the_same_macaddr = false;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("joinbss event call back received with res =%d\n", pnetwork->join_res));
> -
>  	rtw_get_encrypt_decrypt_from_registrypriv(adapter);
>  
>  	if (pmlmepriv->assoc_ssid.SsidLength == 0)
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("@@@@@   joinbss event call back  for Any SSid\n"));
> +		{}
>  	else
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("@@@@@   rtw_joinbss_event_callback for SSid:%s\n", pmlmepriv->assoc_ssid.Ssid));
> +		{}
>  
>  	the_same_macaddr = !memcmp(pnetwork->network.MacAddress, cur_network->network.MacAddress, ETH_ALEN);
>  
>  	pnetwork->network.Length = get_wlan_bssid_ex_sz(&pnetwork->network);
>  	if (pnetwork->network.Length > sizeof(struct wlan_bssid_ex)) {
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("\n\n ***joinbss_evt_callback return a wrong bss ***\n\n"));
>  		return;
>  	}
>  
> @@ -1310,8 +1276,6 @@ void rtw_joinbss_event_prehandle(struct adapter *adapter, u8 *pbuf)
>  	pmlmepriv->LinkDetectInfo.TrafficTransitionCount = 0;
>  	pmlmepriv->LinkDetectInfo.LowPowerTransitionCount = 0;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("\n rtw_joinbss_event_callback !! spin_lock_irqsave\n"));
> -
>  	if (pnetwork->join_res > 0) {
>  		spin_lock_bh(&(pmlmepriv->scanned_queue.lock));
>  		retry = 0;
> @@ -1357,7 +1321,6 @@ void rtw_joinbss_event_prehandle(struct adapter *adapter, u8 *pbuf)
>  			if (check_fwstate(pmlmepriv, WIFI_STATION_STATE) == true) {
>  				ptarget_sta = rtw_joinbss_update_stainfo(adapter, pnetwork);
>  				if (!ptarget_sta) {
> -					RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("Can't update stainfo when joinbss_event callback\n"));
>  					spin_unlock_bh(&(pmlmepriv->scanned_queue.lock));
>  					goto ignore_joinbss_callback;
>  				}
> @@ -1368,17 +1331,12 @@ void rtw_joinbss_event_prehandle(struct adapter *adapter, u8 *pbuf)
>  				pmlmepriv->cur_network_scanned = ptarget_wlan;
>  				rtw_indicate_connect(adapter);
>  			} else {
> -				/* adhoc mode will rtw_indicate_connect when rtw_stassoc_event_callback */
> -				RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("adhoc mode, fw_state:%x", get_fwstate(pmlmepriv)));
>  			}
>  
>  			/* s5. Cancel assoc_timer */
>  			del_timer_sync(&pmlmepriv->assoc_timer);
>  
> -			RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("Cancel assoc_timer\n"));
> -
>  		} else {
> -			RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("rtw_joinbss_event_callback err: fw_state:%x", get_fwstate(pmlmepriv)));
>  			spin_unlock_bh(&(pmlmepriv->scanned_queue.lock));
>  			goto ignore_joinbss_callback;
>  		}
> @@ -1392,7 +1350,6 @@ void rtw_joinbss_event_prehandle(struct adapter *adapter, u8 *pbuf)
>  		/* rtw_free_assoc_resources(adapter, 1); */
>  
>  		if ((check_fwstate(pmlmepriv, _FW_UNDER_LINKING)) == true) {
> -			RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("fail! clear _FW_UNDER_LINKING ^^^fw_state =%x\n", get_fwstate(pmlmepriv)));
>  			_clr_fwstate_(pmlmepriv, _FW_UNDER_LINKING);
>  		}
>  
> @@ -1402,7 +1359,6 @@ void rtw_joinbss_event_prehandle(struct adapter *adapter, u8 *pbuf)
>  		res = _FAIL;
>  		if (retry < 2) {
>  			res = rtw_select_and_join_from_scanned_queue(pmlmepriv);
> -			RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("rtw_select_and_join_from_scanned_queue again! res:%d\n", res));
>  		}
>  
>  		if (res == _SUCCESS) {
> @@ -1413,7 +1369,6 @@ void rtw_joinbss_event_prehandle(struct adapter *adapter, u8 *pbuf)
>  			_clr_fwstate_(pmlmepriv, _FW_UNDER_LINKING);
>  			rtw_indicate_connect(adapter);
>  		} else {
> -			RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("Set Assoc_Timer = 1; can't find match ssid in scanned_q\n"));
>  		#endif
>  
>  			_set_timer(&pmlmepriv->assoc_timer, 1);
> @@ -1501,16 +1456,12 @@ void rtw_stassoc_event_callback(struct adapter *adapter, u8 *pbuf)
>  	/* for AD-HOC mode */
>  	psta = rtw_get_stainfo(&adapter->stapriv, pstassoc->macaddr);
>  	if (psta) {
> -		/* the sta have been in sta_info_queue => do nothing */
> -
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("Error: rtw_stassoc_event_callback: sta has been in sta_hash_queue\n"));
>  
>  		return; /* between drv has received this event before and  fw have not yet to set key to CAM_ENTRY) */
>  	}
>  
>  	psta = rtw_alloc_stainfo(&adapter->stapriv, pstassoc->macaddr);
>  	if (!psta) {
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("Can't alloc sta_info when rtw_stassoc_event_callback\n"));
>  		return;
>  	}
>  
> @@ -1661,7 +1612,7 @@ void rtw_stadel_event_callback(struct adapter *adapter, u8 *pbuf)
>  			}
>  
>  			if (rtw_createbss_cmd(adapter) != _SUCCESS)
> -				RT_TRACE(_module_rtl871x_ioctl_set_c_, _drv_err_, ("***Error =>stadel_event_callback: rtw_createbss_cmd status FAIL***\n "));
> +				{}
>  		}
>  
>  	}
> @@ -1673,7 +1624,6 @@ void rtw_cpwm_event_callback(struct adapter *padapter, u8 *pbuf)
>  {
>  	struct reportpwrstate_parm *preportpwrstate;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("+rtw_cpwm_event_callback !!!\n"));
>  	preportpwrstate = (struct reportpwrstate_parm *)pbuf;
>  	preportpwrstate->state |= (u8)(adapter_to_pwrctl(padapter)->cpwm_tog + 0x80);
>  	cpwm_int_hdl(padapter, preportpwrstate);
> @@ -1939,7 +1889,6 @@ int rtw_select_roaming_candidate(struct mlme_priv *mlme)
>  
>  		pnetwork = container_of(mlme->pscanned, struct wlan_network, list);
>  		if (!pnetwork) {
> -			RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("%s return _FAIL:(pnetwork == NULL)\n", __func__));
>  			ret = _FAIL;
>  			goto exit;
>  		}
> @@ -2069,7 +2018,6 @@ int rtw_select_and_join_from_scanned_queue(struct mlme_priv *pmlmepriv)
>  
>  		pnetwork = container_of(pmlmepriv->pscanned, struct wlan_network, list);
>  		if (!pnetwork) {
> -			RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("%s return _FAIL:(pnetwork == NULL)\n", __func__));
>  			ret = _FAIL;
>  			goto exit;
>  		}
> @@ -2146,8 +2094,6 @@ signed int rtw_set_auth(struct adapter *adapter, struct security_priv *psecurity
>  
>  	INIT_LIST_HEAD(&pcmd->list);
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("after enqueue set_auth_cmd, auth_mode =%x\n", psecuritypriv->dot11AuthAlgrthm));
> -
>  	res = rtw_enqueue_cmd(pcmdpriv, pcmd);
>  
>  exit:
> @@ -2170,10 +2116,8 @@ signed int rtw_set_key(struct adapter *adapter, struct security_priv *psecurityp
>  
>  	if (psecuritypriv->dot11AuthAlgrthm == dot11AuthAlgrthm_8021X) {
>  		psetkeyparm->algorithm = (unsigned char)psecuritypriv->dot118021XGrpPrivacy;
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("\n rtw_set_key: psetkeyparm->algorithm =(unsigned char)psecuritypriv->dot118021XGrpPrivacy =%d\n", psetkeyparm->algorithm));
>  	} else {
>  		psetkeyparm->algorithm = (u8)psecuritypriv->dot11PrivacyAlgrthm;
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("\n rtw_set_key: psetkeyparm->algorithm =(u8)psecuritypriv->dot11PrivacyAlgrthm =%d\n", psetkeyparm->algorithm));
>  
>  	}
>  	psetkeyparm->keyid = (u8)keyid;/* 0~3 */
> @@ -2182,7 +2126,6 @@ signed int rtw_set_key(struct adapter *adapter, struct security_priv *psecurityp
>  		adapter->securitypriv.key_mask |= BIT(psetkeyparm->keyid);
>  
>  	DBG_871X("==> rtw_set_key algorithm(%x), keyid(%x), key_mask(%x)\n", psetkeyparm->algorithm, psetkeyparm->keyid, adapter->securitypriv.key_mask);
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("\n rtw_set_key: psetkeyparm->algorithm =%d psetkeyparm->keyid =(u8)keyid =%d\n", psetkeyparm->algorithm, keyid));
>  
>  	switch (psetkeyparm->algorithm) {
>  
> @@ -2205,7 +2148,6 @@ signed int rtw_set_key(struct adapter *adapter, struct security_priv *psecurityp
>  		psetkeyparm->grpkey = 1;
>  		break;
>  	default:
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("\n rtw_set_key:psecuritypriv->dot11PrivacyAlgrthm = %x (must be 1 or 2 or 4 or 5)\n", psecuritypriv->dot11PrivacyAlgrthm));
>  		res = _FAIL;
>  		kfree(psetkeyparm);
>  		goto exit;
> @@ -2339,9 +2281,6 @@ signed int rtw_restruct_sec_ie(struct adapter *adapter, u8 *in_ie, u8 *out_ie, u
>  	struct security_priv *psecuritypriv = &adapter->securitypriv;
>  	uint	ndisauthmode = psecuritypriv->ndisauthtype;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_notice_,
> -		 ("+rtw_restruct_sec_ie: ndisauthmode =%d\n", ndisauthmode));
> -
>  	/* copy fixed ie only */
>  	memcpy(out_ie, in_ie, 12);
>  	ielength = 12;
> @@ -2438,7 +2377,6 @@ void rtw_update_registrypriv_dev_network(struct adapter *adapter)
>  	}
>  
>  	pdev_network->Configuration.DSConfig = (pregistrypriv->channel);
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("pregistrypriv->channel =%d, pdev_network->Configuration.DSConfig = 0x%x\n", pregistrypriv->channel, pdev_network->Configuration.DSConfig));
>  
>  	if (cur_network->network.InfrastructureMode == Ndis802_11IBSS)
>  		pdev_network->Configuration.ATIMWindow = (0);
> diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
> index 8aadcf72a7ba..ea52e8762372 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
> @@ -526,12 +526,7 @@ void mgt_dispatcher(struct adapter *padapter, union recv_frame *precv_frame)
>  	struct dvobj_priv *psdpriv = padapter->dvobj;
>  	struct debug_priv *pdbgpriv = &psdpriv->drv_dbg;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_,
> -		 ("+mgt_dispatcher: type(0x%x) subtype(0x%x)\n",
> -		  GetFrameType(pframe), GetFrameSubType(pframe)));
> -
>  	if (GetFrameType(pframe) != WIFI_MGT_TYPE) {
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("mgt_dispatcher: type(0x%x) error!\n", GetFrameType(pframe)));
>  		return;
>  	}
>  
> @@ -546,7 +541,6 @@ void mgt_dispatcher(struct adapter *padapter, union recv_frame *precv_frame)
>  	index = GetFrameSubType(pframe) >> 4;
>  
>  	if (index >= ARRAY_SIZE(mlme_sta_tbl)) {
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("Currently we do not support reserved sub-fr-type =%d\n", index));
>  		return;
>  	}
>  	ptable += index;
> @@ -2675,8 +2669,6 @@ static int _issue_probereq(struct adapter *padapter,
>  	int	bssrate_len = 0;
>  	u8 bc_addr[] = {0xff, 0xff, 0xff, 0xff, 0xff, 0xff};
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_notice_, ("+issue_probereq\n"));
> -
>  	pmgntframe = alloc_mgtxmitframe(pxmitpriv);
>  	if (!pmgntframe)
>  		goto exit;
> @@ -2743,8 +2735,6 @@ static int _issue_probereq(struct 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));
> -
>  	if (wait_ack) {
>  		ret = dump_mgntframe_and_wait_ack(padapter, pmgntframe);
>  	} else {
> @@ -4441,8 +4431,6 @@ void start_create_ibss(struct adapter *padapter)
>  
>  		/* issue beacon */
>  		if (send_beacon(padapter) == _FAIL) {
> -			RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("issuing beacon frame fail....\n"));
> -
>  			report_join_res(padapter, -1);
>  			pmlmeinfo->state = WIFI_FW_NULL_STATE;
>  		} else {
> @@ -4622,8 +4610,6 @@ static void process_80211d(struct adapter *padapter, struct wlan_bssid_ex *bssid
>  		memset(country, 0, 4);
>  		memcpy(country, p, 3);
>  		p += 3;
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_notice_,
> -				("%s: 802.11d country =%s\n", __func__, country));
>  
>  		i = 0;
>  		while ((ie - p) >= 3) {
> @@ -4813,9 +4799,6 @@ static void process_80211d(struct adapter *padapter, struct wlan_bssid_ex *bssid
>  					break;
>  
>  				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));
>  			}
>  			break;
>  		}
> @@ -6332,10 +6315,6 @@ u8 mlme_evt_hdl(struct adapter *padapter, unsigned char *pbuf)
>  	#ifdef CHECK_EVENT_SEQ
>  	/*  checking event sequence... */
>  	if (evt_seq != (atomic_read(&pevt_priv->event_seq) & 0x7f)) {
> -		RT_TRACE(_module_rtl871x_cmd_c_, _drv_info_,
> -			 ("Event Seq Error! %d vs %d\n", (evt_seq & 0x7f),
> -			  (atomic_read(&pevt_priv->event_seq) & 0x7f)));
> -
>  		pevt_priv->event_seq = (evt_seq+1)&0x7f;
>  
>  		goto _abort_event_;
> @@ -6344,7 +6323,6 @@ u8 mlme_evt_hdl(struct adapter *padapter, unsigned char *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));
>  		goto _abort_event_;
>  	}
>  
> @@ -6352,8 +6330,6 @@ u8 mlme_evt_hdl(struct adapter *padapter, unsigned char *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));
>  		goto _abort_event_;
>  
>  	}
> diff --git a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> index 8bf80e6f4a11..db43061fc0dd 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> @@ -268,29 +268,18 @@ void rtw_set_rpwm(struct adapter *padapter, u8 pslv)
>  	} else {
>  		if ((pwrpriv->rpwm == pslv)
>  			|| ((pwrpriv->rpwm >= PS_STATE_S2) && (pslv >= PS_STATE_S2))) {
> -			RT_TRACE(_module_rtl871x_pwrctrl_c_, _drv_err_,
> -				("%s: Already set rpwm[0x%02X], new = 0x%02X!\n", __func__, pwrpriv->rpwm, pslv));
>  			return;
>  		}
>  	}
>  
>  	if ((padapter->bSurpriseRemoved) || !(padapter->hw_init_completed)) {
> -		RT_TRACE(_module_rtl871x_pwrctrl_c_, _drv_err_,
> -				 ("%s: SurpriseRemoved(%d) hw_init_completed(%d)\n",
> -				  __func__, padapter->bSurpriseRemoved, padapter->hw_init_completed));
> -
>  		pwrpriv->cpwm = PS_STATE_S4;
>  
>  		return;
>  	}
>  
>  	if (padapter->bDriverStopped) {
> -		RT_TRACE(_module_rtl871x_pwrctrl_c_, _drv_err_,
> -				 ("%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));
>  			return;
>  		}
>  	}
> @@ -299,8 +288,6 @@ void rtw_set_rpwm(struct adapter *padapter, u8 pslv)
>  	/*  only when from PS_STATE S0/S1 to S2 and higher needs ACK */
>  	if ((pwrpriv->cpwm < PS_STATE_S2) && (pslv >= PS_STATE_S2))
>  		rpwm |= PS_ACK;
> -	RT_TRACE(_module_rtl871x_pwrctrl_c_, _drv_notice_,
> -			 ("rtw_set_rpwm: rpwm = 0x%02x cpwm = 0x%02x\n", rpwm, pwrpriv->cpwm));
>  
>  	pwrpriv->rpwm = pslv;
>  
> @@ -382,12 +369,7 @@ void rtw_set_ps_mode(struct adapter *padapter, u8 ps_mode, u8 smart_ps, u8 bcn_a
>  {
>  	struct pwrctrl_priv *pwrpriv = adapter_to_pwrctl(padapter);
>  
> -	RT_TRACE(_module_rtl871x_pwrctrl_c_, _drv_notice_,
> -			 ("%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));
>  		return;
>  	}
>  
> @@ -700,9 +682,6 @@ void cpwm_int_hdl(struct adapter *padapter, struct reportpwrstate_parm *preportp
>  
>  exit:
>  	mutex_unlock(&pwrpriv->lock);
> -
> -	RT_TRACE(_module_rtl871x_pwrctrl_c_, _drv_notice_,
> -			 ("cpwm_int_hdl: cpwm = 0x%02x\n", pwrpriv->cpwm));
>  }
>  
>  static void cpwm_event_callback(struct work_struct *work)
> @@ -817,10 +796,6 @@ s32 rtw_register_task_alive(struct adapter *padapter, u32 task)
>  	register_task_alive(pwrctrl, task);
>  
>  	if (pwrctrl->bFwCurrentInPSMode) {
> -		RT_TRACE(_module_rtl871x_pwrctrl_c_, _drv_notice_,
> -				 ("%s: task = 0x%x cpwm = 0x%02x alives = 0x%08x\n",
> -				  __func__, task, pwrctrl->cpwm, pwrctrl->alives));
> -
>  		if (pwrctrl->cpwm < pslv) {
>  			if (pwrctrl->cpwm < PS_STATE_S2)
>  				res = _FAIL;
> @@ -869,10 +844,6 @@ void rtw_unregister_task_alive(struct adapter *padapter, u32 task)
>  	unregister_task_alive(pwrctrl, task);
>  
>  	if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->bFwCurrentInPSMode) {
> -		RT_TRACE(_module_rtl871x_pwrctrl_c_, _drv_notice_,
> -				 ("%s: cpwm = 0x%02x alives = 0x%08x\n",
> -				  __func__, pwrctrl->cpwm, pwrctrl->alives));
> -
>  		if (pwrctrl->cpwm > pslv)
>  			if ((pslv >= PS_STATE_S2) || (pwrctrl->alives == 0))
>  				rtw_set_rpwm(padapter, pslv);
> @@ -910,10 +881,6 @@ s32 rtw_register_tx_alive(struct adapter *padapter)
>  	register_task_alive(pwrctrl, XMIT_ALIVE);
>  
>  	if (pwrctrl->bFwCurrentInPSMode) {
> -		RT_TRACE(_module_rtl871x_pwrctrl_c_, _drv_notice_,
> -				 ("rtw_register_tx_alive: cpwm = 0x%02x alives = 0x%08x\n",
> -				  pwrctrl->cpwm, pwrctrl->alives));
> -
>  		if (pwrctrl->cpwm < pslv) {
>  			if (pwrctrl->cpwm < PS_STATE_S2)
>  				res = _FAIL;
> @@ -959,10 +926,6 @@ s32 rtw_register_cmd_alive(struct adapter *padapter)
>  	register_task_alive(pwrctrl, CMD_ALIVE);
>  
>  	if (pwrctrl->bFwCurrentInPSMode) {
> -		RT_TRACE(_module_rtl871x_pwrctrl_c_, _drv_info_,
> -				 ("rtw_register_cmd_alive: cpwm = 0x%02x alives = 0x%08x\n",
> -				  pwrctrl->cpwm, pwrctrl->alives));
> -
>  		if (pwrctrl->cpwm < pslv) {
>  			if (pwrctrl->cpwm < PS_STATE_S2)
>  				res = _FAIL;
> @@ -1008,10 +971,6 @@ void rtw_unregister_tx_alive(struct adapter *padapter)
>  	unregister_task_alive(pwrctrl, XMIT_ALIVE);
>  
>  	if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->bFwCurrentInPSMode) {
> -		RT_TRACE(_module_rtl871x_pwrctrl_c_, _drv_notice_,
> -				 ("%s: cpwm = 0x%02x alives = 0x%08x\n",
> -				  __func__, pwrctrl->cpwm, pwrctrl->alives));
> -
>  		if (pwrctrl->cpwm > pslv)
>  			if ((pslv >= PS_STATE_S2) || (pwrctrl->alives == 0))
>  				rtw_set_rpwm(padapter, pslv);
> @@ -1048,10 +1007,6 @@ void rtw_unregister_cmd_alive(struct adapter *padapter)
>  	unregister_task_alive(pwrctrl, CMD_ALIVE);
>  
>  	if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->bFwCurrentInPSMode) {
> -		RT_TRACE(_module_rtl871x_pwrctrl_c_, _drv_info_,
> -				 ("%s: cpwm = 0x%02x alives = 0x%08x\n",
> -				  __func__, pwrctrl->cpwm, pwrctrl->alives));
> -
>  		if (pwrctrl->cpwm > pslv) {
>  			if ((pslv >= PS_STATE_S2) || (pwrctrl->alives == 0))
>  				rtw_set_rpwm(padapter, pslv);
> diff --git a/drivers/staging/rtl8723bs/core/rtw_recv.c b/drivers/staging/rtl8723bs/core/rtw_recv.c
> index 1fa381663b4c..b4ea2c3d0c37 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_recv.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_recv.c
> @@ -326,10 +326,6 @@ signed int recvframe_chkmic(struct adapter *adapter,  union recv_frame *precvfra
>  	stainfo = rtw_get_stainfo(&adapter->stapriv, &prxattrib->ta[0]);
>  
>  	if (prxattrib->encrypt == _TKIP_) {
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("\n recvframe_chkmic:prxattrib->encrypt == _TKIP_\n"));
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("\n recvframe_chkmic:da = 0x%02x:0x%02x:0x%02x:0x%02x:0x%02x:0x%02x\n",
> -			prxattrib->ra[0], prxattrib->ra[1], prxattrib->ra[2], prxattrib->ra[3], prxattrib->ra[4], prxattrib->ra[5]));
> -
>  		/* calculate mic code */
>  		if (stainfo) {
>  			if (IS_MCAST(prxattrib->ra)) {
> @@ -337,28 +333,22 @@ signed int recvframe_chkmic(struct adapter *adapter,  union recv_frame *precvfra
>  				/* iv = precvframe->u.hdr.rx_data+prxattrib->hdrlen; */
>  				/* rxdata_key_idx =(((iv[3])>>6)&0x3) ; */
>  				mickey = &psecuritypriv->dot118021XGrprxmickey[prxattrib->key_index].skey[0];
> -
> -				RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("\n recvframe_chkmic: bcmc key\n"));
>  				/* DBG_871X("\n recvframe_chkmic: bcmc key psecuritypriv->dot118021XGrpKeyid(%d), pmlmeinfo->key_index(%d) , recv key_id(%d)\n", */
>  				/* psecuritypriv->dot118021XGrpKeyid, pmlmeinfo->key_index, rxdata_key_idx); */
>  
>  				if (psecuritypriv->binstallGrpkey == false) {
>  					res = _FAIL;
> -					RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("\n recvframe_chkmic:didn't install group key!!!!!!!!!!\n"));
>  					DBG_871X("\n recvframe_chkmic:didn't install group key!!!!!!!!!!\n");
>  					goto exit;
>  				}
>  			} else {
>  				mickey = &stainfo->dot11tkiprxmickey.skey[0];
> -				RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("\n recvframe_chkmic: unicast key\n"));
>  			}
>  
>  			datalen = precvframe->u.hdr.len-prxattrib->hdrlen-prxattrib->iv_len-prxattrib->icv_len-8;/* icv_len included the mic code */
>  			pframe = precvframe->u.hdr.rx_data;
>  			payload = pframe+prxattrib->hdrlen+prxattrib->iv_len;
>  
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("\n prxattrib->iv_len =%d prxattrib->icv_len =%d\n", prxattrib->iv_len, prxattrib->icv_len));
> -
>  
>  			rtw_seccalctkipmic(mickey, pframe, payload, datalen, &miccode[0], (unsigned char)prxattrib->priority); /* care the length of the data */
>  
> @@ -368,7 +358,6 @@ signed int recvframe_chkmic(struct adapter *adapter,  union recv_frame *precvfra
>  
>  			for (i = 0; i < 8; i++) {
>  				if (miccode[i] != *(pframemic+i)) {
> -					RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("recvframe_chkmic:miccode[%d](%02x) != *(pframemic+%d)(%02x) ", i, miccode[i], i, *(pframemic+i)));
>  					bmic_err = true;
>  				}
>  			}
> @@ -376,29 +365,12 @@ signed int recvframe_chkmic(struct adapter *adapter,  union recv_frame *precvfra
>  
>  			if (bmic_err == true) {
>  
> -				RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("\n *(pframemic-8)-*(pframemic-1) = 0x%02x:0x%02x:0x%02x:0x%02x:0x%02x:0x%02x:0x%02x:0x%02x\n",
> -					*(pframemic-8), *(pframemic-7), *(pframemic-6), *(pframemic-5), *(pframemic-4), *(pframemic-3), *(pframemic-2), *(pframemic-1)));
> -				RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("\n *(pframemic-16)-*(pframemic-9) = 0x%02x:0x%02x:0x%02x:0x%02x:0x%02x:0x%02x:0x%02x:0x%02x\n",
> -					*(pframemic-16), *(pframemic-15), *(pframemic-14), *(pframemic-13), *(pframemic-12), *(pframemic-11), *(pframemic-10), *(pframemic-9)));
> -
>  				{
>  					uint i;
> -					RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("\n ======demp packet (len =%d) ======\n", precvframe->u.hdr.len));
>  					for (i = 0; i < precvframe->u.hdr.len; i = i+8) {
> -						RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("0x%02x:0x%02x:0x%02x:0x%02x:0x%02x:0x%02x:0x%02x:0x%02x",
> -							*(precvframe->u.hdr.rx_data+i), *(precvframe->u.hdr.rx_data+i+1),
> -							*(precvframe->u.hdr.rx_data+i+2), *(precvframe->u.hdr.rx_data+i+3),
> -							*(precvframe->u.hdr.rx_data+i+4), *(precvframe->u.hdr.rx_data+i+5),
> -							*(precvframe->u.hdr.rx_data+i+6), *(precvframe->u.hdr.rx_data+i+7)));
>  					}
> -					RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("\n ======demp packet end [len =%d]======\n", precvframe->u.hdr.len));
> -					RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("\n hrdlen =%d,\n", prxattrib->hdrlen));
>  				}
>  
> -				RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("ra = 0x%.2x 0x%.2x 0x%.2x 0x%.2x 0x%.2x 0x%.2x psecuritypriv->binstallGrpkey =%d ",
> -					prxattrib->ra[0], prxattrib->ra[1], prxattrib->ra[2],
> -					prxattrib->ra[3], prxattrib->ra[4], prxattrib->ra[5], psecuritypriv->binstallGrpkey));
> -
>  				/*  double check key_index for some timing issue , */
>  				/*  cannot compare with psecuritypriv->dot118021XGrpKeyid also cause timing issue */
>  				if ((IS_MCAST(prxattrib->ra) == true)  && (prxattrib->key_index != pmlmeinfo->key_index))
> @@ -406,10 +378,8 @@ signed int recvframe_chkmic(struct adapter *adapter,  union recv_frame *precvfra
>  
>  				if ((prxattrib->bdecrypted == true) && (brpt_micerror == true)) {
>  					rtw_handle_tkip_mic_err(adapter, (u8)IS_MCAST(prxattrib->ra));
> -					RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, (" mic error :prxattrib->bdecrypted =%d ", prxattrib->bdecrypted));
>  					DBG_871X(" mic error :prxattrib->bdecrypted =%d\n", prxattrib->bdecrypted);
>  				} else {
> -					RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, (" mic error :prxattrib->bdecrypted =%d ", prxattrib->bdecrypted));
>  					DBG_871X(" mic error :prxattrib->bdecrypted =%d\n", prxattrib->bdecrypted);
>  				}
>  
> @@ -419,12 +389,11 @@ signed int recvframe_chkmic(struct adapter *adapter,  union recv_frame *precvfra
>  				/* mic checked ok */
>  				if ((psecuritypriv->bcheck_grpkey == false) && (IS_MCAST(prxattrib->ra) == true)) {
>  					psecuritypriv->bcheck_grpkey = true;
> -					RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("psecuritypriv->bcheck_grpkey =true"));
>  				}
>  			}
>  
>  		} else
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("recvframe_chkmic: rtw_get_stainfo == NULL!!!\n"));
> +			{}
>  
>  		recvframe_pull_tail(precvframe, 8);
>  
> @@ -445,8 +414,6 @@ union recv_frame *decryptor(struct adapter *padapter, union recv_frame *precv_fr
>  	union recv_frame *return_packet = precv_frame;
>  	u32  res = _SUCCESS;
>  
> -	RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("prxstat->decrypted =%x prxattrib->encrypt = 0x%03x\n", prxattrib->bdecrypted, prxattrib->encrypt));
> -
>  	if (prxattrib->encrypt > 0) {
>  		u8 *iv = precv_frame->u.hdr.rx_data+prxattrib->hdrlen;
>  		prxattrib->key_index = (((iv[3])>>6)&0x3);
> @@ -555,16 +522,10 @@ union recv_frame *portctrl(struct adapter *adapter, union recv_frame *precv_fram
>  
>  	psta = rtw_get_stainfo(pstapriv, psta_addr);
>  
> -	RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("########portctrl:adapter->securitypriv.dot11AuthAlgrthm =%d\n", adapter->securitypriv.dot11AuthAlgrthm));
> -
>  	if (auth_alg == 2) {
>  		if ((psta) && (psta->ieee8021x_blocked)) {
>  			__be16 be_tmp;
>  
> -			/* blocked */
> -			/* only accept EAPOL frame */
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("########portctrl:psta->ieee8021x_blocked == 1\n"));
> -
>  			prtnframe = precv_frame;
>  
>  			/* get ether_type */
> @@ -580,13 +541,9 @@ union recv_frame *portctrl(struct adapter *adapter, union recv_frame *precv_fram
>  				prtnframe = NULL;
>  			}
>  		} else {
> -			/* allowed */
> -			/* check decryption status, and decrypt the frame if needed */
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("########portctrl:psta->ieee8021x_blocked == 0\n"));
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("portctrl:precv_frame->hdr.attrib.privacy =%x\n", precv_frame->u.hdr.attrib.privacy));
>  
>  			if (pattrib->bdecrypted == 0)
> -				RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("portctrl:prxstat->decrypted =%x\n", pattrib->bdecrypted));
> +				{}
>  
>  			prtnframe = precv_frame;
>  			/* check is the EAPOL frame or not (Rekey) */
> @@ -615,15 +572,11 @@ signed int recv_decache(union recv_frame *precv_frame, u8 bretry, struct stainfo
>  		(precv_frame->u.hdr.attrib.frag_num & 0xf);
>  
>  	if (tid > 15) {
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_notice_, ("recv_decache, (tid>15)! seq_ctrl = 0x%x, tid = 0x%x\n", seq_ctrl, tid));
> -
>  		return _FAIL;
>  	}
>  
>  	if (1) { /* if (bretry) */
>  		if (seq_ctrl == prxcache->tid_rxseq[tid]) {
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_notice_, ("recv_decache, seq_ctrl = 0x%x, tid = 0x%x, tid_rxseq = 0x%x\n", seq_ctrl, tid, prxcache->tid_rxseq[tid]));
> -
>  			return _FAIL;
>  		}
>  	}
> @@ -777,7 +730,6 @@ signed int sta2sta_data_frame(struct adapter *adapter, union recv_frame *precv_f
>  
>  		/*  filter packets that SA is myself or multicast or broadcast */
>  		if (!memcmp(myhwaddr, pattrib->src, ETH_ALEN)) {
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, (" SA ==myself\n"));
>  			ret = _FAIL;
>  			goto exit;
>  		}
> @@ -799,7 +751,6 @@ signed int sta2sta_data_frame(struct adapter *adapter, union recv_frame *precv_f
>  	} else if (check_fwstate(pmlmepriv, WIFI_STATION_STATE) == true) {
>  		/*  For Station mode, sa and bssid should always be BSSID, and DA is my mac-address */
>  		if (memcmp(pattrib->bssid, pattrib->src, ETH_ALEN)) {
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("bssid != TA under STATION_MODE; drop pkt\n"));
>  			ret = _FAIL;
>  			goto exit;
>  		}
> @@ -841,7 +792,6 @@ signed int sta2sta_data_frame(struct adapter *adapter, union recv_frame *precv_f
>  		*psta = rtw_get_stainfo(pstapriv, sta_addr); /*  get ap_info */
>  
>  	if (!*psta) {
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("can't get psta under sta2sta_data_frame ; drop pkt\n"));
>  		ret = _FAIL;
>  		goto exit;
>  	}
> @@ -871,7 +821,6 @@ signed int ap2sta_data_frame(struct adapter *adapter, union recv_frame *precv_fr
>  
>  		/*  filter packets that SA is myself or multicast or broadcast */
>  		if (!memcmp(myhwaddr, pattrib->src, ETH_ALEN)) {
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, (" SA ==myself\n"));
>  			#ifdef DBG_RX_DROP_FRAME
>  			DBG_871X("DBG_RX_DROP_FRAME %s SA =%pM, myhwaddr =%pM\n",
>  				__func__, MAC_ARG(pattrib->src), MAC_ARG(myhwaddr));
> @@ -882,8 +831,6 @@ signed int ap2sta_data_frame(struct adapter *adapter, union recv_frame *precv_fr
>  
>  		/*  da should be for me */
>  		if ((memcmp(myhwaddr, pattrib->dst, ETH_ALEN)) && (!bmcast)) {
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_info_,
> -				(" ap2sta_data_frame:  compare DA fail; DA =%pM\n", MAC_ARG(pattrib->dst)));
>  			#ifdef DBG_RX_DROP_FRAME
>  			DBG_871X("DBG_RX_DROP_FRAME %s DA =%pM\n", __func__, MAC_ARG(pattrib->dst));
>  			#endif
> @@ -896,9 +843,6 @@ signed int ap2sta_data_frame(struct adapter *adapter, union recv_frame *precv_fr
>  		if (!memcmp(pattrib->bssid, "\x0\x0\x0\x0\x0\x0", ETH_ALEN) ||
>  		     !memcmp(mybssid, "\x0\x0\x0\x0\x0\x0", ETH_ALEN) ||
>  		     (memcmp(pattrib->bssid, mybssid, ETH_ALEN))) {
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_info_,
> -				(" ap2sta_data_frame:  compare BSSID fail ; BSSID =%pM\n", MAC_ARG(pattrib->bssid)));
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("mybssid =%pM\n", MAC_ARG(mybssid)));
>  			#ifdef DBG_RX_DROP_FRAME
>  			DBG_871X("DBG_RX_DROP_FRAME %s BSSID =%pM, mybssid =%pM\n",
>  				__func__, MAC_ARG(pattrib->bssid), MAC_ARG(mybssid));
> @@ -920,7 +864,6 @@ signed int ap2sta_data_frame(struct adapter *adapter, union recv_frame *precv_fr
>  			*psta = rtw_get_stainfo(pstapriv, pattrib->bssid); /*  get ap_info */
>  
>  		if (!*psta) {
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("ap2sta: can't get psta under STATION_MODE ; drop pkt\n"));
>  			#ifdef DBG_RX_DROP_FRAME
>  			DBG_871X("DBG_RX_DROP_FRAME %s can't get psta under STATION_MODE ; drop pkt\n", __func__);
>  			#endif
> @@ -949,7 +892,6 @@ signed int ap2sta_data_frame(struct adapter *adapter, union recv_frame *precv_fr
>  
>  		*psta = rtw_get_stainfo(pstapriv, pattrib->bssid); /*  get sta_info */
>  		if (!*psta) {
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("can't get psta under MP_MODE ; drop pkt\n"));
>  			#ifdef DBG_RX_DROP_FRAME
>  			DBG_871X("DBG_RX_DROP_FRAME %s can't get psta under WIFI_MP_STATE ; drop pkt\n", __func__);
>  			#endif
> @@ -1013,7 +955,6 @@ signed int sta2ap_data_frame(struct adapter *adapter, union recv_frame *precv_fr
>  
>  		*psta = rtw_get_stainfo(pstapriv, pattrib->src);
>  		if (!*psta) {
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("can't get psta under AP_MODE; drop pkt\n"));
>  			DBG_871X("issue_deauth to sta =%pM for the reason(7)\n", MAC_ARG(pattrib->src));
>  
>  			issue_deauth(adapter, pattrib->src, WLAN_REASON_CLASS3_FRAME_FROM_NONASSOC_STA);
> @@ -1190,13 +1131,9 @@ union recv_frame *recvframe_chk_defrag(struct adapter *padapter, union recv_fram
>  signed int validate_recv_mgnt_frame(struct adapter *padapter, union recv_frame *precv_frame);
>  signed int validate_recv_mgnt_frame(struct adapter *padapter, union recv_frame *precv_frame)
>  {
> -	/* struct mlme_priv *pmlmepriv = &adapter->mlmepriv; */
> -
> -	RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("+validate_recv_mgnt_frame\n"));
>  
>  	precv_frame = recvframe_chk_defrag(padapter, precv_frame);
>  	if (!precv_frame) {
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_notice_, ("%s: fragment packet\n", __func__));
>  		return _SUCCESS;
>  	}
>  
> @@ -1279,7 +1216,6 @@ signed int validate_recv_data_frame(struct adapter *adapter, union recv_frame *p
>  		memcpy(pattrib->ra, GetAddr1Ptr(ptr), ETH_ALEN);
>  		memcpy(pattrib->ta, GetAddr2Ptr(ptr), ETH_ALEN);
>  		ret = _FAIL;
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, (" case 3\n"));
>  		break;
>  
>  	default:
> @@ -1299,7 +1235,6 @@ signed int validate_recv_data_frame(struct adapter *adapter, union recv_frame *p
>  
>  
>  	if (!psta) {
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, (" after to_fr_ds_chk; psta == NULL\n"));
>  		#ifdef DBG_RX_DROP_FRAME
>  		DBG_871X("DBG_RX_DROP_FRAME %s psta == NULL\n", __func__);
>  		#endif
> @@ -1337,7 +1272,6 @@ signed int validate_recv_data_frame(struct adapter *adapter, union recv_frame *p
>  
>  	/*  decache, drop duplicate recv packets */
>  	if (recv_decache(precv_frame, bretry, &psta->sta_recvpriv.rxcache) == _FAIL) {
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("decache : drop pkt\n"));
>  		#ifdef DBG_RX_DROP_FRAME
>  		DBG_871X("DBG_RX_DROP_FRAME %s recv_decache return _FAIL\n", __func__);
>  		#endif
> @@ -1347,13 +1281,8 @@ signed int validate_recv_data_frame(struct adapter *adapter, union recv_frame *p
>  
>  	if (pattrib->privacy) {
>  
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("validate_recv_data_frame:pattrib->privacy =%x\n", pattrib->privacy));
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("\n ^^^^^^^^^^^IS_MCAST(pattrib->ra(0x%02x)) =%d^^^^^^^^^^^^^^^6\n", pattrib->ra[0], IS_MCAST(pattrib->ra)));
> -
>  		GET_ENCRY_ALGO(psecuritypriv, psta, pattrib->encrypt, IS_MCAST(pattrib->ra));
>  
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("\n pattrib->encrypt =%d\n", pattrib->encrypt));
> -
>  		SET_ICE_IV_LEN(pattrib->iv_len, pattrib->icv_len, pattrib->encrypt);
>  	} else {
>  		pattrib->encrypt = 0;
> @@ -1478,7 +1407,6 @@ signed int validate_recv_frame(struct adapter *adapter, union recv_frame *precv_
>  
>  	/* add version chk */
>  	if (ver != 0) {
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("validate_recv_data_frame fail! (ver!= 0)\n"));
>  		retval = _FAIL;
>  		goto exit;
>  	}
> @@ -1513,13 +1441,13 @@ signed int validate_recv_frame(struct adapter *adapter, union recv_frame *precv_
>  
>  		retval = validate_recv_mgnt_frame(adapter, precv_frame);
>  		if (retval == _FAIL)
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("validate_recv_mgnt_frame fail\n"));
> +			{}
>  		retval = _FAIL; /*  only data frame return _SUCCESS */
>  		break;
>  	case WIFI_CTRL_TYPE: /* ctrl */
>  		retval = validate_recv_ctrl_frame(adapter, precv_frame);
>  		if (retval == _FAIL)
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("validate_recv_ctrl_frame fail\n"));
> +			{}
>  		retval = _FAIL; /*  only data frame return _SUCCESS */
>  		break;
>  	case WIFI_DATA_TYPE: /* data */
> @@ -1545,7 +1473,6 @@ signed int validate_recv_frame(struct adapter *adapter, union recv_frame *precv_
>  		}
>  		break;
>  	default:
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("validate_recv_data_frame fail! type = 0x%x\n", type));
>  		#ifdef DBG_RX_DROP_FRAME
>  		DBG_871X("DBG_RX_DROP_FRAME validate_recv_data_frame fail! type = 0x%x\n", type);
>  		#endif
> @@ -1594,8 +1521,6 @@ signed int wlanhdr_to_ethhdr(union recv_frame *precvframe)
>  	rmv_len = pattrib->hdrlen + pattrib->iv_len + (bsnaphdr?SNAP_SIZE:0);
>  	len = precvframe->u.hdr.len - rmv_len;
>  
> -	RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("\n ===pattrib->hdrlen: %x,  pattrib->iv_len:%x ===\n\n", pattrib->hdrlen,  pattrib->iv_len));
> -
>  	memcpy(&be_tmp, ptr+rmv_len, 2);
>  	eth_type = ntohs(be_tmp); /* pattrib->ether_type */
>  	pattrib->eth_type = eth_type;
> @@ -1699,8 +1624,6 @@ static union recv_frame *recvframe_defrag(struct adapter *adapter,
>  	/* free the defrag_q queue and return the prframe */
>  	rtw_free_recvframe_queue(defrag_q, pfree_recv_queue);
>  
> -	RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("Performance defrag!!!!!\n"));
> -
>  	return prframe;
>  }
>  
> @@ -1758,9 +1681,6 @@ union recv_frame *recvframe_chk_defrag(struct adapter *padapter, union recv_fram
>  			/* spin_lock(&pdefrag_q->lock); */
>  			phead = get_list_head(pdefrag_q);
>  			list_add_tail(&pfhdr->list, phead);
> -			/* spin_unlock(&pdefrag_q->lock); */
> -
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("Enqueuq: ismfrag = %d, fragnum = %d\n", ismfrag, fragnum));
>  
>  			prtnframe = NULL;
>  
> @@ -1768,7 +1688,6 @@ union recv_frame *recvframe_chk_defrag(struct adapter *padapter, union recv_fram
>  			/* can't find this ta's defrag_queue, so free this recv_frame */
>  			rtw_free_recvframe(precv_frame, pfree_recv_queue);
>  			prtnframe = NULL;
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("Free because pdefrag_q == NULL: ismfrag = %d, fragnum = %d\n", ismfrag, fragnum));
>  		}
>  
>  	}
> @@ -1783,7 +1702,6 @@ union recv_frame *recvframe_chk_defrag(struct adapter *padapter, union recv_fram
>  			/* spin_unlock(&pdefrag_q->lock); */
>  
>  			/* call recvframe_defrag to defrag */
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("defrag: ismfrag = %d, fragnum = %d\n", ismfrag, fragnum));
>  			precv_frame = recvframe_defrag(padapter, pdefrag_q);
>  			prtnframe = precv_frame;
>  
> @@ -1791,7 +1709,6 @@ union recv_frame *recvframe_chk_defrag(struct adapter *padapter, union recv_fram
>  			/* can't find this ta's defrag_queue, so free this recv_frame */
>  			rtw_free_recvframe(precv_frame, pfree_recv_queue);
>  			prtnframe = NULL;
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("Free because pdefrag_q == NULL: ismfrag = %d, fragnum = %d\n", ismfrag, fragnum));
>  		}
>  
>  	}
> @@ -1800,7 +1717,6 @@ union recv_frame *recvframe_chk_defrag(struct adapter *padapter, union recv_fram
>  	if ((prtnframe) && (prtnframe->u.hdr.attrib.privacy)) {
>  		/* after defrag we must check tkip mic code */
>  		if (recvframe_chkmic(padapter,  prtnframe) == _FAIL) {
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("recvframe_chkmic(padapter,  prtnframe) == _FAIL\n"));
>  			rtw_free_recvframe(prtnframe, pfree_recv_queue);
>  			prtnframe = NULL;
>  		}
> @@ -2067,10 +1983,6 @@ int recv_indicatepkts_in_order(struct adapter *padapter, struct recv_reorder_ctr
>  		pattrib = &prframe->u.hdr.attrib;
>  
>  		if (!SN_LESS(preorder_ctrl->indicate_seq, pattrib->seq_num)) {
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_notice_,
> -				 ("recv_indicatepkts_in_order: indicate =%d seq =%d amsdu =%d\n",
> -				  preorder_ctrl->indicate_seq, pattrib->seq_num, pattrib->amsdu));
> -
>  			plist = get_next(plist);
>  			list_del_init(&(prframe->u.hdr.list));
>  
> @@ -2142,8 +2054,6 @@ int recv_indicatepkt_reorder(struct adapter *padapter, union recv_frame *prframe
>  		if (pattrib->qos != 1) {
>  			if ((padapter->bDriverStopped == false) &&
>  			    (padapter->bSurpriseRemoved == false)) {
> -				RT_TRACE(_module_rtl871x_recv_c_, _drv_notice_, ("@@@@  recv_indicatepkt_reorder -recv_func recv_indicatepkt\n"));
> -
>  				rtw_recv_indicatepkt(padapter, prframe);
>  				return _SUCCESS;
>  
> @@ -2203,10 +2113,6 @@ int recv_indicatepkt_reorder(struct adapter *padapter, union recv_frame *prframe
>  
>  	spin_lock_bh(&ppending_recvframe_queue->lock);
>  
> -	RT_TRACE(_module_rtl871x_recv_c_, _drv_notice_,
> -		 ("recv_indicatepkt_reorder: indicate =%d seq =%d\n",
> -		  preorder_ctrl->indicate_seq, pattrib->seq_num));
> -
>  	/* s2. check if winstart_b(indicate_seq) needs to been updated */
>  	if (!check_indicate_seq(preorder_ctrl, pattrib->seq_num)) {
>  		pdbgpriv->dbg_rx_ampdu_drop_count++;
> @@ -2305,7 +2211,6 @@ int process_recv_indicatepkts(struct adapter *padapter, union recv_frame *prfram
>  	} else { /* B/G mode */
>  		retval = wlanhdr_to_ethhdr(prframe);
>  		if (retval != _SUCCESS) {
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("wlanhdr_to_ethhdr: drop pkt\n"));
>  			#ifdef DBG_RX_DROP_FRAME
>  			DBG_871X("DBG_RX_DROP_FRAME %s wlanhdr_to_ethhdr error!\n", __func__);
>  			#endif
> @@ -2314,14 +2219,10 @@ int process_recv_indicatepkts(struct adapter *padapter, union recv_frame *prfram
>  
>  		if ((padapter->bDriverStopped == false) && (padapter->bSurpriseRemoved == false)) {
>  			/* indicate this recv_frame */
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_notice_, ("@@@@ process_recv_indicatepkts- recv_func recv_indicatepkt\n"));
>  			rtw_recv_indicatepkt(padapter, prframe);
>  
>  
>  		} else {
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_notice_, ("@@@@ process_recv_indicatepkts- recv_func free_indicatepkt\n"));
> -
> -			RT_TRACE(_module_rtl871x_recv_c_, _drv_notice_, ("recv_func:bDriverStopped(%d) OR bSurpriseRemoved(%d)", padapter->bDriverStopped, padapter->bSurpriseRemoved));
>  			retval = _FAIL;
>  			return retval;
>  		}
> @@ -2340,7 +2241,6 @@ static int recv_func_prehandle(struct adapter *padapter, union recv_frame *rfram
>  	/* check the frame crtl field and decache */
>  	ret = validate_recv_frame(padapter, rframe);
>  	if (ret != _SUCCESS) {
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("recv_func: validate_recv_frame fail! drop pkt\n"));
>  		rtw_free_recvframe(rframe, pfree_recv_queue);/* free this recv_frame */
>  		goto exit;
>  	}
> @@ -2358,7 +2258,6 @@ static int recv_func_posthandle(struct adapter *padapter, union recv_frame *prfr
>  
>  	prframe = decryptor(padapter, prframe);
>  	if (!prframe) {
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("decryptor: drop pkt\n"));
>  		#ifdef DBG_RX_DROP_FRAME
>  		DBG_871X("DBG_RX_DROP_FRAME %s decryptor: drop pkt\n", __func__);
>  		#endif
> @@ -2368,7 +2267,6 @@ static int recv_func_posthandle(struct adapter *padapter, union recv_frame *prfr
>  
>  	prframe = recvframe_chk_defrag(padapter, prframe);
>  	if (!prframe)	{
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("recvframe_chk_defrag: drop pkt\n"));
>  		#ifdef DBG_RX_DROP_FRAME
>  		DBG_871X("DBG_RX_DROP_FRAME %s recvframe_chk_defrag: drop pkt\n", __func__);
>  		#endif
> @@ -2377,7 +2275,6 @@ static int recv_func_posthandle(struct adapter *padapter, union recv_frame *prfr
>  
>  	prframe = portctrl(padapter, prframe);
>  	if (!prframe) {
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("portctrl: drop pkt\n"));
>  		#ifdef DBG_RX_DROP_FRAME
>  		DBG_871X("DBG_RX_DROP_FRAME %s portctrl: drop pkt\n", __func__);
>  		#endif
> @@ -2389,7 +2286,6 @@ static int recv_func_posthandle(struct adapter *padapter, union recv_frame *prfr
>  
>  	ret = process_recv_indicatepkts(padapter, prframe);
>  	if (ret != _SUCCESS) {
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("recv_func: process_recv_indicatepkts fail!\n"));
>  		#ifdef DBG_RX_DROP_FRAME
>  		DBG_871X("DBG_RX_DROP_FRAME %s process_recv_indicatepkts fail!\n", __func__);
>  		#endif
> @@ -2472,7 +2368,6 @@ s32 rtw_recv_entry(union recv_frame *precvframe)
>  
>  	ret = recv_func(padapter, precvframe);
>  	if (ret == _FAIL) {
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_info_, ("rtw_recv_entry: recv_func return fail!!!\n"));
>  		goto _recv_entry_drop;
>  	}
>  
> diff --git a/drivers/staging/rtl8723bs/core/rtw_security.c b/drivers/staging/rtl8723bs/core/rtw_security.c
> index 44e2b362c867..dc35daefd866 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_security.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_security.c
> @@ -296,14 +296,6 @@ void rtw_wep_decrypt(struct adapter  *padapter, u8 *precvframe)
>  		*((u32 *)crc) = le32_to_cpu(getcrc32(payload, length-4));
>  
>  		if (crc[3] != payload[length-1] || crc[2] != payload[length-2] || crc[1] != payload[length-3] || crc[0] != payload[length-4]) {
> -			RT_TRACE(_module_rtl871x_security_c_,
> -				 _drv_err_,
> -				 ("%s:icv error crc[3](%x)!=payload[length-1](%x) || crc[2](%x)!=payload[length-2](%x) || crc[1](%x)!=payload[length-3](%x) || crc[0](%x)!=payload[length-4](%x)\n",
> -					__func__,
> -					crc[3], payload[length - 1],
> -					crc[2], payload[length - 2],
> -					crc[1], payload[length - 3],
> -					crc[0], payload[length - 4]));
>  		}
>  
>  		WEP_SW_DEC_CNT_INC(psecuritypriv, prxattrib->ra);
> @@ -670,8 +662,6 @@ u32 rtw_tkip_encrypt(struct adapter *padapter, u8 *pxmitframe)
>  	if (pattrib->encrypt == _TKIP_) {
>  
>  		{
> -			RT_TRACE(_module_rtl871x_security_c_, _drv_err_, ("%s: stainfo!= NULL!!!\n", __func__));
> -
>  			if (IS_MCAST(pattrib->ra))
>  				prwskey = psecuritypriv->dot118021XGrpKey[psecuritypriv->dot118021XGrpKeyid].skey;
>  			else
> @@ -692,7 +682,6 @@ u32 rtw_tkip_encrypt(struct adapter *padapter, u8 *pxmitframe)
>  
>  				if ((curfragnum+1) == pattrib->nr_frags) {	/* 4 the last fragment */
>  					length = pattrib->last_txcmdsz-pattrib->hdrlen-pattrib->iv_len-pattrib->icv_len;
> -					RT_TRACE(_module_rtl871x_security_c_, _drv_info_, ("pattrib->iv_len =%x, pattrib->icv_len =%x\n", pattrib->iv_len, pattrib->icv_len));
>  					*((__le32 *)crc) = getcrc32(payload, length);/* modified by Amy*/
>  
>  					arcfour_init(&mycontext, rc4key, 16);
> @@ -803,19 +792,11 @@ u32 rtw_tkip_decrypt(struct adapter *padapter, u8 *precvframe)
>  			*((u32 *)crc) = le32_to_cpu(getcrc32(payload, length-4));
>  
>  			if (crc[3] != payload[length-1] || crc[2] != payload[length-2] || crc[1] != payload[length-3] || crc[0] != payload[length-4]) {
> -				RT_TRACE(_module_rtl871x_security_c_,
> -					 _drv_err_,
> -					 ("rtw_wep_decrypt:icv error crc[3](%x)!=payload[length-1](%x) || crc[2](%x)!=payload[length-2](%x) || crc[1](%x)!=payload[length-3](%x) || crc[0](%x)!=payload[length-4](%x)\n",
> -						crc[3], payload[length - 1],
> -						crc[2], payload[length - 2],
> -						crc[1], payload[length - 3],
> -						crc[0], payload[length - 4]));
>  				res = _FAIL;
>  			}
>  
>  			TKIP_SW_DEC_CNT_INC(psecuritypriv, prxattrib->ra);
>  		} else {
> -			RT_TRACE(_module_rtl871x_security_c_, _drv_err_, ("%s: stainfo == NULL!!!\n", __func__));
>  			res = _FAIL;
>  		}
>  	}
> @@ -1445,8 +1426,6 @@ u32 rtw_aes_encrypt(struct adapter *padapter, u8 *pxmitframe)
>  
>  	/* 4 start to encrypt each fragment */
>  	if (pattrib->encrypt == _AES_) {
> -		RT_TRACE(_module_rtl871x_security_c_, _drv_err_, ("%s: stainfo!= NULL!!!\n", __func__));
> -
>  		if (IS_MCAST(pattrib->ra))
>  			prwskey = psecuritypriv->dot118021XGrpKey[psecuritypriv->dot118021XGrpKeyid].skey;
>  		else
> @@ -1678,13 +1657,6 @@ static signed int aes_decipher(u8 *key, uint	hdrlen,
>  	/* compare the mic */
>  	for (i = 0; i < 8; i++) {
>  		if (pframe[hdrlen+8+plen-8+i] != message[hdrlen+8+plen-8+i]) {
> -			RT_TRACE(_module_rtl871x_security_c_,
> -				 _drv_err_,
> -				 ("%s:mic check error mic[%d]: pframe(%x) != message(%x)\n",
> -					__func__,
> -					i,
> -					pframe[hdrlen + 8 + plen - 8 + i],
> -					message[hdrlen + 8 + plen - 8 + i]));
>  			DBG_871X("%s:mic check error mic[%d]: pframe(%x) != message(%x)\n",
>  					__func__,
>  					i,
> @@ -1716,10 +1688,6 @@ u32 rtw_aes_decrypt(struct adapter *padapter, u8 *precvframe)
>  	if (prxattrib->encrypt == _AES_) {
>  		stainfo = rtw_get_stainfo(&padapter->stapriv, &prxattrib->ta[0]);
>  		if (stainfo) {
> -			RT_TRACE(_module_rtl871x_security_c_,
> -				 _drv_err_,
> -				 ("%s: stainfo!= NULL!!!\n", __func__));
> -
>  			if (IS_MCAST(prxattrib->ra)) {
>  				static unsigned long start;
>  				static u32 no_gkey_bc_cnt;
> @@ -1774,9 +1742,6 @@ u32 rtw_aes_decrypt(struct adapter *padapter, u8 *precvframe)
>  
>  			AES_SW_DEC_CNT_INC(psecuritypriv, prxattrib->ra);
>  		} else {
> -			RT_TRACE(_module_rtl871x_security_c_,
> -				 _drv_err_,
> -				 ("%s: stainfo == NULL!!!\n", __func__));
>  			res = _FAIL;
>  		}
>  	}
> diff --git a/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c b/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c
> index f96dd0b40e04..7dcac4dd9de2 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c
> @@ -216,10 +216,7 @@ struct	sta_info *rtw_alloc_stainfo(struct	sta_priv *pstapriv, u8 *hwaddr)
>  
>  		index = wifi_mac_hash(hwaddr);
>  
> -		RT_TRACE(_module_rtl871x_sta_mgt_c_, _drv_info_, ("rtw_alloc_stainfo: index  = %x", index));
> -
>  		if (index >= NUM_STA) {
> -			RT_TRACE(_module_rtl871x_sta_mgt_c_, _drv_err_, ("ERROR => rtw_alloc_stainfo: index >= NUM_STA"));
>  			spin_unlock_bh(&(pstapriv->sta_hash_lock));
>  			psta = NULL;
>  			goto exit;
> @@ -242,17 +239,6 @@ struct	sta_info *rtw_alloc_stainfo(struct	sta_priv *pstapriv, u8 *hwaddr)
>  		for (i = 0; i < 16; i++)
>  			memcpy(&psta->sta_recvpriv.rxcache.tid_rxseq[i], &wRxSeqInitialValue, 2);
>  
> -		RT_TRACE(_module_rtl871x_sta_mgt_c_, _drv_info_,
> -			 ("alloc number_%d stainfo  with hwaddr = %x %x %x %x %x %x \n",
> -			  pstapriv->asoc_sta_count,
> -			  hwaddr[0],
> -			  hwaddr[1],
> -			  hwaddr[2],
> -			  hwaddr[3],
> -			  hwaddr[4],
> -			  hwaddr[5])
> -		);
> -
>  		init_addba_retry_timer(pstapriv->padapter, psta);
>  
>  		/* for A-MPDU Rx reordering buffer control */
> @@ -363,16 +349,6 @@ u32 rtw_free_stainfo(struct adapter *padapter, struct sta_info *psta)
>  	spin_unlock_bh(&pxmitpriv->lock);
>  
>  	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])
> -	);
>  	pstapriv->asoc_sta_count--;
>  
>  	/*  re-init sta_info; 20061114 will be init in alloc_stainfo */
> @@ -543,7 +519,6 @@ u32 rtw_init_bcmc_stainfo(struct adapter *padapter)
>  
>  	if (!psta) {
>  		res = _FAIL;
> -		RT_TRACE(_module_rtl871x_sta_mgt_c_, _drv_err_, ("rtw_alloc_stainfo fail"));
>  		goto exit;
>  	}
>  
> diff --git a/drivers/staging/rtl8723bs/core/rtw_wlan_util.c b/drivers/staging/rtl8723bs/core/rtw_wlan_util.c
> index bfd55a0356f5..fce854e9d5ce 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_wlan_util.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_wlan_util.c
> @@ -1337,11 +1337,6 @@ int rtw_check_bcn_info(struct adapter *Adapter, u8 *pframe, u32 packet_len)
>  	memcpy(bssid->Ssid.Ssid, (p + 2), ssid_len);
>  	bssid->Ssid.SsidLength = ssid_len;
>  
> -	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__, bssid->Ssid.Ssid,
> -				bssid->Ssid.SsidLength, cur_network->network.Ssid.Ssid,
> -				cur_network->network.Ssid.SsidLength));
> -
>  	if (memcmp(bssid->Ssid.Ssid, cur_network->network.Ssid.Ssid, 32) ||
>  			bssid->Ssid.SsidLength != cur_network->network.Ssid.SsidLength) {
>  		if (bssid->Ssid.Ssid[0] != '\0' && bssid->Ssid.SsidLength != 0) { /* not hidden ssid */
> @@ -1358,9 +1353,6 @@ int rtw_check_bcn_info(struct adapter *Adapter, u8 *pframe, u32 packet_len)
>  	else
>  		bssid->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, bssid->Privacy));
>  	if (cur_network->network.Privacy != bssid->Privacy) {
>  		DBG_871X("%s(), privacy is not match\n", __func__);
>  		goto _mismatch;
> @@ -1386,24 +1378,16 @@ int rtw_check_bcn_info(struct adapter *Adapter, u8 *pframe, u32 packet_len)
>  		pbuf = rtw_get_wpa_ie(&bssid->IEs[12], &wpa_ielen, bssid->IELength-12);
>  		if (pbuf && (wpa_ielen > 0)) {
>  			if (_SUCCESS == rtw_parse_wpa_ie(pbuf, wpa_ielen+2, &group_cipher, &pairwise_cipher, &is_8021x)) {
> -				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));
>  			}
>  		} else {
>  			pbuf = rtw_get_wpa2_ie(&bssid->IEs[12], &wpa_ielen, bssid->IELength-12);
>  
>  			if (pbuf && (wpa_ielen > 0)) {
>  				if (_SUCCESS == rtw_parse_wpa2_ie(pbuf, wpa_ielen+2, &group_cipher, &pairwise_cipher, &is_8021x)) {
> -					RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_,
> -							("%s pnetwork->pairwise_cipher: %d, pnetwork->group_cipher is %d, is_802x is %d\n",
> -							 __func__, pairwise_cipher, group_cipher, is_8021x));
>  				}
>  			}
>  		}
>  
> -		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));
>  		if (pairwise_cipher != cur_network->BcnInfo.pairwise_cipher || group_cipher != cur_network->BcnInfo.group_cipher) {
>  			DBG_871X("%s pairwise_cipher(%x:%x) or group_cipher(%x:%x) is not match\n", __func__,
>  					pairwise_cipher, cur_network->BcnInfo.pairwise_cipher,
> diff --git a/drivers/staging/rtl8723bs/core/rtw_xmit.c b/drivers/staging/rtl8723bs/core/rtw_xmit.c
> index 2daf5c461a4d..e453697ffe0c 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_xmit.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_xmit.c
> @@ -69,7 +69,6 @@ s32 _rtw_init_xmit_priv(struct xmit_priv *pxmitpriv, struct adapter *padapter)
>  
>  	if (!pxmitpriv->pallocated_frame_buf) {
>  		pxmitpriv->pxmit_frame_buf = NULL;
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("alloc xmit_frame fail!\n"));
>  		res = _FAIL;
>  		goto exit;
>  	}
> @@ -105,7 +104,6 @@ s32 _rtw_init_xmit_priv(struct xmit_priv *pxmitpriv, struct adapter *padapter)
>  	pxmitpriv->pallocated_xmitbuf = vzalloc(NR_XMITBUFF * sizeof(struct xmit_buf) + 4);
>  
>  	if (!pxmitpriv->pallocated_xmitbuf) {
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("alloc xmit_buf fail!\n"));
>  		res = _FAIL;
>  		goto exit;
>  	}
> @@ -155,7 +153,6 @@ s32 _rtw_init_xmit_priv(struct xmit_priv *pxmitpriv, struct adapter *padapter)
>  
>  	if (!pxmitpriv->xframe_ext_alloc_addr) {
>  		pxmitpriv->xframe_ext = NULL;
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("alloc xframe_ext fail!\n"));
>  		res = _FAIL;
>  		goto exit;
>  	}
> @@ -188,7 +185,6 @@ s32 _rtw_init_xmit_priv(struct xmit_priv *pxmitpriv, struct adapter *padapter)
>  	pxmitpriv->pallocated_xmit_extbuf = vzalloc(NR_XMIT_EXTBUFF * sizeof(struct xmit_buf) + 4);
>  
>  	if (!pxmitpriv->pallocated_xmit_extbuf) {
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("alloc xmit_extbuf fail!\n"));
>  		res = _FAIL;
>  		goto exit;
>  	}
> @@ -481,12 +477,9 @@ static s32 update_attrib_sec_info(struct adapter *padapter, struct pkt_attrib *p
>  	pattrib->mac_id = psta->mac_id;
>  
>  	if (psta->ieee8021x_blocked == true) {
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("\n psta->ieee8021x_blocked == true\n"));
> -
>  		pattrib->encrypt = 0;
>  
>  		if ((pattrib->ether_type != 0x888e) && (check_fwstate(pmlmepriv, WIFI_MP_STATE) == false)) {
> -			RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("\npsta->ieee8021x_blocked == true,  pattrib->ether_type(%.4x) != 0x888e\n", pattrib->ether_type));
>  			#ifdef DBG_TX_DROP_FRAME
>  			DBG_871X("DBG_TX_DROP_FRAME %s psta->ieee8021x_blocked == true,  pattrib->ether_type(%04x) != 0x888e\n", __func__, pattrib->ether_type);
>  			#endif
> @@ -568,19 +561,11 @@ static s32 update_attrib_sec_info(struct adapter *padapter, struct pkt_attrib *p
>  	if (pattrib->encrypt > 0)
>  		memcpy(pattrib->dot118021x_UncstKey.skey, psta->dot118021x_UncstKey.skey, 16);
>  
> -	RT_TRACE(_module_rtl871x_xmit_c_, _drv_info_,
> -		("update_attrib: encrypt =%d  securitypriv.sw_encrypt =%d\n",
> -		pattrib->encrypt, padapter->securitypriv.sw_encrypt));
> -
>  	if (pattrib->encrypt &&
>  		((padapter->securitypriv.sw_encrypt == true) || (psecuritypriv->hw_decrypted == false))) {
>  		pattrib->bswenc = true;
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_,
> -			("update_attrib: encrypt =%d securitypriv.hw_decrypted =%d bswenc =true\n",
> -			pattrib->encrypt, padapter->securitypriv.sw_encrypt));
>  	} else {
>  		pattrib->bswenc = false;
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_info_, ("update_attrib: bswenc =false\n"));
>  	}
>  
>  exit:
> @@ -685,7 +670,6 @@ static s32 update_attrib(struct adapter *padapter, struct sk_buff *pkt, struct p
>  					((tmp[21] == 67) && (tmp[23] == 68))) {
>  					/*  68 : UDP BOOTP client */
>  					/*  67 : UDP BOOTP server */
> -					RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("======================update_attrib: get DHCP Packet\n"));
>  					pattrib->dhcp_pkt = 1;
>  				}
>  			}
> @@ -720,7 +704,6 @@ static s32 update_attrib(struct adapter *padapter, struct sk_buff *pkt, struct p
>  	} else {
>  		psta = rtw_get_stainfo(pstapriv, pattrib->ra);
>  		if (!psta)	{ /*  if we cannot get psta => drop the pkt */
> -			RT_TRACE(_module_rtl871x_xmit_c_, _drv_alert_, ("\nupdate_attrib => get sta_info fail, ra:%pM\n", MAC_ARG(pattrib->ra)));
>  			#ifdef DBG_TX_DROP_FRAME
>  			DBG_871X("DBG_TX_DROP_FRAME %s get sta_info fail, ra:%pM\n", __func__, MAC_ARG(pattrib->ra));
>  			#endif
> @@ -734,7 +717,6 @@ static s32 update_attrib(struct adapter *padapter, struct sk_buff *pkt, struct p
>  
>  	if (!psta) {
>  		/*  if we cannot get psta => drop the pkt */
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_alert_, ("\nupdate_attrib => get sta_info fail, ra:%pM\n", MAC_ARG(pattrib->ra)));
>  		#ifdef DBG_TX_DROP_FRAME
>  		DBG_871X("DBG_TX_DROP_FRAME %s get sta_info fail, ra:%pM\n", __func__, MAC_ARG(pattrib->ra));
>  		#endif
> @@ -842,11 +824,8 @@ static s32 xmitframe_addmic(struct adapter *padapter, struct xmit_frame *pxmitfr
>  
>  			for (curfragnum = 0; curfragnum < pattrib->nr_frags; curfragnum++) {
>  				payload = (u8 *)round_up((SIZE_PTR)(payload), 4);
> -				RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("===curfragnum =%d, pframe = 0x%.2x, 0x%.2x, 0x%.2x, 0x%.2x, 0x%.2x, 0x%.2x, 0x%.2x, 0x%.2x,!!!\n",
> -					curfragnum, *payload, *(payload+1), *(payload+2), *(payload+3), *(payload+4), *(payload+5), *(payload+6), *(payload+7)));
>  
>  				payload = payload+pattrib->hdrlen+pattrib->iv_len;
> -				RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("curfragnum =%d pattrib->hdrlen =%d pattrib->iv_len =%d", curfragnum, pattrib->hdrlen, pattrib->iv_len));
>  				if ((curfragnum+1) == pattrib->nr_frags) {
>  					length = pattrib->last_txcmdsz-pattrib->hdrlen-pattrib->iv_len-((pattrib->bswenc) ? pattrib->icv_len : 0);
>  					rtw_secmicappend(&micdata, payload, length);
> @@ -855,26 +834,17 @@ static s32 xmitframe_addmic(struct adapter *padapter, struct xmit_frame *pxmitfr
>  					length = pxmitpriv->frag_len-pattrib->hdrlen-pattrib->iv_len-((pattrib->bswenc) ? pattrib->icv_len : 0);
>  					rtw_secmicappend(&micdata, payload, length);
>  					payload = payload+length+pattrib->icv_len;
> -					RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("curfragnum =%d length =%d pattrib->icv_len =%d", curfragnum, length, pattrib->icv_len));
>  				}
>  			}
>  			rtw_secgetmic(&micdata, &mic[0]);
> -			RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("xmitframe_addmic: before add mic code!!!\n"));
> -			RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("xmitframe_addmic: pattrib->last_txcmdsz =%d!!!\n", pattrib->last_txcmdsz));
> -			RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("xmitframe_addmic: mic[0]= 0x%.2x , mic[1]= 0x%.2x , mic[2]= 0x%.2x , mic[3]= 0x%.2x\n\
> -  mic[4]= 0x%.2x , mic[5]= 0x%.2x , mic[6]= 0x%.2x , mic[7]= 0x%.2x !!!!\n",
> -				mic[0], mic[1], mic[2], mic[3], mic[4], mic[5], mic[6], mic[7]));
>  			/* add mic code  and add the mic code length in last_txcmdsz */
>  
>  			memcpy(payload, &mic[0], 8);
>  			pattrib->last_txcmdsz += 8;
>  
> -			RT_TRACE(_module_rtl871x_xmit_c_, _drv_info_, ("\n ========last pkt ========\n"));
>  			payload = payload-pattrib->last_txcmdsz+8;
>  			for (curfragnum = 0; curfragnum < pattrib->last_txcmdsz; curfragnum = curfragnum+8)
> -					RT_TRACE(_module_rtl871x_xmit_c_, _drv_info_, (" %.2x,  %.2x,  %.2x,  %.2x,  %.2x,  %.2x,  %.2x,  %.2x ",
> -					*(payload+curfragnum), *(payload+curfragnum+1), *(payload+curfragnum+2), *(payload+curfragnum+3),
> -					*(payload+curfragnum+4), *(payload+curfragnum+5), *(payload+curfragnum+6), *(payload+curfragnum+7)));
> +					{}
>  			}
>  /*
>  */
> @@ -887,7 +857,6 @@ static s32 xmitframe_swencrypt(struct adapter *padapter, struct xmit_frame *pxmi
>  	struct	pkt_attrib	 *pattrib = &pxmitframe->attrib;
>  
>  	if (pattrib->bswenc) {
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_alert_, ("### xmitframe_swencrypt\n"));
>  		switch (pattrib->encrypt) {
>  		case _WEP40_:
>  		case _WEP104_:
> @@ -903,7 +872,6 @@ static s32 xmitframe_swencrypt(struct adapter *padapter, struct xmit_frame *pxmi
>  				break;
>  		}
>  	} else {
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_notice_, ("### xmitframe_hwencrypt\n"));
>  	}
>  
>  	return _SUCCESS;
> @@ -957,7 +925,6 @@ s32 rtw_make_wlanhdr(struct adapter *padapter, u8 *hdr, struct pkt_attrib *pattr
>  			if (pattrib->qos_en)
>  				qos_option = true;
>  		} else {
> -			RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("fw_state:%x is not allowed to xmit frame\n", get_fwstate(pmlmepriv)));
>  			res = _FAIL;
>  			goto exit;
>  		}
> @@ -1111,7 +1078,6 @@ s32 rtw_xmitframe_coalesce(struct adapter *padapter, struct sk_buff *pkt, struct
>  	mem_start = pbuf_start +	hw_hdr_offset;
>  
>  	if (rtw_make_wlanhdr(padapter, mem_start, pattrib) == _FAIL) {
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("rtw_xmitframe_coalesce: rtw_make_wlanhdr fail; drop pkt\n"));
>  		DBG_8192C("rtw_xmitframe_coalesce: rtw_make_wlanhdr fail; drop pkt\n");
>  		res = _FAIL;
>  		goto exit;
> @@ -1139,10 +1105,6 @@ s32 rtw_xmitframe_coalesce(struct adapter *padapter, struct sk_buff *pkt, struct
>  		if (pattrib->iv_len) {
>  			memcpy(pframe, pattrib->iv, pattrib->iv_len);
>  
> -			RT_TRACE(_module_rtl871x_xmit_c_, _drv_notice_,
> -				 ("rtw_xmitframe_coalesce: keyid =%d pattrib->iv[3]=%.2x pframe =%.2x %.2x %.2x %.2x\n",
> -				  padapter->securitypriv.dot11PrivacyKeyIndex, pattrib->iv[3], *pframe, *(pframe+1), *(pframe+2), *(pframe+3)));
> -
>  			pframe += pattrib->iv_len;
>  
>  			mpdu_len -= pattrib->iv_len;
> @@ -1183,7 +1145,6 @@ s32 rtw_xmitframe_coalesce(struct adapter *padapter, struct sk_buff *pkt, struct
>  
>  			break;
>  		} else {
> -			RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("%s: There're still something in packet!\n", __func__));
>  		}
>  
>  		addr = (SIZE_PTR)(pframe);
> @@ -1193,7 +1154,6 @@ s32 rtw_xmitframe_coalesce(struct adapter *padapter, struct sk_buff *pkt, struct
>  	}
>  
>  	if (xmitframe_addmic(padapter, pxmitframe) == _FAIL) {
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("xmitframe_addmic(padapter, pxmitframe) == _FAIL\n"));
>  		DBG_8192C("xmitframe_addmic(padapter, pxmitframe) == _FAIL\n");
>  		res = _FAIL;
>  		goto exit;
> @@ -1718,7 +1678,6 @@ struct xmit_frame *rtw_alloc_xmitframe(struct xmit_priv *pxmitpriv)/* _queue *pf
>  	spin_lock_bh(&pfree_xmit_queue->lock);
>  
>  	if (list_empty(&pfree_xmit_queue->queue)) {
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_info_, ("rtw_alloc_xmitframe:%d\n", pxmitpriv->free_xmitframe_cnt));
>  		pxframe =  NULL;
>  	} else {
>  		phead = get_list_head(pfree_xmit_queue);
> @@ -1729,7 +1688,6 @@ struct xmit_frame *rtw_alloc_xmitframe(struct xmit_priv *pxmitpriv)/* _queue *pf
>  
>  		list_del_init(&pxframe->list);
>  		pxmitpriv->free_xmitframe_cnt--;
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_info_, ("rtw_alloc_xmitframe():free_xmitframe_cnt =%d\n", pxmitpriv->free_xmitframe_cnt));
>  	}
>  
>  	spin_unlock_bh(&pfree_xmit_queue->lock);
> @@ -1747,7 +1705,6 @@ struct xmit_frame *rtw_alloc_xmitframe_ext(struct xmit_priv *pxmitpriv)
>  	spin_lock_bh(&queue->lock);
>  
>  	if (list_empty(&queue->queue)) {
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_info_, ("rtw_alloc_xmitframe_ext:%d\n", pxmitpriv->free_xframe_ext_cnt));
>  		pxframe =  NULL;
>  	} else {
>  		phead = get_list_head(queue);
> @@ -1756,7 +1713,6 @@ struct xmit_frame *rtw_alloc_xmitframe_ext(struct xmit_priv *pxmitpriv)
>  
>  		list_del_init(&pxframe->list);
>  		pxmitpriv->free_xframe_ext_cnt--;
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_info_, ("rtw_alloc_xmitframe_ext():free_xmitframe_cnt =%d\n", pxmitpriv->free_xframe_ext_cnt));
>  	}
>  
>  	spin_unlock_bh(&queue->lock);
> @@ -1802,7 +1758,6 @@ s32 rtw_free_xmitframe(struct xmit_priv *pxmitpriv, struct xmit_frame *pxmitfram
>  	struct sk_buff *pndis_pkt = NULL;
>  
>  	if (!pxmitframe) {
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("======rtw_free_xmitframe():pxmitframe == NULL!!!!!!!!!!\n"));
>  		goto exit;
>  	}
>  
> @@ -1830,10 +1785,8 @@ s32 rtw_free_xmitframe(struct xmit_priv *pxmitpriv, struct xmit_frame *pxmitfram
>  	list_add_tail(&pxmitframe->list, get_list_head(queue));
>  	if (pxmitframe->ext_tag == 0) {
>  		pxmitpriv->free_xmitframe_cnt++;
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_debug_, ("rtw_free_xmitframe():free_xmitframe_cnt =%d\n", pxmitpriv->free_xmitframe_cnt));
>  	} else if (pxmitframe->ext_tag == 1) {
>  		pxmitpriv->free_xframe_ext_cnt++;
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_debug_, ("rtw_free_xmitframe():free_xframe_ext_cnt =%d\n", pxmitpriv->free_xframe_ext_cnt));
>  	} else {
>  	}
>  
> @@ -1871,8 +1824,6 @@ void rtw_free_xmitframe_queue(struct xmit_priv *pxmitpriv, struct __queue *pfram
>  s32 rtw_xmitframe_enqueue(struct adapter *padapter, struct xmit_frame *pxmitframe)
>  {
>  	if (rtw_xmit_classifier(padapter, pxmitframe) == _FAIL) {
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_,
> -			 ("rtw_xmitframe_enqueue: drop xmit pkt for classifier fail\n"));
>  		return _FAIL;
>  	}
>  
> @@ -1888,21 +1839,18 @@ struct tx_servq *rtw_get_sta_pending(struct adapter *padapter, struct sta_info *
>  	case 2:
>  		ptxservq = &psta->sta_xmitpriv.bk_q;
>  		*(ac) = 3;
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_info_, ("rtw_get_sta_pending : BK\n"));
>  		break;
>  
>  	case 4:
>  	case 5:
>  		ptxservq = &psta->sta_xmitpriv.vi_q;
>  		*(ac) = 1;
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_info_, ("rtw_get_sta_pending : VI\n"));
>  		break;
>  
>  	case 6:
>  	case 7:
>  		ptxservq = &psta->sta_xmitpriv.vo_q;
>  		*(ac) = 0;
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_info_, ("rtw_get_sta_pending : VO\n"));
>  		break;
>  
>  	case 0:
> @@ -1910,7 +1858,6 @@ struct tx_servq *rtw_get_sta_pending(struct adapter *padapter, struct sta_info *
>  	default:
>  		ptxservq = &psta->sta_xmitpriv.be_q;
>  		*(ac) = 2;
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_info_, ("rtw_get_sta_pending : BE\n"));
>  	break;
>  	}
>  
> @@ -1939,7 +1886,6 @@ s32 rtw_xmit_classifier(struct adapter *padapter, struct xmit_frame *pxmitframe)
>  	if (!psta) {
>  		res = _FAIL;
>  		DBG_8192C("rtw_xmit_classifier: psta == NULL\n");
> -		RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_, ("rtw_xmit_classifier: psta == NULL\n"));
>  		goto exit;
>  	}
>  
> @@ -2059,7 +2005,6 @@ static void do_queue_select(struct adapter	*padapter, struct pkt_attrib *pattrib
>  	u8 qsel;
>  
>  	qsel = pattrib->priority;
> -	RT_TRACE(_module_rtl871x_xmit_c_, _drv_info_, ("### do_queue_select priority =%d , qsel = %d\n", pattrib->priority, qsel));
>  
>  	pattrib->qsel = qsel;
>  }
> @@ -2096,14 +2041,12 @@ s32 rtw_xmit(struct adapter *padapter, struct sk_buff **ppkt)
>  
>  	if (!pxmitframe) {
>  		drop_cnt++;
> -		RT_TRACE(_module_xmit_osdep_c_, _drv_err_, ("%s: no more pxmitframe\n", __func__));
>  		return -1;
>  	}
>  
>  	res = update_attrib(padapter, *ppkt, &pxmitframe->attrib);
>  
>  	if (res == _FAIL) {
> -		RT_TRACE(_module_xmit_osdep_c_, _drv_err_, ("%s: update attrib fail\n", __func__));
>  		#ifdef DBG_TX_DROP_FRAME
>  		DBG_871X("DBG_TX_DROP_FRAME %s update attrib fail\n", __func__);
>  		#endif
> diff --git a/drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c b/drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c
> index 932b31fda6ad..b1f271c65c3e 100644
> --- a/drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c
> +++ b/drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c
> @@ -53,22 +53,6 @@ u8 HalPwrSeqCmdParsing(
>  	do {
>  		PwrCfgCmd = PwrSeqCmd[AryIdx];
>  
> -		RT_TRACE(
> -			_module_hal_init_c_,
> -			_drv_info_,
> -			(
> -				"HalPwrSeqCmdParsing: 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 */
>  		if (
>  			(GET_PWR_CFG_FAB_MASK(PwrCfgCmd) & FabVersion) &&
> @@ -77,19 +61,9 @@ u8 HalPwrSeqCmdParsing(
>  		) {
>  			switch (GET_PWR_CFG_CMD(PwrCfgCmd)) {
>  			case PWR_CMD_READ:
> -				RT_TRACE(
> -					_module_hal_init_c_,
> -					_drv_info_,
> -					("HalPwrSeqCmdParsing: PWR_CMD_READ\n")
> -				);
>  				break;
>  
>  			case PWR_CMD_WRITE:
> -				RT_TRACE(
> -					_module_hal_init_c_,
> -					_drv_info_,
> -					("HalPwrSeqCmdParsing: PWR_CMD_WRITE\n")
> -				);
>  				offset = GET_PWR_CFG_OFFSET(PwrCfgCmd);
>  
>  				/*  */
> @@ -124,11 +98,6 @@ u8 HalPwrSeqCmdParsing(
>  				break;
>  
>  			case PWR_CMD_POLLING:
> -				RT_TRACE(
> -					_module_hal_init_c_,
> -					_drv_info_,
> -					("HalPwrSeqCmdParsing: PWR_CMD_POLLING\n")
> -				);
>  
>  				bPollingBit = false;
>  				offset = GET_PWR_CFG_OFFSET(PwrCfgCmd);
> @@ -160,11 +129,6 @@ u8 HalPwrSeqCmdParsing(
>  				break;
>  
>  			case PWR_CMD_DELAY:
> -				RT_TRACE(
> -					_module_hal_init_c_,
> -					_drv_info_,
> -					("HalPwrSeqCmdParsing: PWR_CMD_DELAY\n")
> -				);
>  				if (GET_PWR_CFG_VALUE(PwrCfgCmd) == PWRSEQ_DELAY_US)
>  					udelay(GET_PWR_CFG_OFFSET(PwrCfgCmd));
>  				else
> @@ -173,19 +137,9 @@ u8 HalPwrSeqCmdParsing(
>  
>  			case PWR_CMD_END:
>  				/*  When this command is parsed, end the process */
> -				RT_TRACE(
> -					_module_hal_init_c_,
> -					_drv_info_,
> -					("HalPwrSeqCmdParsing: PWR_CMD_END\n")
> -				);
>  				return true;
>  
>  			default:
> -				RT_TRACE(
> -					_module_hal_init_c_,
> -					_drv_err_,
> -					("HalPwrSeqCmdParsing: Unknown CMD!!\n")
> -				);
>  				break;
>  			}
>  		}
> diff --git a/drivers/staging/rtl8723bs/hal/hal_intf.c b/drivers/staging/rtl8723bs/hal/hal_intf.c
> index aa4356327636..7fbac4c8f2f8 100644
> --- a/drivers/staging/rtl8723bs/hal/hal_intf.c
> +++ b/drivers/staging/rtl8723bs/hal/hal_intf.c
> @@ -108,8 +108,6 @@ uint rtw_hal_init(struct adapter *padapter)
>  		DBG_871X("rtw_hal_init: hal__init fail\n");
>  	}
>  
> -	RT_TRACE(_module_hal_init_c_, _drv_err_, ("-rtl871x_hal_init:status = 0x%x\n", status));
> -
>  	return status;
>  }
>  
> diff --git a/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c b/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c
> index 0081fe0a431f..fd5ab45e566e 100644
> --- a/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c
> +++ b/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c
> @@ -65,17 +65,6 @@ static int _BlockWrite(struct adapter *padapter, void *buffer, u32 buffSize)
>  	remainSize_p1 = buffSize % blockSize_p1;
>  
>  	if (blockCount_p1) {
> -		RT_TRACE(
> -			_module_hal_init_c_,
> -			_drv_notice_,
> -			(
> -				"_BlockWrite: [P1] buffSize(%d) blockSize_p1(%d) blockCount_p1(%d) remainSize_p1(%d)\n",
> -				buffSize,
> -				blockSize_p1,
> -				blockCount_p1,
> -				remainSize_p1
> -			)
> -		);
>  	}
>  
>  	for (i = 0; i < blockCount_p1; i++) {
> @@ -94,17 +83,6 @@ static int _BlockWrite(struct adapter *padapter, void *buffer, u32 buffSize)
>  		remainSize_p2 = remainSize_p1%blockSize_p2;
>  
>  		if (blockCount_p2) {
> -				RT_TRACE(
> -					_module_hal_init_c_,
> -					_drv_notice_,
> -					(
> -						"_BlockWrite: [P2] buffSize_p2(%d) blockSize_p2(%d) blockCount_p2(%d) remainSize_p2(%d)\n",
> -						(buffSize-offset),
> -						blockSize_p2,
> -						blockCount_p2,
> -						remainSize_p2
> -					)
> -				);
>  		}
>  
>  	}
> @@ -115,10 +93,6 @@ static int _BlockWrite(struct adapter *padapter, void *buffer, u32 buffSize)
>  
>  		blockCount_p3 = remainSize_p2 / blockSize_p3;
>  
> -		RT_TRACE(_module_hal_init_c_, _drv_notice_,
> -				("_BlockWrite: [P3] buffSize_p3(%d) blockSize_p3(%d) blockCount_p3(%d)\n",
> -				(buffSize-offset), blockSize_p3, blockCount_p3));
> -
>  		for (i = 0; i < blockCount_p3; i++) {
>  			ret = rtw_write8(padapter, (FW_8723B_START_ADDRESS + offset + i), *(bufferPtr + offset + i));
>  
> @@ -181,7 +155,6 @@ static int _WriteFW(struct adapter *padapter, void *buffer, u32 size)
>  			goto exit;
>  		}
>  	}
> -	RT_TRACE(_module_hal_init_c_, _drv_info_, ("_WriteFW Done- for Normal chip.\n"));
>  
>  exit:
>  	return ret;
> @@ -334,10 +307,8 @@ void rtl8723b_FirmwareSelfReset(struct adapter *padapter)
>  			udelay(50);
>  			u1bTmp = rtw_read8(padapter, REG_SYS_FUNC_EN+1);
>  		}
> -		RT_TRACE(_module_hal_init_c_, _drv_notice_, ("-%s: 8051 reset success (%d)\n", __func__, Delay));
>  
>  		if (Delay == 0) {
> -			RT_TRACE(_module_hal_init_c_, _drv_notice_, ("%s: Force 8051 reset!!!\n", __func__));
>  			/* force firmware reset */
>  			u1bTmp = rtw_read8(padapter, REG_SYS_FUNC_EN+1);
>  			rtw_write8(padapter, REG_SYS_FUNC_EN+1, u1bTmp&(~BIT2));
> @@ -368,7 +339,6 @@ s32 rtl8723b_FirmwareDownload(struct adapter *padapter, bool  bUsedWoWLANFw)
>  	struct debug_priv *pdbgpriv = &psdpriv->drv_dbg;
>  	u8 tmp_ps;
>  
> -	RT_TRACE(_module_hal_init_c_, _drv_info_, ("+%s\n", __func__));
>  	pFirmware = kzalloc(sizeof(struct rt_firmware), GFP_KERNEL);
>  	if (!pFirmware)
>  		return _FAIL;
> @@ -409,11 +379,6 @@ s32 rtl8723b_FirmwareDownload(struct adapter *padapter, bool  bUsedWoWLANFw)
>  
>  	if (fw->size > FW_8723B_SIZE) {
>  		rtStatus = _FAIL;
> -		RT_TRACE(
> -			_module_hal_init_c_,
> -			_drv_err_,
> -			("Firmware size exceed 0x%X. Check it.\n", FW_8188E_SIZE)
> -		);
>  		goto exit;
>  	}
>  
> @@ -1948,12 +1913,6 @@ static void ResumeTxBeacon(struct adapter *padapter)
>  {
>  	struct hal_com_data *pHalData = GET_HAL_DATA(padapter);
>  
> -
> -	/*  2010.03.01. Marked by tynli. No need to call workitem beacause we record the value */
> -	/*  which should be read from register to a global variable. */
> -
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("+ResumeTxBeacon\n"));
> -
>  	pHalData->RegFwHwTxQCtrl |= BIT(6);
>  	rtw_write8(padapter, REG_FWHW_TXQ_CTRL+2, pHalData->RegFwHwTxQCtrl);
>  	rtw_write8(padapter, REG_TBTT_PROHIBIT+1, 0xff);
> @@ -1965,12 +1924,6 @@ static void StopTxBeacon(struct adapter *padapter)
>  {
>  	struct hal_com_data *pHalData = GET_HAL_DATA(padapter);
>  
> -
> -	/*  2010.03.01. Marked by tynli. No need to call workitem beacause we record the value */
> -	/*  which should be read from register to a global variable. */
> -
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("+StopTxBeacon\n"));
> -
>  	pHalData->RegFwHwTxQCtrl &= ~BIT(6);
>  	rtw_write8(padapter, REG_FWHW_TXQ_CTRL+2, pHalData->RegFwHwTxQCtrl);
>  	rtw_write8(padapter, REG_TBTT_PROHIBIT+1, 0x64);
> @@ -2314,7 +2267,6 @@ static bool Hal_GetChnlGroup8723B(u8 Channel, u8 *pGroup)
>  		else if (12 <= Channel && Channel <= 14)
>  			*pGroup = 4;
>  		else {
> -			RT_TRACE(_module_hci_hal_init_c_, _drv_notice_, ("==>Hal_GetChnlGroup8723B in 2.4 G, but Channel %d in Group not found\n", Channel));
>  		}
>  	} else {
>  		bIn24G = false;
> @@ -2348,20 +2300,9 @@ static bool Hal_GetChnlGroup8723B(u8 Channel, u8 *pGroup)
>  		else if (173  <= Channel && Channel <= 177)
>  			*pGroup = 13;
>  		else {
> -			RT_TRACE(_module_hci_hal_init_c_, _drv_notice_, ("==>Hal_GetChnlGroup8723B in 5G, but Channel %d in Group not found\n", Channel));
>  		}
>  
>  	}
> -	RT_TRACE(
> -		_module_hci_hal_init_c_,
> -		_drv_info_,
> -		(
> -			"<==Hal_GetChnlGroup8723B,  (%s) Channel = %d, Group =%d,\n",
> -			bIn24G ? "2.4G" : "5G",
> -			Channel,
> -			*pGroup
> -		)
> -	);
>  	return bIn24G;
>  }
>  
> @@ -2376,7 +2317,6 @@ void Hal_InitPGData(struct adapter *padapter, u8 *PROMContent)
>  			memcpy((void *)PROMContent, (void *)pEEPROM->efuse_eeprom_data, HWSET_MAX_SIZE_8723B);
>  		}
>  	} else {/* autoload fail */
> -		RT_TRACE(_module_hci_hal_init_c_, _drv_notice_, ("AutoLoad Fail reported from CR9346!!\n"));
>  		if (!pEEPROM->EepromOrEfuse)
>  			EFUSE_ShadowMapUpdate(padapter, EFUSE_WIFI, false);
>  		memcpy((void *)PROMContent, (void *)pEEPROM->efuse_eeprom_data, HWSET_MAX_SIZE_8723B);
> @@ -2397,8 +2337,6 @@ void Hal_EfuseParseIDCode(struct adapter *padapter, u8 *hwinfo)
>  		pEEPROM->bautoload_fail_flag = true;
>  	} else
>  		pEEPROM->bautoload_fail_flag = false;
> -
> -	RT_TRACE(_module_hal_init_c_, _drv_notice_, ("EEPROM ID = 0x%04x\n", EEPROMId));
>  }
>  
>  static void Hal_ReadPowerValueFromPROM_8723B(
> @@ -2540,9 +2478,6 @@ void Hal_EfuseParseTxPowerInfo_8723B(
>  				pHalData->Index24G_BW40_Base[rfPath][ch] = pwrInfo24G.IndexBW40_Base[rfPath][group];
>  			}
>  #ifdef DEBUG
> -			RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("======= Path %d, ChannelIndex %d, Group %d =======\n", rfPath, ch, group));
> -			RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("Index24G_CCK_Base[%d][%d] = 0x%x\n", rfPath, ch, pHalData->Index24G_CCK_Base[rfPath][ch]));
> -			RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("Index24G_BW40_Base[%d][%d] = 0x%x\n", rfPath, ch, pHalData->Index24G_BW40_Base[rfPath][ch]));
>  #endif
>  		}
>  
> @@ -2553,11 +2488,6 @@ void Hal_EfuseParseTxPowerInfo_8723B(
>  			pHalData->BW40_24G_Diff[rfPath][TxCount] = pwrInfo24G.BW40_Diff[rfPath][TxCount];
>  
>  #ifdef DEBUG
> -			RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("--------------------------------------- 2.4G ---------------------------------------\n"));
> -			RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("CCK_24G_Diff[%d][%d]= %d\n", rfPath, TxCount, pHalData->CCK_24G_Diff[rfPath][TxCount]));
> -			RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("OFDM_24G_Diff[%d][%d]= %d\n", rfPath, TxCount, pHalData->OFDM_24G_Diff[rfPath][TxCount]));
> -			RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("BW20_24G_Diff[%d][%d]= %d\n", rfPath, TxCount, pHalData->BW20_24G_Diff[rfPath][TxCount]));
> -			RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("BW40_24G_Diff[%d][%d]= %d\n", rfPath, TxCount, pHalData->BW40_24G_Diff[rfPath][TxCount]));
>  #endif
>  		}
>  	}
> @@ -2569,8 +2499,6 @@ void Hal_EfuseParseTxPowerInfo_8723B(
>  			pHalData->EEPROMRegulatory = (EEPROM_DEFAULT_BOARD_OPTION&0x7);	/* bit0~2 */
>  	} else
>  		pHalData->EEPROMRegulatory = 0;
> -
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_notice_, ("EEPROMRegulatory = 0x%x\n", pHalData->EEPROMRegulatory));
>  }
>  
>  void Hal_EfuseParseBTCoexistInfo_8723B(
> @@ -2660,8 +2588,6 @@ void Hal_EfuseParseEEPROMVer_8723B(
>  		pHalData->EEPROMVersion = hwinfo[EEPROM_VERSION_8723B];
>  	else
>  		pHalData->EEPROMVersion = 1;
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_notice_, ("Hal_EfuseParseEEPROMVer(), EEVer = %d\n",
> -		pHalData->EEPROMVersion));
>  }
>  
>  
> @@ -2728,8 +2654,6 @@ void Hal_EfuseParseChnlPlan_8723B(
>  	);
>  
>  	Hal_ChannelPlanToRegulation(padapter, padapter->mlmepriv.ChannelPlan);
> -
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_notice_, ("EEPROM ChannelPlan = 0x%02x\n", padapter->mlmepriv.ChannelPlan));
>  }
>  
>  void Hal_EfuseParseCustomerID_8723B(
> @@ -2743,8 +2667,6 @@ void Hal_EfuseParseCustomerID_8723B(
>  		pHalData->EEPROMCustomerID = hwinfo[EEPROM_CustomID_8723B];
>  	else
>  		pHalData->EEPROMCustomerID = 0;
> -
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_notice_, ("EEPROM Customer ID: 0x%2x\n", pHalData->EEPROMCustomerID));
>  }
>  
>  void Hal_EfuseParseAntennaDiversity_8723B(
> @@ -2768,8 +2690,6 @@ void Hal_EfuseParseXtal_8723B(
>  			pHalData->CrystalCap = EEPROM_Default_CrystalCap_8723B;	   /* what value should 8812 set? */
>  	} else
>  		pHalData->CrystalCap = EEPROM_Default_CrystalCap_8723B;
> -
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_notice_, ("EEPROM CrystalCap: 0x%2x\n", pHalData->CrystalCap));
>  }
>  
>  
> @@ -2792,8 +2712,6 @@ void Hal_EfuseParseThermalMeter_8723B(
>  		pHalData->bAPKThermalMeterIgnore = true;
>  		pHalData->EEPROMThermalMeter = EEPROM_Default_ThermalMeter_8723B;
>  	}
> -
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_notice_, ("EEPROM ThermalMeter = 0x%x\n", pHalData->EEPROMThermalMeter));
>  }
>  
>  
> @@ -3111,10 +3029,7 @@ static void rtl8723b_fill_default_txdesc(
>  			ptxdesc->sw_define = (u8)(GET_PRIMARY_ADAPTER(padapter)->xmitpriv.seq_no);
>  		}
>  	} else if (pxmitframe->frame_tag == TXAGG_FRAMETAG) {
> -		RT_TRACE(_module_hal_xmit_c_, _drv_warning_, ("%s: TXAGG_FRAMETAG\n", __func__));
>  	} else {
> -		RT_TRACE(_module_hal_xmit_c_, _drv_warning_, ("%s: frame_tag = 0x%x\n", __func__, pxmitframe->frame_tag));
> -
>  		ptxdesc->macid = pattrib->mac_id; /*  CAM_ID(MAC_ID) */
>  		ptxdesc->rate_id = pattrib->raid; /*  Rate ID */
>  		ptxdesc->qsel = pattrib->qsel;
> @@ -3588,7 +3503,6 @@ s32 c2h_handler_8723b(struct adapter *padapter, u8 *buf)
>  		break;
>  	case C2H_DBG:
>  		{
> -			RT_TRACE(_module_hal_init_c_, _drv_info_, ("c2h_handler_8723b: %s\n", pC2hEvent->payload));
>  		}
>  		break;
>  
> @@ -3601,9 +3515,7 @@ s32 c2h_handler_8723b(struct adapter *padapter, u8 *buf)
>  		break;
>  
>  	case C2H_HW_INFO_EXCH:
> -		RT_TRACE(_module_hal_init_c_, _drv_info_, ("[BT], C2H_HW_INFO_EXCH\n"));
>  		for (index = 0; index < pC2hEvent->plen; index++) {
> -			RT_TRACE(_module_hal_init_c_, _drv_info_, ("[BT], tmpBuf[%d]= 0x%x\n", index, pC2hEvent->payload[index]));
>  		}
>  		break;
>  
> @@ -3637,7 +3549,6 @@ static void process_c2h_event(struct adapter *padapter, struct c2h_evt_hdr_t *pC
>  		break;
>  	case C2H_DBG:
>  		{
> -			RT_TRACE(_module_hal_init_c_, _drv_info_, ("C2HCommandHandler: %s\n", c2hBuf));
>  		}
>  		break;
>  
> @@ -3650,9 +3561,7 @@ static void process_c2h_event(struct adapter *padapter, struct c2h_evt_hdr_t *pC
>  		break;
>  
>  	case C2H_HW_INFO_EXCH:
> -		RT_TRACE(_module_hal_init_c_, _drv_info_, ("[BT], C2H_HW_INFO_EXCH\n"));
>  		for (index = 0; index < pC2hEvent->CmdLen; index++) {
> -			RT_TRACE(_module_hal_init_c_, _drv_info_, ("[BT], tmpBuf[%d]= 0x%x\n", index, c2hBuf[index]));
>  		}
>  		break;
>  
> @@ -4054,7 +3963,6 @@ void SetHwReg8723B(struct adapter *padapter, u8 variable, u8 *val)
>  			u32 usNavUpper = *((u32 *)val);
>  
>  			if (usNavUpper > HAL_NAV_UPPER_UNIT_8723B * 0xFF) {
> -				RT_TRACE(_module_hal_init_c_, _drv_notice_, ("The setting value (0x%08X us) of NAV_UPPER is larger than (%d * 0xFF)!!!\n", usNavUpper, HAL_NAV_UPPER_UNIT_8723B));
>  				break;
>  			}
>  
> @@ -4358,7 +4266,6 @@ void rtl8723b_start_thread(struct adapter *padapter)
>  
>  	xmitpriv->SdioXmitThread = kthread_run(rtl8723bs_xmit_thread, padapter, "RTWHALXT");
>  	if (IS_ERR(xmitpriv->SdioXmitThread)) {
> -		RT_TRACE(_module_hal_xmit_c_, _drv_err_, ("%s: start rtl8723bs_xmit_thread FAIL!!\n", __func__));
>  	}
>  }
>  
> diff --git a/drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c b/drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c
> index 2abff4673be2..e5f0e48db557 100644
> --- a/drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c
> +++ b/drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c
> @@ -551,7 +551,6 @@ void PHY_SetTxPowerIndex(
>  			break;
>  		}
>  	} else {
> -		RT_TRACE(_module_hal_init_c_, _drv_err_, ("Invalid RFPath!!\n"));
>  	}
>  }
>  
> @@ -604,11 +603,7 @@ void PHY_SetTxPowerLevel8723B(struct adapter *Adapter, u8 Channel)
>  		RFPath = pHalData->ant_path;
>  	}
>  
> -	RT_TRACE(_module_hal_init_c_, _drv_info_, ("==>PHY_SetTxPowerLevel8723B()\n"));
> -
>  	PHY_SetTxPowerLevelByPath(Adapter, Channel, RFPath);
> -
> -	RT_TRACE(_module_hal_init_c_, _drv_info_, ("<==PHY_SetTxPowerLevel8723B()\n"));
>  }
>  
>  void PHY_GetTxPowerLevel8723B(struct adapter *Adapter, s32 *powerlevel)
> @@ -648,23 +643,13 @@ static u8 phy_GetSecondaryChnl_8723B(struct adapter *Adapter)
>  	u8 SCSettingOf40 = 0, SCSettingOf20 = 0;
>  	struct hal_com_data *pHalData = GET_HAL_DATA(Adapter);
>  
> -	RT_TRACE(
> -		_module_hal_init_c_,
> -		_drv_info_,
> -		(
> -			"SCMapping: VHT Case: pHalData->CurrentChannelBW %d, pHalData->nCur80MhzPrimeSC %d, pHalData->nCur40MhzPrimeSC %d\n",
> -			pHalData->CurrentChannelBW,
> -			pHalData->nCur80MhzPrimeSC,
> -			pHalData->nCur40MhzPrimeSC
> -		)
> -	);
>  	if (pHalData->CurrentChannelBW == CHANNEL_WIDTH_80) {
>  		if (pHalData->nCur80MhzPrimeSC == HAL_PRIME_CHNL_OFFSET_LOWER)
>  			SCSettingOf40 = VHT_DATA_SC_40_LOWER_OF_80MHZ;
>  		else if (pHalData->nCur80MhzPrimeSC == HAL_PRIME_CHNL_OFFSET_UPPER)
>  			SCSettingOf40 = VHT_DATA_SC_40_UPPER_OF_80MHZ;
>  		else
> -			RT_TRACE(_module_hal_init_c_, _drv_err_, ("SCMapping: Not Correct Primary40MHz Setting\n"));
> +			{}
>  
>  		if (
>  			(pHalData->nCur40MhzPrimeSC == HAL_PRIME_CHNL_OFFSET_LOWER) &&
> @@ -687,27 +672,16 @@ static u8 phy_GetSecondaryChnl_8723B(struct adapter *Adapter)
>  		)
>  			SCSettingOf20 = VHT_DATA_SC_20_UPPERST_OF_80MHZ;
>  		else
> -			RT_TRACE(_module_hal_init_c_, _drv_err_, ("SCMapping: Not Correct Primary40MHz Setting\n"));
> +			{}
>  	} else if (pHalData->CurrentChannelBW == CHANNEL_WIDTH_40) {
> -		RT_TRACE(
> -			_module_hal_init_c_,
> -			_drv_info_,
> -			(
> -				"SCMapping: VHT Case: pHalData->CurrentChannelBW %d, pHalData->nCur40MhzPrimeSC %d\n",
> -				pHalData->CurrentChannelBW,
> -				pHalData->nCur40MhzPrimeSC
> -			)
> -		);
> -
>  		if (pHalData->nCur40MhzPrimeSC == HAL_PRIME_CHNL_OFFSET_UPPER)
>  			SCSettingOf20 = VHT_DATA_SC_20_UPPER_OF_80MHZ;
>  		else if (pHalData->nCur40MhzPrimeSC == HAL_PRIME_CHNL_OFFSET_LOWER)
>  			SCSettingOf20 = VHT_DATA_SC_20_LOWER_OF_80MHZ;
>  		else
> -			RT_TRACE(_module_hal_init_c_, _drv_err_, ("SCMapping: Not Correct Primary40MHz Setting\n"));
> +			{}
>  	}
>  
> -	RT_TRACE(_module_hal_init_c_, _drv_info_, ("SCMapping: SC Value %x\n", ((SCSettingOf40 << 4) | SCSettingOf20)));
>  	return  (SCSettingOf40 << 4) | SCSettingOf20;
>  }
>  
> diff --git a/drivers/staging/rtl8723bs/hal/rtl8723bs_recv.c b/drivers/staging/rtl8723bs/hal/rtl8723bs_recv.c
> index 17a3e7bf66ef..d27d56920a11 100644
> --- a/drivers/staging/rtl8723bs/hal/rtl8723bs_recv.c
> +++ b/drivers/staging/rtl8723bs/hal/rtl8723bs_recv.c
> @@ -346,7 +346,6 @@ static void rtl8723bs_recv_tasklet(struct tasklet_struct *t)
>  						update_recvframe_phyinfo(precvframe, (struct phy_stat *)ptr);
>  
>  					if (rtw_recv_entry(precvframe) != _SUCCESS) {
> -						RT_TRACE(_module_rtl871x_recv_c_, _drv_dump_, ("%s: rtw_recv_entry(precvframe) != _SUCCESS\n", __func__));
>  					}
>  				} else if (pattrib->pkt_rpt_type == C2H_PACKET) {
>  					struct c2h_evt_hdr_t	C2hEvent;
> @@ -404,7 +403,6 @@ s32 rtl8723bs_init_recv_priv(struct adapter *padapter)
>  	precvpriv->pallocated_recv_buf = rtw_zmalloc(n);
>  	if (!precvpriv->pallocated_recv_buf) {
>  		res = _FAIL;
> -		RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("alloc recv_buf fail!\n"));
>  		goto exit;
>  	}
>  
> diff --git a/drivers/staging/rtl8723bs/hal/rtl8723bs_xmit.c b/drivers/staging/rtl8723bs/hal/rtl8723bs_xmit.c
> index 597cf3a88c51..c18587cc78a5 100644
> --- a/drivers/staging/rtl8723bs/hal/rtl8723bs_xmit.c
> +++ b/drivers/staging/rtl8723bs/hal/rtl8723bs_xmit.c
> @@ -102,11 +102,6 @@ static s32 rtl8723_dequeue_writeport(struct adapter *padapter)
>  		(padapter->bSurpriseRemoved) ||
>  		(padapter->bDriverStopped)
>  	) {
> -		RT_TRACE(
> -			_module_hal_xmit_c_,
> -			_drv_notice_,
> -			("%s: bSurpriseRemoved(write port)\n", __func__)
> -		);
>  		goto free_xmitbuf;
>  	}
>  
> @@ -151,16 +146,6 @@ s32 rtl8723bs_xmit_buf_handler(struct adapter *padapter)
>  
>  	ret = (padapter->bDriverStopped) || (padapter->bSurpriseRemoved);
>  	if (ret) {
> -		RT_TRACE(
> -			_module_hal_xmit_c_,
> -			_drv_err_,
> -			(
> -				"%s: bDriverStopped(%d) bSurpriseRemoved(%d)!\n",
> -				__func__,
> -				padapter->bDriverStopped,
> -				padapter->bSurpriseRemoved
> -			)
> -		);
>  		return _FAIL;
>  	}
>  
> @@ -374,8 +359,6 @@ static s32 xmit_xmitframes(struct adapter *padapter, struct xmit_priv *pxmitpriv
>  
>  		/*  dump xmit_buf to hw tx fifo */
>  		if (pxmitbuf) {
> -			RT_TRACE(_module_hal_xmit_c_, _drv_info_, ("pxmitbuf->len =%d enqueue\n", pxmitbuf->len));
> -
>  			if (pxmitbuf->len > 0) {
>  				struct xmit_frame *pframe;
>  				pframe = (struct xmit_frame *)pxmitbuf->priv_data;
> @@ -424,16 +407,6 @@ static s32 rtl8723bs_xmit_handler(struct adapter *padapter)
>  		(padapter->bDriverStopped) ||
>  		(padapter->bSurpriseRemoved)
>  	) {
> -		RT_TRACE(
> -			_module_hal_xmit_c_,
> -			_drv_notice_,
> -			(
> -				"%s: bDriverStopped(%d) bSurpriseRemoved(%d)\n",
> -				__func__,
> -				padapter->bDriverStopped,
> -				padapter->bSurpriseRemoved
> -			)
> -		);
>  		return _FAIL;
>  	}
>  
> @@ -492,8 +465,6 @@ int rtl8723bs_xmit_thread(void *context)
>  
>  	complete(&pxmitpriv->SdioXmitTerminate);
>  
> -	RT_TRACE(_module_hal_xmit_c_, _drv_notice_, ("-%s\n", __func__));
> -
>  	thread_exit();
>  }
>  
> @@ -509,8 +480,6 @@ s32 rtl8723bs_mgnt_xmit(
>  	u8 *pframe = (u8 *)(pmgntframe->buf_addr) + TXDESC_OFFSET;
>  	u8 txdesc_size = TXDESC_SIZE;
>  
> -	RT_TRACE(_module_hal_xmit_c_, _drv_info_, ("+%s\n", __func__));
> -
>  	pattrib = &pmgntframe->attrib;
>  	pxmitbuf = pmgntframe->pxmitbuf;
>  
> @@ -572,7 +541,6 @@ s32 rtl8723bs_hal_xmit(
>  	err = rtw_xmitframe_enqueue(padapter, pxmitframe);
>  	spin_unlock_bh(&pxmitpriv->lock);
>  	if (err != _SUCCESS) {
> -		RT_TRACE(_module_hal_xmit_c_, _drv_err_, ("rtl8723bs_hal_xmit: enqueue xmitframe fail\n"));
>  		rtw_free_xmitframe(pxmitpriv, pxmitframe);
>  
>  		pxmitpriv->tx_drop++;
> diff --git a/drivers/staging/rtl8723bs/hal/sdio_halinit.c b/drivers/staging/rtl8723bs/hal/sdio_halinit.c
> index b1c4bbf29790..faedeeffcead 100644
> --- a/drivers/staging/rtl8723bs/hal/sdio_halinit.c
> +++ b/drivers/staging/rtl8723bs/hal/sdio_halinit.c
> @@ -57,11 +57,6 @@ u8 _InitPowerOn_8723BS(struct adapter *padapter)
>  	/*  only cmd52 can be used before power on(card enable) */
>  	ret = CardEnable(padapter);
>  	if (!ret) {
> -		RT_TRACE(
> -			_module_hci_hal_init_c_,
> -			_drv_emerg_,
> -			("%s: run power on flow fail\n", __func__)
> -		);
>  		return _FAIL;
>  	}
>  
> @@ -680,7 +675,6 @@ static u32 rtl8723bs_hal_init(struct adapter *padapter)
>  
>  	ret = _InitPowerOn_8723BS(padapter);
>  	if (_FAIL == ret) {
> -		RT_TRACE(_module_hci_hal_init_c_, _drv_err_, ("Failed to init Power On!\n"));
>  		return _FAIL;
>  	}
>  
> @@ -688,12 +682,10 @@ static u32 rtl8723bs_hal_init(struct adapter *padapter)
>  
>  	ret = rtl8723b_FirmwareDownload(padapter, false);
>  	if (ret != _SUCCESS) {
> -		RT_TRACE(_module_hci_hal_init_c_, _drv_err_, ("%s: Download Firmware failed!!\n", __func__));
>  		padapter->bFWReady = false;
>  		pHalData->fw_ractrl = false;
>  		return ret;
>  	} else {
> -		RT_TRACE(_module_hci_hal_init_c_, _drv_notice_, ("rtl8723bs_hal_init(): Download Firmware Success!!\n"));
>  		padapter->bFWReady = true;
>  		pHalData->fw_ractrl = true;
>  	}
> @@ -719,7 +711,6 @@ static u32 rtl8723bs_hal_init(struct adapter *padapter)
>  #if (HAL_MAC_ENABLE == 1)
>  	ret = PHY_MACConfig8723B(padapter);
>  	if (ret != _SUCCESS) {
> -		RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("Initializepadapter8192CSdio(): Fail to configure MAC!!\n"));
>  		return ret;
>  	}
>  #endif
> @@ -729,7 +720,6 @@ static u32 rtl8723bs_hal_init(struct adapter *padapter)
>  #if (HAL_BB_ENABLE == 1)
>  	ret = PHY_BBConfig8723B(padapter);
>  	if (ret != _SUCCESS) {
> -		RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("Initializepadapter8192CSdio(): Fail to configure BB!!\n"));
>  		return ret;
>  	}
>  #endif
> @@ -741,7 +731,6 @@ static u32 rtl8723bs_hal_init(struct adapter *padapter)
>  #if (HAL_RF_ENABLE == 1)
>  		ret = PHY_RFConfig8723B(padapter);
>  		if (ret != _SUCCESS) {
> -			RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("Initializepadapter8192CSdio(): Fail to configure RF!!\n"));
>  			return ret;
>  		}
>  #endif
> @@ -893,8 +882,6 @@ static u32 rtl8723bs_hal_init(struct adapter *padapter)
>  	/*  Init BT hw config. */
>  	hal_btcoex_InitHwConfig(padapter, false);
>  
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("-%s\n", __func__));
> -
>  	return _SUCCESS;
>  }
>  
> @@ -1030,10 +1017,6 @@ static u32 rtl8723bs_inirp_init(struct adapter *padapter)
>  
>  static u32 rtl8723bs_inirp_deinit(struct adapter *padapter)
>  {
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("+rtl8723bs_inirp_deinit\n"));
> -
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("-rtl8723bs_inirp_deinit\n"));
> -
>  	return _SUCCESS;
>  }
>  
> @@ -1130,10 +1113,6 @@ static void Hal_EfuseParseMACAddr_8723BS(
>  		/* Read Permanent MAC address */
>  		memcpy(pEEPROM->mac_addr, &hwinfo[EEPROM_MAC_ADDR_8723BS], ETH_ALEN);
>  	}
> -/* 	NicIFSetMacAddress(padapter, padapter->PermanentAddress); */
> -
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_notice_,
> -		 ("Hal_EfuseParseMACAddr_8723BS: Permanent Address = %pM\n", pEEPROM->mac_addr));
>  }
>  
>  static void Hal_EfuseParseBoardType_8723BS(
> @@ -1148,7 +1127,6 @@ static void Hal_EfuseParseBoardType_8723BS(
>  			pHalData->BoardType = (EEPROM_DEFAULT_BOARD_OPTION & 0xE0) >> 5;
>  	} else
>  		pHalData->BoardType = 0;
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("Board Type: 0x%2x\n", pHalData->BoardType));
>  }
>  
>  static void _ReadEfuseInfo8723BS(struct adapter *padapter)
> @@ -1156,8 +1134,6 @@ static void _ReadEfuseInfo8723BS(struct adapter *padapter)
>  	struct eeprom_priv *pEEPROM = GET_EEPROM_EFUSE_PRIV(padapter);
>  	u8 *hwinfo = NULL;
>  
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("====>_ReadEfuseInfo8723BS()\n"));
> -
>  	/*  */
>  	/*  This part read and parse the eeprom/efuse content */
>  	/*  */
> @@ -1191,8 +1167,6 @@ static void _ReadEfuseInfo8723BS(struct adapter *padapter)
>  	Hal_EfuseParseVoltage_8723B(padapter, hwinfo, pEEPROM->bautoload_fail_flag);
>  
>  	Hal_ReadRFGainOffset(padapter, hwinfo, pEEPROM->bautoload_fail_flag);
> -
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("<==== _ReadEfuseInfo8723BS()\n"));
>  }
>  
>  static void _ReadPROMContent(struct adapter *padapter)
> @@ -1205,12 +1179,6 @@ static void _ReadPROMContent(struct adapter *padapter)
>  	pEEPROM->EepromOrEfuse = (eeValue & BOOT_FROM_EEPROM) ? true : false;
>  	pEEPROM->bautoload_fail_flag = (eeValue & EEPROM_EN) ? false : true;
>  
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_info_,
> -		 ("%s: 9346CR = 0x%02X, Boot from %s, Autoload %s\n",
> -		  __func__, eeValue,
> -		  (pEEPROM->EepromOrEfuse ? "EEPROM" : "EFUSE"),
> -		  (pEEPROM->bautoload_fail_flag ? "Fail" : "OK")));
> -
>  /* 	pHalData->EEType = IS_BOOT_FROM_EEPROM(Adapter) ? EEPROM_93C46 : EEPROM_BOOT_EFUSE; */
>  
>  	_ReadEfuseInfo8723BS(padapter);
> @@ -1233,8 +1201,6 @@ static s32 _ReadAdapterInfo8723BS(struct adapter *padapter)
>  	u8 val8;
>  	unsigned long start;
>  
> -	RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("+_ReadAdapterInfo8723BS\n"));
> -
>  	/*  before access eFuse, make sure card enable has been called */
>  	if (!padapter->hw_init_completed)
>  		_InitPowerOn_8723BS(padapter);
> diff --git a/drivers/staging/rtl8723bs/hal/sdio_ops.c b/drivers/staging/rtl8723bs/hal/sdio_ops.c
> index 170a28f4b191..3d2288683461 100644
> --- a/drivers/staging/rtl8723bs/hal/sdio_ops.c
> +++ b/drivers/staging/rtl8723bs/hal/sdio_ops.c
> @@ -777,30 +777,10 @@ void EnableInterrupt8723BSdio(struct adapter *adapter)
>  	himr = cpu_to_le32(haldata->sdio_himr);
>  	sdio_local_write(adapter, SDIO_REG_HIMR, 4, (u8 *)&himr);
>  
> -	RT_TRACE(
> -		_module_hci_ops_c_,
> -		_drv_notice_,
> -		(
> -			"%s: enable SDIO HIMR = 0x%08X\n",
> -			__func__,
> -			haldata->sdio_himr
> -		)
> -	);
> -
>  	/*  Update current system IMR settings */
>  	tmp = rtw_read32(adapter, REG_HSIMR);
>  	rtw_write32(adapter, REG_HSIMR, tmp | haldata->SysIntrMask);
>  
> -	RT_TRACE(
> -		_module_hci_ops_c_,
> -		_drv_notice_,
> -		(
> -			"%s: enable HSIMR = 0x%08X\n",
> -			__func__,
> -			haldata->SysIntrMask
> -		)
> -	);
> -
>  	/*  */
>  	/*  <Roger_Notes> There are some C2H CMDs have been sent before system interrupt is enabled, e.g., C2H, CPWM. */
>  	/*  So we need to clear all C2H events that FW has notified, otherwise FW won't schedule any commands anymore. */
> @@ -894,7 +874,6 @@ static struct recv_buf *sd_recv_rxfifo(struct adapter *adapter, u32 size)
>  	readbuf = recvbuf->pskb->data;
>  	ret = sdio_read_port(&adapter->iopriv.intf, WLAN_RX0FF_DEVICE_ID, readsize, readbuf);
>  	if (ret == _FAIL) {
> -		RT_TRACE(_module_hci_ops_os_c_, _drv_err_, ("%s: read port FAIL!\n", __func__));
>  		return NULL;
>  	}
>  
> @@ -1064,9 +1043,6 @@ void sd_int_hdl(struct adapter *adapter)
>  
>  		sd_int_dpc(adapter);
>  	} else {
> -		RT_TRACE(_module_hci_ops_c_, _drv_err_,
> -				("%s: HISR(0x%08x) and HIMR(0x%08x) not match!\n",
> -				__func__, hal->sdio_hisr, hal->sdio_himr));
>  	}
>  }
>  
> @@ -1090,13 +1066,6 @@ u8 HalQueryTxBufferStatus8723BSdio(struct adapter *adapter)
>  	numof_free_page = SdioLocalCmd53Read4Byte(adapter, SDIO_REG_FREE_TXPG);
>  
>  	memcpy(hal->SdioTxFIFOFreePage, &numof_free_page, 4);
> -	RT_TRACE(_module_hci_ops_c_, _drv_notice_,
> -			("%s: Free page for HIQ(%#x), MIDQ(%#x), LOWQ(%#x), PUBQ(%#x)\n",
> -			__func__,
> -			hal->SdioTxFIFOFreePage[HI_QUEUE_IDX],
> -			hal->SdioTxFIFOFreePage[MID_QUEUE_IDX],
> -			hal->SdioTxFIFOFreePage[LOW_QUEUE_IDX],
> -			hal->SdioTxFIFOFreePage[PUBLIC_QUEUE_IDX]));
>  
>  	return true;
>  }
> diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c
> index a577ddcce8cd..73d525ebe1fc 100644
> --- a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c
> +++ b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c
> @@ -875,7 +875,6 @@ static int rtw_cfg80211_set_encryption(struct net_device *dev, struct ieee_param
>  
>  	if (strcmp(param->u.crypt.alg, "WEP") == 0)
>  	{
> -		RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_err_, ("wpa_set_encryption, crypt.alg = WEP\n"));
>  		DBG_8192C("wpa_set_encryption, crypt.alg = WEP\n");
>  
>  		wep_key_idx = param->u.crypt.idx;
> @@ -1798,7 +1797,6 @@ static int rtw_cfg80211_set_wpa_ie(struct adapter *padapter, u8 *pie, size_t iel
>  	}
>  
>  	if (ielen < RSN_HEADER_LEN) {
> -		RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_err_, ("Ie len too short %d\n", ielen));
>  		ret  = -1;
>  		goto exit;
>  	}
> @@ -1902,10 +1900,6 @@ static int rtw_cfg80211_set_wpa_ie(struct adapter *padapter, u8 *pie, size_t iel
>  		/*  check_fwstate(&padapter->mlmepriv, WIFI_UNDER_WPS) == true) */
>  		rtw_hal_set_hwreg(padapter, HW_VAR_OFF_RCR_AM, null_addr);
>  
> -	RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_info_,
> -		("rtw_set_wpa_ie: pairwise_cipher = 0x%08x padapter->securitypriv.ndisencryptstatus =%d padapter->securitypriv.ndisauthtype =%d\n",
> -		pairwise_cipher, padapter->securitypriv.ndisencryptstatus, padapter->securitypriv.ndisauthtype));
> -
>  exit:
>  	kfree(buf);
>  	if (ret)
> diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c b/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
> index 9909dece0b01..ed0a1c0ec6cb 100644
> --- a/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
> +++ b/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
> @@ -232,8 +232,6 @@ static char *translate_scan(struct adapter *padapter,
>  		u16 wpa_len = 0, rsn_len = 0;
>  		u8 *p;
>  		rtw_get_sec_ie(pnetwork->network.IEs, pnetwork->network.IELength, rsn_ie, &rsn_len, wpa_ie, &wpa_len);
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_wx_get_scan: ssid =%s\n", pnetwork->network.Ssid.Ssid));
> -		RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_wx_get_scan: wpa_len =%d rsn_len =%d\n", wpa_len, rsn_len));
>  
>  		buf = kzalloc(MAX_WPA_IE_LEN*2, GFP_ATOMIC);
>  		if (!buf)
> @@ -423,7 +421,6 @@ static int wpa_set_encryption(struct net_device *dev, struct ieee_param *param,
>  	}
>  
>  	if (strcmp(param->u.crypt.alg, "WEP") == 0) {
> -		RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_err_, ("wpa_set_encryption, crypt.alg = WEP\n"));
>  		DBG_871X("wpa_set_encryption, crypt.alg = WEP\n");
>  
>  		padapter->securitypriv.ndisencryptstatus = Ndis802_11Encryption1Enabled;
> @@ -433,20 +430,16 @@ static int wpa_set_encryption(struct net_device *dev, struct ieee_param *param,
>  		wep_key_idx = param->u.crypt.idx;
>  		wep_key_len = param->u.crypt.key_len;
>  
> -		RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_info_, ("(1)wep_key_idx =%d\n", wep_key_idx));
>  		DBG_871X("(1)wep_key_idx =%d\n", wep_key_idx);
>  
>  		if (wep_key_idx > WEP_KEYS)
>  			return -EINVAL;
>  
> -		RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_info_, ("(2)wep_key_idx =%d\n", wep_key_idx));
> -
>  		if (wep_key_len > 0) {
>  			wep_key_len = wep_key_len <= 5 ? 5 : 13;
>  			wep_total_len = wep_key_len + FIELD_OFFSET(struct ndis_802_11_wep, KeyMaterial);
>  			pwep = kzalloc(wep_total_len, GFP_KERNEL);
>  			if (pwep == NULL) {
> -				RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_err_, (" wpa_set_encryption: pwep allocate fail !!!\n"));
>  				goto exit;
>  			}
>  
> @@ -612,7 +605,6 @@ static int rtw_set_wpa_ie(struct adapter *padapter, char *pie, unsigned short ie
>  		}
>  
>  		if (ielen < RSN_HEADER_LEN) {
> -			RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_err_, ("Ie len too short %d\n", ielen));
>  			ret  = -1;
>  			goto exit;
>  		}
> @@ -715,10 +707,6 @@ static int rtw_set_wpa_ie(struct adapter *padapter, char *pie, unsigned short ie
>                  /*  check_fwstate(&padapter->mlmepriv, WIFI_UNDER_WPS) == true) */
>                  rtw_hal_set_hwreg(padapter, HW_VAR_OFF_RCR_AM, null_addr);
>  
> -	RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_info_,
> -		 ("rtw_set_wpa_ie: pairwise_cipher = 0x%08x padapter->securitypriv.ndisencryptstatus =%d padapter->securitypriv.ndisauthtype =%d\n",
> -		  pairwise_cipher, padapter->securitypriv.ndisencryptstatus, padapter->securitypriv.ndisauthtype));
> -
>  exit:
>  
>  	kfree(buf);
> @@ -738,8 +726,6 @@ static int rtw_wx_get_name(struct net_device *dev,
>  	struct wlan_bssid_ex  *pcur_bss = &pmlmepriv->cur_network.network;
>  	NDIS_802_11_RATES_EX *prates = NULL;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("cmd_code =%x\n", info->cmd));
> -
>  	if (check_fwstate(pmlmepriv, _FW_LINKED|WIFI_ADHOC_MASTER_STATE) == true) {
>  		/* parsing HT_CAP_IE */
>  		p = rtw_get_ie(&pcur_bss->IEs[12], WLAN_EID_HT_CAPABILITY, &ht_ielen, pcur_bss->IELength-12);
> @@ -785,8 +771,6 @@ static int rtw_wx_set_freq(struct net_device *dev,
>  			     struct iw_request_info *info,
>  			     union iwreq_data *wrqu, char *extra)
>  {
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_notice_, ("+rtw_wx_set_freq\n"));
> -
>  	return 0;
>  }
>  
> @@ -850,7 +834,6 @@ static int rtw_wx_set_mode(struct net_device *dev, struct iw_request_info *a,
>  		break;
>  	default:
>  		ret = -EINVAL;
> -		RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_err_, ("\n Mode: %s is not supported \n", iw_operation_mode[wrqu->mode]));
>  		goto exit;
>  	}
>  
> @@ -884,8 +867,6 @@ static int rtw_wx_get_mode(struct net_device *dev, struct iw_request_info *a,
>  	struct adapter *padapter = rtw_netdev_priv(dev);
>  	struct	mlme_priv *pmlmepriv = &(padapter->mlmepriv);
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, (" rtw_wx_get_mode\n"));
> -
>  	if (check_fwstate(pmlmepriv, WIFI_STATION_STATE) == true) {
>  		wrqu->mode = IW_MODE_INFRA;
>  	} else if  ((check_fwstate(pmlmepriv, WIFI_ADHOC_MASTER_STATE) == true) ||
> @@ -999,8 +980,6 @@ static int rtw_wx_get_range(struct net_device *dev,
>  	u16 val;
>  	int i;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_wx_get_range. cmd_code =%x\n", info->cmd));
> -
>  	wrqu->data.length = sizeof(*range);
>  	memset(range, 0, sizeof(*range));
>  
> @@ -1178,8 +1157,6 @@ static int rtw_wx_get_wap(struct net_device *dev,
>  
>  	eth_zero_addr(wrqu->ap_addr.sa_data);
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_wx_get_wap\n"));
> -
>  	if  (((check_fwstate(pmlmepriv, _FW_LINKED)) == true) ||
>  			((check_fwstate(pmlmepriv, WIFI_ADHOC_MASTER_STATE)) == true) ||
>  			((check_fwstate(pmlmepriv, WIFI_AP_STATE)) == true)) {
> @@ -1232,7 +1209,6 @@ static int rtw_wx_set_scan(struct net_device *dev, struct iw_request_info *a,
>  	struct adapter *padapter = rtw_netdev_priv(dev);
>  	struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
>  	struct ndis_802_11_ssid ssid[RTW_SSID_SCAN_AMOUNT];
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_wx_set_scan\n"));
>  
>  	#ifdef DBG_IOCTL
>  	DBG_871X("DBG_IOCTL %s:%d\n", __func__, __LINE__);
> @@ -1393,9 +1369,6 @@ static int rtw_wx_get_scan(struct net_device *dev, struct iw_request_info *a,
>  	u32 ret = 0;
>  	signed int wait_status;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_wx_get_scan\n"));
> -	RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_info_, (" Start of Query SIOCGIWSCAN .\n"));
> -
>  	#ifdef DBG_IOCTL
>  	DBG_871X("DBG_IOCTL %s:%d\n", __func__, __LINE__);
>  	#endif
> @@ -1477,9 +1450,6 @@ static int rtw_wx_set_essid(struct net_device *dev,
>  	DBG_871X("DBG_IOCTL %s:%d\n", __func__, __LINE__);
>  	#endif
>  
> -	RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_info_,
> -		 ("+rtw_wx_set_essid: fw_state = 0x%08x\n", get_fwstate(pmlmepriv)));
> -
>  	rtw_ps_deny(padapter, PS_DENY_JOIN);
>  	if (_FAIL == rtw_pwr_wakeup(padapter)) {
>  		ret = -1;
> @@ -1513,16 +1483,12 @@ static int rtw_wx_set_essid(struct net_device *dev,
>  		memcpy(ndis_ssid.Ssid, extra, len);
>  		src_ssid = ndis_ssid.Ssid;
>  
> -		RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_info_, ("rtw_wx_set_essid: ssid =[%s]\n", src_ssid));
>  		spin_lock_bh(&queue->lock);
>  		phead = get_list_head(queue);
>  		pmlmepriv->pscanned = get_next(phead);
>  
>  		while (1) {
>  			if (phead == pmlmepriv->pscanned) {
> -			        RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_warning_,
> -					 ("rtw_wx_set_essid: scan_q is empty, set ssid to check if scanning again!\n"));
> -
>  				break;
>  			}
>  
> @@ -1532,15 +1498,8 @@ static int rtw_wx_set_essid(struct net_device *dev,
>  
>  			dst_ssid = pnetwork->network.Ssid.Ssid;
>  
> -			RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_info_,
> -				 ("rtw_wx_set_essid: dst_ssid =%s\n",
> -				  pnetwork->network.Ssid.Ssid));
> -
>  			if ((!memcmp(dst_ssid, src_ssid, ndis_ssid.SsidLength)) &&
>  				(pnetwork->network.Ssid.SsidLength == ndis_ssid.SsidLength)) {
> -				RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_info_,
> -					 ("rtw_wx_set_essid: find match, set infra mode\n"));
> -
>  				if (check_fwstate(pmlmepriv, WIFI_ADHOC_STATE) == true) {
>  					if (pnetwork->network.InfrastructureMode != pmlmepriv->cur_network.network.InfrastructureMode)
>  						continue;
> @@ -1556,8 +1515,6 @@ static int rtw_wx_set_essid(struct net_device *dev,
>  			}
>  		}
>  		spin_unlock_bh(&queue->lock);
> -		RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_info_,
> -			 ("set ssid: set_802_11_auth. mode =%d\n", authmode));
>  		rtw_set_802_11_authentication_mode(padapter, authmode);
>  		/* set_802_11_encryption_mode(padapter, padapter->securitypriv.ndisencryptstatus); */
>  		if (rtw_set_802_11_ssid(padapter, &ndis_ssid) == false) {
> @@ -1588,8 +1545,6 @@ static int rtw_wx_get_essid(struct net_device *dev,
>  	struct	mlme_priv *pmlmepriv = &(padapter->mlmepriv);
>  	struct wlan_bssid_ex  *pcur_bss = &pmlmepriv->cur_network.network;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("rtw_wx_get_essid\n"));
> -
>  	if ((check_fwstate(pmlmepriv, _FW_LINKED) == true) ||
>  	      (check_fwstate(pmlmepriv, WIFI_ADHOC_MASTER_STATE) == true)) {
>  		len = pcur_bss->Ssid.SsidLength;
> @@ -1620,9 +1575,6 @@ static int rtw_wx_set_rate(struct net_device *dev,
>  	u32 ratevalue = 0;
>  	u8 mpdatarate[NumRates] = {11, 10, 9, 8, 7, 6, 5, 4, 3, 2, 1, 0, 0xff};
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, (" rtw_wx_set_rate\n"));
> -	RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_info_, ("target_rate = %d, fixed = %d\n", target_rate, fixed));
> -
>  	if (target_rate == -1) {
>  		ratevalue = 11;
>  		goto set_rate;
> @@ -1681,12 +1633,9 @@ static int rtw_wx_set_rate(struct net_device *dev,
>  		} else {
>  			datarates[i] = 0xff;
>  		}
> -
> -		RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_info_, ("datarate_inx =%d\n", datarates[i]));
>  	}
>  
>  	if (rtw_setdatarate_cmd(padapter, datarates) != _SUCCESS) {
> -		RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_err_, ("rtw_wx_set_rate Fail!!!\n"));
>  		ret = -1;
>  	}
>  	return ret;
> @@ -3180,8 +3129,6 @@ static int wpa_set_param(struct net_device *dev, u8 name, u32 value)
>  			break;
>  		}
>  
> -		RT_TRACE(_module_rtl871x_ioctl_os_c, _drv_info_, ("wpa_set_param:padapter->securitypriv.ndisauthtype =%d\n", padapter->securitypriv.ndisauthtype));
> -
>  		break;
>  
>  	case IEEE_PARAM_TKIP_COUNTERMEASURES:
> diff --git a/drivers/staging/rtl8723bs/os_dep/mlme_linux.c b/drivers/staging/rtl8723bs/os_dep/mlme_linux.c
> index ba4d3789a41e..b46c086233bb 100644
> --- a/drivers/staging/rtl8723bs/os_dep/mlme_linux.c
> +++ b/drivers/staging/rtl8723bs/os_dep/mlme_linux.c
> @@ -145,12 +145,8 @@ void rtw_report_sec_ie(struct adapter *adapter, u8 authmode, u8 *sec_ie)
>  	u8 *buff, *p, i;
>  	union iwreq_data wrqu;
>  
> -	RT_TRACE(_module_mlme_osdep_c_, _drv_info_, ("+rtw_report_sec_ie, authmode =%d\n", authmode));
> -
>  	buff = NULL;
>  	if (authmode == WLAN_EID_VENDOR_SPECIFIC) {
> -		RT_TRACE(_module_mlme_osdep_c_, _drv_info_, ("rtw_report_sec_ie, authmode =%d\n", authmode));
> -
>  		buff = rtw_zmalloc(IW_CUSTOM_MAX);
>  		if (NULL == buff) {
>  			DBG_871X(FUNC_ADPT_FMT ": alloc memory FAIL!!\n",
> diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> index 9ae7d46fb501..80bb5a0c0bc8 100644
> --- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> +++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> @@ -481,8 +481,6 @@ struct net_device *rtw_init_netdev(struct adapter *old_padapter)
>  	struct adapter *padapter;
>  	struct net_device *pnetdev;
>  
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("+init_net_dev\n"));
> -
>  	if (old_padapter)
>  		pnetdev = rtw_alloc_etherdev_with_old_priv(sizeof(struct adapter), (void *)old_padapter);
>  	else
> @@ -531,7 +529,6 @@ u32 rtw_start_drv_threads(struct adapter *padapter)
>  {
>  	u32 _status = _SUCCESS;
>  
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("+rtw_start_drv_threads\n"));
>  	padapter->xmitThread = kthread_run(rtw_xmit_thread, padapter, "RTW_XMIT_THREAD");
>  	if (IS_ERR(padapter->xmitThread))
>  		_status = _FAIL;
> @@ -548,14 +545,11 @@ u32 rtw_start_drv_threads(struct adapter *padapter)
>  
>  void rtw_stop_drv_threads(struct adapter *padapter)
>  {
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("+rtw_stop_drv_threads\n"));
> -
>  	rtw_stop_cmd_thread(padapter);
>  
>  	/*  Below is to termindate tx_thread... */
>  	complete(&padapter->xmitpriv.xmit_comp);
>  	wait_for_completion(&padapter->xmitpriv.terminate_xmitthread_comp);
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("\n drv_halt: rtw_xmit_thread can be terminated !\n"));
>  
>  	rtw_hal_stop_thread(padapter);
>  }
> @@ -699,14 +693,11 @@ u8 rtw_init_drv_sw(struct adapter *padapter)
>  {
>  	u8 ret8 = _SUCCESS;
>  
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("+rtw_init_drv_sw\n"));
> -
>  	rtw_init_default_value(padapter);
>  
>  	rtw_init_hal_com_default_value(padapter);
>  
>  	if (rtw_init_cmd_priv(&padapter->cmdpriv)) {
> -		RT_TRACE(_module_os_intfs_c_, _drv_err_, ("\n Can't init cmd_priv\n"));
>  		ret8 = _FAIL;
>  		goto exit;
>  	}
> @@ -714,14 +705,12 @@ u8 rtw_init_drv_sw(struct adapter *padapter)
>  	padapter->cmdpriv.padapter = padapter;
>  
>  	if (rtw_init_evt_priv(&padapter->evtpriv)) {
> -		RT_TRACE(_module_os_intfs_c_, _drv_err_, ("\n Can't init evt_priv\n"));
>  		ret8 = _FAIL;
>  		goto exit;
>  	}
>  
>  
>  	if (rtw_init_mlme_priv(padapter) == _FAIL) {
> -		RT_TRACE(_module_os_intfs_c_, _drv_err_, ("\n Can't init mlme_priv\n"));
>  		ret8 = _FAIL;
>  		goto exit;
>  	}
> @@ -762,29 +751,21 @@ u8 rtw_init_drv_sw(struct adapter *padapter)
>  
>  exit:
>  
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("-rtw_init_drv_sw\n"));
> -
>  	return ret8;
>  }
>  
>  void rtw_cancel_all_timer(struct adapter *padapter)
>  {
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("+rtw_cancel_all_timer\n"));
> -
>  	del_timer_sync(&padapter->mlmepriv.assoc_timer);
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("rtw_cancel_all_timer:cancel association timer complete!\n"));
>  
>  	del_timer_sync(&padapter->mlmepriv.scan_to_timer);
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("rtw_cancel_all_timer:cancel scan_to_timer!\n"));
>  
>  	del_timer_sync(&padapter->mlmepriv.dynamic_chk_timer);
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("rtw_cancel_all_timer:cancel dynamic_chk_timer!\n"));
>  
>  	del_timer_sync(&(adapter_to_pwrctl(padapter)->pwr_state_check_timer));
>  
>  	del_timer_sync(&padapter->mlmepriv.set_scan_deny_timer);
>  	rtw_clear_scan_deny(padapter);
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("rtw_cancel_all_timer:cancel set_scan_deny_timer!\n"));
>  
>  	del_timer_sync(&padapter->recvpriv.signal_stat_timer);
>  
> @@ -794,8 +775,6 @@ void rtw_cancel_all_timer(struct adapter *padapter)
>  
>  u8 rtw_free_drv_sw(struct adapter *padapter)
>  {
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("==>rtw_free_drv_sw"));
> -
>  	free_mlme_ext_priv(&padapter->mlmeextpriv);
>  
>  	rtw_free_cmd_priv(&padapter->cmdpriv);
> @@ -818,8 +797,6 @@ u8 rtw_free_drv_sw(struct adapter *padapter)
>  
>  	rtw_hal_free_data(padapter);
>  
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("<==rtw_free_drv_sw\n"));
> -
>  	/* free the old_pnetdev */
>  	if (padapter->rereg_nd_name_priv.old_pnetdev) {
>  		free_netdev(padapter->rereg_nd_name_priv.old_pnetdev);
> @@ -830,8 +807,6 @@ u8 rtw_free_drv_sw(struct adapter *padapter)
>  	if (padapter->pbuddy_adapter)
>  		padapter->pbuddy_adapter->pbuddy_adapter = NULL;
>  
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("-rtw_free_drv_sw\n"));
> -
>  	return _SUCCESS;
>  }
>  
> @@ -881,7 +856,6 @@ int _netdev_open(struct net_device *pnetdev)
>  	struct adapter *padapter = rtw_netdev_priv(pnetdev);
>  	struct pwrctrl_priv *pwrctrlpriv = adapter_to_pwrctl(padapter);
>  
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("+871x_drv - dev_open\n"));
>  	DBG_871X("+871x_drv - drv_open, bup =%d\n", padapter->bup);
>  
>  	padapter->netif_up = true;
> @@ -898,7 +872,6 @@ int _netdev_open(struct net_device *pnetdev)
>  
>  		status = rtw_hal_init(padapter);
>  		if (status == _FAIL) {
> -			RT_TRACE(_module_os_intfs_c_, _drv_err_, ("rtl871x_hal_init(): Can't init h/w!\n"));
>  			goto netdev_open_error;
>  		}
>  
> @@ -929,7 +902,6 @@ int _netdev_open(struct net_device *pnetdev)
>  
>  netdev_open_normal_process:
>  
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("-871x_drv - dev_open\n"));
>  	DBG_871X("-871x_drv - drv_open, bup =%d\n", padapter->bup);
>  
>  	return 0;
> @@ -941,7 +913,6 @@ int _netdev_open(struct net_device *pnetdev)
>  	netif_carrier_off(pnetdev);
>  	rtw_netif_stop_queue(pnetdev);
>  
> -	RT_TRACE(_module_os_intfs_c_, _drv_err_, ("-871x_drv - dev_open, fail!\n"));
>  	DBG_871X("-871x_drv - drv_open fail, bup =%d\n", padapter->bup);
>  
>  	return (-1);
> @@ -980,7 +951,6 @@ static int  ips_netdrv_open(struct adapter *padapter)
>  
>  	status = rtw_hal_init(padapter);
>  	if (status == _FAIL) {
> -		RT_TRACE(_module_os_intfs_c_, _drv_err_, ("ips_netdrv_open(): Can't init h/w!\n"));
>  		goto netdev_open_error;
>  	}
>  
> @@ -1053,8 +1023,6 @@ static int netdev_close(struct net_device *pnetdev)
>  	struct adapter *padapter = rtw_netdev_priv(pnetdev);
>  	struct pwrctrl_priv *pwrctl = adapter_to_pwrctl(padapter);
>  
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("+871x_drv - drv_close\n"));
> -
>  	if (pwrctl->bInternalAutoSuspend) {
>  		/* rtw_pwr_wakeup(padapter); */
>  		if (pwrctl->rf_pwrstate == rf_off)
> @@ -1095,7 +1063,6 @@ static int netdev_close(struct net_device *pnetdev)
>  	rtw_scan_abort(padapter);
>  	adapter_wdev_data(padapter)->bandroid_scan = false;
>  
> -	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("-871x_drv - drv_close\n"));
>  	DBG_871X("-871x_drv - drv_close, bup =%d\n", padapter->bup);
>  
>  	return 0;
> @@ -1116,8 +1083,6 @@ void rtw_dev_unload(struct adapter *padapter)
>  	struct cmd_priv *pcmdpriv = &padapter->cmdpriv;
>  	u8 cnt = 0;
>  
> -	RT_TRACE(_module_hci_intfs_c_, _drv_notice_, ("+%s\n", __func__));
> -
>  	if (padapter->bup) {
>  
>  		padapter->bDriverStopped = true;
> @@ -1127,8 +1092,6 @@ void rtw_dev_unload(struct adapter *padapter)
>  		if (padapter->intf_stop)
>  			padapter->intf_stop(padapter);
>  
> -		RT_TRACE(_module_hci_intfs_c_, _drv_notice_, ("@ rtw_dev_unload: stop intf complete!\n"));
> -
>  		if (!pwrctl->bInternalAutoSuspend)
>  			rtw_stop_drv_threads(padapter);
>  
> @@ -1143,8 +1106,6 @@ void rtw_dev_unload(struct adapter *padapter)
>  			}
>  		}
>  
> -		RT_TRACE(_module_hci_intfs_c_, _drv_notice_, ("@ %s: stop thread complete!\n", __func__));
> -
>  		/* check the status of IPS */
>  		if (rtw_hal_check_ips_status(padapter) || pwrctl->rf_pwrstate == rf_off) {
>  			/* check HW status and SW state */
> @@ -1163,17 +1124,12 @@ void rtw_dev_unload(struct adapter *padapter)
>  
>  			padapter->bSurpriseRemoved = true;
>  		}
> -		RT_TRACE(_module_hci_intfs_c_, _drv_notice_,
> -			 ("@ %s: deinit hal complete!\n", __func__));
>  
>  		padapter->bup = false;
>  
>  	} else {
> -		RT_TRACE(_module_hci_intfs_c_, _drv_notice_, ("%s: bup ==false\n", __func__));
>  		DBG_871X("%s: bup ==false\n", __func__);
>  	}
> -
> -	RT_TRACE(_module_hci_intfs_c_, _drv_notice_, ("-%s\n", __func__));
>  }
>  
>  static int rtw_suspend_free_assoc_resource(struct adapter *padapter)
> @@ -1332,14 +1288,12 @@ static int rtw_resume_process_normal(struct adapter *padapter)
>  	/* if (sdio_init(adapter_to_dvobj(padapter)) != _SUCCESS) */
>  	if ((padapter->intf_init) && (padapter->intf_init(adapter_to_dvobj(padapter)) != _SUCCESS)) {
>  		ret = -1;
> -		RT_TRACE(_module_hci_intfs_c_, _drv_err_, ("%s: initialize SDIO Failed!!\n", __func__));
>  		goto exit;
>  	}
>  	rtw_hal_disable_interrupt(padapter);
>  	/* if (sdio_alloc_irq(adapter_to_dvobj(padapter)) != _SUCCESS) */
>  	if ((padapter->intf_alloc_irq) && (padapter->intf_alloc_irq(adapter_to_dvobj(padapter)) != _SUCCESS)) {
>  		ret = -1;
> -		RT_TRACE(_module_hci_intfs_c_, _drv_err_, ("%s: sdio_alloc_irq Failed!!\n", __func__));
>  		goto exit;
>  	}
>  
> diff --git a/drivers/staging/rtl8723bs/os_dep/osdep_service.c b/drivers/staging/rtl8723bs/os_dep/osdep_service.c
> index 843003b91ea2..7b2aad346d20 100644
> --- a/drivers/staging/rtl8723bs/os_dep/osdep_service.c
> +++ b/drivers/staging/rtl8723bs/os_dep/osdep_service.c
> @@ -161,7 +161,6 @@ int rtw_change_ifname(struct adapter *padapter, const char *ifname)
>  		ret = register_netdevice(pnetdev);
>  
>  	if (ret != 0) {
> -		RT_TRACE(_module_hci_intfs_c_, _drv_err_, ("register_netdev() failed\n"));
>  		goto error;
>  	}
>  
> diff --git a/drivers/staging/rtl8723bs/os_dep/recv_linux.c b/drivers/staging/rtl8723bs/os_dep/recv_linux.c
> index fbdbcd04d44a..a2a28803c8d9 100644
> --- a/drivers/staging/rtl8723bs/os_dep/recv_linux.c
> +++ b/drivers/staging/rtl8723bs/os_dep/recv_linux.c
> @@ -202,22 +202,15 @@ int rtw_recv_indicatepkt(struct adapter *padapter, union recv_frame *precv_frame
>  
>  	skb = precv_frame->u.hdr.pkt;
>  	if (skb == NULL) {
> -		RT_TRACE(_module_recv_osdep_c_, _drv_err_, ("rtw_recv_indicatepkt():skb == NULL something wrong!!!!\n"));
>  		goto _recv_indicatepkt_drop;
>  	}
>  
> -	RT_TRACE(_module_recv_osdep_c_, _drv_info_, ("rtw_recv_indicatepkt():skb != NULL !!!\n"));
> -	RT_TRACE(_module_recv_osdep_c_, _drv_info_, ("rtw_recv_indicatepkt():precv_frame->u.hdr.rx_head =%p  precv_frame->hdr.rx_data =%p\n", precv_frame->u.hdr.rx_head, precv_frame->u.hdr.rx_data));
> -	RT_TRACE(_module_recv_osdep_c_, _drv_info_, ("precv_frame->hdr.rx_tail =%p precv_frame->u.hdr.rx_end =%p precv_frame->hdr.len =%d\n", precv_frame->u.hdr.rx_tail, precv_frame->u.hdr.rx_end, precv_frame->u.hdr.len));
> -
>  	skb->data = precv_frame->u.hdr.rx_data;
>  
>  	skb_set_tail_pointer(skb, precv_frame->u.hdr.len);
>  
>  	skb->len = precv_frame->u.hdr.len;
>  
> -	RT_TRACE(_module_recv_osdep_c_, _drv_info_, ("\n skb->head =%p skb->data =%p skb->tail =%p skb->end =%p skb->len =%d\n", skb->head, skb->data, skb_tail_pointer(skb), skb_end_pointer(skb), skb->len));
> -
>  	rtw_os_recv_indicate_pkt(padapter, skb, pattrib);
>  
>  	/* pointers to NULL before rtw_free_recvframe() */
> @@ -225,8 +218,6 @@ int rtw_recv_indicatepkt(struct adapter *padapter, union recv_frame *precv_frame
>  
>  	rtw_free_recvframe(precv_frame, pfree_recv_queue);
>  
> -	RT_TRACE(_module_recv_osdep_c_, _drv_info_, ("\n rtw_recv_indicatepkt :after rtw_os_recv_indicate_pkt!!!!\n"));
> -
>  	return _SUCCESS;
>  
>  _recv_indicatepkt_drop:
> diff --git a/drivers/staging/rtl8723bs/os_dep/sdio_intf.c b/drivers/staging/rtl8723bs/os_dep/sdio_intf.c
> index 8f8549eee23e..daca82d4b929 100644
> --- a/drivers/staging/rtl8723bs/os_dep/sdio_intf.c
> +++ b/drivers/staging/rtl8723bs/os_dep/sdio_intf.c
> @@ -159,9 +159,6 @@ static void sdio_deinit(struct dvobj_priv *dvobj)
>  	struct sdio_func *func;
>  	int err;
>  
> -
> -	RT_TRACE(_module_hci_intfs_c_, _drv_notice_, ("+sdio_deinit\n"));
> -
>  	func = dvobj->intf_data.func;
>  
>  	if (func) {
> @@ -201,7 +198,6 @@ static struct dvobj_priv *sdio_dvobj_init(struct sdio_func *func)
>  	psdio->func = func;
>  
>  	if (sdio_init(dvobj) != _SUCCESS) {
> -		RT_TRACE(_module_hci_intfs_c_, _drv_err_, ("%s: initialize SDIO Failed!\n", __func__));
>  		goto free_dvobj;
>  	}
>  	rtw_reset_continual_io_error(dvobj);
> @@ -306,8 +302,6 @@ static struct adapter *rtw_sdio_if1_init(struct dvobj_priv *dvobj, const struct
>  	padapter->intf_free_irq = &sdio_free_irq;
>  
>  	if (rtw_init_io_priv(padapter, sdio_set_intf_ops) == _FAIL) {
> -		RT_TRACE(_module_hci_intfs_c_, _drv_err_,
> -			("rtw_drv_init: Can't init io_priv\n"));
>  		goto free_hal_data;
>  	}
>  
> @@ -322,8 +316,6 @@ static struct adapter *rtw_sdio_if1_init(struct dvobj_priv *dvobj, const struct
>  
>  	/* 3 7. init driver common data */
>  	if (rtw_init_drv_sw(padapter) == _FAIL) {
> -		RT_TRACE(_module_hci_intfs_c_, _drv_err_,
> -			 ("rtw_drv_init: Initialize driver software resource Failed!\n"));
>  		goto free_hal_data;
>  	}
>  
> @@ -406,7 +398,6 @@ static int rtw_drv_init(
>  
>  	dvobj = sdio_dvobj_init(func);
>  	if (dvobj == NULL) {
> -		RT_TRACE(_module_hci_intfs_c_, _drv_err_, ("initialize device object priv Failed!\n"));
>  		goto exit;
>  	}
>  
> @@ -424,8 +415,6 @@ static int rtw_drv_init(
>  	if (sdio_alloc_irq(dvobj) != _SUCCESS)
>  		goto free_if2;
>  
> -	RT_TRACE(_module_hci_intfs_c_, _drv_err_, ("-871x_drv - drv_init, success!\n"));
> -
>  	rtw_ndev_notifier_register();
>  	status = _SUCCESS;
>  
> @@ -447,8 +436,6 @@ static void rtw_dev_remove(struct sdio_func *func)
>  	struct dvobj_priv *dvobj = sdio_get_drvdata(func);
>  	struct adapter *padapter = dvobj->if1;
>  
> -	RT_TRACE(_module_hci_intfs_c_, _drv_notice_, ("+rtw_dev_remove\n"));
> -
>  	dvobj->processing_dev_remove = true;
>  
>  	rtw_unregister_netdevs(dvobj);
> @@ -478,8 +465,6 @@ static void rtw_dev_remove(struct sdio_func *func)
>  	rtw_sdio_if1_deinit(padapter);
>  
>  	sdio_dvobj_deinit(func);
> -
> -	RT_TRACE(_module_hci_intfs_c_, _drv_notice_, ("-rtw_dev_remove\n"));
>  }
>  
>  extern int pm_netdev_open(struct net_device *pnetdev, u8 bnormal);
> diff --git a/drivers/staging/rtl8723bs/os_dep/xmit_linux.c b/drivers/staging/rtl8723bs/os_dep/xmit_linux.c
> index a3b6584ca0d0..610da93ac188 100644
> --- a/drivers/staging/rtl8723bs/os_dep/xmit_linux.c
> +++ b/drivers/staging/rtl8723bs/os_dep/xmit_linux.c
> @@ -196,10 +196,7 @@ int _rtw_xmit_entry(struct sk_buff *pkt, struct net_device *pnetdev)
>  	struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
>  	s32 res = 0;
>  
> -	RT_TRACE(_module_rtl871x_mlme_c_, _drv_info_, ("+xmit_enry\n"));
> -
>  	if (rtw_if_up(padapter) == false) {
> -		RT_TRACE(_module_xmit_osdep_c_, _drv_err_, ("rtw_xmit_entry: rtw_if_up fail\n"));
>  		#ifdef DBG_TX_DROP_FRAME
>  		DBG_871X("DBG_TX_DROP_FRAME %s if_up fail\n", __func__);
>  		#endif
> @@ -232,13 +229,11 @@ int _rtw_xmit_entry(struct sk_buff *pkt, struct net_device *pnetdev)
>  		goto drop_packet;
>  	}
>  
> -	RT_TRACE(_module_xmit_osdep_c_, _drv_info_, ("rtw_xmit_entry: tx_pkts =%d\n", (u32)pxmitpriv->tx_pkts));
>  	goto exit;
>  
>  drop_packet:
>  	pxmitpriv->tx_drop++;
>  	dev_kfree_skb_any(pkt);
> -	RT_TRACE(_module_xmit_osdep_c_, _drv_notice_, ("rtw_xmit_entry: drop, tx_drop =%d\n", (u32)pxmitpriv->tx_drop));
>  
>  exit:
>  	return 0;
> 
> $ $ git diff | ./scripts/checkpatch.pl
> ERROR: that open brace { should be on the previous line
> #82: FILE: drivers/staging/rtl8723bs/core/rtw_cmd.c:665:
>  	if (pmlmepriv->assoc_ssid.SsidLength == 0)
> +		{}
> 
> ERROR: that open brace { should be on the previous line
> #85: FILE: drivers/staging/rtl8723bs/core/rtw_cmd.c:667:
>  	else
> +		{}
> 
> ERROR: that open brace { should be on the previous line
> #94: FILE: drivers/staging/rtl8723bs/core/rtw_cmd.c:755:
>  	if (pmlmepriv->assoc_ssid.SsidLength == 0)
> +		{}
> 
> ERROR: that open brace { should be on the previous line
> #97: FILE: drivers/staging/rtl8723bs/core/rtw_cmd.c:757:
>  	else
> +		{}
> 
> ERROR: that open brace { should be on the previous line
> #613: FILE: drivers/staging/rtl8723bs/core/rtw_ioctl_set.c:464:
>  		if (check_fwstate(pmlmepriv, (_FW_UNDER_SURVEY|_FW_UNDER_LINKING)) == true)
> +			{}
> 
> ERROR: that open brace { should be on the previous line
> #616: FILE: drivers/staging/rtl8723bs/core/rtw_ioctl_set.c:466:
>  		else
> +			{}
> 
> ERROR: that open brace { should be on the previous line
> #771: FILE: drivers/staging/rtl8723bs/core/rtw_mlme.c:841:
>  					if (rtw_createbss_cmd(adapter) != _SUCCESS)
> +						{}
> 
> ERROR: that open brace { should be on the previous line
> #788: FILE: drivers/staging/rtl8723bs/core/rtw_mlme.c:950:
>  	else
> +		{}
> 
> ERROR: that open brace { should be on the previous line
> #858: FILE: drivers/staging/rtl8723bs/core/rtw_mlme.c:1262:
>  	if (pmlmepriv->assoc_ssid.SsidLength == 0)
> +		{}
> 
> ERROR: that open brace { should be on the previous line
> #861: FILE: drivers/staging/rtl8723bs/core/rtw_mlme.c:1264:
>  	else
> +		{}
> 
> ERROR: that open brace { should be on the previous line
> #952: FILE: drivers/staging/rtl8723bs/core/rtw_mlme.c:1614:
>  			if (rtw_createbss_cmd(adapter) != _SUCCESS)
> +				{}
> 
> ERROR: that open brace { should be on the previous line
> #1368: FILE: drivers/staging/rtl8723bs/core/rtw_recv.c:395:
>  		} else
> +			{}
> 
> ERROR: that open brace { should be on the previous line
> #1409: FILE: drivers/staging/rtl8723bs/core/rtw_recv.c:545:
>  			if (pattrib->bdecrypted == 0)
> +				{}
> 
> ERROR: that open brace { should be on the previous line
> #1569: FILE: drivers/staging/rtl8723bs/core/rtw_recv.c:1443:
>  		if (retval == _FAIL)
> +			{}
> 
> ERROR: that open brace { should be on the previous line
> #1576: FILE: drivers/staging/rtl8723bs/core/rtw_recv.c:1449:
>  		if (retval == _FAIL)
> +			{}
> 
> ERROR: that open brace { should be on the previous line
> #2086: FILE: drivers/staging/rtl8723bs/core/rtw_xmit.c:846:
>  			for (curfragnum = 0; curfragnum < pattrib->last_txcmdsz; curfragnum = curfragnum+8)
> +					{}
> 
> ERROR: that open brace { should be on the previous line
> #2722: FILE: drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c:651:
>  		else
> +			{}
> 
> ERROR: that open brace { should be on the previous line
> #2731: FILE: drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c:674:
>  		else
> +			{}
> 
> ERROR: that open brace { should be on the previous line
> #2749: FILE: drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c:681:
>  		else
> +			{}
> 
> total: 19 errors, 0 warnings, 0 checks, 3295 lines checked
> 
> NOTE: For some of the reported defects, checkpatch may be able to
>       mechanically convert to the typical style using --fix or --fix-inplace.
> 
> Your patch has style problems, please review.
> 
> NOTE: If any of the errors are false positives, please report
>       them to the maintainer, see CHECKPATCH in MAINTAINERS.
> 
> 

thank you Joe, this mail is so precious ;)

regards,

fabio





[Index of Archives]     [Linux Driver Development]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux