Re: [PATCH] staging: rtl8192u: remove blank lines after braces (opening)

2016-04-07 Thread Joe Perches
On Thu, 2016-04-07 at 21:06 +0300, Andy Shevchenko wrote: > On Thu, Apr 7, 2016 at 9:04 PM, Joe Perches wrote: > > On Thu, 2016-04-07 at 20:59 +0300, Andy Shevchenko wrote: > > > On Tue, Apr 5, 2016 at 7:07 PM, Nicholas Sim wrote: > > > > > > > > Remove unneeded blank lines

Re: [PATCH] staging: rtl8192u: remove blank lines after braces (opening)

2016-04-07 Thread Joe Perches
On Thu, 2016-04-07 at 21:06 +0300, Andy Shevchenko wrote: > On Thu, Apr 7, 2016 at 9:04 PM, Joe Perches wrote: > > On Thu, 2016-04-07 at 20:59 +0300, Andy Shevchenko wrote: > > > On Tue, Apr 5, 2016 at 7:07 PM, Nicholas Sim wrote: > > > > > > > > Remove unneeded blank lines appearing after

Re: [PATCH] staging: rtl8192u: remove blank lines after braces (opening)

2016-04-07 Thread Andy Shevchenko
On Thu, Apr 7, 2016 at 9:04 PM, Joe Perches wrote: > On Thu, 2016-04-07 at 20:59 +0300, Andy Shevchenko wrote: >> On Tue, Apr 5, 2016 at 7:07 PM, Nicholas Sim wrote: >> > >> > Remove unneeded blank lines appearing after opening braces as suggested >> >

Re: [PATCH] staging: rtl8192u: remove blank lines after braces (opening)

2016-04-07 Thread Andy Shevchenko
On Thu, Apr 7, 2016 at 9:04 PM, Joe Perches wrote: > On Thu, 2016-04-07 at 20:59 +0300, Andy Shevchenko wrote: >> On Tue, Apr 5, 2016 at 7:07 PM, Nicholas Sim wrote: >> > >> > Remove unneeded blank lines appearing after opening braces as suggested >> > by checkpatch.pl >> > >> You have to

Re: [PATCH] staging: rtl8192u: remove blank lines after braces (opening)

2016-04-07 Thread Joe Perches
On Thu, 2016-04-07 at 20:59 +0300, Andy Shevchenko wrote: > On Tue, Apr 5, 2016 at 7:07 PM, Nicholas Sim wrote: > > > > Remove unneeded blank lines appearing after opening braces as suggested > > by checkpatch.pl > > > You have to combine this with the rest so called

Re: [PATCH] staging: rtl8192u: remove blank lines after braces (opening)

2016-04-07 Thread Joe Perches
On Thu, 2016-04-07 at 20:59 +0300, Andy Shevchenko wrote: > On Tue, Apr 5, 2016 at 7:07 PM, Nicholas Sim wrote: > > > > Remove unneeded blank lines appearing after opening braces as suggested > > by checkpatch.pl > > > You have to combine this with the rest so called 'indentation' fixes, > e.g.

Re: [PATCH] staging: rtl8192u: remove blank lines after braces (opening)

2016-04-07 Thread Andy Shevchenko
On Tue, Apr 5, 2016 at 7:07 PM, Nicholas Sim wrote: > Remove unneeded blank lines appearing after opening braces as suggested > by checkpatch.pl > You have to combine this with the rest so called 'indentation' fixes, e.g. +- empty lines, tabs vs. spaces, etc. P.S. By

Re: [PATCH] staging: rtl8192u: remove blank lines after braces (opening)

2016-04-07 Thread Andy Shevchenko
On Tue, Apr 5, 2016 at 7:07 PM, Nicholas Sim wrote: > Remove unneeded blank lines appearing after opening braces as suggested > by checkpatch.pl > You have to combine this with the rest so called 'indentation' fixes, e.g. +- empty lines, tabs vs. spaces, etc. P.S. By the way. are they first

[PATCH] staging: rtl8192u: remove blank lines after braces (opening)

2016-04-05 Thread Nicholas Sim
Remove unneeded blank lines appearing after opening braces as suggested by checkpatch.pl Signed-off-by: Nicholas Sim --- drivers/staging/rtl8192u/r8192U_core.c | 14 -- 1 file changed, 14 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_core.c

[PATCH] staging: rtl8192u: remove blank lines after braces (opening)

2016-04-05 Thread Nicholas Sim
Remove unneeded blank lines appearing after opening braces as suggested by checkpatch.pl Signed-off-by: Nicholas Sim --- drivers/staging/rtl8192u/r8192U_core.c | 14 -- 1 file changed, 14 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_core.c