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 <fabian_hue...@web.de>
This patch properly left aligns all member identifiers in every
struct defined in obd_class.h for better readability.
Signed-off-by: Fabian Huegel <fabian_hue...@web.de>
Signed-off-by: Christoph Volkert <li...@christoph-volkert.de>
---
drivers/staging/lustre/lustre/include/obd_
This patch right aligns all backslashes in multi-line macros
in obd_class.h for better readability.
Signed-off-by: Fabian Huegel <fabian_hue...@web.de>
Signed-off-by: Christoph Volkert <li...@christoph-volkert.de>
---
drivers/staging/lustre/lustre/include/obd_c
Fixed four lines that went over the 80 character limit
to reduce checkpatch warnings.
Signed-off-by: Fabian Huegel <fabian_hue...@web.de>
Signed-off-by: Christoph Volkert <li...@christoph-volkert.de>
---
drivers/staging/lustre/lustre/include/obd_class.h | 14 ++
1 file
This patch properly left aligns all member identifiers in every
struct defined in obd_class.h for better readability.
Signed-off-by: Fabian Huegel <fabian_hue...@web.de>
Signed-off-by: Christoph Volkert <li...@christoph-volkert.de>
---
drivers/staging/lustre/lustre/include/obd_
it imp everywhere.
Signed-off-by: Fabian Huegel <fabian_hue...@web.de>
Signed-off-by: Christoph Volkert <li...@christoph-volkert.de>
---
drivers/staging/lustre/lustre/include/obd_class.h | 22 --
1 file changed, 12 insertions(+), 10 deletions(-)
diff --git a/drivers/st
Some complex multi-line macros were not enclosed by a do-while(0),
so we fixed that.
Signed-off-by: Fabian Huegel <fabian_hue...@web.de>
Signed-off-by: Christoph Volkert <li...@christoph-volkert.de>
---
drivers/staging/lustre/lustre/include/obd_class.h | 16
1 file
Checkpatch wants complex macros to be enclosed in parentheses, so we
put parentheses around these four macros.
Signed-off-by: Fabian Huegel <fabian_hue...@web.de>
Signed-off-by: Christoph Volkert <li...@christoph-volkert.de>
---
drivers/staging/lustre/lustre/include/obd_class.h | 8
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
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 <fabian_hue...@web.de>
Checkpatch wants complex macros to be enclosed in parantheses, so we
put parantheses around these four macros.
Signed-off-by: Fabian Huegel <fabian_hue...@web.de>
Signed-off-by: Christoph Volkert <li...@christoph-volkert.de>
---
drivers/staging/lustre/lustre/include/obd_class.h | 8
it imp everywhere.
Signed-off-by: Fabian Huegel <fabian_hue...@web.de>
Signed-off-by: Christoph Volkert <li...@christoph-volkert.de>
---
drivers/staging/lustre/lustre/include/obd_class.h | 22 --
1 file changed, 12 insertions(+), 10 deletions(-)
diff --git a/drivers/st
Fixed four lines that went over the 80 character limit
to reduce checkpatch warnings.
Signed-off-by: Fabian Huegel <fabian_hue...@web.de>
Signed-off-by: Christoph Volkert <li...@christoph-volkert.de>
---
drivers/staging/lustre/lustre/include/obd_class.h | 14 ++
1 file
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
Some complex multi-line macros were not enclosed by a do-while(0),
so we fixed that.
Signed-off-by: Fabian Huegel <fabian_hue...@web.de>
Signed-off-by: Christoph Volkert <li...@christoph-volkert.de>
---
drivers/staging/lustre/lustre/include/obd_class.h | 16
1 file
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 <fabian_hue...@web.de>
Signed-off-by: Christoph Volkert <li...@christoph-volkert.de>
---
drivers/staging/lustre/lustre/include/obd_c
This patch properly left aligns all member identifiers in every
struct defined in obd_class.h for better readability.
Signed-off-by: Fabian Huegel <fabian_hue...@web.de>
Signed-off-by: Christoph Volkert <li...@christoph-volkert.de>
---
drivers/staging/lustre/lustre/include/obd_
Checkpatch wants complex macros to be enclosed in parentheses, so we
put parentheses around these four macros.
Signed-off-by: Fabian Huegel <fabian_hue...@web.de>
Signed-off-by: Christoph Volkert <li...@christoph-volkert.de>
---
drivers/staging/lustre/lustre/include/obd_class.h | 8
21 matches
Mail list logo