/staging//lustre/lustre/obdclass/obd_config.c:1055:74:got char
*[assigned] sval
Signed-off-by: Mario Bambagini
---
drivers/staging/lustre/lustre/obdclass/obd_config.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/lustre/lustre/obdclass/obd_config.c
b/drivers
.
Signed-off-by: Mario Bambagini
---
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..24590ae 100644
--- a
This patch fixes 4 comments without */ on a new line
Signed-off-by: Mario Bambagini
---
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
b/drivers/staging/lustre
The explicit comparisons against NULL has been modified to be shorter.
Signed-off-by: Mario Bambagini
---
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
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
---
drivers/staging/lustre/lustre/llite/llite_capa.c | 6 --
1 file changed, 4 insertions
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
---
drivers/staging/lustre/lustre/llite/llite_capa.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a
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
---
drivers/staging/lustre/lustre/llite/llite_capa.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
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
---
drivers/staging/lustre
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
---
drivers/staging/lustre/lustre/llite/llite_capa.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git
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
---
drivers/staging/lustre/lustre/llite/llite_capa.c | 6 --
1 file changed, 4 insertions
NULL
This patch fixes 4 comments without */ on a new line
Signed-off-by: Mario Bambagini
---
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
b/drivers/staging/lustre
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 se
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
---
drivers/staging/lustre/lustre/llite
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
---
drivers/staging/lustre/lustre/llite/llite_capa.c | 12
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.
> >
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
---
drivers/staging/ft1000/ft1000-pcmcia/ft1000_cs.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers
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
---
.../lustre/include/linux/libcfs/libcfs_debug.h | 20
From: mbambagini
Signed-off-by: Mario Bambagini
---
drivers/staging/sm750fb/sm750_accel.c | 695 +-
1 file changed, 354 insertions(+), 341 deletions(-)
diff --git a/drivers/staging/sm750fb/sm750_accel.c
b/drivers/staging/sm750fb/sm750_accel.c
index 6a04ef8
18 matches
Mail list logo