Here are the remaining patches rebased on the current staging-testing.
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Most multi-line comments started on the first line, but the preferred
linux kernel style is to start multi-line comments on the second line.
Some comments became less readable after the change, so we changed them
to single-line comments.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph
This patch properly left aligns all member identifiers in every
struct defined in obd_class.h for better readability.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph Volkert
---
drivers/staging/lustre/lustre/include/obd_class.h | 44 +++
1 file changed, 22 insertions
This patch right aligns all backslashes in multi-line macros
in obd_class.h for better readability.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph Volkert
---
drivers/staging/lustre/lustre/include/obd_class.h | 166 +++---
1 file changed, 83 insertions(+), 83 deletions
Fixed four lines that went over the 80 character limit
to reduce checkpatch warnings.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph Volkert
---
drivers/staging/lustre/lustre/include/obd_class.h | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/drivers
This patch properly left aligns all member identifiers in every
struct defined in obd_class.h for better readability.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph Volkert
---
drivers/staging/lustre/lustre/include/obd_class.h | 44 +++
1 file changed, 22 insertions
imp everywhere.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph Volkert
---
drivers/staging/lustre/lustre/include/obd_class.h | 22 --
1 file changed, 12 insertions(+), 10 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/obd_class.h
b/drivers/staging
Some complex multi-line macros were not enclosed by a do-while(0),
so we fixed that.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph Volkert
---
drivers/staging/lustre/lustre/include/obd_class.h | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/drivers
Most multi-line comments started on the first line, but the preferred
linux kernel style is to start multi-line comments on the second line.
Some comments became less readable after the change, so we changed them
to single-line comments.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph
Checkpatch wants complex macros to be enclosed in parentheses, so we
put parentheses around these four macros.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph Volkert
---
drivers/staging/lustre/lustre/include/obd_class.h | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff
This macro was only used in four places to declare two variables.
It saved one line of code, but in our opinion hurt readability.
So we removed the macro, substituting every occurrence with the
declaration of the two variables (like the preprocessor would have done).
Signed-off-by: Fabian Huegel
We cleaned up a lot of checkpatch errors and warnings in obd_class.h,
but there are still some CHECKs and two warnings about flow control
inside macros left.
Changing those macros to inline functions would probably
be a good idea, unfortunatly it's not straightforward since they use
'#op' to print
Checkpatch wants complex macros to be enclosed in parantheses, so we
put parantheses around these four macros.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph Volkert
---
drivers/staging/lustre/lustre/include/obd_class.h | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff
Fixed four lines that went over the 80 character limit
to reduce checkpatch warnings.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph Volkert
---
drivers/staging/lustre/lustre/include/obd_class.h | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/drivers
imp everywhere.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph Volkert
---
drivers/staging/lustre/lustre/include/obd_class.h | 22 --
1 file changed, 12 insertions(+), 10 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/obd_class.h
b/drivers/staging
We cleaned up a lot of checkpatch errors and warnings in obd_class.h,
but there are still some CHECKs and two warnings about flow control
inside macros left.
Changing those macros to inline functions would probably
be a good idea, unfortunatly it's not straightforward since they use
'#op' to print
Most multi-line comments started on the first line, but the preferred
linux kernel style is to start multi-line comments on the second line.
Some comments became less readable after the change, so we changed them
to single-line comments.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph
Some complex multi-line macros were not enclosed by a do-while(0),
so we fixed that.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph Volkert
---
drivers/staging/lustre/lustre/include/obd_class.h | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/drivers
This macro was only used in four places to declare two variables.
It saved one line of code, but in our opinion hurt readability.
So we removed the macro, substituting every occurrence with the
declaration of the two variables (like the preprocessor would have done).
Signed-off-by: Fabian Huegel
This patch right aligns all backslashes in multi-line macros
in obd_class.h for better readability.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph Volkert
---
drivers/staging/lustre/lustre/include/obd_class.h | 166 +++---
1 file changed, 83 insertions(+), 83 deletions
This patch properly left aligns all member identifiers in every
struct defined in obd_class.h for better readability.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph Volkert
---
drivers/staging/lustre/lustre/include/obd_class.h | 44 +++
1 file changed, 22 insertions
Checkpatch wants complex macros to be enclosed in parentheses, so we
put parentheses around these four macros.
Signed-off-by: Fabian Huegel
Signed-off-by: Christoph Volkert
---
drivers/staging/lustre/lustre/include/obd_class.h | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff
22 matches
Mail list logo
