Commit dcc48e08 authored by Fabio Aiuto's avatar Fabio Aiuto Committed by Greg Kroah-Hartman
Browse files

staging: rtl8723bs: remove all DBG_8192C logs 



remove all DBG_8192C logs.

Macro DBG_8192C belongs to a family of verbose
private tracing macros.

The default behaviour is _do nothing_, to activate
it one should define DEBUG symbol by hand.

So just remove it with the following semantic patch:

@@
expression a, b, c, d, e, f, g, h, i, j, k;
constant B, C, D, E;
@@

(
-	DBG_8192C(a);
|
-	DBG_8192C(a, b);
|
-	DBG_8192C(a, B);
|
-	DBG_8192C(a, b, c);
|
-	DBG_8192C(a, B, c);
|
-	DBG_8192C(a, b, C);
|
-	DBG_8192C(a, B, C);
|
-	DBG_8192C(a, b, c, d);
|
-	DBG_8192C(a, B, c, d);
|
-	DBG_8192C(a, b, C, d);
|
-	DBG_8192C(a, b, c, D);
|
-	DBG_8192C(a, B, C, d);
|
-	DBG_8192C(a, B, c, D);
|
-	DBG_8192C(a, b, C, D);
|
-	DBG_8192C(a, B, C, D);
|
-	DBG_8192C(a, b, c, d, e);
|
-	DBG_8192C(a, B, c, d, e);
|
-	DBG_8192C(a, b, C, d, e);
|
-	DBG_8192C(a, b, c, D, e);
|
-	DBG_8192C(a, b, c, d, E);
|
-	DBG_8192C(a, B, C, d, e);
|
-	DBG_8192C(a, B, c, D, e);
|
-	DBG_8192C(a, B, c, d, E);
|
-	DBG_8192C(a, b, C, D, e);
|
-	DBG_8192C(a, b, C, d, E);
|
-	DBG_8192C(a, b, c, D, E);
|
-	DBG_8192C(a, B, C, D, e);
|
-	DBG_8192C(a, B, C, d, E);
|
-	DBG_8192C(a, B, c, D, E);
|
-	DBG_8192C(a, b, C, D, E);
|
-	DBG_8192C(a, B, C, D, E);
|
-	DBG_8192C(a, b, c, d, e, f);
|
-	DBG_8192C(a, b, c, d, e, f, g);
|
-	DBG_8192C(a, b, c, d, e, f, g, h);
|
-	DBG_8192C(a, b, c, d, e, f, g, h, i);
|
-	DBG_8192C(a, b, c, d, e, f, g, h, i, j);
|
-	DBG_8192C(a, b, c, d, e, f, g, h, i, j, k);
)

Signed-off-by: default avatarFabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/9b338b3781e40c04104f26832add075e7f72d890.1618064274.git.fabioaiuto83@gmail.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 426ceac4

drivers/staging/rtl8723bs/core/rtw_mlme_ext.c

+0 −1
Original line number Diff line number Diff line
@@ -4405,7 +4405,6 @@ static void process_80211d(struct adapter *padapter, struct wlan_bssid_ex *bssid 
#ifdef DEBUG_RTL871X

		i = 0;

		while ((i < chplan_ap.Len) && (chplan_ap.Channel[i] != 0)) {

			DBG_8192C("%02d,", chplan_ap.Channel[i]);

			i++;

		}

#endif

drivers/staging/rtl8723bs/core/rtw_pwrctrl.c

+0 −7
Original line number Diff line number Diff line
@@ -1072,9 +1072,7 @@ int _rtw_pwr_wakeup(struct adapter *padapter, u32 ips_deffer_ms, const char *cal 


	if (rf_off == pwrpriv->rf_pwrstate) {

		{

			DBG_8192C("%s call ips_leave....\n", __func__);

			if (ips_leave(padapter) == _FAIL) {

				DBG_8192C("======> ips_leave fail.............\n");

				ret = _FAIL;

				goto exit;

			}
@@ -1083,11 +1081,6 @@ int _rtw_pwr_wakeup(struct adapter *padapter, u32 ips_deffer_ms, const char *cal 


	/* TODO: the following checking need to be merged... */

	if (padapter->bDriverStopped || !padapter->bup || !padapter->hw_init_completed) {

		DBG_8192C("%s: bDriverStopped =%d, bup =%d, hw_init_completed =%u\n"

			, caller

			, padapter->bDriverStopped

			, padapter->bup

			, padapter->hw_init_completed);

		ret = false;

		goto exit;

	}

drivers/staging/rtl8723bs/core/rtw_xmit.c

+0 −4
Original line number Diff line number Diff line
@@ -1036,7 +1036,6 @@ s32 rtw_xmitframe_coalesce(struct adapter *padapter, struct sk_buff *pkt, struct 
	s32 res = _SUCCESS;



	if (!pxmitframe->buf_addr) {

		DBG_8192C("==> %s buf_addr == NULL\n", __func__);

		return _FAIL;

	}
@@ -1046,7 +1045,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) {

		DBG_8192C("rtw_xmitframe_coalesce: rtw_make_wlanhdr fail; drop pkt\n");

		res = _FAIL;

		goto exit;

	}
@@ -1121,7 +1119,6 @@ s32 rtw_xmitframe_coalesce(struct adapter *padapter, struct sk_buff *pkt, struct 
	}



	if (xmitframe_addmic(padapter, pxmitframe) == _FAIL) {

		DBG_8192C("xmitframe_addmic(padapter, pxmitframe) == _FAIL\n");

		res = _FAIL;

		goto exit;

	}
@@ -1808,7 +1805,6 @@ s32 rtw_xmit_classifier(struct adapter *padapter, struct xmit_frame *pxmitframe) 


	if (!psta) {

		res = _FAIL;

		DBG_8192C("rtw_xmit_classifier: psta == NULL\n");

		goto exit;

	}

drivers/staging/rtl8723bs/hal/hal_com.c

+0 −7
Original line number Diff line number Diff line
@@ -19,7 +19,6 @@ u8 rtw_hal_data_init(struct adapter *padapter) 
		padapter->hal_data_sz = sizeof(struct hal_com_data);

		padapter->HalData = vzalloc(padapter->hal_data_sz);

		if (!padapter->HalData) {

			DBG_8192C("cannot alloc memory for HAL DATA\n");

			return _FAIL;

		}

	}
@@ -1129,15 +1128,12 @@ u8 SetHalDefVar( 


		if (dm_func == 0) { /* disable all dynamic func */

			odm->SupportAbility = DYNAMIC_FUNC_DISABLE;

			DBG_8192C("==> Disable all dynamic function...\n");

		} else if (dm_func == 1) {/* disable DIG */

			odm->SupportAbility  &= (~DYNAMIC_BB_DIG);

			DBG_8192C("==> Disable DIG...\n");

		} else if (dm_func == 2) {/* disable High power */

			odm->SupportAbility  &= (~DYNAMIC_BB_DYNAMIC_TXPWR);

		} else if (dm_func == 3) {/* disable tx power tracking */

			odm->SupportAbility  &= (~DYNAMIC_RF_CALIBRATION);

			DBG_8192C("==> Disable tx power tracking...\n");

		} else if (dm_func == 4) {/* disable BT coexistence */

			dm->DMFlag &= (~DYNAMIC_FUNC_BT);

		} else if (dm_func == 5) {/* disable antenna diversity */
@@ -1149,7 +1145,6 @@ u8 SetHalDefVar( 
			}

			dm->DMFlag |= DYNAMIC_FUNC_BT;

			odm->SupportAbility = DYNAMIC_ALL_FUNC_ENABLE;

			DBG_8192C("==> Turn on all dynamic function...\n");

		}

	}

		break;
@@ -1254,10 +1249,8 @@ void SetHalODMVar( 
		{

			struct sta_info *psta = pValue1;

			if (bSet) {

				DBG_8192C("### Set STA_(%d) info ###\n", psta->mac_id);

				ODM_CmnInfoPtrArrayHook(podmpriv, ODM_CMNINFO_STA_STATUS, psta->mac_id, psta);

			} else {

				DBG_8192C("### Clean STA_(%d) info ###\n", psta->mac_id);

				/* spin_lock_bh(&pHalData->odm_stainfo_lock); */

				ODM_CmnInfoPtrArrayHook(podmpriv, ODM_CMNINFO_STA_STATUS, psta->mac_id, NULL);

drivers/staging/rtl8723bs/hal/rtl8723b_cmd.c

+0 −21
Original line number Diff line number Diff line
@@ -77,7 +77,6 @@ s32 FillH2CCmd8723B(struct adapter *padapter, u8 ElementID, u32 CmdLen, u8 *pCmd 
		h2c_box_num = pHalData->LastHMEBoxNum;



		if (!_is_fw_read_cmd_down(padapter, h2c_box_num)) {

			DBG_8192C(" fw read cmd failed...\n");

			/* DBG_8192C(" 0x1c0: 0x%8x\n", rtw_read32(padapter, 0x1c0)); */

			/* DBG_8192C(" 0x1c4: 0x%8x\n", rtw_read32(padapter, 0x1c4)); */

			goto exit;
@@ -667,9 +666,6 @@ void rtl8723b_download_rsvd_page(struct adapter *padapter, u8 mstatus) 
	u32 poll = 0;

	u8 val8;



	DBG_8192C("+" FUNC_ADPT_FMT ": iface_type =%d mstatus(%x)\n",

		FUNC_ADPT_ARG(padapter), get_iface_type(padapter), mstatus);



	if (mstatus == RT_MEDIA_CONNECT) {

		bool bRecover = false;

		u8 v8;
@@ -869,7 +865,6 @@ static void SetFwRsvdPagePkt_BTCoex(struct adapter *padapter) 


	pcmdframe = rtw_alloc_cmdxmitframe(pxmitpriv);

	if (!pcmdframe) {

		DBG_8192C("%s: alloc ReservedPagePacket fail!\n", __func__);

		return;

	}
@@ -913,8 +908,6 @@ static void SetFwRsvdPagePkt_BTCoex(struct adapter *padapter) 


	TotalPacketLen = BufIndex + BTQosNullLength;

	if (TotalPacketLen > MaxRsvdPageBufSize) {

		DBG_8192C(FUNC_ADPT_FMT ": ERROR: The rsvd page size is not enough!!TotalPacketLen %d, MaxRsvdPageBufSize %d\n",

			FUNC_ADPT_ARG(padapter), TotalPacketLen, MaxRsvdPageBufSize);

		goto error;

	}
@@ -947,14 +940,8 @@ void rtl8723b_download_BTCoex_AP_mode_rsvd_page(struct adapter *padapter) 
	u32 poll = 0;

	u8 val8;





	DBG_8192C("+" FUNC_ADPT_FMT ": iface_type =%d fw_state = 0x%08X\n",

		FUNC_ADPT_ARG(padapter), get_iface_type(padapter), get_fwstate(&padapter->mlmepriv));



#ifdef DEBUG

	if (check_fwstate(&padapter->mlmepriv, WIFI_AP_STATE) == false) {

		DBG_8192C(FUNC_ADPT_FMT ": [WARNING] not in AP mode!!\n",

			FUNC_ADPT_ARG(padapter));

	}

#endif /*  DEBUG */
@@ -1008,15 +995,7 @@ void rtl8723b_download_BTCoex_AP_mode_rsvd_page(struct adapter *padapter) 
	if (bcn_valid) {

		struct pwrctrl_priv *pwrctl = adapter_to_pwrctl(padapter);

		pwrctl->fw_psmode_iface_id = padapter->iface_id;

		DBG_8192C("%s: DL RSVD page success! DLBcnCount:%d, poll:%d\n",

			ADPT_ARG(padapter), DLBcnCount, poll);

	} else {

		DBG_8192C("%s: DL RSVD page fail! DLBcnCount:%d, poll:%d\n",

			ADPT_ARG(padapter), DLBcnCount, poll);

		DBG_8192C("%s: DL RSVD page fail! bSurpriseRemoved =%d\n",

			ADPT_ARG(padapter), padapter->bSurpriseRemoved);

		DBG_8192C("%s: DL RSVD page fail! bDriverStopped =%d\n",

			ADPT_ARG(padapter), padapter->bDriverStopped);

	}



	/*  2010.05.11. Added by tynli. */