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 b/drivers/staging/
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 --git a/drivers/staging/i4l
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 a/drivers/staging/fsl-mc/bus/dpmng-cmd.h
b/drivers/st
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 a/drivers/staging/fsl-mc/bus/dprc-cmd.h
b/drivers/stag
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 ---
drivers/staging/fsl-mc/
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, 2 insertions(+), 2 deletio
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 a/drivers/staging/comedi/drivers/ni_at_a215
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 a/drivers/staging/comedi/drivers/jr3_pci.c
b/drivers/staging/co
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 insertions(+), 7 deletions(-)
diff
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 a/drivers/staging/comedi/driver
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 ++---
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 a/drivers/staging/comedi/driv
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 deletions(-)
diff --git a/d
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 s
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.
__
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: jr3_pci.h
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 whitesp
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.
>>
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 based on the key inde
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 a/drivers/staging/comedi/drivers/ni_at_a215
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 a/drivers/staging/comedi/drivers/jr3_pci.c
b/drivers/staging/co
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 a/drivers/staging/comedi/driver
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 insertions(+), 7 deletions(-)
diff
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 ++---
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 a/drivers/staging/comedi/driv
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 deletions(-)
diff --git a/d
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 s
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 DIDmib_dot11smt_dot1
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
---
drivers/staging/wlan-ng/cfg80211.c | 85 ++
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 deletion(-)
diff --git a/drivers/staging/rtl8188eu/os_d
30 matches
Mail list logo
