Re: [PATCH] staging: rtl8188eu: coding style fixup

2014-10-10 Thread Sudip Mukherjee
On Fri, Oct 10, 2014 at 11:09:20AM +0400, Igor Bogomazov wrote: > checkpatch.pl tiny fix > get rid of 2 warnings and 2 errors for hal/fw.c: > > WARNING: braces {} are not necessary for single statement blocks > WARNING: line over 80 characters > ERROR: space required before the open brace '{' > ER

[PATCH] staging: rtl8188eu: coding style fixup

2014-10-10 Thread Igor Bogomazov
checkpatch.pl tiny fix get rid of 2 warnings and 2 errors for hal/fw.c: WARNING: braces {} are not necessary for single statement blocks WARNING: line over 80 characters ERROR: space required before the open brace '{' ERROR: space required after that ',' (ctx:VxV) Signed-off-by: Igor Bogomazov C

Re: [PATCH] staging: rtl8188eu: coding style fixup

2014-10-09 Thread Greg KH
On Thu, Oct 09, 2014 at 10:30:06AM +0400, Igor Bogomazov wrote: > checkpatch.pl tiny fix > get rid of 2 warnings and 2 errors for hal/fw.c What warnings? What errors? Be specific please. > Trailing whitespace :( ___ devel mailing list de...@linu

Re: [PATCH] staging: rtl8188eu: coding style fixup

2014-10-08 Thread Igor Bogomazov
you are right, would be wise to do that, added On Thu, 9 Oct 2014 10:32:37 +0400 Denis Kirjanov wrote: > But you haven't CCed people in the CC list > > On 10/9/14, Igor Bogomazov wrote: > > checkpatch.pl tiny fix > > get rid of 2 warnings and 2 errors for hal/fw.c > > > > Signed-off-by: Igor B

Re: [PATCH] staging: rtl8188eu: coding style fixup

2014-10-08 Thread Denis Kirjanov
But you haven't CCed people in the CC list On 10/9/14, Igor Bogomazov wrote: > checkpatch.pl tiny fix > get rid of 2 warnings and 2 errors for hal/fw.c > > Signed-off-by: Igor Bogomazov > Cc: Greg Kroah-Hartman > Cc: navin patidar > Cc: Stephen Rothwell > --- > for linux-next 3.17.0 > > this

[PATCH] staging: rtl8188eu: coding style fixup

2014-10-08 Thread Igor Bogomazov
checkpatch.pl tiny fix get rid of 2 warnings and 2 errors for hal/fw.c Signed-off-by: Igor Bogomazov Cc: Greg Kroah-Hartman Cc: navin patidar Cc: Stephen Rothwell --- for linux-next 3.17.0 this is an updated version of the patch I've sent two days ago, that one did not apply cleanly diff

Re: [PATCH] staging: rtl8188eu: coding style fixup

2014-10-08 Thread Greg KH
On Tue, Oct 07, 2014 at 01:11:42PM +0400, Igor Bogomazov wrote: > checkpatch.pk tiny fix > get rid of 2 warnings and 2 errors for hal/fw.c > > Signed-off-by: Igor Bogomazov > Cc: Greg Kroah-Hartman > Cc: navin patidar > Cc: Stephen Rothwell > --- > linux-next 3.17.0 I think someone else a

[PATCH] staging: rtl8188eu: coding style fixup

2014-10-07 Thread Igor Bogomazov
checkpatch.pk tiny fix get rid of 2 warnings and 2 errors for hal/fw.c Signed-off-by: Igor Bogomazov Cc: Greg Kroah-Hartman Cc: navin patidar Cc: Stephen Rothwell --- linux-next 3.17.0 diff --git a/drivers/staging/rtl8188eu/hal/fw.c b/drivers/staging/rtl8188eu/hal/fw.c index 17b7f37..c42