ERROR: space required after that ',' (ctx:VxV) #49: FILE: fs/configfs/dir.c:405: + __simple_rmdir(d_inode(parent),d); ^ WARNING: Prefer using '"%s...", __func__' to using 'remove_dir', this function's name, in a string #52: FILE: fs/configfs/dir.c:408: + pr_warn("remove_dir (%pd): attributes remain", d); CHECK: extern prototypes should be avoided in .h files #137: FILE: include/linux/fs.h:3624: +extern void __simple_unlink(struct inode *, struct dentry *); WARNING: function definition argument 'struct inode *' should also have an identifier name #137: FILE: include/linux/fs.h:3624: +extern void __simple_unlink(struct inode *, struct dentry *); WARNING: function definition argument 'struct dentry *' should also have an identifier name #137: FILE: include/linux/fs.h:3624: +extern void __simple_unlink(struct inode *, struct dentry *); CHECK: extern prototypes should be avoided in .h files #138: FILE: include/linux/fs.h:3625: +extern void __simple_rmdir(struct inode *, struct dentry *); WARNING: function definition argument 'struct inode *' should also have an identifier name #138: FILE: include/linux/fs.h:3625: +extern void __simple_rmdir(struct inode *, struct dentry *); WARNING: function definition argument 'struct dentry *' should also have an identifier name #138: FILE: include/linux/fs.h:3625: +extern void __simple_rmdir(struct inode *, struct dentry *); total: 1 errors, 5 warnings, 2 checks, 98 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. Commit 75bc7edd9590 ("d_make_discardable(): warn if given a non-persistent dentry") has style problems, please review. NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE FILE_PATH_CHANGES GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.