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

2015-06-21 Thread Abdul, Hussain (H.)
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

2015-06-21 Thread Abdul, Hussain (H.)
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

2015-06-21 Thread Abdul, Hussain (H.)
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

2015-06-21 Thread Abdul, Hussain (H.)
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

2015-06-21 Thread Abdul, Hussain (H.)
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

2015-06-21 Thread Abdul, Hussain (H.)
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/2] staging: sm750fb: fix typo in debug statement

2015-06-21 Thread Gujulan Elango, Hari Prasath (H.)
From: Hari Prasath Gujulan Elango This patch fixes a typo in the debug statement Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/sm750fb/sm750_accel.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/sm750fb/sm750_accel.c b/drivers/staging/sm7

[PATCH 1/2] staging: sm750fb: remove redundant __func__ in debug statement

2015-06-21 Thread Gujulan Elango, Hari Prasath (H.)
From: Hari Prasath Gujulan Elango This patch removes the redundant __func__ from dynamic debug prints as the pr_xxx set of functions can be dynamically controlled to include function name as well Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/sm750fb/sm750_accel.c | 2 +- 1 fil

[PATCH 2/5] staging: wilc1000: wilc_wfi_netdevice.c: remove prohibited space

2015-06-21 Thread Chaehyun Lim
Fix checkpatch warning found by checkpatch.pl WARNING: space prohibited between function name and open parenthesis '(' Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/wilc_wfi_netdevice.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/wi

[PATCH 3/5] staging: wilc1000: wilc_wfi_netdevice.c: remove braces for single statement block

2015-06-21 Thread Chaehyun Lim
Fix checkpatch warning found by checkpatch.pl WARNING: braces {} are not necessary for single statement blocks Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/wilc_wfi_netdevice.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wfi_ne

[PATCH 5/5] staging: wilc1000: wilc_wfi_netdevice.c: move statement after declarations

2015-06-21 Thread Chaehyun Lim
Fix checkpatch warning found by checkpatch.pl WARNING: Missing a blank line after declarations Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/wilc_wfi_netdevice.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/wilc_wfi_netdevice.c b/driv

[PATCH 4/5] staging: wilc1000: wilc_wfi_netdevice.c: Insert blank line after declarations

2015-06-21 Thread Chaehyun Lim
Fix checkpatch warning found by checkpatch.pl WARNING: Missing a blank line after declarations Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/wilc_wfi_netdevice.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/staging/wilc1000/wilc_wfi_netdevice.c b/drivers/staging/wil

[PATCH 1/5] staging: wilc1000: wilc_wfi_netdevice.c: remove prohibited space before semicolon

2015-06-21 Thread Chaehyun Lim
Fix checkpatch warning found by checkpatch.pl WARNING: space prohibited before semicolon Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/wilc_wfi_netdevice.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/wilc_wfi_netdevice.c b/drivers/stag

[PATCH 1/1] cpu-hotplug: export cpu_hotplug_enable/cpu_hotplug_disable

2015-06-21 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov Loaded Hyper-V module will use these functions to disable CPU hotplug under certain circumstances. Convert cpu_hotplug_disabled to a counter (protected by cpu_add_remove_lock) to support e.g. disable -> disable -> enable call sequences. Signed-off-by: Vitaly Kuznetsov Sig

[PATCH] staging: rtl8192u: ieee80211_rx: Fix incorrect type in assignments

2015-06-21 Thread Gaston Gonzalez
Fix the following incorrect type in assignments detected by sparse: drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:571:37: warning: incorrect type in assignment (different base types) drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:571:37:expected unsigned short [unsigned] [usertype] le

[PATCH 1/2] staging: ft1000-usb: fixed table alignment

2015-06-21 Thread Peter Karlsson
Fixed alignment to 8 bytes per line. Signed-off-by: Peter Karlsson --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 19 +-- 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c b/drivers/staging/ft1000/ft1000-usb/f

[PATCH v2 0/2] checkpatch fixes

2015-06-21 Thread Peter Karlsson
This patch set includes 2 patches fixing checkpatch warnings about lines over 80 characters. Changes since v1: * Split table lines on 8 bytes instead of 7. * Better commit message. Peter Karlsson (2): staging: ft1000-usb: fixed table alignment staging: ft1000-usb: shorten lines to under 80 c

[PATCH 2/2] staging: ft1000-usb: shorten lines to under 80 characters

2015-06-21 Thread Peter Karlsson
Wrap function arguments to shorten lines to under 80 characters. Signed-off-by: Peter Karlsson --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c| 12 drivers/staging/ft1000/ft1000-usb/ft1000_download.c | 21 ++--- 2 files changed, 22 insertions(+), 11 deletion

Re: [PATCH 09/12] staging: lustre: obdclass: Use !x to check for kzalloc failure

2015-06-21 Thread Julia Lawall
> > @@ -885,7 +885,7 @@ static int lmd_parse_mgssec(struct lustr > > length = tail - ptr; > > > > lmd->lmd_mgssec = kzalloc(length + 1, GFP_NOFS); > > - if (lmd->lmd_mgssec == NULL) > > + if (!lmd->lmd_mgssec) > > return -ENOMEM; > > > > memcpy(lmd->lmd_mgsse

Re: [PATCH 09/12] staging: lustre: obdclass: Use !x to check for kzalloc failure

2015-06-21 Thread walter harms
Am 20.06.2015 18:59, schrieb Julia Lawall: > !x is more normal for kzalloc failure in the kernel. > > The semantic patch that makes this change is as follows: > (http://coccinelle.lip6.fr/) > > // > @@ > expression x; > statement S1, S2; > @@ > > x = kzalloc(...); > if ( > - x == NULL > + !x

[PATCH] staging : Comedi : comedi_fops : Fixed the return error code

2015-06-21 Thread Santosh
This patch fixes the checkpatch.pl warning: WARNING: ENOSYS means 'invalid syscall nr' and nothing else Regards, Santosh Pai >From 502464417255edb52a1db71146e2f33e67df87ce Mon Sep 17 00:00:00 2001 From: sanpai Date: Sun, 21 Jun 2015 15:05:00 +0530 Subject: [PATCH] staging : Comedi : comedi_fop

[PATCH] ion_chunk_heap.c: Fixed line over 80 characters

2015-06-21 Thread Guillermo O. Freschi
Simple style fix. Signed-off-by: Guillermo O. Freschi --- drivers/staging/android/ion/ion_chunk_heap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/android/ion/ion_chunk_heap.c b/drivers/staging/android/ion/ion_chunk_heap.c index 5474615..0813163 10064