Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1560833 > unrolled thread

[PATCH] drivers: staging: Fix "space required after that ','" errors

Started bysimran singhal <singhalsimran0@gmail.com>
First post2017-01-17 18:30 +0100
Last post2017-01-19 10:50 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] drivers: staging: Fix "space required after that ','" errors simran singhal <singhalsimran0@gmail.com> - 2017-01-17 18:30 +0100
    Re: [PATCH] drivers: staging: Fix "space required after that ','"  errors Greg KH <gregkh@linuxfoundation.org> - 2017-01-19 10:50 +0100

#1560833 — [PATCH] drivers: staging: Fix "space required after that ','" errors

Fromsimran singhal <singhalsimran0@gmail.com>
Date2017-01-17 18:30 +0100
Subject[PATCH] drivers: staging: Fix "space required after that ','" errors
Message-ID<t0FK2-1rB-23@gated-at.bofh.it>
Fix checkpatch.pl "space required after that ','" errors

Signed-off-by: simran singhal <singhalsimran0@gmail.com>

diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
index d7d85b3..6380b41 100644
--- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
+++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
@@ -3177,7 +3177,7 @@ int ieee80211_wpa_supplicant_ioctl(struct ieee80211_device *ieee, struct iw_poin
 		break;

 	default:
-		printk("Unknown WPA supplicant request: %d\n",param->cmd);
+		printk("Unknown WPA supplicant request: %d\n", param->cmd);
 		ret = -EOPNOTSUPP;
 		break;
 	}
---
 drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
index d7d85b3..6380b41 100644
--- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
+++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
@@ -3177,7 +3177,7 @@ int ieee80211_wpa_supplicant_ioctl(struct ieee80211_device *ieee, struct iw_poin
 		break;
 
 	default:
-		printk("Unknown WPA supplicant request: %d\n",param->cmd);
+		printk("Unknown WPA supplicant request: %d\n", param->cmd);
 		ret = -EOPNOTSUPP;
 		break;
 	}
-- 
2.7.4

[toc] | [next] | [standalone]


#1562536 — Re: [PATCH] drivers: staging: Fix "space required after that ','" errors

FromGreg KH <gregkh@linuxfoundation.org>
Date2017-01-19 10:50 +0100
SubjectRe: [PATCH] drivers: staging: Fix "space required after that ','" errors
Message-ID<t1hvY-8e7-33@gated-at.bofh.it>
In reply to#1560833
On Tue, Jan 17, 2017 at 10:56:31PM +0530, simran singhal wrote:
> Fix checkpatch.pl "space required after that ','" errors
> 
> Signed-off-by: simran singhal <singhalsimran0@gmail.com>
> 
> diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
> index d7d85b3..6380b41 100644
> --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
> +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
> @@ -3177,7 +3177,7 @@ int ieee80211_wpa_supplicant_ioctl(struct ieee80211_device *ieee, struct iw_poin
>  		break;
> 
>  	default:
> -		printk("Unknown WPA supplicant request: %d\n",param->cmd);
> +		printk("Unknown WPA supplicant request: %d\n", param->cmd);
>  		ret = -EOPNOTSUPP;
>  		break;
>  	}
> ---
>  drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
> index d7d85b3..6380b41 100644
> --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
> +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
> @@ -3177,7 +3177,7 @@ int ieee80211_wpa_supplicant_ioctl(struct ieee80211_device *ieee, struct iw_poin
>  		break;
>  
>  	default:
> -		printk("Unknown WPA supplicant request: %d\n",param->cmd);
> +		printk("Unknown WPA supplicant request: %d\n", param->cmd);
>  		ret = -EOPNOTSUPP;
>  		break;
>  	}
> -- 
> 2.7.4

You seem to have duplicated the patch twice here, please fix up and
resend.

thanks,

greg k-h

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web