[PATCH] staging: dgnc: Let line have less 80 char

This patch fix a minor checkpath warming: "WARNING: line over 80 characters" Signed-off-by: Sean --- drivers/staging/dgnc/dgnc_neo.c | 116 1 file changed, 82 insertions(+), 34 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_neo.c

[PATCH] staging: i4l: act2000: capi: Fix checkpatch warning

Fix checkpath.pl warning: trailing statements should be on next line open brace '{' following function declarations go on the next line Signed-off-by: Anson Jacob --- drivers/staging/i4l/act2000/capi.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff

[PATCH 1/2] staging: fsl-mc: dpmng-cmd: Fix checkpatch warning

Fix checkpatch.pl warning: Block comments use * on subsequent lines Fix Commenting Style Signed-off-by: Anson Jacob --- drivers/staging/fsl-mc/bus/dpmng-cmd.h | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git

[PATCH 2/2] staging: fsl-mc: dprc-cmd: Fix checkpatch warning

Fix checkpatch.pl warning: Block comments use * on subsequent lines Fix Commenting Style Signed-off-by: Anson Jacob --- drivers/staging/fsl-mc/bus/dprc-cmd.h | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git

[PATCH 0/2] staging: fsl-mc: Fix checkpatch warning

Fix checkpatch.pl warning: Block comments use * on subsequent lines Fix Commenting Style Anson Jacob (2): staging: fsl-mc: dpmng-cmd: Fix checkpatch warning staging: fsl-mc: dprc-cmd: Fix checkpatch warning drivers/staging/fsl-mc/bus/dpmng-cmd.h | 15 ---

[PATCH v3 8/8] staging: comedi: cb_pcidas64: Fix checkpath warning

Fix checkpatch.pl warning: Please use a blank line after function/struct/union/enum declarations Move macro definition to the top of the file Aligne macro definitions Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/cb_pcidas64.c | 4 ++-- 1 file changed,

[PATCH v3 7/8] staging: comedi: ni_at_a2150: Fix checkpatch warning

Fix checkpatch.pl warning 'line over 80 characters' Remove extra space after /* Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/ni_at_a2150.c | 170 ++- 1 file changed, 89 insertions(+), 81 deletions(-) diff --git

[PATCH v3 6/8] staging: comedi: jr3_pci.c: Fix checkpatch warning

Fix checkpatch.pl warning 'line over 80 characters' Remove extra space after /* Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/jr3_pci.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git

[PATCH v3 4/8] staging: comedi: ni_atmio.c: Fix checkpatch warning

Fix checkpatch.pl warning for 'Statements should start on a tabstop' Convert single line block comment to single line comment Align single line comment Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/ni_atmio.c | 11 --- 1 file changed, 4

[PATCH v3 5/8] staging: comedi: s626.h: Fix checkpatch warning

Fix checkpatch.pl warning for Comparisons should place the constant on the right side of the test Remove extra space after /* Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/s626.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v3 3/8] staging: comedi: jr3_pci.h: Fix checkpatch warning

Fix checkpatch.pl warning: Block comments use * on subsequent lines Block comments use a trailing */ on a separate line Block comments starts with /* followed by comments on the next line Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/jr3_pci.h | 290

[PATCH v3 2/8] staging: comedi: cb_pcidas64: Fix commenting style

Remove additional space after /* Convert single line block comment into single line comment Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/cb_pcidas64.c | 431 +-- 1 file changed, 214 insertions(+), 217 deletions(-) diff --git

[PATCH v3 1/8] staging: comedi: cb_pcidas64.c: Fix checkpatch warning

Fix checkpatch.pl warning: Block comments use * on subsequent lines Block comments use a trailing */ on a separate line Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/cb_pcidas64.c | 148 ++- 1 file changed, 77 insertions(+), 71

[PATCH v3 0/8] staging: comedi: Fix checkpatch warning

Fix checkpath.pl warning: line over 80 characters Block comments use * on subsequent lines Block comments use a trailing */ on a separate line Statements should start on a tabstop Comparisons should place the constant on the right side of the test Changes in V2: Remove extra space after /* Align

Re: [PATCH v2 3/7] staging: comedi: jr3_pci.h: Fix checkpatch warning

On Fri, Aug 26, 2016 at 06:35:51PM -0400, Anson Jacob wrote: > I only found one trailing whitespace issue in the patchset. Yes, there's just one. > Are my commits are too long. Am I supposed to submit shorter ones. In my opinion your commits are generally fine.

Re: [PATCH v2 3/7] staging: comedi: jr3_pci.h: Fix checkpatch warning

On Sat, Aug 27, 2016 at 12:23:58AM +0300, Andrey Utkin wrote: > On Fri, Aug 26, 2016 at 02:33:08PM -0400, Anson Jacob wrote: > > > > - /* Default_FS contains the full scale that is used if the user does */ > > - /* not set a full scale. */ > > + /* > > Applying: staging: comedi:

Re: [PATCH v2 3/7] staging: comedi: jr3_pci.h: Fix checkpatch warning

On Fri, Aug 26, 2016 at 02:33:08PM -0400, Anson Jacob wrote: > > - /* Default_FS contains the full scale that is used if the user does */ > - /* not set a full scale. */ > + /* Applying: staging: comedi: jr3_pci.h: Fix checkpatch warning .git/rebase-apply/patch:226: trailing

Re: [PATCH] staging/lustre: avoid zero buf for the first time

Hello! On Aug 22, 2016, at 6:04 AM, Greg Kroah-Hartman wrote: > On Mon, Aug 22, 2016 at 04:46:04PM +0800, Shawn Lin wrote: >> We only need to zero it when repeating in order to >> avoid old garbage. Let's improve it by moving this >> before we repeat the calculation to save some cpu >> cycle. >>

Re: [PATCH v2] Staging: wlan-ng: replace switch-case statements with macro

I will do it soon. Thanks, Claudiu On Fri, Aug 26, 2016 at 9:07 PM, Joe Perches wrote: > On Fri, 2016-08-26 at 20:58 +0300, Claudiu Beznea wrote: >> This patch removes multiple switch-case statements >> with a new macro. The macro will generate the >> corresponding bit mask

[PATCH v2 7/7] staging: comedi: ni_at_a2150: Fix checkpatch warning

Fix checkpatch.pl warning 'line over 80 characters' Remove extra space after /* Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/ni_at_a2150.c | 170 ++- 1 file changed, 89 insertions(+), 81 deletions(-) diff --git

[PATCH v2 6/7] staging: comedi: jr3_pci.c: Fix checkpatch warning

Fix checkpatch.pl warning 'line over 80 characters' Remove extra space after /* Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/jr3_pci.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git

[PATCH v2 5/7] staging: comedi: s626.h: Fix checkpatch warning

Fix checkpatch.pl warning for Comparisons should place the constant on the right side of the test Remove extra space after /* Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/s626.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v2 4/7] staging: comedi: ni_atmio.c: Fix checkpatch warning

Fix checkpatch.pl warning for 'Statements should start on a tabstop' Convert single line block comment to single line comment Align single line comment Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/ni_atmio.c | 11 --- 1 file changed, 4

[PATCH v2 3/7] staging: comedi: jr3_pci.h: Fix checkpatch warning

Fix checkpatch.pl warning: Block comments use * on subsequent lines Block comments use a trailing */ on a separate line Block comments starts with /* followed by comments on the next line Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/jr3_pci.h | 290

[PATCH v2 2/7] staging: comedi: cb_pcidas64: Fix commenting style

Remove additional space after /* Convert single line block comment into single line comment Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/cb_pcidas64.c | 431 +-- 1 file changed, 214 insertions(+), 217 deletions(-) diff --git

[PATCH v2 1/7] staging: comedi: cb_pcidas64.c: Fix checkpatch warning

Fix checkpatch.pl warning: Block comments use * on subsequent lines Block comments use a trailing */ on a separate line Signed-off-by: Anson Jacob --- drivers/staging/comedi/drivers/cb_pcidas64.c | 148 ++- 1 file changed, 77 insertions(+), 71

[PATCH v2 0/7] staging: comedi: Fix checkpatch warning

Fix checkpath.pl warning: line over 80 characters Block comments use * on subsequent lines Block comments use a trailing */ on a separate line Statements should start on a tabstop Comparisons should place the constant on the right side of the test Changes in V2: Remove extra space after /* Align

Re: [PATCH v2] Staging: wlan-ng: replace switch-case statements with macro

On Fri, 2016-08-26 at 20:58 +0300, Claudiu Beznea wrote: > This patch removes multiple switch-case statements > with a new macro. The macro will generate the > corresponding bit mask based on the key index > received as input. Can you go a little farther and remove the #define

[PATCH v2] Staging: wlan-ng: replace switch-case statements with macro

This patch removes multiple switch-case statements with a new macro. The macro will generate the corresponding bit mask based on the key index received as input. Chances since v1: Corrected patch title Signed-off-by: Claudiu Beznea ---

[PATCH -next] staging: rtl8188eu: fix missing unlock on error in rtw_resume_process()

From: Wei Yongjun Add the missing unlock before return from function rtw_resume_process() in the error handling case. Signed-off-by: Wei Yongjun --- drivers/staging/rtl8188eu/os_dep/usb_intf.c | 4 +++- 1 file changed, 3 insertions(+), 1