Re: [PATCH] drivers: staging: rtl8192u: Fix "space required before the open parenthesis '('" errors

2015-07-14 Thread Greg Kroah-Hartman
On Sat, Jun 20, 2015 at 03:56:08PM -0500, Greg Donald wrote: > Fix checkpatch.pl "space required before the open parenthesis '('" errors > > Signed-off-by: Greg Donald > --- > .../staging/rtl8192u/ieee80211/rtl819x_HTProc.c| 97 > ++ > 1 file changed, 44 insertions(+),

Re: [PATCH] drivers: staging: rtl8192u: Fix space required before the open parenthesis '(' errors

2015-07-14 Thread Greg Kroah-Hartman
On Sat, Jun 20, 2015 at 03:56:08PM -0500, Greg Donald wrote: Fix checkpatch.pl space required before the open parenthesis '(' errors Signed-off-by: Greg Donald gdon...@gmail.com --- .../staging/rtl8192u/ieee80211/rtl819x_HTProc.c| 97 ++ 1 file changed, 44

Re: [PATCH] drivers: staging: rtl8192u: Fix "space required before the open parenthesis '('" errors

2015-06-23 Thread Greg Donald
On Tue, Jun 23, 2015 at 3:01 AM, Dan Carpenter wrote: > Actually, Greg complains about these all the time. All I know is he's yet to turn away a single one of my patches. > And here is a useless URL that you can look at: http://slashdot.org > I'm sure it supports my argument. > > But

Re: [PATCH] drivers: staging: rtl8192u: Fix "space required before the open parenthesis '('" errors

2015-06-23 Thread Dan Carpenter
On Mon, Jun 22, 2015 at 06:25:11AM -0500, Greg Donald wrote: > Umm.. you have to fix more than one error if there's more than one > error on or very near the same line you are already fixing, else > checkpatch.pl complains that your patch has errors. Not to mention > Greg KH has never complained:

Re: [PATCH] drivers: staging: rtl8192u: Fix space required before the open parenthesis '(' errors

2015-06-23 Thread Dan Carpenter
On Mon, Jun 22, 2015 at 06:25:11AM -0500, Greg Donald wrote: Umm.. you have to fix more than one error if there's more than one error on or very near the same line you are already fixing, else checkpatch.pl complains that your patch has errors. Not to mention Greg KH has never complained:

Re: [PATCH] drivers: staging: rtl8192u: Fix space required before the open parenthesis '(' errors

2015-06-23 Thread Greg Donald
On Tue, Jun 23, 2015 at 3:01 AM, Dan Carpenter dan.carpen...@oracle.com wrote: Actually, Greg complains about these all the time. All I know is he's yet to turn away a single one of my patches. And here is a useless URL that you can look at: http://slashdot.org I'm sure it supports my

Re: [PATCH] drivers: staging: rtl8192u: Fix "space required before the open parenthesis '('" errors

2015-06-22 Thread Greg Donald
Umm.. you have to fix more than one error if there's more than one error on or very near the same line you are already fixing, else checkpatch.pl complains that your patch has errors. Not to mention Greg KH has never complained:

Re: [PATCH] drivers: staging: rtl8192u: Fix "space required before the open parenthesis '('" errors

2015-06-22 Thread Sudip Mukherjee
On Sat, Jun 20, 2015 at 03:56:08PM -0500, Greg Donald wrote: > Fix checkpatch.pl "space required before the open parenthesis '('" errors > > Signed-off-by: Greg Donald > --- > .../staging/rtl8192u/ieee80211/rtl819x_HTProc.c| 97 > ++ > 1 file changed, 44 insertions(+),

Re: [PATCH] drivers: staging: rtl8192u: Fix space required before the open parenthesis '(' errors

2015-06-22 Thread Greg Donald
Umm.. you have to fix more than one error if there's more than one error on or very near the same line you are already fixing, else checkpatch.pl complains that your patch has errors. Not to mention Greg KH has never complained:

Re: [PATCH] drivers: staging: rtl8192u: Fix space required before the open parenthesis '(' errors

2015-06-22 Thread Sudip Mukherjee
On Sat, Jun 20, 2015 at 03:56:08PM -0500, Greg Donald wrote: Fix checkpatch.pl space required before the open parenthesis '(' errors Signed-off-by: Greg Donald gdon...@gmail.com --- .../staging/rtl8192u/ieee80211/rtl819x_HTProc.c| 97 ++ 1 file changed, 44

[PATCH] drivers: staging: rtl8192u: Fix "space required before the open parenthesis '('" errors

2015-06-20 Thread Greg Donald
Fix checkpatch.pl "space required before the open parenthesis '('" errors Signed-off-by: Greg Donald --- .../staging/rtl8192u/ieee80211/rtl819x_HTProc.c| 97 ++ 1 file changed, 44 insertions(+), 53 deletions(-) diff --git

[PATCH] drivers: staging: rtl8192u: Fix space required before the open parenthesis '(' errors

2015-06-20 Thread Greg Donald
Fix checkpatch.pl space required before the open parenthesis '(' errors Signed-off-by: Greg Donald gdon...@gmail.com --- .../staging/rtl8192u/ieee80211/rtl819x_HTProc.c| 97 ++ 1 file changed, 44 insertions(+), 53 deletions(-) diff --git