Re: [PATCH 3/6] Staging: comedi: daqboard2000: Simplify a trivial if-return sequence

On Friday 19 June 2015 07:38 PM, Sudip Mukherjee wrote: > On Fri, Jun 19, 2015 at 06:00:06AM +0000, Abdul, Hussain (H.) wrote: >> From: Abdul Hussain >> >> This patch simplify a trivial if-return sequence. Possibly combine with >> a preceding function call. >>

[PATCH v2 6/6] Staging: comedi: s626: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/s626.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers/

[PATCH v2 3/6] Staging: combine: daqboard2000: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/daqboard2000.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/staging/come

[PATCH v2 2/6] Staging: comedi: fl512: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/fl512.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers

[PATCH v2 4/6] Staging: comedi: dac02: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/dac02.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers

[PATCH v2 5/6] Staging: comedi: ni_daq_dio24: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/ni_daq_dio24.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/

[PATCH v2 1/6] Staging: comedi: dmm32at: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/dmm32at.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drive

Re: [PATCH 1/6] Staging: comedi: dmm32at: Simplify a trivial if-return sequence

On Friday 19 June 2015 04:30 AM, gre...@linuxfoundation.org wrote: > On Tue, Jun 16, 2015 at 02:03:44PM +0000, Abdul, Hussain (H.) wrote: >> From: Abdul Hussain >> >> This patch simplify a trivial if-return sequence. Possibly combine with >> a preceding function call.

[PATCH 5/6] Staging: comedi: ni_daq_dio24: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/ni_daq_dio24.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/

[PATCH 6/6] Staging: comedi: s626: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/s626.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers/

[PATCH 3/6] Staging: comedi: daqboard2000: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/daqboard2000.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/staging/comedi

[PATCH 4/6] Staging: comedi: dac02: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/dac02.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers

[PATCH 1/6] Staging: comedi: dmm32at: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/dmm32at.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drive

[PATCH 2/6] Staging: comedi: fl512: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/fl512.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers

[PATCH 0/6] Staging: comedi: Simplify a trivial if-return sequence

This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Abdul Hussain (6): staging: comedi: dmm32at: Simplify a trivial if-return sequence staging: comedi: fl512: Simplify a trivial if-return sequence staging: comedi: daqboard2000: Simplify

[PATCH] Staging: wilc1000: NULL check before some freeing functions is not needed

From: Abdul Hussain This patch removes check before freeing the memory since kfree(NULL) is safe Signed-off-by: Abdul Hussain --- drivers/staging/wilc1000/linux_wlan_spi.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/wilc1000/linux_wlan_spi.c b/dr

[PATCH v2] Staging: wilc1000: Remove casting the values returned by kmalloc()

From: Abdul Hussain This patch removes casting the values returned by memory allocation functions. Signed-off-by: Abdul Hussain --- drivers/staging/wilc1000/linux_mon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/linux_mon.c b/drivers/staging/w

Re: [PATCH] Staging: wilc1000: Remove casting the values returned by kmalloc()

On Wednesday 17 June 2015 07:48 AM, gre...@linuxfoundation.org wrote: > On Tue, Jun 16, 2015 at 07:09:56AM +0000, Abdul, Hussain (H.) wrote: >> From: Abdul Hussain >> >> This patch removes casting the values returned by memory allocation >> functions. >>

Re: [PATCH] Staging: comedi: Simplify a trivial if-return sequence

On Tuesday 16 June 2015 06:42 PM, Ian Abbott wrote: > On 16/06/15 08:58, Abdul, Hussain (H.) wrote: >> From: Abdul Hussain >> >> This patch simplify a trivial if-return sequence. Possibly combine with >> a preceding function call. >> >> Signed-off-by: Abd

[PATCH 2/6] Staging: comedi: fl512: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/fl512.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers

[PATCH 5/6] Staging: comedi: ni_daq_dio24: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/ni_daq_dio24.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/

[PATCH 4/6] Staging: comedi: dac02: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/dac02.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers

[PATCH 6/6] Staging: comedi: s626: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/s626.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers/

[PATCH 0/6] Staging: comedi: Simplify a trivial if-return sequence

This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Abdul Hussain (6): staging: comedi: dmm32at: Simplify a trivial if-return sequence staging: comedi: fl512: Simplify a trivial if-return sequence staging: comedi: daqboard2000: Simplify a

[PATCH 3/6] Staging: comedi: daqboard2000: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/daqboard2000.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/staging/comedi

[PATCH 1/6] Staging: comedi: dmm32at: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/dmm32at.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drive

Re: [PATCH] Staging: wilc1000: Boolean tests don't need comparisons

On Tuesday 16 June 2015 04:41 PM, Sudip Mukherjee wrote: > On Tue, Jun 16, 2015 at 08:55:13AM +0000, Abdul, Hussain (H.) wrote: >> On Tuesday 16 June 2015 01:40 PM, Sudip Mukherjee wrote: >>> On Tue, Jun 16, 2015 at 07:33:42AM +0000, Abdul, Hussain (H.) wrote: >>>> Fr

[PATCH] Staging: wilc1000: Assign proper boolean value

From: Abdul Hussain This patch assign proper boolean value to boolean variable. Signed-off-by: Abdul Hussain --- drivers/staging/wilc1000/host_interface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/h

[PATCH v2] Staging: wilc1000: Boolean tests don't need comparisons

From: Abdul Hussain This patch removes unwanted true and false from boolean tests. Signed-off-by: Abdul Hussain --- drivers/staging/wilc1000/coreconfigurator.c | 8 drivers/staging/wilc1000/linux_mon.c | 2 +- drivers/staging/wilc1000/linux_wlan.c | 2 +-

Re: [PATCH] Staging: wilc1000: Boolean tests don't need comparisons

On Tuesday 16 June 2015 01:40 PM, Sudip Mukherjee wrote: > On Tue, Jun 16, 2015 at 07:33:42AM +0000, Abdul, Hussain (H.) wrote: >> From: Abdul Hussain >> >> This patch removes unwanted true and false from boolean tests. >> >> Signed-off-by: Abdul Hussain >>

[PATCH] Staging: comedi: Simplify a trivial if-return sequence

From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/dac02.c| 6 +- drivers/staging/comedi/drivers/daqboard2000.c | 6 +- drivers/staging/comedi/dr

[PATCH] Staging: wilc1000: Boolean tests don't need comparisons

From: Abdul Hussain This patch removes unwanted true and false from boolean tests. Signed-off-by: Abdul Hussain --- drivers/staging/wilc1000/coreconfigurator.c | 8 drivers/staging/wilc1000/host_interface.c | 2 +- drivers/staging/wilc1000/linux_mon.c | 2 +-

[PATCH v2] Staging: lusture: Remove an open coded simple_open() function

From: Abdul Hussain This patch removes an open coded simple_open() function and replace file operations references to the function with simple_open() instead Signed-off-by: Abdul Hussain --- drivers/staging/lustre/lustre/fld/lproc_fld.c | 9 + 1 file changed, 1 insertion(+), 8 deletion

[PATCH] Staging: wilc1000: Remove casting the values returned by kmalloc()

From: Abdul Hussain This patch removes casting the values returned by memory allocation functions. Signed-off-by: Abdul Hussain --- drivers/staging/wilc1000/linux_wlan.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/s

[PATCH] Staging: wilc1000: Remove casting the values returned by kmalloc()

From: Abdul Hussain This patch removes casting the values returned by memory allocation functions. Signed-off-by: Abdul Hussain --- drivers/staging/wilc1000/linux_mon.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/linux_mon.c b/drivers/stagi

[PATCH] Staging: lustre: Use memdup_user rather than duplicating its implementation

From: Abdul Hussain This patch uses memdup_user rather than duplicating its implementation Signed-off-by: Abdul Hussain --- drivers/staging/lustre/lustre/llite/file.c | 22 ++ 1 file changed, 6 insertions(+), 16 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite

[PATCH] Staging: lustre: Use memdup_user rather than duplicating its implementation

From: Abdul Hussain This patch uses memdup_user rather than duplicating its implementation Signed-off-by: Abdul Hussain --- drivers/staging/lustre/lustre/llite/dir.c | 32 +-- 1 file changed, 9 insertions(+), 23 deletions(-) diff --git a/drivers/staging/lustre/lust

[PATCH] Staging: lusture: Remove an open coded simple_open() function

From: Abdul Hussain This patch removes an open coded simple_open() function and replace file operations references to the function with simple_open() instead Signed-off-by: Abdul Hussain --- drivers/staging/lustre/lustre/fld/lproc_fld.c | 9 + 1 file changed, 1 insertion(+), 8 deletion

[PATCH] staging: vt6656: Boolean tests don't need comparisons.

From: Abdul Hussain This patch remove true and false from boolean tests. Signed-off-by: Abdul Hussain --- drivers/staging/vt6656/card.c | 2 +- drivers/staging/vt6656/main_usb.c | 2 +- drivers/staging/vt6656/rxtx.c | 2 +- drivers/staging/vt6656/usbpipe.c | 2 +- 4 files changed, 4 i

[PATCH] staging: vt6655: Boolean tests don't need comparisons

From: Abdul Hussain This patch removes unwanted true from boolean tests. Signed-off-by: Abdul Hussain --- 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 index 33c4aa4..53433

Re: [PATCH] staging: vt6655: Boolean tests don't need comparisons

On Thursday 11 June 2015 05:24 PM, Sudip Mukherjee wrote: > On Thu, Jun 11, 2015 at 09:51:15AM +0000, Abdul, Hussain (H.) wrote: >> This patch removes unwnated true from boolean tests. > did u mean "unwanted"? > > and for all your four patches the email header From

Re: [PATCH] staging: ste_rmi4: Add IRQF_ONESHOT flag

On Thursday 11 June 2015 08:06 PM, gre...@linuxfoundation.org wrote: > On Thu, Jun 11, 2015 at 10:14:49AM +0000, Abdul, Hussain (H.) wrote: >> This patch add IRQF_ONESHOT flag in threaded IRQs request without a primary >> handler. >> >> Signed-off-by: Abdul Hussain

[PATCH] staging: ste_rmi4: Add IRQF_ONESHOT flag

This patch add IRQF_ONESHOT flag in threaded IRQs request without a primary handler. Signed-off-by: Abdul Hussain --- drivers/staging/ste_rmi4/synaptics_i2c_rmi4.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/ste_rmi4/synaptics_i2c_rmi4.c b/drivers/stagin

[PATCH] staging: unisys: Remove unneeded variable

This patch remove unneeded variable used to store return value. Signed-off-by: Abdul Hussain --- drivers/staging/unisys/visorbus/visorbus_main.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/unisys/visorbus/visorbus_main.c b/drivers/staging/unisys/visorbu

[PATCH] staging: vt6655: Boolean tests don't need comparisons

This patch removes unwnated true from boolean tests. Signed-off-by: Abdul Hussain --- 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 index 33c4aa4..534338c 100644 --- a/driver

[PATCH] staging: vt6656: Boolean tests don't need comparisons.

This patch remove true and false from boolean tests. Signed-off-by: Abdul Hussain --- drivers/staging/vt6656/card.c | 2 +- drivers/staging/vt6656/main_usb.c | 2 +- drivers/staging/vt6656/rxtx.c | 2 +- drivers/staging/vt6656/usbpipe.c | 2 +- 4 files changed, 4 insertions(+), 4 deleti