[PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style

All of lore.kernel.org
 help / color / mirror / Atom feed

* [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style
@ 2017-09-10 15:10 Harsha Sharma
  2017-09-10 15:15 ` [PATCH 01/10] staging: rtl8723bs: Add spaces around '=' Harsha Sharma
                   ` (12 more replies)
  0 siblings, 13 replies; 30+ messages in thread
From: Harsha Sharma @ 2017-09-10 15:10 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, outreachy-kernel, Harsha Sharma

Linux kernel coding style uses one space around most binary and ternary
operators and spaces are prohibited at the start of a line. Concatenated
strings should have space between elements and space is required after ','.
In conditional statements, do not use unnecessary braces where a single
statement will do and put the opening brace last on the line, and put
the closing brace first. Unnecessary spaces and blank lines should be
avoided. 

Harsha Sharma (10):
  staging: rtl8723bs: Add spaces around '='
  staging: rtl8723bs: Add spaces around '+', '|', '*'
  staging: rtl8723bs: Add spaces around '?', '==', '||', '!='
  staging: rtl8723bs: Remove unnecessary spaces at the start of line
  staging: rtl8723bs: Add space between concatenated strings
  staging: rtl8723bs: Add space after ','
  staging: rtl8723bs: Removed unneccesary space
  staging: rtl8723bs: Remove unnecessary space
  staging: rtl8723bs: Remove unneccesary braces and change position of
    open brace
  staging: rtl8723bs: Remove unnecessary blank lines

 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 190 ++++++++++++----------------
 1 file changed, 78 insertions(+), 112 deletions(-)

-- 
1.9.1



^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH 01/10] staging: rtl8723bs: Add spaces around '='
  2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
@ 2017-09-10 15:15 ` Harsha Sharma
  2017-09-10 19:44   ` [Outreachy kernel] " Julia Lawall
  2017-09-10 15:17 ` [PATCH 02/10] staging: rtl8723bs: Add spaces around '+', '|', '*' Harsha Sharma
                   ` (11 subsequent siblings)
  12 siblings, 1 reply; 30+ messages in thread
From: Harsha Sharma @ 2017-09-10 15:15 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, outreachy-kernel, Harsha Sharma

Use one space around (on each side of) '=' operator

Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
---
 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
index 0215899..7f22ca9 100644
--- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
+++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
@@ -121,7 +121,7 @@
 
 static int rtw_enusbss = 0;/* 0:disable, 1:enable */
 
-static int rtw_hwpdn_mode =2;/* 0:disable, 1:enable, 2: by EFUSE config */
+static int rtw_hwpdn_mode = 2;/* 0:disable, 1:enable, 2: by EFUSE config */
 
 #ifdef CONFIG_HW_PWRP_DETECTION
 static int rtw_hwpwrp_detect = 1;
@@ -183,7 +183,7 @@
 
 module_param(rtw_hw_wps_pbc, int, 0644);
 
-static uint rtw_max_roaming_times =2;
+static uint rtw_max_roaming_times = 2;
 module_param(rtw_max_roaming_times, uint, 0644);
 MODULE_PARM_DESC(rtw_max_roaming_times,"The max roaming times to try");
 
@@ -255,8 +255,8 @@ static uint loadparam(struct adapter *padapter, _nic_hdl pnetdev)
 
 	registry_par->vrtl_carrier_sense = (u8)rtw_vrtl_carrier_sense ;
 	registry_par->vcs_type = (u8)rtw_vcs_type;
-	registry_par->rts_thresh =(u16)rtw_rts_thresh;
-	registry_par->frag_thresh =(u16)rtw_frag_thresh;
+	registry_par->rts_thresh = (u16)rtw_rts_thresh;
+	registry_par->frag_thresh = (u16)rtw_frag_thresh;
 	registry_par->preamble = (u8)rtw_preamble;
 	registry_par->scan_mode = (u8)rtw_scan_mode;
 	registry_par->adhoc_tx_pwr = (u8)rtw_adhoc_tx_pwr;
@@ -652,8 +652,8 @@ static u8 rtw_init_default_value(struct adapter *padapter)
 #ifdef CONFIG_GTK_OL
 	psecuritypriv->binstallKCK_KEK = _FAIL;
 #endif /* CONFIG_GTK_OL */
-	psecuritypriv->sw_encrypt =pregistrypriv->software_encrypt;
-	psecuritypriv->sw_decrypt =pregistrypriv->software_decrypt;
+	psecuritypriv->sw_encrypt = pregistrypriv->software_encrypt;
+	psecuritypriv->sw_decrypt = pregistrypriv->software_decrypt;
 
 	psecuritypriv->dot11AuthAlgrthm = dot11AuthAlgrthm_Open; /* open system */
 	psecuritypriv->dot11PrivacyAlgrthm = _NO_PRIVACY_;
@@ -777,7 +777,7 @@ u8 rtw_init_drv_sw(struct adapter *padapter)
 		goto exit;
 	}
 
-	padapter->cmdpriv.padapter =padapter;
+	padapter->cmdpriv.padapter = padapter;
 
 	if ((rtw_init_evt_priv(&padapter->evtpriv)) == _FAIL) {
 		RT_TRACE(_module_os_intfs_c_, _drv_err_, ("\n Can't init evt_priv\n"));
@@ -986,7 +986,7 @@ int _netdev_open(struct net_device *pnetdev)
 
 		DBG_871X("MAC Address = "MAC_FMT"\n", MAC_ARG(pnetdev->dev_addr));
 
-		status =rtw_start_drv_threads(padapter);
+		status = rtw_start_drv_threads(padapter);
 		if (status == _FAIL) {
 			DBG_871X("Initialize driver software resource Failed!\n");
 			goto netdev_open_error;
@@ -1641,7 +1641,7 @@ int rtw_resume_process_wow(struct adapter *padapter)
 		}
 
 		/* Disable WOW, set H2C command */
-		poidparam.subcode =WOWLAN_DISABLE;
+		poidparam.subcode = WOWLAN_DISABLE;
 		padapter->HalFunc.SetHwRegHandler(padapter, HW_VAR_WOWLAN, (u8 *)&poidparam);
 
 		psta = rtw_get_stainfo(&padapter->stapriv, get_bssid(&padapter->mlmepriv));
@@ -1705,7 +1705,7 @@ int rtw_resume_process_wow(struct adapter *padapter)
 		DBG_871X_LEVEL(_drv_always_, "do not reset timer\n");
 	}
 
-	pwrpriv->wowlan_mode =false;
+	pwrpriv->wowlan_mode = false;
 
 	/* clean driver side wake up reason. */
 	pwrpriv->wowlan_wake_reason = 0;
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 30+ messages in thread

* [PATCH 02/10] staging: rtl8723bs: Add spaces around '+', '|', '*'
  2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
  2017-09-10 15:15 ` [PATCH 01/10] staging: rtl8723bs: Add spaces around '=' Harsha Sharma
@ 2017-09-10 15:17 ` Harsha Sharma
  2017-09-10 19:36   ` [Outreachy kernel] " Julia Lawall
  2017-09-10 17:21 ` [PATCH 03/10] staging: rtl8723bs: Add spaces around '?', '==', '||', '!=' Harsha Sharma
                   ` (10 subsequent siblings)
  12 siblings, 1 reply; 30+ messages in thread
From: Harsha Sharma @ 2017-09-10 15:17 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, outreachy-kernel, Harsha Sharma

Use one space around most binary operators

Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
---
 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
index 7f22ca9..18f0ac2 100644
--- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
+++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
@@ -222,7 +222,7 @@
 /*  BIT4 - RF,					0: non-support, 1: support */
 /*  BIT5 - RF_TXPWR_TRACK,	0: non-support, 1: support */
 /*  BIT6 - RF_TXPWR_LMT,		0: non-support, 1: support */
-static int rtw_load_phy_file = (BIT2|BIT6);
+static int rtw_load_phy_file = (BIT2 | BIT6);
 module_param(rtw_load_phy_file, int, 0644);
 MODULE_PARM_DESC(rtw_load_phy_file,"PHY File Bit Map");
 static int rtw_decrypt_phy_file = 0;
@@ -440,12 +440,12 @@ u16 rtw_recv_select_queue(struct sk_buff *skb)
 	u32 priority;
 	u8 *pdata = skb->data;
 
-	memcpy(&eth_type, pdata+(ETH_ALEN<<1), 2);
+	memcpy(&eth_type, pdata + (ETH_ALEN << 1), 2);
 
 	switch (be16_to_cpu(eth_type)) {
 		case ETH_P_IP:
 
-			piphdr = (struct iphdr *)(pdata+ETH_HLEN);
+			piphdr = (struct iphdr *)(pdata + ETH_HLEN);
 
 			dscp = piphdr->tos & 0xfc;
 
@@ -561,7 +561,7 @@ struct net_device *rtw_init_netdev(struct adapter *old_padapter)
 	pnetdev->netdev_ops = &rtw_netdev_ops;
 
 	/* pnetdev->tx_timeout = NULL; */
-	pnetdev->watchdog_timeo = HZ*3; /* 3 second timeout */
+	pnetdev->watchdog_timeo = HZ * 3; /* 3 second timeout */
 	pnetdev->wireless_handlers = (struct iw_handler_def *)&rtw_handlers_def;
 
 	/* step 2. */
@@ -748,7 +748,7 @@ u8 rtw_reset_drv_sw(struct adapter *padapter)
 	pmlmepriv->LinkDetectInfo.TrafficTransitionCount = 0;
 	pmlmepriv->LinkDetectInfo.LowPowerTransitionCount = 0;
 
-	_clr_fwstate_(pmlmepriv, _FW_UNDER_SURVEY |_FW_UNDER_LINKING);
+	_clr_fwstate_(pmlmepriv, _FW_UNDER_SURVEY | _FW_UNDER_LINKING);
 
 	pwrctrlpriv->pwr_state_check_cnts = 0;
 
@@ -935,7 +935,7 @@ static int _rtw_drv_register_netdev(struct adapter *padapter, char *name)
 		goto error_register_netdev;
 	}
 
-	DBG_871X("%s, MAC Address (if%d) = " MAC_FMT "\n", __func__, (padapter->iface_id+1), MAC_ARG(pnetdev->dev_addr));
+	DBG_871X("%s, MAC Address (if%d) = " MAC_FMT "\n", __func__, (padapter->iface_id + 1), MAC_ARG(pnetdev->dev_addr));
 
 	return ret;
 
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 30+ messages in thread

* [PATCH 03/10] staging: rtl8723bs: Add spaces around '?', '==', '||', '!='
  2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
  2017-09-10 15:15 ` [PATCH 01/10] staging: rtl8723bs: Add spaces around '=' Harsha Sharma
  2017-09-10 15:17 ` [PATCH 02/10] staging: rtl8723bs: Add spaces around '+', '|', '*' Harsha Sharma
@ 2017-09-10 17:21 ` Harsha Sharma
  2017-09-10 20:01   ` [Outreachy kernel] " Julia Lawall
  2017-09-10 17:23 ` [PATCH 04/10] staging: rtl8723bs: Remove unnecessary spaces at the start of line Harsha Sharma
                   ` (9 subsequent siblings)
  12 siblings, 1 reply; 30+ messages in thread
From: Harsha Sharma @ 2017-09-10 17:21 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, outreachy-kernel, Harsha Sharma

Use spaces around most binary operators

Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
---
 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
index 18f0ac2..5ceccd7 100644
--- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
+++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
@@ -1140,7 +1140,7 @@ static int pm_netdev_open(struct net_device *pnetdev, u8 bnormal)
 		}
 	}
 	else
-		status =  (_SUCCESS == ips_netdrv_open(padapter))?(0):(-1);
+		status =  (_SUCCESS == ips_netdrv_open(padapter)) ? (0) : (-1);
 
 	return status;
 }
@@ -1262,7 +1262,7 @@ void rtw_dev_unload(struct adapter *padapter)
 			rtw_btcoex_IpsNotify(padapter, pwrctl->ips_mode_req);
 #ifdef CONFIG_WOWLAN
 			if (pwrctl->bSupportRemoteWakeup == true &&
-				pwrctl->wowlan_mode ==true) {
+				pwrctl->wowlan_mode == true) {
 				DBG_871X_LEVEL(_drv_always_, "%s bSupportRemoteWakeup ==true  do not run rtw_hal_deinit()\n", __func__);
 			}
 			else
@@ -1524,7 +1524,7 @@ int rtw_suspend_common(struct adapter *padapter)
 	while (pwrpriv->bips_processing == true)
 		msleep(1);
 
-	if ((!padapter->bup) || (padapter->bDriverStopped)||(padapter->bSurpriseRemoved))
+	if ((!padapter->bup) || (padapter->bDriverStopped) || (padapter->bSurpriseRemoved))
 	{
 		DBG_871X("%s bup =%d bDriverStopped =%d bSurpriseRemoved = %d\n", __func__
 			, padapter->bup, padapter->bDriverStopped, padapter->bSurpriseRemoved);
@@ -1671,7 +1671,7 @@ int rtw_resume_process_wow(struct adapter *padapter)
 		DBG_871X_LEVEL(_drv_always_, "%s: ### ERROR ### wowlan_mode =%d\n", __func__, pwrpriv->wowlan_mode);
 	}
 
-	if (padapter->pid[1]!= 0) {
+	if (padapter->pid[1] != 0) {
 		DBG_871X("pid[1]:%d\n", padapter->pid[1]);
 		rtw_signal_process(padapter->pid[1], SIGUSR2);
 	}
@@ -1774,7 +1774,7 @@ int rtw_resume_process_ap_wow(struct adapter *padapter)
 			rtw_netif_wake_queue(pnetdev);
 	}
 
-	if (padapter->pid[1]!= 0) {
+	if (padapter->pid[1] != 0) {
 		DBG_871X("pid[1]:%d\n", padapter->pid[1]);
 		rtw_signal_process(padapter->pid[1], SIGUSR2);
 	}
@@ -1842,7 +1842,7 @@ static int rtw_resume_process_normal(struct adapter *padapter)
 	netif_device_attach(pnetdev);
 	netif_carrier_on(pnetdev);
 
-	if (padapter->pid[1]!= 0) {
+	if (padapter->pid[1] != 0) {
 		DBG_871X("pid[1]:%d\n", padapter->pid[1]);
 		rtw_signal_process(padapter->pid[1], SIGUSR2);
 	}
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 30+ messages in thread

* [PATCH 04/10] staging: rtl8723bs: Remove unnecessary spaces at the start of line
  2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
                   ` (2 preceding siblings ...)
  2017-09-10 17:21 ` [PATCH 03/10] staging: rtl8723bs: Add spaces around '?', '==', '||', '!=' Harsha Sharma
@ 2017-09-10 17:23 ` Harsha Sharma
  2017-09-10 20:33   ` [Outreachy kernel] " Julia Lawall
  2017-09-10 17:29 ` [PATCH 05/10] staging: rtl8723bs: Add space between concatenated strings Harsha Sharma
                   ` (8 subsequent siblings)
  12 siblings, 1 reply; 30+ messages in thread
From: Harsha Sharma @ 2017-09-10 17:23 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, outreachy-kernel, Harsha Sharma

No spaces at the start of a line

Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
---
 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
index 5ceccd7..97ff3ab 100644
--- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
+++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
@@ -597,7 +597,7 @@ u32 rtw_start_drv_threads(struct adapter *padapter)
 		_status = _FAIL;
 
 	padapter->cmdThread = kthread_run(rtw_cmd_thread, padapter, "RTW_CMD_THREAD");
-        if (IS_ERR(padapter->cmdThread))
+	if (IS_ERR(padapter->cmdThread))
 		_status = _FAIL;
 	else
 		down(&padapter->cmdpriv.terminate_cmdthread_sema); /* wait for cmd_thread to run */
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 30+ messages in thread

* [PATCH 05/10] staging: rtl8723bs: Add space between concatenated strings
  2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
                   ` (3 preceding siblings ...)
  2017-09-10 17:23 ` [PATCH 04/10] staging: rtl8723bs: Remove unnecessary spaces at the start of line Harsha Sharma
@ 2017-09-10 17:29 ` Harsha Sharma
  2017-09-11 11:09   ` [Outreachy kernel] " Julia Lawall
  2017-09-10 17:32 ` [PATCH 06/10] staging: rtl8723bs: Add space after ',' Harsha Sharma
                   ` (7 subsequent siblings)
  12 siblings, 1 reply; 30+ messages in thread
From: Harsha Sharma @ 2017-09-10 17:29 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, outreachy-kernel, Harsha Sharma

Use spaces between concatenated strings

Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
---
 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 50 ++++++++++++++---------------
 1 file changed, 25 insertions(+), 25 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
index 97ff3ab..b320df8 100644
--- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
+++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
@@ -467,7 +467,7 @@ static int rtw_ndev_notifier_call(struct notifier_block * nb, unsigned long stat
 	if (dev->netdev_ops->ndo_do_ioctl != rtw_ioctl)
 		return NOTIFY_DONE;
 
-	DBG_871X_LEVEL(_drv_info_, FUNC_NDEV_FMT" state:%lu\n", FUNC_NDEV_ARG(dev), state);
+	DBG_871X_LEVEL(_drv_info_, FUNC_NDEV_FMT " state:%lu\n", FUNC_NDEV_ARG(dev), state);
 
 	switch (state) {
 	case NETDEV_CHANGENAME:
@@ -497,7 +497,7 @@ static int rtw_ndev_init(struct net_device *dev)
 {
 	struct adapter *adapter = rtw_netdev_priv(dev);
 
-	DBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT"\n", FUNC_ADPT_ARG(adapter));
+	DBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT "\n", FUNC_ADPT_ARG(adapter));
 	strncpy(adapter->old_ifname, dev->name, IFNAMSIZ);
 	rtw_adapter_proc_init(dev);
 
@@ -508,7 +508,7 @@ static void rtw_ndev_uninit(struct net_device *dev)
 {
 	struct adapter *adapter = rtw_netdev_priv(dev);
 
-	DBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT"\n", FUNC_ADPT_ARG(adapter));
+	DBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT "\n", FUNC_ADPT_ARG(adapter));
 	rtw_adapter_proc_deinit(dev);
 }
 
@@ -984,7 +984,7 @@ int _netdev_open(struct net_device *pnetdev)
 			goto netdev_open_error;
 		}
 
-		DBG_871X("MAC Address = "MAC_FMT"\n", MAC_ARG(pnetdev->dev_addr));
+		DBG_871X("MAC Address = " MAC_FMT "\n", MAC_ARG(pnetdev->dev_addr));
 
 		status = rtw_start_drv_threads(padapter);
 		if (status == _FAIL) {
@@ -1203,7 +1203,7 @@ static int netdev_close(struct net_device *pnetdev)
 
 void rtw_ndev_destructor(struct net_device *ndev)
 {
-	DBG_871X(FUNC_NDEV_FMT"\n", FUNC_NDEV_ARG(ndev));
+	DBG_871X(FUNC_NDEV_FMT "\n", FUNC_NDEV_ARG(ndev));
 
 	if (ndev->ieee80211_ptr)
 		kfree((u8 *)ndev->ieee80211_ptr);
@@ -1292,7 +1292,7 @@ static int rtw_suspend_free_assoc_resource(struct adapter *padapter)
 {
 	struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
 
-	DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", FUNC_ADPT_ARG(padapter));
+	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
 
 	if (rtw_chk_roam_flags(padapter, RTW_ROAM_ON_RESUME)) {
 		if (check_fwstate(pmlmepriv, WIFI_STATION_STATE)
@@ -1333,7 +1333,7 @@ static int rtw_suspend_free_assoc_resource(struct adapter *padapter)
 		rtw_indicate_disconnect(padapter);
 	}
 
-	DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", FUNC_ADPT_ARG(padapter));
+	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
 	return _SUCCESS;
 }
 
@@ -1347,7 +1347,7 @@ int rtw_suspend_wow(struct adapter *padapter)
 	struct wowlan_ioctl_param poidparam;
 	int ret = _SUCCESS;
 
-	DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", FUNC_ADPT_ARG(padapter));
+	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
 
 
 	DBG_871X("wowlan_mode: %d\n", pwrpriv->wowlan_mode);
@@ -1401,7 +1401,7 @@ int rtw_suspend_wow(struct adapter *padapter)
 		}
 
 		if (rtw_get_ch_setting_union(padapter, &ch, &bw, &offset) != 0) {
-			DBG_871X(FUNC_ADPT_FMT" back to linked/linking union - ch:%u, bw:%u, offset:%u\n",
+			DBG_871X(FUNC_ADPT_FMT " back to linked/linking union - ch:%u, bw:%u, offset:%u\n",
 				FUNC_ADPT_ARG(padapter), ch, bw, offset);
 			set_channel_bwmode(padapter, ch, offset, bw);
 		}
@@ -1416,7 +1416,7 @@ int rtw_suspend_wow(struct adapter *padapter)
 	{
 		DBG_871X_LEVEL(_drv_always_, "%s: ### ERROR ### wowlan_mode =%d\n", __func__, pwrpriv->wowlan_mode);
 	}
-	DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", FUNC_ADPT_ARG(padapter));
+	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
 	return ret;
 }
 #endif /* ifdef CONFIG_WOWLAN */
@@ -1430,7 +1430,7 @@ int rtw_suspend_ap_wow(struct adapter *padapter)
 	struct wowlan_ioctl_param poidparam;
 	int ret = _SUCCESS;
 
-	DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", FUNC_ADPT_ARG(padapter));
+	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
 
 	pwrpriv->wowlan_ap_mode = true;
 
@@ -1462,14 +1462,14 @@ int rtw_suspend_ap_wow(struct adapter *padapter)
 	DBG_871X_LEVEL(_drv_always_, "%s: wowmode suspending\n", __func__);
 
 	if (rtw_get_ch_setting_union(padapter, &ch, &bw, &offset) != 0) {
-		DBG_871X(FUNC_ADPT_FMT" back to linked/linking union - ch:%u, bw:%u, offset:%u\n",
+		DBG_871X(FUNC_ADPT_FMT " back to linked/linking union - ch:%u, bw:%u, offset:%u\n",
 			 FUNC_ADPT_ARG(padapter), ch, bw, offset);
 		set_channel_bwmode(padapter, ch, offset, bw);
 	}
 
 	rtw_set_ps_mode(padapter, PS_MODE_MIN, 0, 0, "AP-WOWLAN");
 
-	DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", FUNC_ADPT_ARG(padapter));
+	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
 	return ret;
 }
 #endif /* ifdef CONFIG_AP_WOWLAN */
@@ -1480,7 +1480,7 @@ static int rtw_suspend_normal(struct adapter *padapter)
 	struct net_device *pnetdev = padapter->pnetdev;
 	int ret = _SUCCESS;
 
-	DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", FUNC_ADPT_ARG(padapter));
+	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
 	if (pnetdev) {
 		netif_carrier_off(pnetdev);
 		rtw_netif_stop_queue(pnetdev);
@@ -1501,7 +1501,7 @@ static int rtw_suspend_normal(struct adapter *padapter)
 	if (padapter->intf_deinit)
 		padapter->intf_deinit(adapter_to_dvobj(padapter));
 
-	DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", FUNC_ADPT_ARG(padapter));
+	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
 	return ret;
 }
 
@@ -1599,7 +1599,7 @@ int rtw_resume_process_wow(struct adapter *padapter)
 	struct sta_info *psta = NULL;
 	int ret = _SUCCESS;
 
-	DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", FUNC_ADPT_ARG(padapter));
+	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
 
 	if (padapter) {
 		pnetdev = padapter->pnetdev;
@@ -1710,7 +1710,7 @@ int rtw_resume_process_wow(struct adapter *padapter)
 	/* clean driver side wake up reason. */
 	pwrpriv->wowlan_wake_reason = 0;
 exit:
-	DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", FUNC_ADPT_ARG(padapter));
+	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
 	return ret;
 }
 #endif /* ifdef CONFIG_WOWLAN */
@@ -1725,7 +1725,7 @@ int rtw_resume_process_ap_wow(struct adapter *padapter)
 	struct wowlan_ioctl_param poidparam;
 	int ret = _SUCCESS;
 
-	DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", FUNC_ADPT_ARG(padapter));
+	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
 
 	if (padapter) {
 		pnetdev = padapter->pnetdev;
@@ -1785,7 +1785,7 @@ int rtw_resume_process_ap_wow(struct adapter *padapter)
 	/* clean driver side wake up reason. */
 	pwrpriv->wowlan_wake_reason = 0;
 exit:
-	DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", FUNC_ADPT_ARG(padapter));
+	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
 	return ret;
 }
 #endif /* ifdef CONFIG_APWOWLAN */
@@ -1811,7 +1811,7 @@ static int rtw_resume_process_normal(struct adapter *padapter)
 	psdpriv = padapter->dvobj;
 	pdbgpriv = &psdpriv->drv_dbg;
 
-	DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", FUNC_ADPT_ARG(padapter));
+	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
 	/*  interface init */
 	/* if (sdio_init(adapter_to_dvobj(padapter)) != _SUCCESS) */
 	if ((padapter->intf_init) && (padapter->intf_init(adapter_to_dvobj(padapter)) != _SUCCESS))
@@ -1849,21 +1849,21 @@ static int rtw_resume_process_normal(struct adapter *padapter)
 
 
 	if (check_fwstate(pmlmepriv, WIFI_STATION_STATE)) {
-		DBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x - WIFI_STATION_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
+		DBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x - WIFI_STATION_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
 
 		if (rtw_chk_roam_flags(padapter, RTW_ROAM_ON_RESUME))
 			rtw_roaming(padapter, NULL);
 
 	} else if (check_fwstate(pmlmepriv, WIFI_AP_STATE)) {
-		DBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x - WIFI_AP_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
+		DBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x - WIFI_AP_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
 		rtw_ap_restore_network(padapter);
 	} else if (check_fwstate(pmlmepriv, WIFI_ADHOC_STATE)) {
-		DBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x - WIFI_ADHOC_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
+		DBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x - WIFI_ADHOC_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
 	} else {
-		DBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x - ???\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
+		DBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x - ???\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
 	}
 
-	DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", FUNC_ADPT_ARG(padapter));
+	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
 
 exit:
 	return ret;
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 30+ messages in thread

* [PATCH 06/10] staging: rtl8723bs: Add space after ','
  2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
                   ` (4 preceding siblings ...)
  2017-09-10 17:29 ` [PATCH 05/10] staging: rtl8723bs: Add space between concatenated strings Harsha Sharma
@ 2017-09-10 17:32 ` Harsha Sharma
  2017-09-11 11:13   ` [Outreachy kernel] " Julia Lawall
  2017-09-11 16:44   ` Greg KH
  2017-09-10 17:34 ` [PATCH 07/10] staging: rtl8723bs: Removed unneccesary space Harsha Sharma
                   ` (6 subsequent siblings)
  12 siblings, 2 replies; 30+ messages in thread
From: Harsha Sharma @ 2017-09-10 17:32 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, outreachy-kernel, Harsha Sharma

Space required after ','

Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
---
 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
index b320df8..ecb01cf 100644
--- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
+++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
@@ -45,7 +45,7 @@
 static int rtw_power_mgnt = 1;
 static int rtw_ips_mode = IPS_NORMAL;
 module_param(rtw_ips_mode, int, 0644);
-MODULE_PARM_DESC(rtw_ips_mode,"The default IPS mode");
+MODULE_PARM_DESC(rtw_ips_mode, "The default IPS mode");
 
 static int rtw_smart_ps = 2;
 
@@ -185,7 +185,7 @@
 
 static uint rtw_max_roaming_times = 2;
 module_param(rtw_max_roaming_times, uint, 0644);
-MODULE_PARM_DESC(rtw_max_roaming_times,"The max roaming times to try");
+MODULE_PARM_DESC(rtw_max_roaming_times, "The max roaming times to try");
 
 module_param(rtw_mc2u_disable, int, 0644);
 
@@ -206,10 +206,10 @@
 static int rtw_tx_pwr_by_rate = 0;
 
 module_param(rtw_tx_pwr_lmt_enable, int, 0644);
-MODULE_PARM_DESC(rtw_tx_pwr_lmt_enable,"0:Disable, 1:Enable, 2: Depend on efuse");
+MODULE_PARM_DESC(rtw_tx_pwr_lmt_enable, "0:Disable, 1:Enable, 2: Depend on efuse");
 
 module_param(rtw_tx_pwr_by_rate, int, 0644);
-MODULE_PARM_DESC(rtw_tx_pwr_by_rate,"0:Disable, 1:Enable, 2: Depend on efuse");
+MODULE_PARM_DESC(rtw_tx_pwr_by_rate, "0:Disable, 1:Enable, 2: Depend on efuse");
 
 char *rtw_phy_file_path = "";
 module_param(rtw_phy_file_path, charp, 0644);
@@ -224,10 +224,10 @@
 /*  BIT6 - RF_TXPWR_LMT,		0: non-support, 1: support */
 static int rtw_load_phy_file = (BIT2 | BIT6);
 module_param(rtw_load_phy_file, int, 0644);
-MODULE_PARM_DESC(rtw_load_phy_file,"PHY File Bit Map");
+MODULE_PARM_DESC(rtw_load_phy_file, "PHY File Bit Map");
 static int rtw_decrypt_phy_file = 0;
 module_param(rtw_decrypt_phy_file, int, 0644);
-MODULE_PARM_DESC(rtw_decrypt_phy_file,"Enable Decrypt PHY File");
+MODULE_PARM_DESC(rtw_decrypt_phy_file, "Enable Decrypt PHY File");
 
 int _netdev_open(struct net_device *pnetdev);
 int netdev_open (struct net_device *pnetdev);
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 30+ messages in thread

* [PATCH 07/10] staging: rtl8723bs: Removed unneccesary space
  2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
                   ` (5 preceding siblings ...)
  2017-09-10 17:32 ` [PATCH 06/10] staging: rtl8723bs: Add space after ',' Harsha Sharma
@ 2017-09-10 17:34 ` Harsha Sharma
  2017-09-11 16:46   ` Greg KH
  2017-09-10 17:36 ` [PATCH 08/10] staging: rtl8723bs: Remove unnecessary space Harsha Sharma
                   ` (5 subsequent siblings)
  12 siblings, 1 reply; 30+ messages in thread
From: Harsha Sharma @ 2017-09-10 17:34 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, outreachy-kernel, Harsha Sharma

Remove space between function name and open parenthesis '('

Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
---
 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
index ecb01cf..8e13fb3 100644
--- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
+++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
@@ -230,8 +230,8 @@
 MODULE_PARM_DESC(rtw_decrypt_phy_file, "Enable Decrypt PHY File");
 
 int _netdev_open(struct net_device *pnetdev);
-int netdev_open (struct net_device *pnetdev);
-static int netdev_close (struct net_device *pnetdev);
+int netdev_open(struct net_device *pnetdev);
+static int netdev_close(struct net_device *pnetdev);
 
 static uint loadparam(struct adapter *padapter, _nic_hdl pnetdev)
 {
@@ -606,7 +606,7 @@ u32 rtw_start_drv_threads(struct adapter *padapter)
 	return _status;
 }
 
-void rtw_stop_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"));
 
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 30+ messages in thread

* [PATCH 08/10] staging: rtl8723bs: Remove unnecessary space
  2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
                   ` (6 preceding siblings ...)
  2017-09-10 17:34 ` [PATCH 07/10] staging: rtl8723bs: Removed unneccesary space Harsha Sharma
@ 2017-09-10 17:36 ` Harsha Sharma
  2017-09-10 17:39 ` [PATCH 09/10] staging: rtl8723bs: Remove unneccesary braces and change position of open brace Harsha Sharma
                   ` (4 subsequent siblings)
  12 siblings, 0 replies; 30+ messages in thread
From: Harsha Sharma @ 2017-09-10 17:36 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, outreachy-kernel, Harsha Sharma

Remove space before ';', '++', ','

Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
---
 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
index 8e13fb3..7a2d088 100644
--- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
+++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
@@ -253,7 +253,7 @@ static uint loadparam(struct adapter *padapter, _nic_hdl pnetdev)
 	if (registry_par->channel > 14)
 		registry_par->channel = 1;
 
-	registry_par->vrtl_carrier_sense = (u8)rtw_vrtl_carrier_sense ;
+	registry_par->vrtl_carrier_sense = (u8)rtw_vrtl_carrier_sense;
 	registry_par->vcs_type = (u8)rtw_vcs_type;
 	registry_par->rts_thresh = (u16)rtw_rts_thresh;
 	registry_par->frag_thresh = (u16)rtw_frag_thresh;
@@ -1240,7 +1240,7 @@ void rtw_dev_unload(struct adapter *padapter)
 				DBG_871X("stop cmdthd timeout\n");
 				break;
 			} else {
-				cnt ++;
+				cnt++;
 				DBG_871X("cmdthd is running(%d)\n", cnt);
 				msleep(10);
 			}
@@ -1907,7 +1907,7 @@ int rtw_resume_common(struct adapter *padapter)
 		pwrpriv->pno_in_resume = false;
 	#endif
 	}
-	DBG_871X_LEVEL(_drv_always_, "%s:%d in %d ms\n", __func__ , ret,
+	DBG_871X_LEVEL(_drv_always_, "%s:%d in %d ms\n", __func__, ret,
 		jiffies_to_msecs(jiffies - start_time));
 
 	return ret;
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 30+ messages in thread

* [PATCH 09/10] staging: rtl8723bs: Remove unneccesary braces and change position of open brace
  2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
                   ` (7 preceding siblings ...)
  2017-09-10 17:36 ` [PATCH 08/10] staging: rtl8723bs: Remove unnecessary space Harsha Sharma
@ 2017-09-10 17:39 ` Harsha Sharma
  2017-09-10 20:18   ` [PATCH] spelling.txt: Add "unnecessary" typo variants Joe Perches
  2017-09-10 17:41 ` [PATCH 10/10] staging: rtl8723bs: Remove unnecessary blank lines Harsha Sharma
                   ` (3 subsequent siblings)
  12 siblings, 1 reply; 30+ messages in thread
From: Harsha Sharma @ 2017-09-10 17:39 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, outreachy-kernel, Harsha Sharma

Follow linux-kernel code style for conditional statements

Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
---
 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 63 +++++++++--------------------
 1 file changed, 19 insertions(+), 44 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
index 7a2d088..fb8502a 100644
--- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
+++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
@@ -351,8 +351,7 @@ static int rtw_net_set_mac_address(struct net_device *pnetdev, void *p)
 	struct adapter *padapter = (struct adapter *)rtw_netdev_priv(pnetdev);
 	struct sockaddr *addr = p;
 
-	if (padapter->bup == false)
-	{
+	if (padapter->bup == false) {
 		/* DBG_871X("r8711_net_set_mac_address(), MAC =%x:%x:%x:%x:%x:%x\n", addr->sa_data[0], addr->sa_data[1], addr->sa_data[2], addr->sa_data[3], */
 		/* addr->sa_data[4], addr->sa_data[5]); */
 		memcpy(padapter->eeprompriv.mac_addr, addr->sa_data, ETH_ALEN);
@@ -425,9 +424,7 @@ static u16 rtw_select_queue(struct net_device *dev, struct sk_buff *skb
 	skb->priority = rtw_classify8021d(skb);
 
 	if (pmlmepriv->acm_mask != 0)
-	{
 		skb->priority = qos_acm(pmlmepriv->acm_mask, skb->priority);
-	}
 
 	return rtw_1d_to_queue[skb->priority];
 }
@@ -1036,8 +1033,7 @@ int netdev_open(struct net_device *pnetdev)
 	struct adapter *padapter = (struct adapter *)rtw_netdev_priv(pnetdev);
 	struct pwrctrl_priv *pwrctrlpriv = adapter_to_pwrctl(padapter);
 
-	if (pwrctrlpriv->bInSuspend == true)
-	{
+	if (pwrctrlpriv->bInSuspend == true) {
 		DBG_871X("+871x_drv - drv_open, bInSuspend =%d\n", pwrctrlpriv->bInSuspend);
 		return 0;
 	}
@@ -1066,16 +1062,13 @@ static int  ips_netdrv_open(struct adapter *padapter)
 	/* padapter->bup = true; */
 
 	status = rtw_hal_init(padapter);
-	if (status == _FAIL)
-	{
+	if (status == _FAIL) {
 		RT_TRACE(_module_os_intfs_c_, _drv_err_, ("ips_netdrv_open(): Can't init h/w!\n"));
 		goto netdev_open_error;
 	}
 
 	if (padapter->intf_start)
-	{
 		padapter->intf_start(padapter);
-	}
 
 	_set_timer(&padapter->mlmepriv.dynamic_chk_timer, 2000);
 
@@ -1119,9 +1112,7 @@ void rtw_ips_dev_unload(struct adapter *padapter)
 
 
 	if (padapter->bSurpriseRemoved == false)
-	{
 		rtw_hal_deinit(padapter);
-	}
 
 }
 
@@ -1132,8 +1123,7 @@ static int pm_netdev_open(struct net_device *pnetdev, u8 bnormal)
 
 	struct adapter *padapter = (struct adapter *)rtw_netdev_priv(pnetdev);
 
-	if (true == bnormal)
-	{
+	if (true == bnormal) {
 		if (mutex_lock_interruptible(&(adapter_to_dvobj(padapter)->hw_init_mutex)) == 0) {
 			status = _netdev_open(pnetdev);
 			mutex_unlock(&(adapter_to_dvobj(padapter)->hw_init_mutex));
@@ -1152,8 +1142,7 @@ static int netdev_close(struct net_device *pnetdev)
 
 	RT_TRACE(_module_os_intfs_c_, _drv_info_, ("+871x_drv - drv_close\n"));
 
-	if (pwrctl->bInternalAutoSuspend == true)
-	{
+	if (pwrctl->bInternalAutoSuspend == true) {
 		/* rtw_pwr_wakeup(padapter); */
 		if (pwrctl->rf_pwrstate == rf_off)
 			pwrctl->ps_flag = true;
@@ -1174,8 +1163,7 @@ static int netdev_close(struct net_device *pnetdev)
 		DBG_871X("(2)871x_drv - drv_close, bup =%d, hw_init_completed =%d\n", padapter->bup, padapter->hw_init_completed);
 
 		/* s1. */
-		if (pnetdev)
-		{
+		if (pnetdev) {
 			if (!rtw_netif_queue_stopped(pnetdev))
 				rtw_netif_stop_queue(pnetdev);
 		}
@@ -1219,8 +1207,7 @@ void rtw_dev_unload(struct adapter *padapter)
 
 	RT_TRACE(_module_hci_intfs_c_, _drv_notice_, ("+%s\n", __func__));
 
-	if (padapter->bup == true)
-	{
+	if (padapter->bup == true) {
 		DBG_871X("===> %s\n", __func__);
 
 		padapter->bDriverStopped = true;
@@ -1257,8 +1244,7 @@ void rtw_dev_unload(struct adapter *padapter)
 			DBG_871X_LEVEL(_drv_always_, "%s: driver not in IPS\n", __func__);
 		}
 
-		if (padapter->bSurpriseRemoved == false)
-		{
+		if (padapter->bSurpriseRemoved == false) {
 			rtw_btcoex_IpsNotify(padapter, pwrctl->ips_mode_req);
 #ifdef CONFIG_WOWLAN
 			if (pwrctl->bSupportRemoteWakeup == true &&
@@ -1296,8 +1282,7 @@ static int rtw_suspend_free_assoc_resource(struct adapter *padapter)
 
 	if (rtw_chk_roam_flags(padapter, RTW_ROAM_ON_RESUME)) {
 		if (check_fwstate(pmlmepriv, WIFI_STATION_STATE)
-			&& check_fwstate(pmlmepriv, _FW_LINKED))
-		{
+			&& check_fwstate(pmlmepriv, _FW_LINKED)) {
 			DBG_871X("%s %s(" MAC_FMT "), length:%d assoc_ssid.length:%d\n", __func__,
 					pmlmepriv->cur_network.network.Ssid.Ssid,
 					MAC_ARG(pmlmepriv->cur_network.network.MacAddress),
@@ -1307,14 +1292,12 @@ static int rtw_suspend_free_assoc_resource(struct adapter *padapter)
 		}
 	}
 
-	if (check_fwstate(pmlmepriv, WIFI_STATION_STATE) && check_fwstate(pmlmepriv, _FW_LINKED))
-	{
+	if (check_fwstate(pmlmepriv, WIFI_STATION_STATE) && check_fwstate(pmlmepriv, _FW_LINKED)) {
 		rtw_disassoc_cmd(padapter, 0, false);
 		/* s2-2.  indicate disconnect to os */
 		rtw_indicate_disconnect(padapter);
 	}
-	else if (check_fwstate(pmlmepriv, WIFI_AP_STATE))
-	{
+	else if (check_fwstate(pmlmepriv, WIFI_AP_STATE)) {
 		rtw_sta_flush(padapter);
 	}
 
@@ -1327,8 +1310,7 @@ static int rtw_suspend_free_assoc_resource(struct adapter *padapter)
 	if (check_fwstate(pmlmepriv, _FW_UNDER_SURVEY))
 		rtw_indicate_scan_done(padapter, 1);
 
-	if (check_fwstate(pmlmepriv, _FW_UNDER_LINKING) == true)
-	{
+	if (check_fwstate(pmlmepriv, _FW_UNDER_LINKING) == true) {
 		DBG_871X_LEVEL(_drv_always_, "%s: fw_under_linking\n", __func__);
 		rtw_indicate_disconnect(padapter);
 	}
@@ -1379,8 +1361,7 @@ int rtw_suspend_wow(struct adapter *padapter)
 		padapter->HalFunc.SetHwRegHandler(padapter, HW_VAR_WOWLAN, (u8 *)&poidparam);
 		if (rtw_chk_roam_flags(padapter, RTW_ROAM_ON_RESUME)) {
 			if (check_fwstate(pmlmepriv, WIFI_STATION_STATE)
-				&& check_fwstate(pmlmepriv, _FW_LINKED))
-			{
+				&& check_fwstate(pmlmepriv, _FW_LINKED)) {
 				DBG_871X("%s %s(" MAC_FMT "), length:%d assoc_ssid.length:%d\n", __func__,
 						pmlmepriv->cur_network.network.Ssid.Ssid,
 						MAC_ARG(pmlmepriv->cur_network.network.MacAddress),
@@ -1393,8 +1374,7 @@ int rtw_suspend_wow(struct adapter *padapter)
 
 		DBG_871X_LEVEL(_drv_always_, "%s: wowmode suspending\n", __func__);
 
-		if (check_fwstate(pmlmepriv, _FW_UNDER_SURVEY) == true)
-		{
+		if (check_fwstate(pmlmepriv, _FW_UNDER_SURVEY) == true) {
 			DBG_871X_LEVEL(_drv_always_, "%s: fw_under_survey\n", __func__);
 			rtw_indicate_scan_done(padapter, 1);
 			clr_fwstate(pmlmepriv, _FW_UNDER_SURVEY);
@@ -1412,8 +1392,7 @@ int rtw_suspend_wow(struct adapter *padapter)
 			rtw_set_ps_mode(padapter, PS_MODE_DTIM, 0, 0, "WOWLAN");
 
 	}
-	else
-	{
+	else {
 		DBG_871X_LEVEL(_drv_always_, "%s: ### ERROR ### wowlan_mode =%d\n", __func__, pwrpriv->wowlan_mode);
 	}
 	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
@@ -1489,8 +1468,7 @@ static int rtw_suspend_normal(struct adapter *padapter)
 	rtw_suspend_free_assoc_resource(padapter);
 
 	if ((rtw_hal_check_ips_status(padapter) == true)
-		|| (adapter_to_pwrctl(padapter)->rf_pwrstate == rf_off))
-	{
+		|| (adapter_to_pwrctl(padapter)->rf_pwrstate == rf_off)) {
 		DBG_871X_LEVEL(_drv_always_, "%s: ### ERROR #### driver in IPS ####ERROR###!!!\n", __func__);
 
 	}
@@ -1524,8 +1502,7 @@ int rtw_suspend_common(struct adapter *padapter)
 	while (pwrpriv->bips_processing == true)
 		msleep(1);
 
-	if ((!padapter->bup) || (padapter->bDriverStopped) || (padapter->bSurpriseRemoved))
-	{
+	if ((!padapter->bup) || (padapter->bDriverStopped) || (padapter->bSurpriseRemoved)) {
 		DBG_871X("%s bup =%d bDriverStopped =%d bSurpriseRemoved = %d\n", __func__
 			, padapter->bup, padapter->bDriverStopped, padapter->bSurpriseRemoved);
 		pdbgpriv->dbg_suspend_error_cnt++;
@@ -1814,16 +1791,14 @@ static int rtw_resume_process_normal(struct adapter *padapter)
 	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
 	/*  interface init */
 	/* if (sdio_init(adapter_to_dvobj(padapter)) != _SUCCESS) */
-	if ((padapter->intf_init) && (padapter->intf_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))
-	{
+	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;
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 30+ messages in thread

* [PATCH 10/10] staging: rtl8723bs: Remove unnecessary blank lines
  2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
                   ` (8 preceding siblings ...)
  2017-09-10 17:39 ` [PATCH 09/10] staging: rtl8723bs: Remove unneccesary braces and change position of open brace Harsha Sharma
@ 2017-09-10 17:41 ` Harsha Sharma
  2017-09-12 10:08 ` [PATCH v2 07/10] staging: rtl8723bs: Remove unneccesary space Harsha Sharma
                   ` (2 subsequent siblings)
  12 siblings, 0 replies; 30+ messages in thread
From: Harsha Sharma @ 2017-09-10 17:41 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, outreachy-kernel, Harsha Sharma

Blank lines aren't necessary after an open brace and before a close brace

Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
---
 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
index fb8502a..5cca7a9 100644
--- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
+++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
@@ -454,7 +454,6 @@ u16 rtw_recv_select_queue(struct sk_buff *skb)
 	}
 
 	return rtw_1d_to_queue[priority];
-
 }
 
 static int rtw_ndev_notifier_call(struct notifier_block * nb, unsigned long state, void *ptr)
@@ -1024,7 +1023,6 @@ int _netdev_open(struct net_device *pnetdev)
 	DBG_871X("-871x_drv - drv_open fail, bup =%d\n", padapter->bup);
 
 	return (-1);
-
 }
 
 int netdev_open(struct net_device *pnetdev)
@@ -1091,7 +1089,6 @@ int rtw_ips_pwr_up(struct adapter *padapter)
 
 	DBG_871X("<===  rtw_ips_pwr_up..............\n");
 	return result;
-
 }
 
 void rtw_ips_pwr_down(struct adapter *padapter)
@@ -1113,7 +1110,6 @@ void rtw_ips_dev_unload(struct adapter *padapter)
 
 	if (padapter->bSurpriseRemoved == false)
 		rtw_hal_deinit(padapter);
-
 }
 
 
@@ -1186,7 +1182,6 @@ static int netdev_close(struct net_device *pnetdev)
 	DBG_871X("-871x_drv - drv_close, bup =%d\n", padapter->bup);
 
 	return 0;
-
 }
 
 void rtw_ndev_destructor(struct net_device *ndev)
@@ -1390,7 +1385,6 @@ int rtw_suspend_wow(struct adapter *padapter)
 			DBG_871X_LEVEL(_drv_always_, "%s: pno: %d\n", __func__, pwrpriv->wowlan_pno_enable);
 		else
 			rtw_set_ps_mode(padapter, PS_MODE_DTIM, 0, 0, "WOWLAN");
-
 	}
 	else {
 		DBG_871X_LEVEL(_drv_always_, "%s: ### ERROR ### wowlan_mode =%d\n", __func__, pwrpriv->wowlan_mode);
@@ -1470,7 +1464,6 @@ static int rtw_suspend_normal(struct adapter *padapter)
 	if ((rtw_hal_check_ips_status(padapter) == true)
 		|| (adapter_to_pwrctl(padapter)->rf_pwrstate == rf_off)) {
 		DBG_871X_LEVEL(_drv_always_, "%s: ### ERROR #### driver in IPS ####ERROR###!!!\n", __func__);
-
 	}
 
 	rtw_dev_unload(padapter);
@@ -1644,7 +1637,6 @@ int rtw_resume_process_wow(struct adapter *padapter)
 		}
 	}
 	else {
-
 		DBG_871X_LEVEL(_drv_always_, "%s: ### ERROR ### wowlan_mode =%d\n", __func__, pwrpriv->wowlan_mode);
 	}
 
@@ -1657,7 +1649,6 @@ int rtw_resume_process_wow(struct adapter *padapter)
 		if (pwrpriv->wowlan_wake_reason == FWDecisionDisconnect ||
 			pwrpriv->wowlan_wake_reason == Rx_DisAssoc ||
 			pwrpriv->wowlan_wake_reason == Rx_DeAuth) {
-
 			DBG_871X("%s: disconnect reason: %02x\n", __func__,
 						pwrpriv->wowlan_wake_reason);
 			rtw_indicate_disconnect(padapter);
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 30+ messages in thread

* Re: [Outreachy kernel] [PATCH 02/10] staging: rtl8723bs: Add spaces around '+', '|', '*'
  2017-09-10 15:17 ` [PATCH 02/10] staging: rtl8723bs: Add spaces around '+', '|', '*' Harsha Sharma
@ 2017-09-10 19:36   ` Julia Lawall
  0 siblings, 0 replies; 30+ messages in thread
From: Julia Lawall @ 2017-09-10 19:36 UTC (permalink / raw)
  To: Harsha Sharma; +Cc: gregkh, devel, linux-kernel, outreachy-kernel



On Sun, 10 Sep 2017, Harsha Sharma wrote:

> Use one space around most binary operators
>
> Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>

Acked-by: Julia Lawall <julia.lawall@lip6.fr>

> ---
>  drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> index 7f22ca9..18f0ac2 100644
> --- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> +++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> @@ -222,7 +222,7 @@
>  /*  BIT4 - RF,					0: non-support, 1: support */
>  /*  BIT5 - RF_TXPWR_TRACK,	0: non-support, 1: support */
>  /*  BIT6 - RF_TXPWR_LMT,		0: non-support, 1: support */
> -static int rtw_load_phy_file = (BIT2|BIT6);
> +static int rtw_load_phy_file = (BIT2 | BIT6);
>  module_param(rtw_load_phy_file, int, 0644);
>  MODULE_PARM_DESC(rtw_load_phy_file,"PHY File Bit Map");
>  static int rtw_decrypt_phy_file = 0;
> @@ -440,12 +440,12 @@ u16 rtw_recv_select_queue(struct sk_buff *skb)
>  	u32 priority;
>  	u8 *pdata = skb->data;
>
> -	memcpy(&eth_type, pdata+(ETH_ALEN<<1), 2);
> +	memcpy(&eth_type, pdata + (ETH_ALEN << 1), 2);
>
>  	switch (be16_to_cpu(eth_type)) {
>  		case ETH_P_IP:
>
> -			piphdr = (struct iphdr *)(pdata+ETH_HLEN);
> +			piphdr = (struct iphdr *)(pdata + ETH_HLEN);
>
>  			dscp = piphdr->tos & 0xfc;
>
> @@ -561,7 +561,7 @@ struct net_device *rtw_init_netdev(struct adapter *old_padapter)
>  	pnetdev->netdev_ops = &rtw_netdev_ops;
>
>  	/* pnetdev->tx_timeout = NULL; */
> -	pnetdev->watchdog_timeo = HZ*3; /* 3 second timeout */
> +	pnetdev->watchdog_timeo = HZ * 3; /* 3 second timeout */
>  	pnetdev->wireless_handlers = (struct iw_handler_def *)&rtw_handlers_def;
>
>  	/* step 2. */
> @@ -748,7 +748,7 @@ u8 rtw_reset_drv_sw(struct adapter *padapter)
>  	pmlmepriv->LinkDetectInfo.TrafficTransitionCount = 0;
>  	pmlmepriv->LinkDetectInfo.LowPowerTransitionCount = 0;
>
> -	_clr_fwstate_(pmlmepriv, _FW_UNDER_SURVEY |_FW_UNDER_LINKING);
> +	_clr_fwstate_(pmlmepriv, _FW_UNDER_SURVEY | _FW_UNDER_LINKING);
>
>  	pwrctrlpriv->pwr_state_check_cnts = 0;
>
> @@ -935,7 +935,7 @@ static int _rtw_drv_register_netdev(struct adapter *padapter, char *name)
>  		goto error_register_netdev;
>  	}
>
> -	DBG_871X("%s, MAC Address (if%d) = " MAC_FMT "\n", __func__, (padapter->iface_id+1), MAC_ARG(pnetdev->dev_addr));
> +	DBG_871X("%s, MAC Address (if%d) = " MAC_FMT "\n", __func__, (padapter->iface_id + 1), MAC_ARG(pnetdev->dev_addr));
>
>  	return ret;
>
> --
> 1.9.1
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/28be543667f44c31e38872ec4a96bad8c167e3d8.1505052882.git.harshasharmaiitr%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>


^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [Outreachy kernel] [PATCH 01/10] staging: rtl8723bs: Add spaces around '='
  2017-09-10 15:15 ` [PATCH 01/10] staging: rtl8723bs: Add spaces around '=' Harsha Sharma
@ 2017-09-10 19:44   ` Julia Lawall
  0 siblings, 0 replies; 30+ messages in thread
From: Julia Lawall @ 2017-09-10 19:44 UTC (permalink / raw)
  To: Harsha Sharma; +Cc: gregkh, devel, linux-kernel, outreachy-kernel



On Sun, 10 Sep 2017, Harsha Sharma wrote:

> Use one space around (on each side of) '=' operator
>
> Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>

Acked-by: Julia Lawall <julia.lawall@lip6.fr>

> ---
>  drivers/staging/rtl8723bs/os_dep/os_intfs.c | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> index 0215899..7f22ca9 100644
> --- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> +++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> @@ -121,7 +121,7 @@
>
>  static int rtw_enusbss = 0;/* 0:disable, 1:enable */
>
> -static int rtw_hwpdn_mode =2;/* 0:disable, 1:enable, 2: by EFUSE config */
> +static int rtw_hwpdn_mode = 2;/* 0:disable, 1:enable, 2: by EFUSE config */
>
>  #ifdef CONFIG_HW_PWRP_DETECTION
>  static int rtw_hwpwrp_detect = 1;
> @@ -183,7 +183,7 @@
>
>  module_param(rtw_hw_wps_pbc, int, 0644);
>
> -static uint rtw_max_roaming_times =2;
> +static uint rtw_max_roaming_times = 2;
>  module_param(rtw_max_roaming_times, uint, 0644);
>  MODULE_PARM_DESC(rtw_max_roaming_times,"The max roaming times to try");
>
> @@ -255,8 +255,8 @@ static uint loadparam(struct adapter *padapter, _nic_hdl pnetdev)
>
>  	registry_par->vrtl_carrier_sense = (u8)rtw_vrtl_carrier_sense ;
>  	registry_par->vcs_type = (u8)rtw_vcs_type;
> -	registry_par->rts_thresh =(u16)rtw_rts_thresh;
> -	registry_par->frag_thresh =(u16)rtw_frag_thresh;
> +	registry_par->rts_thresh = (u16)rtw_rts_thresh;
> +	registry_par->frag_thresh = (u16)rtw_frag_thresh;
>  	registry_par->preamble = (u8)rtw_preamble;
>  	registry_par->scan_mode = (u8)rtw_scan_mode;
>  	registry_par->adhoc_tx_pwr = (u8)rtw_adhoc_tx_pwr;
> @@ -652,8 +652,8 @@ static u8 rtw_init_default_value(struct adapter *padapter)
>  #ifdef CONFIG_GTK_OL
>  	psecuritypriv->binstallKCK_KEK = _FAIL;
>  #endif /* CONFIG_GTK_OL */
> -	psecuritypriv->sw_encrypt =pregistrypriv->software_encrypt;
> -	psecuritypriv->sw_decrypt =pregistrypriv->software_decrypt;
> +	psecuritypriv->sw_encrypt = pregistrypriv->software_encrypt;
> +	psecuritypriv->sw_decrypt = pregistrypriv->software_decrypt;
>
>  	psecuritypriv->dot11AuthAlgrthm = dot11AuthAlgrthm_Open; /* open system */
>  	psecuritypriv->dot11PrivacyAlgrthm = _NO_PRIVACY_;
> @@ -777,7 +777,7 @@ u8 rtw_init_drv_sw(struct adapter *padapter)
>  		goto exit;
>  	}
>
> -	padapter->cmdpriv.padapter =padapter;
> +	padapter->cmdpriv.padapter = padapter;
>
>  	if ((rtw_init_evt_priv(&padapter->evtpriv)) == _FAIL) {
>  		RT_TRACE(_module_os_intfs_c_, _drv_err_, ("\n Can't init evt_priv\n"));
> @@ -986,7 +986,7 @@ int _netdev_open(struct net_device *pnetdev)
>
>  		DBG_871X("MAC Address = "MAC_FMT"\n", MAC_ARG(pnetdev->dev_addr));
>
> -		status =rtw_start_drv_threads(padapter);
> +		status = rtw_start_drv_threads(padapter);
>  		if (status == _FAIL) {
>  			DBG_871X("Initialize driver software resource Failed!\n");
>  			goto netdev_open_error;
> @@ -1641,7 +1641,7 @@ int rtw_resume_process_wow(struct adapter *padapter)
>  		}
>
>  		/* Disable WOW, set H2C command */
> -		poidparam.subcode =WOWLAN_DISABLE;
> +		poidparam.subcode = WOWLAN_DISABLE;
>  		padapter->HalFunc.SetHwRegHandler(padapter, HW_VAR_WOWLAN, (u8 *)&poidparam);
>
>  		psta = rtw_get_stainfo(&padapter->stapriv, get_bssid(&padapter->mlmepriv));
> @@ -1705,7 +1705,7 @@ int rtw_resume_process_wow(struct adapter *padapter)
>  		DBG_871X_LEVEL(_drv_always_, "do not reset timer\n");
>  	}
>
> -	pwrpriv->wowlan_mode =false;
> +	pwrpriv->wowlan_mode = false;
>
>  	/* clean driver side wake up reason. */
>  	pwrpriv->wowlan_wake_reason = 0;
> --
> 1.9.1
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/7b2a57c6f3eb01142a73e92124fdf06a7504e68e.1505052882.git.harshasharmaiitr%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>


^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [Outreachy kernel] [PATCH 03/10] staging: rtl8723bs: Add spaces around '?', '==', '||', '!='
  2017-09-10 17:21 ` [PATCH 03/10] staging: rtl8723bs: Add spaces around '?', '==', '||', '!=' Harsha Sharma
@ 2017-09-10 20:01   ` Julia Lawall
  0 siblings, 0 replies; 30+ messages in thread
From: Julia Lawall @ 2017-09-10 20:01 UTC (permalink / raw)
  To: Harsha Sharma; +Cc: gregkh, devel, linux-kernel, outreachy-kernel



On Sun, 10 Sep 2017, Harsha Sharma wrote:

> Use spaces around most binary operators
>
> Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>

Acked-by: Julia Lawall <julia.lawall@lip6.fr>

> ---
>  drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> index 18f0ac2..5ceccd7 100644
> --- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> +++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> @@ -1140,7 +1140,7 @@ static int pm_netdev_open(struct net_device *pnetdev, u8 bnormal)
>  		}
>  	}
>  	else
> -		status =  (_SUCCESS == ips_netdrv_open(padapter))?(0):(-1);
> +		status =  (_SUCCESS == ips_netdrv_open(padapter)) ? (0) : (-1);
>
>  	return status;
>  }
> @@ -1262,7 +1262,7 @@ void rtw_dev_unload(struct adapter *padapter)
>  			rtw_btcoex_IpsNotify(padapter, pwrctl->ips_mode_req);
>  #ifdef CONFIG_WOWLAN
>  			if (pwrctl->bSupportRemoteWakeup == true &&
> -				pwrctl->wowlan_mode ==true) {
> +				pwrctl->wowlan_mode == true) {
>  				DBG_871X_LEVEL(_drv_always_, "%s bSupportRemoteWakeup ==true  do not run rtw_hal_deinit()\n", __func__);
>  			}
>  			else
> @@ -1524,7 +1524,7 @@ int rtw_suspend_common(struct adapter *padapter)
>  	while (pwrpriv->bips_processing == true)
>  		msleep(1);
>
> -	if ((!padapter->bup) || (padapter->bDriverStopped)||(padapter->bSurpriseRemoved))
> +	if ((!padapter->bup) || (padapter->bDriverStopped) || (padapter->bSurpriseRemoved))
>  	{
>  		DBG_871X("%s bup =%d bDriverStopped =%d bSurpriseRemoved = %d\n", __func__
>  			, padapter->bup, padapter->bDriverStopped, padapter->bSurpriseRemoved);
> @@ -1671,7 +1671,7 @@ int rtw_resume_process_wow(struct adapter *padapter)
>  		DBG_871X_LEVEL(_drv_always_, "%s: ### ERROR ### wowlan_mode =%d\n", __func__, pwrpriv->wowlan_mode);
>  	}
>
> -	if (padapter->pid[1]!= 0) {
> +	if (padapter->pid[1] != 0) {
>  		DBG_871X("pid[1]:%d\n", padapter->pid[1]);
>  		rtw_signal_process(padapter->pid[1], SIGUSR2);
>  	}
> @@ -1774,7 +1774,7 @@ int rtw_resume_process_ap_wow(struct adapter *padapter)
>  			rtw_netif_wake_queue(pnetdev);
>  	}
>
> -	if (padapter->pid[1]!= 0) {
> +	if (padapter->pid[1] != 0) {
>  		DBG_871X("pid[1]:%d\n", padapter->pid[1]);
>  		rtw_signal_process(padapter->pid[1], SIGUSR2);
>  	}
> @@ -1842,7 +1842,7 @@ static int rtw_resume_process_normal(struct adapter *padapter)
>  	netif_device_attach(pnetdev);
>  	netif_carrier_on(pnetdev);
>
> -	if (padapter->pid[1]!= 0) {
> +	if (padapter->pid[1] != 0) {
>  		DBG_871X("pid[1]:%d\n", padapter->pid[1]);
>  		rtw_signal_process(padapter->pid[1], SIGUSR2);
>  	}
> --
> 1.9.1
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/be22d7c63780098571fec9d490b1a83039ddbb75.1505052882.git.harshasharmaiitr%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>


^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH] spelling.txt: Add "unnecessary" typo variants
  2017-09-10 17:39 ` [PATCH 09/10] staging: rtl8723bs: Remove unneccesary braces and change position of open brace Harsha Sharma
@ 2017-09-10 20:18   ` Joe Perches
  0 siblings, 0 replies; 30+ messages in thread
From: Joe Perches @ 2017-09-10 20:18 UTC (permalink / raw)
  To: Andrew Morton; +Cc: LKML

Add unnecessary typos by copying the necessary typos.

Signed-off-by: Joe Perches <joe@perches.com>
---
 scripts/spelling.txt | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/scripts/spelling.txt b/scripts/spelling.txt
index 400ef35169c5..e16d8f8e8bea 100644
--- a/scripts/spelling.txt
+++ b/scripts/spelling.txt
@@ -1155,6 +1155,10 @@ unknonw||unknown
 unknow||unknown
 unkown||unknown
 unneded||unneeded
+unneccecary||unnecessary
+unneccesary||unnecessary
+unneccessary||unnecessary
+unnecesary||unnecessary
 unneedingly||unnecessarily
 unnsupported||unsupported
 unmached||unmatched

^ permalink raw reply related	[flat|nested] 30+ messages in thread

* Re: [Outreachy kernel] [PATCH 04/10] staging: rtl8723bs: Remove unnecessary spaces at the start of line
  2017-09-10 17:23 ` [PATCH 04/10] staging: rtl8723bs: Remove unnecessary spaces at the start of line Harsha Sharma
@ 2017-09-10 20:33   ` Julia Lawall
  0 siblings, 0 replies; 30+ messages in thread
From: Julia Lawall @ 2017-09-10 20:33 UTC (permalink / raw)
  To: Harsha Sharma; +Cc: gregkh, devel, linux-kernel, outreachy-kernel



On Sun, 10 Sep 2017, Harsha Sharma wrote:

> No spaces at the start of a line
>
> Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>

Acked-by: Julia Lawall <julia.lawall@lip6.fr>

> ---
>  drivers/staging/rtl8723bs/os_dep/os_intfs.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> index 5ceccd7..97ff3ab 100644
> --- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> +++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> @@ -597,7 +597,7 @@ u32 rtw_start_drv_threads(struct adapter *padapter)
>  		_status = _FAIL;
>
>  	padapter->cmdThread = kthread_run(rtw_cmd_thread, padapter, "RTW_CMD_THREAD");
> -        if (IS_ERR(padapter->cmdThread))
> +	if (IS_ERR(padapter->cmdThread))
>  		_status = _FAIL;
>  	else
>  		down(&padapter->cmdpriv.terminate_cmdthread_sema); /* wait for cmd_thread to run */
> --
> 1.9.1
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/94120b866c10e83ec66c8b96522ae7d265d254cc.1505052882.git.harshasharmaiitr%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>


^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [Outreachy kernel] [PATCH 05/10] staging: rtl8723bs: Add space between concatenated strings
  2017-09-10 17:29 ` [PATCH 05/10] staging: rtl8723bs: Add space between concatenated strings Harsha Sharma
@ 2017-09-11 11:09   ` Julia Lawall
  2017-09-11 11:44     ` Harsha Sharma
  0 siblings, 1 reply; 30+ messages in thread
From: Julia Lawall @ 2017-09-11 11:09 UTC (permalink / raw)
  To: Harsha Sharma; +Cc: gregkh, devel, linux-kernel, outreachy-kernel



On Sun, 10 Sep 2017, Harsha Sharma wrote:

> Use spaces between concatenated strings

Perhaps all the entry/exit ones could be dropped, and the ohters could be
rewritten to use more standard kernel debugging functionalities.

julia

>
> Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
> ---
>  drivers/staging/rtl8723bs/os_dep/os_intfs.c | 50 ++++++++++++++---------------
>  1 file changed, 25 insertions(+), 25 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> index 97ff3ab..b320df8 100644
> --- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> +++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> @@ -467,7 +467,7 @@ static int rtw_ndev_notifier_call(struct notifier_block * nb, unsigned long stat
>  	if (dev->netdev_ops->ndo_do_ioctl != rtw_ioctl)
>  		return NOTIFY_DONE;
>
> -	DBG_871X_LEVEL(_drv_info_, FUNC_NDEV_FMT" state:%lu\n", FUNC_NDEV_ARG(dev), state);
> +	DBG_871X_LEVEL(_drv_info_, FUNC_NDEV_FMT " state:%lu\n", FUNC_NDEV_ARG(dev), state);
>
>  	switch (state) {
>  	case NETDEV_CHANGENAME:
> @@ -497,7 +497,7 @@ static int rtw_ndev_init(struct net_device *dev)
>  {
>  	struct adapter *adapter = rtw_netdev_priv(dev);
>
> -	DBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT"\n", FUNC_ADPT_ARG(adapter));
> +	DBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT "\n", FUNC_ADPT_ARG(adapter));
>  	strncpy(adapter->old_ifname, dev->name, IFNAMSIZ);
>  	rtw_adapter_proc_init(dev);
>
> @@ -508,7 +508,7 @@ static void rtw_ndev_uninit(struct net_device *dev)
>  {
>  	struct adapter *adapter = rtw_netdev_priv(dev);
>
> -	DBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT"\n", FUNC_ADPT_ARG(adapter));
> +	DBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT "\n", FUNC_ADPT_ARG(adapter));
>  	rtw_adapter_proc_deinit(dev);
>  }
>
> @@ -984,7 +984,7 @@ int _netdev_open(struct net_device *pnetdev)
>  			goto netdev_open_error;
>  		}
>
> -		DBG_871X("MAC Address = "MAC_FMT"\n", MAC_ARG(pnetdev->dev_addr));
> +		DBG_871X("MAC Address = " MAC_FMT "\n", MAC_ARG(pnetdev->dev_addr));
>
>  		status = rtw_start_drv_threads(padapter);
>  		if (status == _FAIL) {
> @@ -1203,7 +1203,7 @@ static int netdev_close(struct net_device *pnetdev)
>
>  void rtw_ndev_destructor(struct net_device *ndev)
>  {
> -	DBG_871X(FUNC_NDEV_FMT"\n", FUNC_NDEV_ARG(ndev));
> +	DBG_871X(FUNC_NDEV_FMT "\n", FUNC_NDEV_ARG(ndev));
>
>  	if (ndev->ieee80211_ptr)
>  		kfree((u8 *)ndev->ieee80211_ptr);
> @@ -1292,7 +1292,7 @@ static int rtw_suspend_free_assoc_resource(struct adapter *padapter)
>  {
>  	struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
>
> -	DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", FUNC_ADPT_ARG(padapter));
> +	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
>
>  	if (rtw_chk_roam_flags(padapter, RTW_ROAM_ON_RESUME)) {
>  		if (check_fwstate(pmlmepriv, WIFI_STATION_STATE)
> @@ -1333,7 +1333,7 @@ static int rtw_suspend_free_assoc_resource(struct adapter *padapter)
>  		rtw_indicate_disconnect(padapter);
>  	}
>
> -	DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", FUNC_ADPT_ARG(padapter));
> +	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
>  	return _SUCCESS;
>  }
>
> @@ -1347,7 +1347,7 @@ int rtw_suspend_wow(struct adapter *padapter)
>  	struct wowlan_ioctl_param poidparam;
>  	int ret = _SUCCESS;
>
> -	DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", FUNC_ADPT_ARG(padapter));
> +	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
>
>
>  	DBG_871X("wowlan_mode: %d\n", pwrpriv->wowlan_mode);
> @@ -1401,7 +1401,7 @@ int rtw_suspend_wow(struct adapter *padapter)
>  		}
>
>  		if (rtw_get_ch_setting_union(padapter, &ch, &bw, &offset) != 0) {
> -			DBG_871X(FUNC_ADPT_FMT" back to linked/linking union - ch:%u, bw:%u, offset:%u\n",
> +			DBG_871X(FUNC_ADPT_FMT " back to linked/linking union - ch:%u, bw:%u, offset:%u\n",
>  				FUNC_ADPT_ARG(padapter), ch, bw, offset);
>  			set_channel_bwmode(padapter, ch, offset, bw);
>  		}
> @@ -1416,7 +1416,7 @@ int rtw_suspend_wow(struct adapter *padapter)
>  	{
>  		DBG_871X_LEVEL(_drv_always_, "%s: ### ERROR ### wowlan_mode =%d\n", __func__, pwrpriv->wowlan_mode);
>  	}
> -	DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", FUNC_ADPT_ARG(padapter));
> +	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
>  	return ret;
>  }
>  #endif /* ifdef CONFIG_WOWLAN */
> @@ -1430,7 +1430,7 @@ int rtw_suspend_ap_wow(struct adapter *padapter)
>  	struct wowlan_ioctl_param poidparam;
>  	int ret = _SUCCESS;
>
> -	DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", FUNC_ADPT_ARG(padapter));
> +	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
>
>  	pwrpriv->wowlan_ap_mode = true;
>
> @@ -1462,14 +1462,14 @@ int rtw_suspend_ap_wow(struct adapter *padapter)
>  	DBG_871X_LEVEL(_drv_always_, "%s: wowmode suspending\n", __func__);
>
>  	if (rtw_get_ch_setting_union(padapter, &ch, &bw, &offset) != 0) {
> -		DBG_871X(FUNC_ADPT_FMT" back to linked/linking union - ch:%u, bw:%u, offset:%u\n",
> +		DBG_871X(FUNC_ADPT_FMT " back to linked/linking union - ch:%u, bw:%u, offset:%u\n",
>  			 FUNC_ADPT_ARG(padapter), ch, bw, offset);
>  		set_channel_bwmode(padapter, ch, offset, bw);
>  	}
>
>  	rtw_set_ps_mode(padapter, PS_MODE_MIN, 0, 0, "AP-WOWLAN");
>
> -	DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", FUNC_ADPT_ARG(padapter));
> +	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
>  	return ret;
>  }
>  #endif /* ifdef CONFIG_AP_WOWLAN */
> @@ -1480,7 +1480,7 @@ static int rtw_suspend_normal(struct adapter *padapter)
>  	struct net_device *pnetdev = padapter->pnetdev;
>  	int ret = _SUCCESS;
>
> -	DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", FUNC_ADPT_ARG(padapter));
> +	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
>  	if (pnetdev) {
>  		netif_carrier_off(pnetdev);
>  		rtw_netif_stop_queue(pnetdev);
> @@ -1501,7 +1501,7 @@ static int rtw_suspend_normal(struct adapter *padapter)
>  	if (padapter->intf_deinit)
>  		padapter->intf_deinit(adapter_to_dvobj(padapter));
>
> -	DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", FUNC_ADPT_ARG(padapter));
> +	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
>  	return ret;
>  }
>
> @@ -1599,7 +1599,7 @@ int rtw_resume_process_wow(struct adapter *padapter)
>  	struct sta_info *psta = NULL;
>  	int ret = _SUCCESS;
>
> -	DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", FUNC_ADPT_ARG(padapter));
> +	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
>
>  	if (padapter) {
>  		pnetdev = padapter->pnetdev;
> @@ -1710,7 +1710,7 @@ int rtw_resume_process_wow(struct adapter *padapter)
>  	/* clean driver side wake up reason. */
>  	pwrpriv->wowlan_wake_reason = 0;
>  exit:
> -	DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", FUNC_ADPT_ARG(padapter));
> +	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
>  	return ret;
>  }
>  #endif /* ifdef CONFIG_WOWLAN */
> @@ -1725,7 +1725,7 @@ int rtw_resume_process_ap_wow(struct adapter *padapter)
>  	struct wowlan_ioctl_param poidparam;
>  	int ret = _SUCCESS;
>
> -	DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", FUNC_ADPT_ARG(padapter));
> +	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
>
>  	if (padapter) {
>  		pnetdev = padapter->pnetdev;
> @@ -1785,7 +1785,7 @@ int rtw_resume_process_ap_wow(struct adapter *padapter)
>  	/* clean driver side wake up reason. */
>  	pwrpriv->wowlan_wake_reason = 0;
>  exit:
> -	DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", FUNC_ADPT_ARG(padapter));
> +	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
>  	return ret;
>  }
>  #endif /* ifdef CONFIG_APWOWLAN */
> @@ -1811,7 +1811,7 @@ static int rtw_resume_process_normal(struct adapter *padapter)
>  	psdpriv = padapter->dvobj;
>  	pdbgpriv = &psdpriv->drv_dbg;
>
> -	DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", FUNC_ADPT_ARG(padapter));
> +	DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", FUNC_ADPT_ARG(padapter));
>  	/*  interface init */
>  	/* if (sdio_init(adapter_to_dvobj(padapter)) != _SUCCESS) */
>  	if ((padapter->intf_init) && (padapter->intf_init(adapter_to_dvobj(padapter)) != _SUCCESS))
> @@ -1849,21 +1849,21 @@ static int rtw_resume_process_normal(struct adapter *padapter)
>
>
>  	if (check_fwstate(pmlmepriv, WIFI_STATION_STATE)) {
> -		DBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x - WIFI_STATION_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
> +		DBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x - WIFI_STATION_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
>
>  		if (rtw_chk_roam_flags(padapter, RTW_ROAM_ON_RESUME))
>  			rtw_roaming(padapter, NULL);
>
>  	} else if (check_fwstate(pmlmepriv, WIFI_AP_STATE)) {
> -		DBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x - WIFI_AP_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
> +		DBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x - WIFI_AP_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
>  		rtw_ap_restore_network(padapter);
>  	} else if (check_fwstate(pmlmepriv, WIFI_ADHOC_STATE)) {
> -		DBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x - WIFI_ADHOC_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
> +		DBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x - WIFI_ADHOC_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
>  	} else {
> -		DBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x - ???\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
> +		DBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x - ???\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
>  	}
>
> -	DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", FUNC_ADPT_ARG(padapter));
> +	DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", FUNC_ADPT_ARG(padapter));
>
>  exit:
>  	return ret;
> --
> 1.9.1
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/86078625c574018711c48bf66ebb5f62d540ac79.1505052882.git.harshasharmaiitr%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>


^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [Outreachy kernel] [PATCH 06/10] staging: rtl8723bs: Add space after ','
  2017-09-10 17:32 ` [PATCH 06/10] staging: rtl8723bs: Add space after ',' Harsha Sharma
@ 2017-09-11 11:13   ` Julia Lawall
  2017-09-11 16:44   ` Greg KH
  1 sibling, 0 replies; 30+ messages in thread
From: Julia Lawall @ 2017-09-11 11:13 UTC (permalink / raw)
  To: Harsha Sharma; +Cc: gregkh, devel, linux-kernel, outreachy-kernel



On Sun, 10 Sep 2017, Harsha Sharma wrote:

> Space required after ','
>
> Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>

Acked-by: Julia Lawall <julia.lawall@lip6.fr>


> ---
>  drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> index b320df8..ecb01cf 100644
> --- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> +++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> @@ -45,7 +45,7 @@
>  static int rtw_power_mgnt = 1;
>  static int rtw_ips_mode = IPS_NORMAL;
>  module_param(rtw_ips_mode, int, 0644);
> -MODULE_PARM_DESC(rtw_ips_mode,"The default IPS mode");
> +MODULE_PARM_DESC(rtw_ips_mode, "The default IPS mode");
>
>  static int rtw_smart_ps = 2;
>
> @@ -185,7 +185,7 @@
>
>  static uint rtw_max_roaming_times = 2;
>  module_param(rtw_max_roaming_times, uint, 0644);
> -MODULE_PARM_DESC(rtw_max_roaming_times,"The max roaming times to try");
> +MODULE_PARM_DESC(rtw_max_roaming_times, "The max roaming times to try");
>
>  module_param(rtw_mc2u_disable, int, 0644);
>
> @@ -206,10 +206,10 @@
>  static int rtw_tx_pwr_by_rate = 0;
>
>  module_param(rtw_tx_pwr_lmt_enable, int, 0644);
> -MODULE_PARM_DESC(rtw_tx_pwr_lmt_enable,"0:Disable, 1:Enable, 2: Depend on efuse");
> +MODULE_PARM_DESC(rtw_tx_pwr_lmt_enable, "0:Disable, 1:Enable, 2: Depend on efuse");
>
>  module_param(rtw_tx_pwr_by_rate, int, 0644);
> -MODULE_PARM_DESC(rtw_tx_pwr_by_rate,"0:Disable, 1:Enable, 2: Depend on efuse");
> +MODULE_PARM_DESC(rtw_tx_pwr_by_rate, "0:Disable, 1:Enable, 2: Depend on efuse");
>
>  char *rtw_phy_file_path = "";
>  module_param(rtw_phy_file_path, charp, 0644);
> @@ -224,10 +224,10 @@
>  /*  BIT6 - RF_TXPWR_LMT,		0: non-support, 1: support */
>  static int rtw_load_phy_file = (BIT2 | BIT6);
>  module_param(rtw_load_phy_file, int, 0644);
> -MODULE_PARM_DESC(rtw_load_phy_file,"PHY File Bit Map");
> +MODULE_PARM_DESC(rtw_load_phy_file, "PHY File Bit Map");
>  static int rtw_decrypt_phy_file = 0;
>  module_param(rtw_decrypt_phy_file, int, 0644);
> -MODULE_PARM_DESC(rtw_decrypt_phy_file,"Enable Decrypt PHY File");
> +MODULE_PARM_DESC(rtw_decrypt_phy_file, "Enable Decrypt PHY File");
>
>  int _netdev_open(struct net_device *pnetdev);
>  int netdev_open (struct net_device *pnetdev);
> --
> 1.9.1
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/f38c5d1ba5518dce605cd9eb5d7adbdee3f7544d.1505052882.git.harshasharmaiitr%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>


^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [Outreachy kernel] [PATCH 05/10] staging: rtl8723bs: Add space between concatenated strings
  2017-09-11 11:09   ` [Outreachy kernel] " Julia Lawall
@ 2017-09-11 11:44     ` Harsha Sharma
  2017-09-11 12:11       ` Julia Lawall
  0 siblings, 1 reply; 30+ messages in thread
From: Harsha Sharma @ 2017-09-11 11:44 UTC (permalink / raw)
  To: outreachy-kernel


[-- Attachment #1.1: Type: text/plain, Size: 12549 bytes --]



On Monday, September 11, 2017 at 4:40:30 PM UTC+5:30, Julia Lawall wrote:
>
>
>
> On Sun, 10 Sep 2017, Harsha Sharma wrote: 
>
> > Use spaces between concatenated strings 
>
> Perhaps all the entry/exit ones could be dropped, and the ohters could be 
> rewritten to use more standard kernel debugging functionalities. 
>
> julia 
>
Hi,
Sorry but I didn't get you.
Thanks a lot for your time.

Regards,
Harsha Sharma 

>
> > 
> > Signed-off-by: Harsha Sharma <harshash...@gmail.com <javascript:>> 
> > --- 
> >  drivers/staging/rtl8723bs/os_dep/os_intfs.c | 50 
> ++++++++++++++--------------- 
> >  1 file changed, 25 insertions(+), 25 deletions(-) 
> > 
> > diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c 
> b/drivers/staging/rtl8723bs/os_dep/os_intfs.c 
> > index 97ff3ab..b320df8 100644 
> > --- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c 
> > +++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c 
> > @@ -467,7 +467,7 @@ static int rtw_ndev_notifier_call(struct 
> notifier_block * nb, unsigned long stat 
> >          if (dev->netdev_ops->ndo_do_ioctl != rtw_ioctl) 
> >                  return NOTIFY_DONE; 
> > 
> > -        DBG_871X_LEVEL(_drv_info_, FUNC_NDEV_FMT" state:%lu\n", 
> FUNC_NDEV_ARG(dev), state); 
> > +        DBG_871X_LEVEL(_drv_info_, FUNC_NDEV_FMT " state:%lu\n", 
> FUNC_NDEV_ARG(dev), state); 
> > 
> >          switch (state) { 
> >          case NETDEV_CHANGENAME: 
> > @@ -497,7 +497,7 @@ static int rtw_ndev_init(struct net_device *dev) 
> >  { 
> >          struct adapter *adapter = rtw_netdev_priv(dev); 
> > 
> > -        DBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT"\n", 
> FUNC_ADPT_ARG(adapter)); 
> > +        DBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT "\n", 
> FUNC_ADPT_ARG(adapter)); 
> >          strncpy(adapter->old_ifname, dev->name, IFNAMSIZ); 
> >          rtw_adapter_proc_init(dev); 
> > 
> > @@ -508,7 +508,7 @@ static void rtw_ndev_uninit(struct net_device *dev) 
> >  { 
> >          struct adapter *adapter = rtw_netdev_priv(dev); 
> > 
> > -        DBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT"\n", 
> FUNC_ADPT_ARG(adapter)); 
> > +        DBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT "\n", 
> FUNC_ADPT_ARG(adapter)); 
> >          rtw_adapter_proc_deinit(dev); 
> >  } 
> > 
> > @@ -984,7 +984,7 @@ int _netdev_open(struct net_device *pnetdev) 
> >                          goto netdev_open_error; 
> >                  } 
> > 
> > -                DBG_871X("MAC Address = "MAC_FMT"\n", 
> MAC_ARG(pnetdev->dev_addr)); 
> > +                DBG_871X("MAC Address = " MAC_FMT "\n", 
> MAC_ARG(pnetdev->dev_addr)); 
> > 
> >                  status = rtw_start_drv_threads(padapter); 
> >                  if (status == _FAIL) { 
> > @@ -1203,7 +1203,7 @@ static int netdev_close(struct net_device 
> *pnetdev) 
> > 
> >  void rtw_ndev_destructor(struct net_device *ndev) 
> >  { 
> > -        DBG_871X(FUNC_NDEV_FMT"\n", FUNC_NDEV_ARG(ndev)); 
> > +        DBG_871X(FUNC_NDEV_FMT "\n", FUNC_NDEV_ARG(ndev)); 
> > 
> >          if (ndev->ieee80211_ptr) 
> >                  kfree((u8 *)ndev->ieee80211_ptr); 
> > @@ -1292,7 +1292,7 @@ static int rtw_suspend_free_assoc_resource(struct 
> adapter *padapter) 
> >  { 
> >          struct mlme_priv *pmlmepriv = &padapter->mlmepriv; 
> > 
> > -        DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > +        DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > 
> >          if (rtw_chk_roam_flags(padapter, RTW_ROAM_ON_RESUME)) { 
> >                  if (check_fwstate(pmlmepriv, WIFI_STATION_STATE) 
> > @@ -1333,7 +1333,7 @@ static int rtw_suspend_free_assoc_resource(struct 
> adapter *padapter) 
> >                  rtw_indicate_disconnect(padapter); 
> >          } 
> > 
> > -        DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > +        DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", 
> FUNC_ADPT_ARG(padapter)); 
> >          return _SUCCESS; 
> >  } 
> > 
> > @@ -1347,7 +1347,7 @@ int rtw_suspend_wow(struct adapter *padapter) 
> >          struct wowlan_ioctl_param poidparam; 
> >          int ret = _SUCCESS; 
> > 
> > -        DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > +        DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > 
> > 
> >          DBG_871X("wowlan_mode: %d\n", pwrpriv->wowlan_mode); 
> > @@ -1401,7 +1401,7 @@ int rtw_suspend_wow(struct adapter *padapter) 
> >                  } 
> > 
> >                  if (rtw_get_ch_setting_union(padapter, &ch, &bw, 
> &offset) != 0) { 
> > -                        DBG_871X(FUNC_ADPT_FMT" back to linked/linking 
> union - ch:%u, bw:%u, offset:%u\n", 
> > +                        DBG_871X(FUNC_ADPT_FMT " back to linked/linking 
> union - ch:%u, bw:%u, offset:%u\n", 
> >                                  FUNC_ADPT_ARG(padapter), ch, bw, 
> offset); 
> >                          set_channel_bwmode(padapter, ch, offset, bw); 
> >                  } 
> > @@ -1416,7 +1416,7 @@ int rtw_suspend_wow(struct adapter *padapter) 
> >          { 
> >                  DBG_871X_LEVEL(_drv_always_, "%s: ### ERROR ### 
> wowlan_mode =%d\n", __func__, pwrpriv->wowlan_mode); 
> >          } 
> > -        DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > +        DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", 
> FUNC_ADPT_ARG(padapter)); 
> >          return ret; 
> >  } 
> >  #endif /* ifdef CONFIG_WOWLAN */ 
> > @@ -1430,7 +1430,7 @@ int rtw_suspend_ap_wow(struct adapter *padapter) 
> >          struct wowlan_ioctl_param poidparam; 
> >          int ret = _SUCCESS; 
> > 
> > -        DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > +        DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > 
> >          pwrpriv->wowlan_ap_mode = true; 
> > 
> > @@ -1462,14 +1462,14 @@ int rtw_suspend_ap_wow(struct adapter *padapter) 
> >          DBG_871X_LEVEL(_drv_always_, "%s: wowmode suspending\n", 
> __func__); 
> > 
> >          if (rtw_get_ch_setting_union(padapter, &ch, &bw, &offset) != 0) 
> { 
> > -                DBG_871X(FUNC_ADPT_FMT" back to linked/linking union - 
> ch:%u, bw:%u, offset:%u\n", 
> > +                DBG_871X(FUNC_ADPT_FMT " back to linked/linking union - 
> ch:%u, bw:%u, offset:%u\n", 
> >                           FUNC_ADPT_ARG(padapter), ch, bw, offset); 
> >                  set_channel_bwmode(padapter, ch, offset, bw); 
> >          } 
> > 
> >          rtw_set_ps_mode(padapter, PS_MODE_MIN, 0, 0, "AP-WOWLAN"); 
> > 
> > -        DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > +        DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", 
> FUNC_ADPT_ARG(padapter)); 
> >          return ret; 
> >  } 
> >  #endif /* ifdef CONFIG_AP_WOWLAN */ 
> > @@ -1480,7 +1480,7 @@ static int rtw_suspend_normal(struct adapter 
> *padapter) 
> >          struct net_device *pnetdev = padapter->pnetdev; 
> >          int ret = _SUCCESS; 
> > 
> > -        DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > +        DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", 
> FUNC_ADPT_ARG(padapter)); 
> >          if (pnetdev) { 
> >                  netif_carrier_off(pnetdev); 
> >                  rtw_netif_stop_queue(pnetdev); 
> > @@ -1501,7 +1501,7 @@ static int rtw_suspend_normal(struct adapter 
> *padapter) 
> >          if (padapter->intf_deinit) 
> >                  padapter->intf_deinit(adapter_to_dvobj(padapter)); 
> > 
> > -        DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > +        DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", 
> FUNC_ADPT_ARG(padapter)); 
> >          return ret; 
> >  } 
> > 
> > @@ -1599,7 +1599,7 @@ int rtw_resume_process_wow(struct adapter 
> *padapter) 
> >          struct sta_info *psta = NULL; 
> >          int ret = _SUCCESS; 
> > 
> > -        DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > +        DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > 
> >          if (padapter) { 
> >                  pnetdev = padapter->pnetdev; 
> > @@ -1710,7 +1710,7 @@ int rtw_resume_process_wow(struct adapter 
> *padapter) 
> >          /* clean driver side wake up reason. */ 
> >          pwrpriv->wowlan_wake_reason = 0; 
> >  exit: 
> > -        DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > +        DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", 
> FUNC_ADPT_ARG(padapter)); 
> >          return ret; 
> >  } 
> >  #endif /* ifdef CONFIG_WOWLAN */ 
> > @@ -1725,7 +1725,7 @@ int rtw_resume_process_ap_wow(struct adapter 
> *padapter) 
> >          struct wowlan_ioctl_param poidparam; 
> >          int ret = _SUCCESS; 
> > 
> > -        DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > +        DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > 
> >          if (padapter) { 
> >                  pnetdev = padapter->pnetdev; 
> > @@ -1785,7 +1785,7 @@ int rtw_resume_process_ap_wow(struct adapter 
> *padapter) 
> >          /* clean driver side wake up reason. */ 
> >          pwrpriv->wowlan_wake_reason = 0; 
> >  exit: 
> > -        DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > +        DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", 
> FUNC_ADPT_ARG(padapter)); 
> >          return ret; 
> >  } 
> >  #endif /* ifdef CONFIG_APWOWLAN */ 
> > @@ -1811,7 +1811,7 @@ static int rtw_resume_process_normal(struct 
> adapter *padapter) 
> >          psdpriv = padapter->dvobj; 
> >          pdbgpriv = &psdpriv->drv_dbg; 
> > 
> > -        DBG_871X("==> "FUNC_ADPT_FMT" entry....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > +        DBG_871X("==> " FUNC_ADPT_FMT " entry....\n", 
> FUNC_ADPT_ARG(padapter)); 
> >          /*  interface init */ 
> >          /* if (sdio_init(adapter_to_dvobj(padapter)) != _SUCCESS) */ 
> >          if ((padapter->intf_init) && 
> (padapter->intf_init(adapter_to_dvobj(padapter)) != _SUCCESS)) 
> > @@ -1849,21 +1849,21 @@ static int rtw_resume_process_normal(struct 
> adapter *padapter) 
> > 
> > 
> >          if (check_fwstate(pmlmepriv, WIFI_STATION_STATE)) { 
> > -                DBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x - 
> WIFI_STATION_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv)); 
> > +                DBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x - 
> WIFI_STATION_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv)); 
> > 
> >                  if (rtw_chk_roam_flags(padapter, RTW_ROAM_ON_RESUME)) 
> >                          rtw_roaming(padapter, NULL); 
> > 
> >          } else if (check_fwstate(pmlmepriv, WIFI_AP_STATE)) { 
> > -                DBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x - 
> WIFI_AP_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv)); 
> > +                DBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x - 
> WIFI_AP_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv)); 
> >                  rtw_ap_restore_network(padapter); 
> >          } else if (check_fwstate(pmlmepriv, WIFI_ADHOC_STATE)) { 
> > -                DBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x - 
> WIFI_ADHOC_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv)); 
> > +                DBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x - 
> WIFI_ADHOC_STATE\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv)); 
> >          } else { 
> > -                DBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x - ???\n", 
> FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv)); 
> > +                DBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x - ???\n", 
> FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv)); 
> >          } 
> > 
> > -        DBG_871X("<== "FUNC_ADPT_FMT" exit....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > +        DBG_871X("<== " FUNC_ADPT_FMT " exit....\n", 
> FUNC_ADPT_ARG(padapter)); 
> > 
> >  exit: 
> >          return ret; 
> > -- 
> > 1.9.1 
> > 
> > -- 
> > You received this message because you are subscribed to the Google 
> Groups "outreachy-kernel" group. 
> > To unsubscribe from this group and stop receiving emails from it, send 
> an email to outreachy-kern...@googlegroups.com <javascript:>. 
> > To post to this group, send email to outreach...@googlegroups.com 
> <javascript:>. 
> > To view this discussion on the web visit 
> https://groups.google.com/d/msgid/outreachy-kernel/86078625c574018711c48bf66ebb5f62d540ac79.1505052882.git.harshasharmaiitr%40gmail.com. 
>
> > For more options, visit https://groups.google.com/d/optout. 
> > 
>

[-- Attachment #1.2: Type: text/html, Size: 18074 bytes --]

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [Outreachy kernel] [PATCH 05/10] staging: rtl8723bs: Add space between concatenated strings
  2017-09-11 11:44     ` Harsha Sharma
@ 2017-09-11 12:11       ` Julia Lawall
  0 siblings, 0 replies; 30+ messages in thread
From: Julia Lawall @ 2017-09-11 12:11 UTC (permalink / raw)
  To: Harsha Sharma; +Cc: outreachy-kernel

[-- Attachment #1: Type: text/plain, Size: 17966 bytes --]



On Mon, 11 Sep 2017, Harsha Sharma wrote:

>
>
> On Monday, September 11, 2017 at 4:40:30 PM UTC+5:30, Julia Lawall wrote:
>
>
>       On Sun, 10 Sep 2017, Harsha Sharma wrote:
>
>       > Use spaces between concatenated strings
>
>       Perhaps all the entry/exit ones could be dropped, and the ohters
>       could be
>       rewritten to use more standard kernel debugging functionalities.
>
>       julia
>
> Hi,
> Sorry but I didn't get you.
> Thanks a lot for your time.

There are functions like dev_dbg that are used many other places in the
kernel.  This driver defines its own macros for printing debugging
messages.

julia

>
> Regards,
> Harsha Sharmaï¿œ
>
>       >
>       > Signed-off-by: Harsha Sharma <harshash...@gmail.com>
>       > ---
>       > ï¿œdrivers/staging/rtl8723bs/os_dep/os_intfs.c | 50
>       ++++++++++++++---------------
>       > ï¿œ1 file changed, 25 insertions(+), 25 deletions(-)
>       >
>       > diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
>       b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
>       > index 97ff3ab..b320df8 100644
>       > --- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
>       > +++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
>       > @@ -467,7 +467,7 @@ static int rtw_ndev_notifier_call(struct
>       notifier_block * nb, unsigned long stat
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œif (dev->netdev_ops->ndo_do_ioctl != rtw_ioctl)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œreturn NOTIFY_DONE;
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X_LEVEL(_drv_info_, FUNC_NDEV_FMT"
>       state:%lu\n", FUNC_NDEV_ARG(dev), state);
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X_LEVEL(_drv_info_, FUNC_NDEV_FMT "
>       state:%lu\n", FUNC_NDEV_ARG(dev), state);
>       >
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œswitch (state) {
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œcase NETDEV_CHANGENAME:
>       > @@ -497,7 +497,7 @@ static int rtw_ndev_init(struct net_device
>       *dev)
>       > ï¿œ{
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œstruct adapter *adapter = rtw_netdev_priv(dev);
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT"\n",
>       FUNC_ADPT_ARG(adapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT "\n",
>       FUNC_ADPT_ARG(adapter));
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œstrncpy(adapter->old_ifname, dev->name, IFNAMSIZ);
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œrtw_adapter_proc_init(dev);
>       >
>       > @@ -508,7 +508,7 @@ static void rtw_ndev_uninit(struct
>       net_device *dev)
>       > ï¿œ{
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œstruct adapter *adapter = rtw_netdev_priv(dev);
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT"\n",
>       FUNC_ADPT_ARG(adapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X_LEVEL(_drv_always_, FUNC_ADPT_FMT "\n",
>       FUNC_ADPT_ARG(adapter));
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œrtw_adapter_proc_deinit(dev);
>       > ï¿œ}
>       >
>       > @@ -984,7 +984,7 @@ int _netdev_open(struct net_device
>       *pnetdev)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œgoto netdev_open_error;
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ}
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("MAC Address = "MAC_FMT"\n",
>       MAC_ARG(pnetdev->dev_addr));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("MAC Address = " MAC_FMT "\n",
>       MAC_ARG(pnetdev->dev_addr));
>       >
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œstatus = rtw_start_drv_threads(padapter);
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œif (status == _FAIL) {
>       > @@ -1203,7 +1203,7 @@ static int netdev_close(struct
>       net_device *pnetdev)
>       >
>       > ï¿œvoid rtw_ndev_destructor(struct net_device *ndev)
>       > ï¿œ{
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X(FUNC_NDEV_FMT"\n", FUNC_NDEV_ARG(ndev));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X(FUNC_NDEV_FMT "\n", FUNC_NDEV_ARG(ndev));
>       >
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œif (ndev->ieee80211_ptr)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œkfree((u8 *)ndev->ieee80211_ptr);
>       > @@ -1292,7 +1292,7 @@ static int
>       rtw_suspend_free_assoc_resource(struct adapter *padapter)
>       > ï¿œ{
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œstruct mlme_priv *pmlmepriv = &padapter->mlmepriv;
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("==> "FUNC_ADPT_FMT" entry....\n",
>       FUNC_ADPT_ARG(padapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("==> " FUNC_ADPT_FMT " entry....\n",
>       FUNC_ADPT_ARG(padapter));
>       >
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œif (rtw_chk_roam_flags(padapter, RTW_ROAM_ON_RESUME))
>       {
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œif (check_fwstate(pmlmepriv,
>       WIFI_STATION_STATE)
>       > @@ -1333,7 +1333,7 @@ static int
>       rtw_suspend_free_assoc_resource(struct adapter *padapter)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œrtw_indicate_disconnect(padapter);
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ}
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("<== "FUNC_ADPT_FMT" exit....\n",
>       FUNC_ADPT_ARG(padapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("<== " FUNC_ADPT_FMT " exit....\n",
>       FUNC_ADPT_ARG(padapter));
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œreturn _SUCCESS;
>       > ï¿œ}
>       >
>       > @@ -1347,7 +1347,7 @@ int rtw_suspend_wow(struct adapter
>       *padapter)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œstruct wowlan_ioctl_param poidparam;
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œint ret = _SUCCESS;
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("==> "FUNC_ADPT_FMT" entry....\n",
>       FUNC_ADPT_ARG(padapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("==> " FUNC_ADPT_FMT " entry....\n",
>       FUNC_ADPT_ARG(padapter));
>       >
>       >
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("wowlan_mode: %d\n", pwrpriv->wowlan_mode);
>       > @@ -1401,7 +1401,7 @@ int rtw_suspend_wow(struct adapter
>       *padapter)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ}
>       >
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œif (rtw_get_ch_setting_union(padapter, &ch,
>       &bw, &offset) != 0) {
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X(FUNC_ADPT_FMT" back to
>       linked/linking union - ch:%u, bw:%u, offset:%u\n",
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X(FUNC_ADPT_FMT " back to
>       linked/linking union - ch:%u, bw:%u, offset:%u\n",
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œFUNC_ADPT_ARG(padapter), ch,
>       bw, offset);
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œset_channel_bwmode(padapter, ch,
>       offset, bw);
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ}
>       > @@ -1416,7 +1416,7 @@ int rtw_suspend_wow(struct adapter
>       *padapter)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ{
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X_LEVEL(_drv_always_, "%s: ### ERROR
>       ### wowlan_mode =%d\n", __func__, pwrpriv->wowlan_mode);
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ}
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("<== "FUNC_ADPT_FMT" exit....\n",
>       FUNC_ADPT_ARG(padapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("<== " FUNC_ADPT_FMT " exit....\n",
>       FUNC_ADPT_ARG(padapter));
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œreturn ret;
>       > ï¿œ}
>       > ï¿œ#endif /* ifdef CONFIG_WOWLAN */
>       > @@ -1430,7 +1430,7 @@ int rtw_suspend_ap_wow(struct adapter
>       *padapter)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œstruct wowlan_ioctl_param poidparam;
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œint ret = _SUCCESS;
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("==> "FUNC_ADPT_FMT" entry....\n",
>       FUNC_ADPT_ARG(padapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("==> " FUNC_ADPT_FMT " entry....\n",
>       FUNC_ADPT_ARG(padapter));
>       >
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œpwrpriv->wowlan_ap_mode = true;
>       >
>       > @@ -1462,14 +1462,14 @@ int rtw_suspend_ap_wow(struct adapter
>       *padapter)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X_LEVEL(_drv_always_, "%s: wowmode
>       suspending\n", __func__);
>       >
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œif (rtw_get_ch_setting_union(padapter, &ch, &bw,
>       &offset) != 0) {
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X(FUNC_ADPT_FMT" back to
>       linked/linking union - ch:%u, bw:%u, offset:%u\n",
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X(FUNC_ADPT_FMT " back to
>       linked/linking union - ch:%u, bw:%u, offset:%u\n",
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ FUNC_ADPT_ARG(padapter), ch, bw,
>       offset);
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œset_channel_bwmode(padapter, ch, offset, bw);
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ}
>       >
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œrtw_set_ps_mode(padapter, PS_MODE_MIN, 0, 0,
>       "AP-WOWLAN");
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("<== "FUNC_ADPT_FMT" exit....\n",
>       FUNC_ADPT_ARG(padapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("<== " FUNC_ADPT_FMT " exit....\n",
>       FUNC_ADPT_ARG(padapter));
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œreturn ret;
>       > ï¿œ}
>       > ï¿œ#endif /* ifdef CONFIG_AP_WOWLAN */
>       > @@ -1480,7 +1480,7 @@ static int rtw_suspend_normal(struct
>       adapter *padapter)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œstruct net_device *pnetdev = padapter->pnetdev;
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œint ret = _SUCCESS;
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("==> "FUNC_ADPT_FMT" entry....\n",
>       FUNC_ADPT_ARG(padapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("==> " FUNC_ADPT_FMT " entry....\n",
>       FUNC_ADPT_ARG(padapter));
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œif (pnetdev) {
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œnetif_carrier_off(pnetdev);
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œrtw_netif_stop_queue(pnetdev);
>       > @@ -1501,7 +1501,7 @@ static int rtw_suspend_normal(struct
>       adapter *padapter)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œif (padapter->intf_deinit)
>       >
>       ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œpadapter->intf_deinit(adapter_to_dvobj(padapter));
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("<== "FUNC_ADPT_FMT" exit....\n",
>       FUNC_ADPT_ARG(padapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("<== " FUNC_ADPT_FMT " exit....\n",
>       FUNC_ADPT_ARG(padapter));
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œreturn ret;
>       > ï¿œ}
>       >
>       > @@ -1599,7 +1599,7 @@ int rtw_resume_process_wow(struct
>       adapter *padapter)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œstruct sta_info *psta = NULL;
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œint ret = _SUCCESS;
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("==> "FUNC_ADPT_FMT" entry....\n",
>       FUNC_ADPT_ARG(padapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("==> " FUNC_ADPT_FMT " entry....\n",
>       FUNC_ADPT_ARG(padapter));
>       >
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œif (padapter) {
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œpnetdev = padapter->pnetdev;
>       > @@ -1710,7 +1710,7 @@ int rtw_resume_process_wow(struct
>       adapter *padapter)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ/* clean driver side wake up reason. */
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œpwrpriv->wowlan_wake_reason = 0;
>       > ï¿œexit:
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("<== "FUNC_ADPT_FMT" exit....\n",
>       FUNC_ADPT_ARG(padapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("<== " FUNC_ADPT_FMT " exit....\n",
>       FUNC_ADPT_ARG(padapter));
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œreturn ret;
>       > ï¿œ}
>       > ï¿œ#endif /* ifdef CONFIG_WOWLAN */
>       > @@ -1725,7 +1725,7 @@ int rtw_resume_process_ap_wow(struct
>       adapter *padapter)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œstruct wowlan_ioctl_param poidparam;
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œint ret = _SUCCESS;
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("==> "FUNC_ADPT_FMT" entry....\n",
>       FUNC_ADPT_ARG(padapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("==> " FUNC_ADPT_FMT " entry....\n",
>       FUNC_ADPT_ARG(padapter));
>       >
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œif (padapter) {
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œpnetdev = padapter->pnetdev;
>       > @@ -1785,7 +1785,7 @@ int rtw_resume_process_ap_wow(struct
>       adapter *padapter)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ/* clean driver side wake up reason. */
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œpwrpriv->wowlan_wake_reason = 0;
>       > ï¿œexit:
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("<== "FUNC_ADPT_FMT" exit....\n",
>       FUNC_ADPT_ARG(padapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("<== " FUNC_ADPT_FMT " exit....\n",
>       FUNC_ADPT_ARG(padapter));
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œreturn ret;
>       > ï¿œ}
>       > ï¿œ#endif /* ifdef CONFIG_APWOWLAN */
>       > @@ -1811,7 +1811,7 @@ static int
>       rtw_resume_process_normal(struct adapter *padapter)
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œpsdpriv = padapter->dvobj;
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œpdbgpriv = &psdpriv->drv_dbg;
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("==> "FUNC_ADPT_FMT" entry....\n",
>       FUNC_ADPT_ARG(padapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("==> " FUNC_ADPT_FMT " entry....\n",
>       FUNC_ADPT_ARG(padapter));
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ/* ï¿œinterface init */
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ/* if (sdio_init(adapter_to_dvobj(padapter)) !=
>       _SUCCESS) */
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œif ((padapter->intf_init) &&
>       (padapter->intf_init(adapter_to_dvobj(padapter)) != _SUCCESS))
>       > @@ -1849,21 +1849,21 @@ static int
>       rtw_resume_process_normal(struct adapter *padapter)
>       >
>       >
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œif (check_fwstate(pmlmepriv, WIFI_STATION_STATE)) {
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x -
>       WIFI_STATION_STATE\n", FUNC_ADPT_ARG(padapter),
>       get_fwstate(pmlmepriv));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x -
>       WIFI_STATION_STATE\n", FUNC_ADPT_ARG(padapter),
>       get_fwstate(pmlmepriv));
>       >
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œif (rtw_chk_roam_flags(padapter,
>       RTW_ROAM_ON_RESUME))
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œrtw_roaming(padapter, NULL);
>       >
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ} else if (check_fwstate(pmlmepriv, WIFI_AP_STATE)) {
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x -
>       WIFI_AP_STATE\n", FUNC_ADPT_ARG(padapter),
>       get_fwstate(pmlmepriv));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x -
>       WIFI_AP_STATE\n", FUNC_ADPT_ARG(padapter),
>       get_fwstate(pmlmepriv));
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œrtw_ap_restore_network(padapter);
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ} else if (check_fwstate(pmlmepriv,
>       WIFI_ADHOC_STATE)) {
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x -
>       WIFI_ADHOC_STATE\n", FUNC_ADPT_ARG(padapter),
>       get_fwstate(pmlmepriv));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x -
>       WIFI_ADHOC_STATE\n", FUNC_ADPT_ARG(padapter),
>       get_fwstate(pmlmepriv));
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ} else {
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X(FUNC_ADPT_FMT" fwstate:0x%08x -
>       ???\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X(FUNC_ADPT_FMT " fwstate:0x%08x -
>       ???\n", FUNC_ADPT_ARG(padapter), get_fwstate(pmlmepriv));
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œ}
>       >
>       > -ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("<== "FUNC_ADPT_FMT" exit....\n",
>       FUNC_ADPT_ARG(padapter));
>       > +ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œDBG_871X("<== " FUNC_ADPT_FMT " exit....\n",
>       FUNC_ADPT_ARG(padapter));
>       >
>       > ï¿œexit:
>       > ï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œï¿œreturn ret;
>       > --
>       > 1.9.1
>       >
>       > --
>       > You received this message because you are subscribed to the
>       Google Groups "outreachy-kernel" group.
>       > To unsubscribe from this group and stop receiving emails from
>       it, send an email to outreachy-kern...@googlegroups.com.
>       > To post to this group, send email to
>       outreach...@googlegroups.com.
>       > To view this discussion on the web visithttps://groups.google.com/d/msgid/outreachy-kernel/86078625c574018711c48bf6
>       6ebb5f62d540ac79.1505052882.git.harshasharmaiitr%40gmail.com.
>       > For more options, visit https://groups.google.com/d/optout.
>       >
>
> --
> You received this message because you are subscribed to the Google Groups
> "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visithttps://groups.google.com/d/msgid/outreachy-kernel/331e385f-53e0-486d-9457-
> 3afe47e26240%40googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
>
>

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [Outreachy kernel] [PATCH 06/10] staging: rtl8723bs: Add space after ','
  2017-09-10 17:32 ` [PATCH 06/10] staging: rtl8723bs: Add space after ',' Harsha Sharma
  2017-09-11 11:13   ` [Outreachy kernel] " Julia Lawall
@ 2017-09-11 16:44   ` Greg KH
  1 sibling, 0 replies; 30+ messages in thread
From: Greg KH @ 2017-09-11 16:44 UTC (permalink / raw)
  To: Harsha Sharma; +Cc: devel, linux-kernel, outreachy-kernel

On Sun, Sep 10, 2017 at 11:02:02PM +0530, Harsha Sharma wrote:
> Space required after ','
> 
> Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
> Acked-by: Julia Lawall <julia.lawall@lip6.fr>

This patch does not apply, can you rebase and resend?

thanks,

greg k-h


^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 07/10] staging: rtl8723bs: Removed unneccesary space
  2017-09-10 17:34 ` [PATCH 07/10] staging: rtl8723bs: Removed unneccesary space Harsha Sharma
@ 2017-09-11 16:46   ` Greg KH
  2017-09-11 18:59     ` Harsha Sharma
  0 siblings, 1 reply; 30+ messages in thread
From: Greg KH @ 2017-09-11 16:46 UTC (permalink / raw)
  To: Harsha Sharma; +Cc: devel, outreachy-kernel, linux-kernel

On Sun, Sep 10, 2017 at 11:04:34PM +0530, Harsha Sharma wrote:
> Remove space between function name and open parenthesis '('
> 
> Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
> ---
>  drivers/staging/rtl8723bs/os_dep/os_intfs.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

This and patch 8/10, have the same subject line (well, one letter
different), yet they do different things, please either merge them
together or give them better names.

thanks,

greg k-h


^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH 07/10] staging: rtl8723bs: Removed unneccesary space
  2017-09-11 16:46   ` Greg KH
@ 2017-09-11 18:59     ` Harsha Sharma
  2017-09-11 19:34       ` [Outreachy kernel] " Greg KH
  0 siblings, 1 reply; 30+ messages in thread
From: Harsha Sharma @ 2017-09-11 18:59 UTC (permalink / raw)
  To: outreachy-kernel


[-- Attachment #1.1: Type: text/plain, Size: 835 bytes --]



On Monday, September 11, 2017 at 10:16:58 PM UTC+5:30, gregkh wrote:
>
> On Sun, Sep 10, 2017 at 11:04:34PM +0530, Harsha Sharma wrote: 
> > Remove space between function name and open parenthesis '(' 
> > 
> > Signed-off-by: Harsha Sharma <harshash...@gmail.com <javascript:>> 
> > --- 
> >  drivers/staging/rtl8723bs/os_dep/os_intfs.c | 6 +++--- 
> >  1 file changed, 3 insertions(+), 3 deletions(-) 
>
> This and patch 8/10, have the same subject line (well, one letter 
> different), yet they do different things, please either merge them 
> together or give them better names. 
>
> thanks, 
>
> Hello,
Thanks a lot for your feedback.
Sorry but I have a doubt. Should I send the whole patchset again with 
required changes ? Also I was working on branch first-patchset for this 
patchset.

Regards,
Harsha Sharma 

> greg k-h 
>

[-- Attachment #1.2: Type: text/html, Size: 1445 bytes --]

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [Outreachy kernel] Re: [PATCH 07/10] staging: rtl8723bs: Removed unneccesary space
  2017-09-11 18:59     ` Harsha Sharma
@ 2017-09-11 19:34       ` Greg KH
  0 siblings, 0 replies; 30+ messages in thread
From: Greg KH @ 2017-09-11 19:34 UTC (permalink / raw)
  To: Harsha Sharma; +Cc: outreachy-kernel

On Mon, Sep 11, 2017 at 11:59:23AM -0700, Harsha Sharma wrote:
> 

You are still using html email, and you stripped off the email of the
person who originally sent you the message, which wasn't good :(

> On Monday, September 11, 2017 at 10:16:58 PM UTC+5:30, gregkh wrote:
> 
>     On Sun, Sep 10, 2017 at 11:04:34PM +0530, Harsha Sharma wrote:
>     > Remove space between function name and open parenthesis '('
>     >
>     > Signed-off-by: Harsha Sharma <harshash...@gmail.com>
>     > ---
>     > ï¿½drivers/staging/rtl8723bs/os_dep/os_intfs.c | 6 +++---
>     > ï¿½1 file changed, 3 insertions(+), 3 deletions(-)
> 
>     This and patch 8/10, have the same subject line (well, one letter
>     different), yet they do different things, please either merge them
>     together or give them better names.
> 
>     thanks,
> 
> 
> Hello,
> Thanks a lot for your feedback.
> Sorry but I have a doubt. Should I send the whole patchset again with required
> changes ? Also I was working on branch first-patchset for this patchset.

You will note that I did take some patches in this series, you have
those emails.  So rebase your series on my latest tree and the applied
patches will go away, and you will be left with what I did not take.

Work from them, I can't do anything with a patch I have already
accepted, right?  :)

thanks

greg k-h


^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH v2 07/10] staging: rtl8723bs: Remove unneccesary space
  2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
                   ` (9 preceding siblings ...)
  2017-09-10 17:41 ` [PATCH 10/10] staging: rtl8723bs: Remove unnecessary blank lines Harsha Sharma
@ 2017-09-12 10:08 ` Harsha Sharma
  2017-09-12 10:22   ` [Outreachy kernel] " Julia Lawall
  2017-09-12 10:33 ` [PATCH v3 07/10] staging: rtl8723bs: Remove unnecessary space Harsha Sharma
  2017-09-12 13:35 ` [PATCH v2 06/10] staging: rtl8723bs: Add space after ',' Harsha Sharma
  12 siblings, 1 reply; 30+ messages in thread
From: Harsha Sharma @ 2017-09-12 10:08 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, outreachy-kernel, Harsha Sharma

Remove space between function name and open parenthesis '('
Remove space before ';', '++', ','

Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
---
Change in v2:
 -Merge patches 07/10 and 08/10

 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
index eb21821..61be2b8 100644
--- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
+++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
@@ -235,8 +235,8 @@
 MODULE_PARM_DESC(rtw_decrypt_phy_file, "Enable Decrypt PHY File");
 
 int _netdev_open(struct net_device *pnetdev);
-int netdev_open (struct net_device *pnetdev);
-static int netdev_close (struct net_device *pnetdev);
+int netdev_open(struct net_device *pnetdev);
+static int netdev_close(struct net_device *pnetdev);
 
 static uint loadparam(struct adapter *padapter, _nic_hdl pnetdev)
 {
@@ -258,7 +258,7 @@ static uint loadparam(struct adapter *padapter, _nic_hdl pnetdev)
 	if (registry_par->channel > 14)
 		registry_par->channel = 1;
 
-	registry_par->vrtl_carrier_sense = (u8)rtw_vrtl_carrier_sense ;
+	registry_par->vrtl_carrier_sense = (u8)rtw_vrtl_carrier_sense;
 	registry_par->vcs_type = (u8)rtw_vcs_type;
 	registry_par->rts_thresh = (u16)rtw_rts_thresh;
 	registry_par->frag_thresh = (u16)rtw_frag_thresh;
@@ -607,7 +607,7 @@ u32 rtw_start_drv_threads(struct adapter *padapter)
 	return _status;
 }
 
-void rtw_stop_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"));
 
@@ -1228,7 +1228,7 @@ void rtw_dev_unload(struct adapter *padapter)
 				DBG_871X("stop cmdthd timeout\n");
 				break;
 			} else {
-				cnt ++;
+				cnt++;
 				DBG_871X("cmdthd is running(%d)\n", cnt);
 				msleep(10);
 			}
@@ -1879,7 +1879,7 @@ int rtw_resume_common(struct adapter *padapter)
 		pwrpriv->pno_in_resume = false;
 	#endif
 	}
-	DBG_871X_LEVEL(_drv_always_, "%s:%d in %d ms\n", __func__ , ret,
+	DBG_871X_LEVEL(_drv_always_, "%s:%d in %d ms\n", __func__, ret,
 		jiffies_to_msecs(jiffies - start_time));
 
 	return ret;
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 30+ messages in thread

* Re: [Outreachy kernel] [PATCH v2 07/10] staging: rtl8723bs: Remove unneccesary space
  2017-09-12 10:08 ` [PATCH v2 07/10] staging: rtl8723bs: Remove unneccesary space Harsha Sharma
@ 2017-09-12 10:22   ` Julia Lawall
  0 siblings, 0 replies; 30+ messages in thread
From: Julia Lawall @ 2017-09-12 10:22 UTC (permalink / raw)
  To: Harsha Sharma; +Cc: gregkh, devel, linux-kernel, outreachy-kernel

It'sa small detail, but perhaps you could fix the spelling of unnecessary
(1 c and 2 s's).

julia

On Tue, 12 Sep 2017, Harsha Sharma wrote:

> Remove space between function name and open parenthesis '('
> Remove space before ';', '++', ','
>
> Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
> ---
> Change in v2:
>  -Merge patches 07/10 and 08/10
>
>  drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> index eb21821..61be2b8 100644
> --- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> +++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> @@ -235,8 +235,8 @@
>  MODULE_PARM_DESC(rtw_decrypt_phy_file, "Enable Decrypt PHY File");
>
>  int _netdev_open(struct net_device *pnetdev);
> -int netdev_open (struct net_device *pnetdev);
> -static int netdev_close (struct net_device *pnetdev);
> +int netdev_open(struct net_device *pnetdev);
> +static int netdev_close(struct net_device *pnetdev);
>
>  static uint loadparam(struct adapter *padapter, _nic_hdl pnetdev)
>  {
> @@ -258,7 +258,7 @@ static uint loadparam(struct adapter *padapter, _nic_hdl pnetdev)
>  	if (registry_par->channel > 14)
>  		registry_par->channel = 1;
>
> -	registry_par->vrtl_carrier_sense = (u8)rtw_vrtl_carrier_sense ;
> +	registry_par->vrtl_carrier_sense = (u8)rtw_vrtl_carrier_sense;
>  	registry_par->vcs_type = (u8)rtw_vcs_type;
>  	registry_par->rts_thresh = (u16)rtw_rts_thresh;
>  	registry_par->frag_thresh = (u16)rtw_frag_thresh;
> @@ -607,7 +607,7 @@ u32 rtw_start_drv_threads(struct adapter *padapter)
>  	return _status;
>  }
>
> -void rtw_stop_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"));
>
> @@ -1228,7 +1228,7 @@ void rtw_dev_unload(struct adapter *padapter)
>  				DBG_871X("stop cmdthd timeout\n");
>  				break;
>  			} else {
> -				cnt ++;
> +				cnt++;
>  				DBG_871X("cmdthd is running(%d)\n", cnt);
>  				msleep(10);
>  			}
> @@ -1879,7 +1879,7 @@ int rtw_resume_common(struct adapter *padapter)
>  		pwrpriv->pno_in_resume = false;
>  	#endif
>  	}
> -	DBG_871X_LEVEL(_drv_always_, "%s:%d in %d ms\n", __func__ , ret,
> +	DBG_871X_LEVEL(_drv_always_, "%s:%d in %d ms\n", __func__, ret,
>  		jiffies_to_msecs(jiffies - start_time));
>
>  	return ret;
> --
> 1.9.1
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/1505210936-24971-1-git-send-email-harshasharmaiitr%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>


^ permalink raw reply	[flat|nested] 30+ messages in thread

* [PATCH v3 07/10] staging: rtl8723bs: Remove unnecessary space
  2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
                   ` (10 preceding siblings ...)
  2017-09-12 10:08 ` [PATCH v2 07/10] staging: rtl8723bs: Remove unneccesary space Harsha Sharma
@ 2017-09-12 10:33 ` Harsha Sharma
  2017-09-12 13:35 ` [PATCH v2 06/10] staging: rtl8723bs: Add space after ',' Harsha Sharma
  12 siblings, 0 replies; 30+ messages in thread
From: Harsha Sharma @ 2017-09-12 10:33 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, outreachy-kernel, Harsha Sharma

Remove space between function name and open parenthesis '('
Remove space before ';', '++', ','

Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
---
Change in v3:
 -Fix small spelling mistake
Change in v2:
 -Merge patches 07/10 and 08/10

 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
index eb21821..61be2b8 100644
--- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
+++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
@@ -235,8 +235,8 @@
 MODULE_PARM_DESC(rtw_decrypt_phy_file, "Enable Decrypt PHY File");
 
 int _netdev_open(struct net_device *pnetdev);
-int netdev_open (struct net_device *pnetdev);
-static int netdev_close (struct net_device *pnetdev);
+int netdev_open(struct net_device *pnetdev);
+static int netdev_close(struct net_device *pnetdev);
 
 static uint loadparam(struct adapter *padapter, _nic_hdl pnetdev)
 {
@@ -258,7 +258,7 @@ static uint loadparam(struct adapter *padapter, _nic_hdl pnetdev)
 	if (registry_par->channel > 14)
 		registry_par->channel = 1;
 
-	registry_par->vrtl_carrier_sense = (u8)rtw_vrtl_carrier_sense ;
+	registry_par->vrtl_carrier_sense = (u8)rtw_vrtl_carrier_sense;
 	registry_par->vcs_type = (u8)rtw_vcs_type;
 	registry_par->rts_thresh = (u16)rtw_rts_thresh;
 	registry_par->frag_thresh = (u16)rtw_frag_thresh;
@@ -607,7 +607,7 @@ u32 rtw_start_drv_threads(struct adapter *padapter)
 	return _status;
 }
 
-void rtw_stop_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"));
 
@@ -1228,7 +1228,7 @@ void rtw_dev_unload(struct adapter *padapter)
 				DBG_871X("stop cmdthd timeout\n");
 				break;
 			} else {
-				cnt ++;
+				cnt++;
 				DBG_871X("cmdthd is running(%d)\n", cnt);
 				msleep(10);
 			}
@@ -1879,7 +1879,7 @@ int rtw_resume_common(struct adapter *padapter)
 		pwrpriv->pno_in_resume = false;
 	#endif
 	}
-	DBG_871X_LEVEL(_drv_always_, "%s:%d in %d ms\n", __func__ , ret,
+	DBG_871X_LEVEL(_drv_always_, "%s:%d in %d ms\n", __func__, ret,
 		jiffies_to_msecs(jiffies - start_time));
 
 	return ret;
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 30+ messages in thread

* [PATCH v2 06/10] staging: rtl8723bs: Add space after ','
  2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
                   ` (11 preceding siblings ...)
  2017-09-12 10:33 ` [PATCH v3 07/10] staging: rtl8723bs: Remove unnecessary space Harsha Sharma
@ 2017-09-12 13:35 ` Harsha Sharma
  2017-09-12 20:41   ` Greg KH
  12 siblings, 1 reply; 30+ messages in thread
From: Harsha Sharma @ 2017-09-12 13:35 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, outreachy-kernel, Harsha Sharma

Space required after ','

Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
---
Changes in v2:
 -Rebase against staging-testing and solve merge conflicts

 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
index b3635e2..22b9e1e 100644
--- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
+++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
@@ -45,7 +45,7 @@
 static int rtw_power_mgnt = 1;
 static int rtw_ips_mode = IPS_NORMAL;
 module_param(rtw_ips_mode, int, 0644);
-MODULE_PARM_DESC(rtw_ips_mode,"The default IPS mode");
+MODULE_PARM_DESC(rtw_ips_mode, "The default IPS mode");
 
 static int rtw_smart_ps = 2;
 
@@ -185,7 +185,7 @@
 
 static uint rtw_max_roaming_times = 2;
 module_param(rtw_max_roaming_times, uint, 0644);
-MODULE_PARM_DESC(rtw_max_roaming_times,"The max roaming times to try");
+MODULE_PARM_DESC(rtw_max_roaming_times, "The max roaming times to try");
 
 module_param(rtw_mc2u_disable, int, 0644);
 
@@ -206,10 +206,10 @@
 static int rtw_tx_pwr_by_rate;
 
 module_param(rtw_tx_pwr_lmt_enable, int, 0644);
-MODULE_PARM_DESC(rtw_tx_pwr_lmt_enable,"0:Disable, 1:Enable, 2: Depend on efuse");
+MODULE_PARM_DESC(rtw_tx_pwr_lmt_enable, "0:Disable, 1:Enable, 2: Depend on efuse");
 
 module_param(rtw_tx_pwr_by_rate, int, 0644);
-MODULE_PARM_DESC(rtw_tx_pwr_by_rate,"0:Disable, 1:Enable, 2: Depend on efuse");
+MODULE_PARM_DESC(rtw_tx_pwr_by_rate, "0:Disable, 1:Enable, 2: Depend on efuse");
 
 char *rtw_phy_file_path = "";
 module_param(rtw_phy_file_path, charp, 0644);
@@ -224,10 +224,10 @@
 /*  BIT6 - RF_TXPWR_LMT,		0: non-support, 1: support */
 static int rtw_load_phy_file = (BIT2 | BIT6);
 module_param(rtw_load_phy_file, int, 0644);
-MODULE_PARM_DESC(rtw_load_phy_file,"PHY File Bit Map");
+MODULE_PARM_DESC(rtw_load_phy_file, "PHY File Bit Map");
 static int rtw_decrypt_phy_file;
 module_param(rtw_decrypt_phy_file, int, 0644);
-MODULE_PARM_DESC(rtw_decrypt_phy_file,"Enable Decrypt PHY File");
+MODULE_PARM_DESC(rtw_decrypt_phy_file, "Enable Decrypt PHY File");
 
 int _netdev_open(struct net_device *pnetdev);
 int netdev_open (struct net_device *pnetdev);
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 30+ messages in thread

* Re: [PATCH v2 06/10] staging: rtl8723bs: Add space after ','
  2017-09-12 13:35 ` [PATCH v2 06/10] staging: rtl8723bs: Add space after ',' Harsha Sharma
@ 2017-09-12 20:41   ` Greg KH
  2017-10-08 12:44     ` Harsha Sharma
  0 siblings, 1 reply; 30+ messages in thread
From: Greg KH @ 2017-09-12 20:41 UTC (permalink / raw)
  To: Harsha Sharma; +Cc: devel, outreachy-kernel, linux-kernel

On Tue, Sep 12, 2017 at 07:05:23PM +0530, Harsha Sharma wrote:
> Space required after ','
> 
> Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
> ---
> Changes in v2:
>  -Rebase against staging-testing and solve merge conflicts
> 
>  drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)

Where are the rest of the patches in this series, I only seem to have 2
(patch 6 and patch 7 out of 10.)

If you have to resend patches, please always do so as a unique series,
If I have taken some of a previous series, please then send the new ones
out as a totally unique series, as that is what they now are.

Remember, make it totally obvious what I, as a maintainer, need to do
here.  I have no memory of previous patch submissions, or what happened
on something else somewhere else.  I deal with thousands of patches, and
as such, have no short-term memory :)

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 30+ messages in thread

* Re: [PATCH v2 06/10] staging: rtl8723bs: Add space after ','
  2017-09-12 20:41   ` Greg KH
@ 2017-10-08 12:44     ` Harsha Sharma
  0 siblings, 0 replies; 30+ messages in thread
From: Harsha Sharma @ 2017-10-08 12:44 UTC (permalink / raw)
  To: Greg KH; +Cc: devel, outreachy-kernel, linux-kernel

On Wed, Sep 13, 2017 at 2:11 AM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Tue, Sep 12, 2017 at 07:05:23PM +0530, Harsha Sharma wrote:
>> Space required after ','
>>
>> Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
>> ---
>> Changes in v2:
>>  -Rebase against staging-testing and solve merge conflicts
>>
>>  drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++++++------
>>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> Where are the rest of the patches in this series, I only seem to have 2
> (patch 6 and patch 7 out of 10.)
>
> If you have to resend patches, please always do so as a unique series,
> If I have taken some of a previous series, please then send the new ones
> out as a totally unique series, as that is what they now are.
>
> Remember, make it totally obvious what I, as a maintainer, need to do
> here.  I have no memory of previous patch submissions, or what happened
> on something else somewhere else.  I deal with thousands of patches, and
> as such, have no short-term memory :)
>
Hi,
The other patches in this series have already been merged and only this one is
remaining, so should I still send whole patchset again or this patch can be
merged ?
Thanks for your time.

Regards,
Harsha Sharma

> thanks,
>
> greg k-h


^ permalink raw reply	[flat|nested] 30+ messages in thread

end of thread, other threads:[~2017-10-08 12:44 UTC | newest]

Thread overview: 30+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-10 15:10 [PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style Harsha Sharma
2017-09-10 15:15 ` [PATCH 01/10] staging: rtl8723bs: Add spaces around '=' Harsha Sharma
2017-09-10 19:44   ` [Outreachy kernel] " Julia Lawall
2017-09-10 15:17 ` [PATCH 02/10] staging: rtl8723bs: Add spaces around '+', '|', '*' Harsha Sharma
2017-09-10 19:36   ` [Outreachy kernel] " Julia Lawall
2017-09-10 17:21 ` [PATCH 03/10] staging: rtl8723bs: Add spaces around '?', '==', '||', '!=' Harsha Sharma
2017-09-10 20:01   ` [Outreachy kernel] " Julia Lawall
2017-09-10 17:23 ` [PATCH 04/10] staging: rtl8723bs: Remove unnecessary spaces at the start of line Harsha Sharma
2017-09-10 20:33   ` [Outreachy kernel] " Julia Lawall
2017-09-10 17:29 ` [PATCH 05/10] staging: rtl8723bs: Add space between concatenated strings Harsha Sharma
2017-09-11 11:09   ` [Outreachy kernel] " Julia Lawall
2017-09-11 11:44     ` Harsha Sharma
2017-09-11 12:11       ` Julia Lawall
2017-09-10 17:32 ` [PATCH 06/10] staging: rtl8723bs: Add space after ',' Harsha Sharma
2017-09-11 11:13   ` [Outreachy kernel] " Julia Lawall
2017-09-11 16:44   ` Greg KH
2017-09-10 17:34 ` [PATCH 07/10] staging: rtl8723bs: Removed unneccesary space Harsha Sharma
2017-09-11 16:46   ` Greg KH
2017-09-11 18:59     ` Harsha Sharma
2017-09-11 19:34       ` [Outreachy kernel] " Greg KH
2017-09-10 17:36 ` [PATCH 08/10] staging: rtl8723bs: Remove unnecessary space Harsha Sharma
2017-09-10 17:39 ` [PATCH 09/10] staging: rtl8723bs: Remove unneccesary braces and change position of open brace Harsha Sharma
2017-09-10 20:18   ` [PATCH] spelling.txt: Add "unnecessary" typo variants Joe Perches
2017-09-10 17:41 ` [PATCH 10/10] staging: rtl8723bs: Remove unnecessary blank lines Harsha Sharma
2017-09-12 10:08 ` [PATCH v2 07/10] staging: rtl8723bs: Remove unneccesary space Harsha Sharma
2017-09-12 10:22   ` [Outreachy kernel] " Julia Lawall
2017-09-12 10:33 ` [PATCH v3 07/10] staging: rtl8723bs: Remove unnecessary space Harsha Sharma
2017-09-12 13:35 ` [PATCH v2 06/10] staging: rtl8723bs: Add space after ',' Harsha Sharma
2017-09-12 20:41   ` Greg KH
2017-10-08 12:44     ` Harsha Sharma

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.