[PATCH v2] Staging: comedi: dt2814: remove unused assignments

function ‘dt2814_attach’: drivers/staging/comedi/drivers/dt2814.c:232:6: warning: variable ‘i’ set but not used [-Wunused-but-set-variable] int i; ^ Signed-off-by: Gaurav Pathak --- v2: Update subject and commit message. drivers/staging/comedi/drivers/dt2814.c | 10 -- 1 file

[PATCH v2] staging: greybus: Fix warning to limit chars per line

Signed-off-by: Gaurav Dhingra --- Changes in v2: - use correct format for multiline comment --- drivers/staging/greybus/audio_codec.h | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/staging/greybus/audio_codec.h b/drivers/staging/greybus/audio_codec.h index

Re: [PATCH v2] staging: greybus: Fix warning to limit chars per line

Hi, On Friday 06 April 2018 03:52 PM, Viresh Kumar wrote: On Fri, Apr 6, 2018 at 3:39 PM, Gaurav Dhingra wrote: Why did you remove the commit log? You had the right one in v1. I didn't realize that it would be necessary to add it to this patch set. I thought you guys will do squashi

[PATCH v3] staging: greybus: Fix warning to limit chars per line

Wrap comment to fix warning "prefer a maximum 75 chars per line" Signed-off-by: Gaurav Dhingra --- Changes in v2: - use correct format for multiline comment Changes in v3: - include commit log --- drivers/staging/greybus/audio_codec.h | 5 - 1 file changed, 4 insert

[PATCH 35/35] Staging:dgap:dgap_tty.c: Upgraded driver to use Mutex instead of semaphore

dgap driver uses semaphore for obtaining lock. I upgraded it to use MUTEX. Signed-off-by: Kumar Gaurav --- drivers/staging/dgap/dgap_tty.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/drivers/staging/dgap/dgap_tty.c b/drivers/staging/dgap/dgap_tty.c

Re: [PATCH 35/35] Staging:dgap:dgap_tty.c: Upgraded driver to use Mutex instead of semaphore

Hi All, Please ignore this patch. I'm sending with correct Subject Line On Thursday 17 April 2014 09:21 PM, Kumar Gaurav wrote: dgap driver uses semaphore for obtaining lock. I upgraded it to use MUTEX. Signed-off-by: Kumar Gaurav --- drivers/staging/dgap/dgap_tty.c |

[PATCH] Staging:dgap:dgap_tty.c: Upgraded driver to use Mutex instead of semaphore

dgap driver uses semaphore for obtaining lock. I upgraded it to use MUTEX. Signed-off-by: Kumar Gaurav --- drivers/staging/dgap/dgap_tty.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/drivers/staging/dgap/dgap_tty.c b/drivers/staging/dgap/dgap_tty.c

Re: [PATCH] Staging:dgap:dgap_tty.c: Upgraded driver to use Mutex instead of semaphore

Hi All, Any update on below patch. Do i need to make any modification? On Thursday 17 April 2014 09:25 PM, Kumar Gaurav wrote: dgap driver uses semaphore for obtaining lock. I upgraded it to use MUTEX. Signed-off-by: Kumar Gaurav --- drivers/staging/dgap/dgap_tty.c | 17

[PATCH v1] staging: rts5208: Change fixed function names with "%s: ", __func__

staging: rts5208: Refactored code to avoid few defects. Signed-off-by: Gaurav Pathak --- Hi, Gaurav hope this text appaers in the email, and i put it at the right place in patch. drivers/staging/rts5208/ms.c| 5 +++-- drivers/staging/rts5208/rtsx.c | 4 ++-- drivers/staging

[PATCH v1] staging: rts5208: Change fixed function names with "%s: ", __func__

staging: rts5208: Fix coding style issue caught by checkpatch.pl related to function name in debug print and shorten few lines over 80 char Signed-off-by: Gaurav Pathak --- drivers/staging/rts5208/ms.c| 5 +++-- drivers/staging/rts5208/rtsx.c | 4 ++-- drivers/staging/rts5208

[PATCH v1] staging: rts5208: Change fixed function names with "%s: ", __func__

staging: rts5208: Fix coding style issue caught by checkpatch.pl related to function name in debug print Signed-off-by: Gaurav Pathak --- drivers/staging/rts5208/ms.c| 5 +++-- drivers/staging/rts5208/rtsx.c | 4 ++-- drivers/staging/rts5208/rtsx_chip.c | 4 ++-- drivers/staging

Query:Regarding object poison overwritten in binder_transaction

0: 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a     368.362215:   <6> Padding ffc5a0692fd0: 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a     368.371681:   <6> Padding ffc5a0692fe0: 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a 5a

Re: Query:Regarding object poison overwritten in binder_transaction

Thanks Greg, I will check the common tree, and it seems to me a new bug , will file a bug if won't be able to resolve from android-common tree. Regards Gaurav On 3/4/2018 12:57 AM, Greg Kroah-Hartman wrote: On Sat, Mar 03, 2018 at 08:22:35PM +0530, Kohli, Gaurav wrote: HI , Is

[PATCH] Removing ununsed variable int lo, hi, int data and int i from comedi/drivers/dt2814.c.

function ‘dt2814_attach’: drivers/staging/comedi/drivers/dt2814.c:232:6: warning: variable ‘i’ set but not used [-Wunused-but-set-variable] int i; ^ Signed-off-by: Gaurav Pathak --- drivers/staging/comedi/drivers/dt2814.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers

Re: [PATCH] Removing ununsed variable int lo, hi, int data and int i from comedi/drivers/dt2814.c.

:44PM +0200, Luc Van Oostenryck wrote: > Hi, > > On Sun, May 24, 2020 at 05:06:18PM +0530, Gaurav Pathak wrote: > > Silence following sparse warning: > > drivers/staging/comedi/drivers/dt2814.c: In function ‘dt2814_interrupt’: > > drivers/staging/comedi/drivers/dt2814

Re: [PATCH] Removing ununsed variable int lo, hi, int data and int i from comedi/drivers/dt2814.c.

said. > > regards, > dan carpenter > Hello Dan, Thank you for reviewing and suggesting changes. I have modified the patch and attached it with the e-mail to keep this thread conversation. I hope I have made the changes correctly. Thanks, Gaurav >From ef4a63e66fc09052c4cad808755d1

[PATCH v3] Staging: comedi: dt2814: remove unused assignments

From: Gaurav Pathak Silence following compiler warning: drivers/staging/comedi/drivers/dt2814.c: In function ‘dt2814_interrupt’: drivers/staging/comedi/drivers/dt2814.c:193:6: warning: variable ‘data’ set but not used [-Wunused-but-set-variable] int data; ^~~~ drivers/staging/comedi

[PATCH] USBIP:stub_dev.c fixing string splitted into multiple line issue

Fixed String splitted into multiple line issue using macro Signed-off-by: Kumar Gaurav --- drivers/staging/usbip/stub_dev.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/drivers/staging/usbip/stub_dev.c b/drivers/staging/usbip/stub_dev.c index 83d629a

Re: [PATCH] USBIP:stub_dev.c fixing string splitted into multiple line issue

the macro is using. 2013/8/1 Julia Lawall mailto:julia.law...@lip6.fr>> On Wed, 31 Jul 2013, Kumar Gaurav wrote: > Fixed String splitted into multiple line issue using macro I'm not an expert on this kind of style issue, but I prefer strings that loo

[PATCH] Staging:zram:zram_drv.c Fixed Error of trailing whitespace

Fixed by removing trailing whitespace Signed-off-by: Kumar Gaurav --- drivers/staging/zram/zram_drv.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/zram/zram_drv.c b/drivers/staging/zram/zram_drv.c index 82c7202..f590715 100644 --- a/drivers/staging/zram