[PATCH] staging: lustre: fix sparse warning about different address spaces

2017-03-01 Thread Mario Bambagini
/staging//lustre/lustre/obdclass/obd_config.c:1055:74:got char *[assigned] sval Signed-off-by: Mario Bambagini <mario.bambag...@gmail.com> --- drivers/staging/lustre/lustre/obdclass/obd_config.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/lustre/

[PATCH v3 1/5] staging: lustre: fixed lines longer than 80 chars

2015-07-28 Thread Mario Bambagini
strings This patch fixes 2 lines longer than 80 chars The first one is a function whose argument has been moved to next line. The second one is a comment split on two lines Signed-off-by: Mario Bambagini mario.bambag...@gmail.com --- drivers/staging/lustre/lustre/llite/llite_capa.c | 6 -- 1 file

[PATCH v3 3/5] staging: lustre: fixed bad alignment

2015-07-28 Thread Mario Bambagini
This patch fixes 7 bad alignments. When a line is split on more than one line, the other lines must be aligned with paranthesis. Signed-off-by: Mario Bambagini mario.bambag...@gmail.com --- drivers/staging/lustre/lustre/llite/llite_capa.c | 14 +++--- 1 file changed, 7 insertions(+), 7

[PATCH v3 4/5] staging: lustre: modified comparisons against NULL

2015-07-28 Thread Mario Bambagini
The explicit comparisons against NULL has been modified to be shorter. Signed-off-by: Mario Bambagini mario.bambag...@gmail.com --- drivers/staging/lustre/lustre/llite/llite_capa.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite

[PATCH v3 5/5] staging: lustre: added a space between concatenated strings

2015-07-28 Thread Mario Bambagini
. Signed-off-by: Mario Bambagini mario.bambag...@gmail.com --- drivers/staging/lustre/lustre/llite/llite_capa.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/llite_capa.c b/drivers/staging/lustre/lustre/llite/llite_capa.c index 7768c6d8

Re: [PATCH 2/2] staging: lustre: fixed 80-char rule

2015-07-27 Thread Mario Bambagini
On Sun, Jul 26, 2015 at 10:14:11PM +0300, Dan Carpenter wrote: On Sat, Jul 25, 2015 at 06:22:52PM +0200, Mario Bambagini wrote: This patch fixes 2 lines longer than 80 chars The first one is a function whose argument has been moved to next line. The second one is a comment split on two

[PATCH v2 3/5] staging: lustre: fix bad alignment

2015-07-27 Thread Mario Bambagini
This patch fixes 8 bad alignments. When a line is split on more than one line, the other lines must be aligned with paranthesis. Signed-off-by: Mario Bambagini mario.bambag...@gmail.com --- drivers/staging/lustre/lustre/llite/llite_capa.c | 16 1 file changed, 8 insertions(+), 8

[PATCH v2 2/5] staging: lustre: fixed 80-char rule

2015-07-27 Thread Mario Bambagini
This patch fixes 2 lines longer than 80 chars The first one is a function whose argument has been moved to next line. The second one is a comment split on two lines Signed-off-by: Mario Bambagini mario.bambag...@gmail.com --- drivers/staging/lustre/lustre/llite/llite_capa.c | 6 -- 1 file

[PATCH v2 4/5] staging: lustre: added a space between concatenated strings

2015-07-27 Thread Mario Bambagini
A space has been inserted between two concatenated strings as required from checkpatch.pl These two updates do not lead to any problem as DFID is defined as a string in ./drivers/staging/lustre/lustre/include/lustre/lustre_user.h Signed-off-by: Mario Bambagini mario.bambag...@gmail.com

[PATCH v2 5/5] staging: lustre: modified comparisons against NULL

2015-07-27 Thread Mario Bambagini
The explicit comparisons against NULL has been modified to be shorter. The script checkpatch.pl does not return any other warning/error. MB Signed-off-by: Mario Bambagini mario.bambag...@gmail.com --- drivers/staging/lustre/lustre/llite/llite_capa.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH v2 1/5] staging: lustre: fixed comments without */ on a separate line

2015-07-27 Thread Mario Bambagini
NULL This patch fixes 4 comments without */ on a new line Signed-off-by: Mario Bambagini mario.bambag...@gmail.com --- drivers/staging/lustre/lustre/llite/llite_capa.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/llite_capa.c

[PATCH 2/2] staging: lustre: fixed 80-char rule

2015-07-25 Thread Mario Bambagini
This patch fixes 2 lines longer than 80 chars The first one is a function whose argument has been moved to next line. The second one is a comment split on two lines The script checkpatch does not complain anymore Signed-off-by: Mario Bambagini mario.bambag...@gmail.com --- drivers/staging

[PATCH 1/2] staging: lustre: fixed comments without */ on a separate line

2015-07-25 Thread Mario Bambagini
this set of patches fixes several code style problems: -patch 1: comments without */ on a separate line -patch 2: lines with more than 80 chars This patch fixes 4 comments without */ on a new line Signed-off-by: Mario Bambagini mario.bambag...@gmail.com --- drivers/staging/lustre/lustre/llite

Re: [PATCH] staging: fix to match 80-char rule

2015-07-20 Thread Mario Bambagini
On Sun, Jul 19, 2015 at 12:28:01PM -0700, Joe Perches wrote: On Sun, 2015-07-19 at 20:41 +0200, Mario Bambagini wrote: Fixed comment line to match the 80-char rules. The line contained 81 characters. The file is fully compliant with the code style, now. Signed-off-by: Mario Bambagini

[PATCH] staging: fix to match 80-char rule

2015-07-19 Thread Mario Bambagini
Fixed comment line to match the 80-char rules. The line contained 81 characters. The file is fully compliant with the code style, now. Signed-off-by: Mario Bambagini mario.bambag...@gmail.com --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_cs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH] defines modified to match the 80-char rule

2015-06-23 Thread Mario Bambagini
Defines have been written in more than one line to match the 80-character rule. This error has been fixed 6 times in this file. The file is fully compliant with respect to the coding rules now. Signed-off-by: Mario Bambagini mario.bambag...@gmail.com --- .../lustre/include/linux/libcfs

[PATCH] fixing code style in sm750_accel.c

2015-05-27 Thread Mario Bambagini
From: mbambagini mario.bambag...@gmail.com Signed-off-by: Mario Bambagini mario.bambag...@gmail.com --- drivers/staging/sm750fb/sm750_accel.c | 695 +- 1 file changed, 354 insertions(+), 341 deletions(-) diff --git a/drivers/staging/sm750fb/sm750_accel.c b