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 devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel