[PATCH] staging: lustre: Fix function declaration/definition mismatch

2016-12-04 Thread sandeepjain . linux
From: Sandeep Jain Fixes following Sparse errors. lprocfs_status.c:1568:5: error: symbol 'lprocfs_wr_root_squash' redeclared with different type... lprocfs_status.c:1632:5: error: symbol 'lprocfs_wr_nosquash_nids' redeclared with different type... Signed-off-by:

[PATCH] staging: lustre: mgc: make llog_process_lock static

2016-12-04 Thread sandeepjain . linux
From: Sandeep Jain Fix following sparse warning. mgc_request.c:376:1: warning: symbol 'llog_process_lock' was not declared. Should it be static? Signed-off-by: Sandeep Jain --- drivers/staging/lustre/lustre/mgc/mgc_request.c | 2 +- 1

[PATCH V2] staging: dgnc: Fix lines longer than 80 characters

2016-12-04 Thread Fernando Apesteguia
For the first lines of the patch, I opted to create a small function instead of breaking the the line in a weird way. This is v2 of the patch with the name of the function changed from v1 The other changes are simple ones. Signed-off-by: Fernando Apesteguia ---

Re: [PATCH] staging: dgnc: Fix lines longer than 80 characters

2016-12-04 Thread Fernando Apesteguia
On Sat, Dec 03, 2016 at 11:11:23AM +0100, Greg KH wrote: > On Sat, Dec 03, 2016 at 10:56:54AM +0100, Fernando Apesteguia wrote: > > On Sat, Dec 03, 2016 at 09:51:13AM +0100, Greg KH wrote: > > > On Fri, Dec 02, 2016 at 08:13:49PM +0100, Fernando Apesteguia wrote: > > > > For the first lines of the

Re: [PATCH 01/19] staging: iio: isl29028: remove nested if statements

2016-12-04 Thread Jonathan Cameron
On 04/12/16 02:19, Brian Masney wrote: > There are two callers to the function isl29028_set_als_ir_mode() and > both instances use a nested if statement to only change the chip state > if it is not in the proper mode. This patch moves this check into the > isl29028_set_als_ir_mode() function to

Re: [PATCH 1/1] staging: iio: cdc: fix improper return value

2016-12-04 Thread Jonathan Cameron
On 03/12/16 13:44, Pan Bian wrote: > From: Pan Bian > > At the end of function ad7150_write_event_config(), directly returns 0. > As a result, the errors will be ignored by the callers. It may be better > to return variable "ret". > > Signed-off-by: Pan Bian

[PATCH 2/2] Staging: ks7010: eap_packet.h: fixed coding style issue

2016-12-04 Thread Manoj Sawai
Rearranged comments so that lines are less than 80 characters long Signed-off-by: Manoj Sawai --- drivers/staging/ks7010/eap_packet.h | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/staging/ks7010/eap_packet.h

[PATCH] Staging: ks7010: ks7010_stio.h: fixed coding style error

2016-12-04 Thread Manoj Sawai
Fixed 2 coding style errors about trailing whitespaces. Signed-off-by: Manoj Sawai --- drivers/staging/ks7010/ks7010_sdio.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ks7010/ks7010_sdio.h b/drivers/staging/ks7010/ks7010_sdio.h