[PATCH 0/4] staging/lustre: checkpatch cleanup
As Dan Carpenter (thank you!) noticed, my original patchset had an extra from field and lacked the signed-off-by. I'm not sure the first was fixed, but I would be really confused if it isn't. I checked that the signed-off-by was generated this time. As a reminder, this patchset aims to fix the following checkpatch warnings: occurences of trailing semicolons in macros uses of __attribute__(format(printf, ...)) where __printf(...) can be used uses of __attribute__(aligned(size)) where __aligned(size) can be used uses of __attribute__(packed) where __packed can be used Regards, Mario. Mario J. Rugiero (4): staging/lustre: checkpatch cleanup: macros should not use a trailing semicolon staging/lustre: checkpatch cleanup: __printf(string-index, first-to-check) is preferred over __attribute__((format(printf, string-index, first-to-check))) staging/lustre: checkpatch cleanup: __aligned(size) is preferred over __attribute__((aligned(size))) staging/lustre: checkpatch cleanup: __packed is preferred over __attribute__((packed)) .../lustre/include/linux/libcfs/libcfs_debug.h | 6 +-- .../include/linux/libcfs/libcfs_kernelcomm.h | 4 +- .../lustre/include/linux/libcfs/libcfs_private.h | 4 +- .../staging/lustre/include/linux/lnet/lib-types.h | 2 +- .../staging/lustre/lustre/include/lprocfs_status.h | 6 +-- drivers/staging/lustre/lustre/include/lu_object.h | 2 +- .../lustre/lustre/include/lustre/lustre_idl.h | 44 +++--- .../lustre/lustre/include/lustre/lustre_user.h | 14 +++ drivers/staging/lustre/lustre/include/lustre_dlm.h | 2 +- drivers/staging/lustre/lustre/include/lustre_net.h | 2 +- 10 files changed, 43 insertions(+), 43 deletions(-) -- 2.3.2 ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[PATCH 0/4] staging/lustre: checkpatch cleanup
This patches cleans the following checkpatch issues: trailing semicolons in macros __attribute__(format(printf,...)) instead of __printf(...) __attribute__(aligned(size)) instead of __aligned(size) __attribute__(packed) instead of __packed Mario J. Rugiero (4): staging/lustre: checkpatch cleanup: macros should not use a trailing semicolon staging/lustre: checkpatch cleanup: __printf(string-index, first-to-check) is preferred over __attribute__((format(printf, string-index, first-to-check))) staging/lustre: checkpatch cleanup: __aligned(size) is preferred over __attribute__((aligned(size))) staging/lustre: checkpatch cleanup: __packed is preferred over __attribute__((packed)) .../lustre/include/linux/libcfs/libcfs_debug.h | 6 +-- .../include/linux/libcfs/libcfs_kernelcomm.h | 4 +- .../lustre/include/linux/libcfs/libcfs_private.h | 4 +- .../staging/lustre/include/linux/lnet/lib-types.h | 2 +- .../staging/lustre/lustre/include/lprocfs_status.h | 6 +-- drivers/staging/lustre/lustre/include/lu_object.h | 2 +- .../lustre/lustre/include/lustre/lustre_idl.h | 44 +++--- .../lustre/lustre/include/lustre/lustre_user.h | 14 +++ drivers/staging/lustre/lustre/include/lustre_dlm.h | 2 +- drivers/staging/lustre/lustre/include/lustre_net.h | 2 +- 10 files changed, 43 insertions(+), 43 deletions(-) -- 2.3.2 ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[PATCH 0/4] staging/lustre: checkpatch cleanup
This patchset attempts to fix the following issues catched by checkpatch: trailing semicolons in macros use of __attribute__(format(printf,...)) where __printf(...) would suffice use of __attribute__(aligned(size)) where __aligned(size) would suffice use of __attribute__(packed) where __packed would suffice Mario J. Rugiero (4): staging/lustre: checkpatch cleanup: macros should not use a trailing semicolon staging/lustre: checkpatch cleanup: __printf(string-index, first-to-check) is preferred over __attribute__((format(printf, string-index, first-to-check))) staging/lustre: checkpatch cleanup: __aligned(size) is preferred over __attribute__((aligned(size))) staging/lustre: checkpatch cleanup: __packed is preferred over __attribute__((packed)) .../lustre/include/linux/libcfs/libcfs_debug.h | 6 +-- .../include/linux/libcfs/libcfs_kernelcomm.h | 4 +- .../lustre/include/linux/libcfs/libcfs_private.h | 4 +- .../staging/lustre/include/linux/lnet/lib-types.h | 2 +- .../staging/lustre/lustre/include/lprocfs_status.h | 6 +-- drivers/staging/lustre/lustre/include/lu_object.h | 2 +- .../lustre/lustre/include/lustre/lustre_idl.h | 44 +++--- .../lustre/lustre/include/lustre/lustre_user.h | 14 +++ drivers/staging/lustre/lustre/include/lustre_dlm.h | 2 +- drivers/staging/lustre/lustre/include/lustre_net.h | 2 +- 10 files changed, 43 insertions(+), 43 deletions(-) -- 2.3.2 ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[PATCH 0/4] staging/lustre: checkpatch cleanup
From: Mario J. Rugiero mrugi...@gmail.com Hi, as listed in the todo for this driver is fixing checkpatch warnings, I wrote this patchset which fixes the following issues on all of the driver files, one patch per issue: trailing semicolons in macros uses of __attribute__((format(printf,...)) instead of __printf(...) uses of __attribute__(aligned(size)) insted of __aligned(size) uses of __attribute__((packed)) instead of __packed Each patch should be pretty much self-contained. As I'm aware that my communication skills aren't the best, aside from the obvious criticism of patches, constructive criticism on how to communicate better with the list is accepted as well. If I broke any netiquette (I did my best, but something might have slipped), I'm also open to hearing. Thanks a lot in advance, Mario. Mario J. Rugiero (4): staging/lustre: checkpatch cleanup: macros should not use a trailing semicolon staging/lustre: checkpatch cleanup: __printf(string-index, first-to-check) is preferred over __attribute__((format(printf, string-index, first-to-check))) staging/lustre: checkpatch cleanup: __aligned(size) is preferred over __attribute__((aligned(size))) staging/lustre: checkpatch cleanup: __packed is preferred over __attribute__((packed)) .../lustre/include/linux/libcfs/libcfs_debug.h | 6 +-- .../include/linux/libcfs/libcfs_kernelcomm.h | 4 +- .../lustre/include/linux/libcfs/libcfs_private.h | 4 +- .../staging/lustre/include/linux/lnet/lib-types.h | 2 +- .../staging/lustre/lustre/include/lprocfs_status.h | 6 +-- drivers/staging/lustre/lustre/include/lu_object.h | 2 +- .../lustre/lustre/include/lustre/lustre_idl.h | 44 +++--- .../lustre/lustre/include/lustre/lustre_user.h | 14 +++ drivers/staging/lustre/lustre/include/lustre_dlm.h | 2 +- drivers/staging/lustre/lustre/include/lustre_net.h | 2 +- 10 files changed, 43 insertions(+), 43 deletions(-) -- 2.3.2 ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
