[PATCH] Staging: android: timed_gpio: fixed bare use of 'unsigned'

2016-04-04 Thread Shyam Saini
'unsigned int gpio' is added in place of 'unsigned gpio' Signed-off-by: Shyam Saini <mayhs11sa...@gmail.com> --- drivers/staging/android/timed_gpio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/android/timed_gpio.c b/drivers/staging/android/timed_

[PATCH 1/1] Staging: rdma: hfi1: sdma.c: Fixed coding style issues

2016-05-23 Thread Shyam Saini
Fixed following checkpatch.pl warnings: Prefer 'unsigned int' to bare use of 'unsigned' Prefer READ_ONCE() over ACCESS_ONCE() Signed-off-by: Shyam Saini <mayhs11sa...@gmail.com> --- drivers/staging/rdma/hfi1/sdma.c | 30 +++--- 1 file changed, 15 insertions(

[PATCH 1/1] Staging: rdma: hfi1: rc.c: Fixed coding style issues

2016-05-23 Thread Shyam Saini
Fixed following checkpatch.pl warnings: Prefer 'unsigned int' to bare use of 'unsigned' Prefer READ_ONCE() over ACCESS_ONCE() Signed-off-by: Shyam Saini <mayhs11sa...@gmail.com> --- drivers/staging/rdma/hfi1/rc.c | 12 ++-- drivers/staging/rdma/hfi1/ruc.c | 6 +++--- 2

[PATCH 1/1] Staging: comedi: comedi_fops.c: Fixed coding style issue

2016-05-10 Thread Shyam Saini
Fixed following checkpatch.pl warnings WARNING: Prefer WRITE_ONCE(, ) over ACCESS_ONCE() = WARNING: Prefer READ_ONCE() over ACCESS_ONCE() Signed-off-by: Shyam Saini <mayhs11sa...@gmail.com> --- drivers/staging/comedi/comedi_fops.c | 12 ++-- 1 file changed, 6 insertions(+), 6 del

[PATCH 1/1] Staging: emxx_udc: emxx_udc: fixed coding style issue

2016-05-07 Thread Shyam Saini
fixed checkpatch.pl warning about 'Prefer 'unsigned int' to bare use of 'unsigned' Signed-off-by: Shyam Saini <mayhs11sa...@gmail.com> --- drivers/staging/emxx_udc/emxx_udc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/d

[PATCH 1651/1651] staging: i4l: pcbit: Remove explicit NULL comparison

2016-08-02 Thread shyam saini
Remove the explicit NULL comparison and rewrite in a compact form using Coccinelle Signed-off-by: shyam saini <mayhs11sa...@gmail.com> --- drivers/staging/i4l/pcbit/callbacks.c | 4 ++-- drivers/staging/i4l/pcbit/capi.c | 18 +- drivers/staging/i4l/pcbit/drv.c

[PATCH 1/1] staging: i4l: pcbit: Remove explicit NULL comparison

2016-08-02 Thread shyam saini
Remove the explicit NULL comparison and rewrite in a compact form using Coccinelle Signed-off-by: shyam saini <mayhs11sa...@gmail.com> --- drivers/staging/i4l/pcbit/callbacks.c | 4 ++-- drivers/staging/i4l/pcbit/capi.c | 18 +- drivers/staging/i4l/pcbit/drv.c

[PATCH 1/1] Staging: android: ion: Fixed coding style issues

2016-09-28 Thread shyam saini
Fixed switch case indentation issue and void function return statement issue Signed-off-by: shyam saini <mayhs11sa...@gmail.com> --- drivers/staging/android/ion/ion_of.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging/andro

[PATCH 1/1] Staging: vt6655: Remove explicit NULL comparison using Coccinelle

2016-09-28 Thread shyam saini
Remove the explicit NULL comparison and rewrite in a compact form. Signed-off-by: shyam saini <mayhs11sa...@gmail.com> --- drivers/staging/vt6655/rxtx.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c

[PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle

2016-09-28 Thread shyam saini
Remove the explicit NULL comparison and rewrite in a compact form. Signed-off-by: shyam saini <mayhs11sa...@gmail.com> --- drivers/staging/android/ion/ion_of.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/android/ion/ion_of.c b/drivers/staging/andro

[PATCH 1/1] Staging: android: ion : Remove explicit NULL comparison and Fixed coding style

2016-09-28 Thread shyam saini
Remove the explicit NULL comparison and rewrite in a compact form and fixed switch case indentation issue. Signed-off-by: shyam saini <mayhs11sa...@gmail.com> --- drivers/staging/android/ion/ion_of.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff

Re: [PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle

2016-10-02 Thread Shyam Saini
On Sun, 2016-10-02 at 16:58 +0200, Greg KH wrote: > On Wed, Sep 28, 2016 at 08:58:51PM +0530, shyam saini wrote: > > > > Remove the explicit NULL comparison and rewrite in a compact form. > > > > Signed-off-by: shyam saini <mayhs11sa...@gmail.com> > &g

Re: [PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle

2016-10-02 Thread Shyam Saini
On Sun, 2016-10-02 at 17:37 +0200, Greg KH wrote: > On Sun, Oct 02, 2016 at 08:51:29PM +0530, Shyam Saini wrote: > > > > On Sun, 2016-10-02 at 16:58 +0200, Greg KH wrote: > > > > > > On Wed, Sep 28, 2016 at 08:58:51PM +0530, shyam saini wrote: > > > &g

[PATCH 1/1] Staging: android: ion: Fixed coding style issues

2016-09-28 Thread shyam saini
Fixed checkpatch coding style issues Signed-off-by: shyam saini <mayhs11sa...@gmail.com> --- drivers/staging/android/ion/ion_of.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging/android/ion/ion_of.c b/drivers/staging/andro

Re: [PATCH] Staging: media: bcm2048: Use octal permissions '0444'

2016-10-18 Thread Shyam Saini
On Mon, 2016-10-17 at 21:37 +0200, Greg KH wrote: > On Mon, Oct 17, 2016 at 10:10:08PM +0530, Shyam Saini wrote: > > > > On Mon, 2016-10-17 at 18:07 +0200, Greg KH wrote: > > > > > > On Mon, Oct 17, 2016 at 09:24:19PM +0530, Shyam Saini wrote: > > > >

Re: [PATCH] Staging: media: bcm2048: Use octal permissions '0444'

2016-10-18 Thread Shyam Saini
On Tue, 2016-10-18 at 14:36 +0200, Greg KH wrote: > On Tue, Oct 18, 2016 at 05:52:21PM +0530, Shyam Saini wrote: > > > > On Mon, 2016-10-17 at 21:37 +0200, Greg KH wrote: > > > > > > On Mon, Oct 17, 2016 at 10:10:08PM +0530, Shyam Saini wrote: > > > &g

[PATCH 1/1] Staging: gs_fpgaboot: Use octal permissions '0444'

2016-10-17 Thread Shyam Saini
Fixed following checkpatch warning WARNING: Symbolic permissions 'S_IRUGO' are not preferred. Consider using octal permissions '0444'. Signed-off-by: Shyam Saini <mayhs11sa...@gmail.com> --- drivers/staging/gs_fpgaboot/gs_fpgaboot.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH] Staging: media: bcm2048: Use octal permissions '0644'

2016-10-17 Thread Shyam Saini
Fixed following checkpatch.pl warning WARNING: Symbolic permissions 'S_IRUGO | S_IWUSR' are not preferred. Consider using octal permissions '0644'. Signed-off-by: Shyam Saini <mayhs11sa...@gmail.com> --- drivers/staging/media/bcm2048/radio-bcm2048.c | 40 +--

[PATCH] Staging: media: bcm2048: Use octal permissions '0444'

2016-10-17 Thread Shyam Saini
Fixed following checkpatch warning Symbolic permissions 'S_IRUGO' are not preferred. Consider using octal permissions '0444' Signed-off-by: Shyam Saini <mayhs11sa...@gmail.com> --- drivers/staging/media/bcm2048/radio-bcm2048.c | 18 +- 1 file changed, 9 insertions(+), 9 del

Re: [PATCH] Staging: media: bcm2048: Use octal permissions '0444'

2016-10-17 Thread Shyam Saini
On Mon, 2016-10-17 at 18:07 +0200, Greg KH wrote: > On Mon, Oct 17, 2016 at 09:24:19PM +0530, Shyam Saini wrote: > > > > Fixed following checkpatch warning > > Symbolic permissions 'S_IRUGO' are not preferred. Consider using > > octal > > permissions '0444' &g

[PATCH] staging: rtl8192e: rtl8192e: Remove NULL test before vfree

2017-01-15 Thread Shyam Saini
vfree frees the virtually continuous block of memory beginning at addr. If addr is NULL, no operation is performed. So, NULL test is not needed before vfree. Signed-off-by: Shyam Saini <mayhs11sa...@gmail.com> --- drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 6 ++ 1 file chan

[PATCH] rtl8188eu: core: Remove NULL test before vfree

2017-01-15 Thread Shyam Saini
vfree frees the virtually continuous block of memory beginning at addr. If addr is NULL, no operation is performed. So, NULL test is not needed before vfree(). Signed-off-by: Shyam Saini <mayhs11sa...@gmail.com> --- drivers/staging/rtl8188eu/core/rtw_xmit.c | 3 +-- 1 file changed, 1 ins

Re: [PATCH] Staging: irda: Prefer 'unsigned int' to bare use of 'unsigned'

2018-01-05 Thread Shyam Saini
Hi, Did you read this? https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/staging/irda/TODO Thanks, Shyam > This patch fixes the following checkpatch.pl issue at multiple lines: > > WARNING: Prefer 'unsigned int' to bare use of 'unsigned' > > Signed-off-by: Sumit