CHECK: Please use a blank line after function/struct/union/enum declarations
#135: FILE: drivers/fwctl/main.c:15:
+};
+static dev_t fwctl_dev;

CHECK: Please use a blank line after function/struct/union/enum declarations
#359: FILE: include/linux/fwctl.h:59:
+}
+static inline void fwctl_put(struct fwctl_device *fwctl)

CHECK: Please use a blank line after function/struct/union/enum declarations
#363: FILE: include/linux/fwctl.h:63:
+}
+DEFINE_FREE(fwctl, struct fwctl_device *, if (_T) fwctl_put(_T));

ERROR: trailing statements should be on next line
#363: FILE: include/linux/fwctl.h:63:
+DEFINE_FREE(fwctl, struct fwctl_device *, if (_T) fwctl_put(_T));

total: 1 errors, 0 warnings, 3 checks, 284 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 662380785e24 ("fwctl: Add basic structure for a class subsystem with a cdev") has style problems, please review.

NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE 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.