[PATCH RESEND 0/8] configfs checkpatch cleanup

From: Andrzej Pietrasiewicz
Date: Thu Jan 03 2013 - 06:42:47 EST


Dear All,

I noticed that configfs implementation has many style problems:

(checkpatch output edited for clarity)

total: 7 errors, 11 warnings, 260 lines checked
include/linux/configfs.h has style problems, please review.

total: 22 errors, 7 warnings, 160 lines checked
fs/configfs/configfs_internal.h has style problems, please review.

total: 53 errors, 20 warnings, 1756 lines checked
fs/configfs/dir.c has style problems, please review.

total: 45 errors, 16 warnings, 344 lines checked
fs/configfs/file.c has style problems, please review.

total: 19 errors, 7 warnings, 296 lines checked
fs/configfs/inode.c has style problems, please review.

total: 30 errors, 8 warnings, 216 lines checked
fs/configfs/item.c has style problems, please review.

total: 3 errors, 3 warnings, 183 lines checked
fs/configfs/mount.c has style problems, please review.

total: 14 errors, 3 warnings, 314 lines checked
fs/configfs/symlink.c has style problems, please review.

TOTAL: 193 errors, 75 warnings

I had some time at the end of the last year so I made this patch series.

After applying it:

total: 0 errors, 0 warnings, 269 lines checked
include/linux/configfs.h has no obvious style problems and is ready for submission.

total: 0 errors, 0 warnings, 166 lines checked
fs/configfs/configfs_internal.h has no obvious style problems and is ready for submission.

total: 0 errors, 0 warnings, 1766 lines checked
fs/configfs/dir.c has no obvious style problems and is ready for submission.

total: 0 errors, 0 warnings, 354 lines checked
fs/configfs/file.c has no obvious style problems and is ready for submission.

total: 0 errors, 0 warnings, 300 lines checked
fs/configfs/inode.c has no obvious style problems and is ready for submission.

total: 0 errors, 0 warnings, 214 lines checked
fs/configfs/item.c has no obvious style problems and is ready for submission.

total: 0 errors, 0 warnings, 183 lines checked
fs/configfs/mount.c has no obvious style problems and is ready for submission.

total: 0 errors, 0 warnings, 315 lines checked
fs/configfs/symlink.c has no obvious style problems and is ready for submission.

Andrzej Pietrasiewicz (8):
fs/configfs: include/linux/configfs.h checkpatch cleanup
fs/configfs: configfs_internal.h checkpatch cleanup
fs/configfs: dir.c checkpatch cleanup
fs/configfs: file.c checkpatch cleanup
fs/configfs: inode.c checkpatch cleanup
fs/configfs: item.c checkpatch cleanup
fs/configfs: mount.c checkpatch cleanup
fs/configfs: symlink.c checkpatch cleanup

fs/configfs/configfs_internal.h | 54 ++++----
fs/configfs/dir.c | 282 ++++++++++++++++++++-------------------
fs/configfs/file.c | 112 +++++++++-------
fs/configfs/inode.c | 46 ++++---
fs/configfs/item.c | 60 ++++-----
fs/configfs/mount.c | 12 +-
fs/configfs/symlink.c | 29 ++--
include/linux/configfs.h | 37 +++--
8 files changed, 335 insertions(+), 297 deletions(-)

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/