Re: [PATCH] staging: ozwpan: Fix coding style warnings

On Fri, Sep 05, 2014 at 10:08:23PM -0400, anicoara wrote: > The warnings addressed: > 1. Missing a blank line after declarations. > 2. else is not generally useful after a break or return Please break this up into 2 patches, each patch should only do one thing at a time. thanks, greg k-h ___

[PATCH] drivers: staging: rtl8821ae: Fix '"(foo*)" should be "(foo *)"' errors

Fix checkpatch.pl '"(foo*)" should be "(foo *)"' errors Signed-off-by: Greg Donald --- drivers/staging/rtl8821ae/base.h | 26 +++--- drivers/staging/rtl8821ae/btcoexist/halbtcoutsrc.c | 8 +++ drivers/staging/rtl8821ae/btcoexist/halbtcoutsrc.h | 4 ++-- dr

Re: [PATCH 2/3] staging: lustre: fix pointer whitespace style

On Sat, 2014-09-06 at 13:38 -0400, Spencer Baugh wrote: > Fix errors reported by checkpatch of this kind: [] > diff --git a/drivers/staging/lustre/lustre/include/lustre_import.h > b/drivers/staging/lustre/lustre/include/lustre_import.h [] > @@ -103,9 +103,9 @@ enum lustre_imp_state { > }; > >

[PATCH 1/3] staging: lustre: fix function definition style

Fix errors reported by checkpatch of this kind: ERROR: open brace '{' following function declarations go on the next line Signed-off-by: Spencer Baugh --- drivers/staging/lustre/lustre/include/lustre_import.h | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/st

[PATCH 2/3] staging: lustre: fix pointer whitespace style

Fix errors reported by checkpatch of this kind: ERROR: "foo * bar" should be "foo *bar" Signed-off-by: Spencer Baugh --- drivers/staging/lustre/lustre/include/lustre_import.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre_impo

[PATCH 3/3] staging: lustre: remove trailing whitespace

Fix errors reported by checkpatch of this kind: ERROR: trailing whitespace Signed-off-by: Spencer Baugh --- drivers/staging/lustre/lustre/include/lustre_import.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre_import.h b/dri

Re: [PATCH] staging: bcm: sort.c: fix coding style

Reviewed-by: Matthias Beyer On 06-09-2014 01:59:37, Omar Sandoval wrote: > Fix checkpatch.pl "Alignment should match open parenthesis". > > Signed-off-by: Omar Sandoval > --- > drivers/staging/bcm/sort.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/stag

Re: [PATCH 1/2] staging: dgnc: Make the struct dgnc_board possible to use list facility

On 09/02, Greg KH wrote: > On Tue, Sep 02, 2014 at 07:55:49PM +0200, Konrad Zapalowicz wrote: > > On 09/02, Greg KH wrote: > > > On Sun, Aug 31, 2014 at 11:23:57PM +0200, Konrad Zapalowicz wrote: > > > > This commit adds the struct list_head to the struct dgnc_board so that > > > > it is now possib

[PATCH] staging: bcm: sort.c: fix coding style

Fix checkpatch.pl "Alignment should match open parenthesis". Signed-off-by: Omar Sandoval --- drivers/staging/bcm/sort.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/bcm/sort.c b/drivers/staging/bcm/sort.c index d518c42..ca0b179 100644 --- a/drivers/sta

[PATCH] staging: rtl8821ae: Fix sparse warning by making functions static.

Signed-off-by: Andreas Schlick --- drivers/staging/rtl8821ae/rtl8821ae/hw.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8821ae/rtl8821ae/hw.c b/drivers/staging/rtl8821ae/rtl8821ae/hw.c index 1aa1661..0539105 100644 --- a/drivers/staging/rtl8821ae