ERROR: trailing statements should be on next line #32: FILE: drivers/fwctl/main.c:20: +DEFINE_FREE(kfree_errptr, void *, if (!IS_ERR_OR_NULL(_T)) kfree(_T)); WARNING: line length of 81 exceeds 80 columns #100: FILE: drivers/fwctl/main.c:100: + IOCTL_OP(FWCTL_INFO, fwctl_cmd_info, struct fwctl_info, out_device_data), CHECK: Please use a blank line after function/struct/union/enum declarations #178: FILE: include/uapi/fwctl/fwctl.h:68: +}; +#define FWCTL_INFO _IO(FWCTL_TYPE, FWCTL_CMD_INFO) total: 1 errors, 1 warnings, 1 checks, 139 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 3333ca9c705f ("fwctl: FWCTL_INFO to return basic information about the device") 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.