Re: [PATCH] Staging: dgnc: fix line should not end with a paranthesis in dgnc_tty.c

2018-02-14 Thread Greg KH
On Thu, Feb 15, 2018 at 11:05:08AM +0530, Yash Omer wrote: > This is a patch to the dgnc_tty.c file that fixes up a paranthesis warning > found by the checkpatch.pl tool. Please wrap your changelog text at 72 columns, and run it through a spell checker if possible. thanks, greg k-h ___

Re: [PATCH] Staging: comedi: fix camelcase issue in range.c

2018-02-14 Thread Greg KH
On Thu, Feb 15, 2018 at 01:36:43AM +0530, Yash Omer wrote: > This is a patch to the range.c file that fixes up a camelcase warning found > bythe checkpatch.pl tool. > > Signed-off-by: Yash Omer > --- > drivers/staging/comedi/range.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-

[PATCH] Staging: dgnc: fix line should not end with a paranthesis in dgnc_tty.c

2018-02-14 Thread Yash Omer
This is a patch to the dgnc_tty.c file that fixes up a paranthesis warning found by the checkpatch.pl tool. Signed-off-by: Yash Omer --- drivers/staging/dgnc/dgnc_tty.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgn

Re: [PATCH] staging: media: reformat line to 80 chars or less

2018-02-14 Thread Steve Longerbeam
Hi Parthiban, please rename the commit title to "media: imx: capture: reformat line to 80 chars or less" Otherwise it is fine with me. Steve On 02/12/2018 04:05 AM, Parthiban Nallathambi wrote: This is a cleanup patch to fix line length issue found by checkpatch.pl script. In this patch, li

Re: [PATCH] staging: imx-media-vdic: fix inconsistent IS_ERR and PTR_ERR

2018-02-14 Thread Gustavo A. R. Silva
Hi all, I was just wondering about the status of this patch. Thanks -- Gustavo On 01/24/2018 06:14 PM, Steve Longerbeam wrote: Acked-by: Steve Longerbeam On 01/23/2018 04:43 PM, Gustavo A. R. Silva wrote: Fix inconsistent IS_ERR and PTR_ERR in vdic_get_ipu_resources. The proper pointer to

[PATCH] Staging: comedi: fix camelcase issue in range.c

2018-02-14 Thread Yash Omer
This is a patch to the range.c file that fixes up a camelcase warning found by the checkpatch.pl tool. Signed-off-by: Yash Omer --- drivers/staging/comedi/range.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/comedi/range.c b/drivers/staging/comedi/range.c

Re: [PATCH char-misc 1/1] Drivers: hv: vmbus: Fix ring buffer signaling

2018-02-14 Thread Stephen Hemminger
What about the following simplification. If on older hosts you can avoid more code and the additional barrier. Also cleanup comment wording somewhat. >From de2d566d9092cbbc8e5974dea581617ef787ff69 Mon Sep 17 00:00:00 2001 From: Michael Kelley Date: Sat, 10 Feb 2018 20:48:49 + Subject: [PAT

[PATCH 3/6] staging: wilc1000: fix line over 80 characters in spi_cmd_complete()

2018-02-14 Thread Ajay Singh
Refactor spi_cmd_complete() to fix the line over 80 char issues reported by checkpatch.pl script. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/wilc_spi.c | 250 ++-- 1 file changed, 124 insertions(+), 126 deletions(-) diff --git a/drivers/staging/wilc10

[PATCH 4/6] staging: wilc1000: fix line over 80 characters in wilc_spi_init()

2018-02-14 Thread Ajay Singh
Modified wilc_spi_init() to fix the line over 80 char issues reported by checkpatch.pl script. To overcome the checkpatch.pl reported issue modified debug logs and comments used in wilc_spi_init(). Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/wilc_spi.c | 18 +++--- 1 file

[PATCH 6/6] staging: wilc1000: fix line over 80 chars in wilc_spi_clear_int_ext()

2018-02-14 Thread Ajay Singh
Refactor wilc_spi_clear_int_ext() to fix the "line over 80 char" issue reported by checkpatch.pl script. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/wilc_spi.c | 113 +--- 1 file changed, 54 insertions(+), 59 deletions(-) diff --git a/drivers/staging/w

[PATCH 5/6] staging: wilc1000: fix line over 80 characters in wilc_spi_read_int()

2018-02-14 Thread Ajay Singh
Refactor wilc_spi_read_int() to fix the line over 80 char issues reported by checkpatch.pl script. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/wilc_spi.c | 57 +++-- 1 file changed, 29 insertions(+), 28 deletions(-) diff --git a/drivers/staging/wilc100

[PATCH 1/6] staging: wilc1000: modified code comments as per linux coding style

2018-02-14 Thread Ajay Singh
Cleanup patch to follow the comments style as per the Linux coding style. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/wilc_spi.c | 151 1 file changed, 82 insertions(+), 69 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_spi.c b/drivers/s

[PATCH 2/6] staging: wilc1000: removed the unnecessary commented code

2018-02-14 Thread Ajay Singh
Cleanup patch to remove the unused commented code. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/wilc_spi.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_spi.c b/drivers/staging/wilc1000/wilc_spi.c index 5d0de4e..66b6aea 100644 --- a/drivers/sta

[PATCH 0/6] fix line over 80 char & coding style in wilc_spi.c

2018-02-14 Thread Ajay Singh
This patch series contains changes to remove "fix line over 80 char" issues found by checkpatch.pl script. Also, few changes code changes are done to follow Linux coding style. Ajay Singh (6): staging: wilc1000: modified code comments as per linux coding style staging: wilc1000: removed the un