ERROR: trailing statements should be on next line #45: FILE: drivers/fwctl/main.c:24: +DEFINE_FREE(kvfree_errptr, void *, if (!IS_ERR_OR_NULL(_T)) kvfree(_T)); CHECK: Lines should not end with a '(' #73: FILE: drivers/fwctl/main.c:102: + dev_warn( CHECK: Lines should not end with a '(' #92: FILE: drivers/fwctl/main.c:121: + void *outbuf __free(kvfree_errptr) = fwctl->ops->fw_rpc( WARNING: line length of 91 exceeds 80 columns #176: FILE: include/uapi/fwctl/fwctl.h:91: + * @FWCTL_RPC_DEBUG_WRITE: Writable access to lockdown compatible debug information CHECK: Please use a blank line after function/struct/union/enum declarations #219: FILE: include/uapi/fwctl/fwctl.h:134: +}; +#define FWCTL_RPC _IO(FWCTL_TYPE, FWCTL_CMD_RPC) total: 1 errors, 1 warnings, 3 checks, 180 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 5ed44aee42a5 ("fwctl: FWCTL_RPC to execute a Remote Procedure Call to device firmware") 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.