WARNING: please write a paragraph that describes the config symbol fully #22: FILE: drivers/rpmsg/Kconfig:25: +config RPMSG_HOSTFS WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #45: new file mode 100644 WARNING: Missing or malformed SPDX-License-Identifier tag in line 1 #50: FILE: drivers/rpmsg/rpmsg_hostfs.c:1: +/* SPDX-License-Identifier: GPL-2.0 */ CHECK: Prefer using the BIT macro #98: FILE: drivers/rpmsg/rpmsg_hostfs.c:49: +#define HOSTFS_S_IFCHR (1 << 11) CHECK: Prefer using the BIT macro #103: FILE: drivers/rpmsg/rpmsg_hostfs.c:54: +#define HOSTFS_S_IFLNK (1 << 15) WARNING: line over 80 characters #107: FILE: drivers/rpmsg/rpmsg_hostfs.c:58: +#define HOSTFS_O_RDONLY (1 << 0) /* Open for read access (only) */ CHECK: Prefer using the BIT macro #107: FILE: drivers/rpmsg/rpmsg_hostfs.c:58: +#define HOSTFS_O_RDONLY (1 << 0) /* Open for read access (only) */ WARNING: line over 80 characters #108: FILE: drivers/rpmsg/rpmsg_hostfs.c:59: +#define HOSTFS_O_WRONLY (1 << 1) /* Open for write access (only) */ CHECK: Prefer using the BIT macro #108: FILE: drivers/rpmsg/rpmsg_hostfs.c:59: +#define HOSTFS_O_WRONLY (1 << 1) /* Open for write access (only) */ WARNING: line over 80 characters #109: FILE: drivers/rpmsg/rpmsg_hostfs.c:60: +#define HOSTFS_O_CREAT (1 << 2) /* Create file/sem/mq object */ CHECK: Prefer using the BIT macro #109: FILE: drivers/rpmsg/rpmsg_hostfs.c:60: +#define HOSTFS_O_CREAT (1 << 2) /* Create file/sem/mq object */ WARNING: line over 80 characters #110: FILE: drivers/rpmsg/rpmsg_hostfs.c:61: +#define HOSTFS_O_EXCL (1 << 3) /* Name must not exist when opened */ CHECK: Prefer using the BIT macro #110: FILE: drivers/rpmsg/rpmsg_hostfs.c:61: +#define HOSTFS_O_EXCL (1 << 3) /* Name must not exist when opened */ WARNING: line over 80 characters #111: FILE: drivers/rpmsg/rpmsg_hostfs.c:62: +#define HOSTFS_O_APPEND (1 << 4) /* Keep contents, append to end */ CHECK: Prefer using the BIT macro #111: FILE: drivers/rpmsg/rpmsg_hostfs.c:62: +#define HOSTFS_O_APPEND (1 << 4) /* Keep contents, append to end */ CHECK: Prefer using the BIT macro #112: FILE: drivers/rpmsg/rpmsg_hostfs.c:63: +#define HOSTFS_O_TRUNC (1 << 5) /* Delete contents */ CHECK: Prefer using the BIT macro #113: FILE: drivers/rpmsg/rpmsg_hostfs.c:64: +#define HOSTFS_O_NONBLOCK (1 << 6) /* Don't wait for data */ WARNING: line over 80 characters #114: FILE: drivers/rpmsg/rpmsg_hostfs.c:65: +#define HOSTFS_O_SYNC (1 << 7) /* Synchronize output on write */ CHECK: Prefer using the BIT macro #114: FILE: drivers/rpmsg/rpmsg_hostfs.c:65: +#define HOSTFS_O_SYNC (1 << 7) /* Synchronize output on write */ WARNING: line over 80 characters #115: FILE: drivers/rpmsg/rpmsg_hostfs.c:66: +#define HOSTFS_O_BINARY (1 << 8) /* Open the file in binary mode. */ CHECK: Prefer using the BIT macro #115: FILE: drivers/rpmsg/rpmsg_hostfs.c:66: +#define HOSTFS_O_BINARY (1 << 8) /* Open the file in binary mode. */ CHECK: Prefer kernel type 'u32' over 'uint32_t' #120: FILE: drivers/rpmsg/rpmsg_hostfs.c:71: + uint32_t f_type; /* Type of filesystem */ WARNING: line over 80 characters #121: FILE: drivers/rpmsg/rpmsg_hostfs.c:72: + uint32_t f_namelen; /* Maximum length of filenames */ CHECK: Prefer kernel type 'u32' over 'uint32_t' #121: FILE: drivers/rpmsg/rpmsg_hostfs.c:72: + uint32_t f_namelen; /* Maximum length of filenames */ WARNING: line over 80 characters #122: FILE: drivers/rpmsg/rpmsg_hostfs.c:73: + uint32_t f_bsize; /* Optimal block size for transfers */ CHECK: Prefer kernel type 'u32' over 'uint32_t' #122: FILE: drivers/rpmsg/rpmsg_hostfs.c:73: + uint32_t f_bsize; /* Optimal block size for transfers */ WARNING: line over 80 characters #123: FILE: drivers/rpmsg/rpmsg_hostfs.c:74: + int32_t f_blocks; /* Total data blocks in the file system of this size */ CHECK: Prefer kernel type 's32' over 'int32_t' #123: FILE: drivers/rpmsg/rpmsg_hostfs.c:74: + int32_t f_blocks; /* Total data blocks in the file system of this size */ WARNING: line over 80 characters #124: FILE: drivers/rpmsg/rpmsg_hostfs.c:75: + int32_t f_bfree; /* Free blocks in the file system */ CHECK: Prefer kernel type 's32' over 'int32_t' #124: FILE: drivers/rpmsg/rpmsg_hostfs.c:75: + int32_t f_bfree; /* Free blocks in the file system */ WARNING: line over 80 characters #125: FILE: drivers/rpmsg/rpmsg_hostfs.c:76: + int32_t f_bavail; /* Free blocks avail to non-superuser */ CHECK: Prefer kernel type 's32' over 'int32_t' #125: FILE: drivers/rpmsg/rpmsg_hostfs.c:76: + int32_t f_bavail; /* Free blocks avail to non-superuser */ WARNING: line over 80 characters #126: FILE: drivers/rpmsg/rpmsg_hostfs.c:77: + int32_t f_files; /* Total file nodes in the file system */ CHECK: Prefer kernel type 's32' over 'int32_t' #126: FILE: drivers/rpmsg/rpmsg_hostfs.c:77: + int32_t f_files; /* Total file nodes in the file system */ WARNING: line over 80 characters #127: FILE: drivers/rpmsg/rpmsg_hostfs.c:78: + int32_t f_ffree; /* Free file nodes in the file system */ CHECK: Prefer kernel type 's32' over 'int32_t' #127: FILE: drivers/rpmsg/rpmsg_hostfs.c:78: + int32_t f_ffree; /* Free file nodes in the file system */ WARNING: 'atributes' may be misspelled - perhaps 'attributes'? #133: FILE: drivers/rpmsg/rpmsg_hostfs.c:84: + uint32_t st_mode; /* File type, atributes, and access mode bits */ WARNING: line over 80 characters #133: FILE: drivers/rpmsg/rpmsg_hostfs.c:84: + uint32_t st_mode; /* File type, atributes, and access mode bits */ CHECK: Prefer kernel type 'u32' over 'uint32_t' #133: FILE: drivers/rpmsg/rpmsg_hostfs.c:84: + uint32_t st_mode; /* File type, atributes, and access mode bits */ WARNING: line over 80 characters #134: FILE: drivers/rpmsg/rpmsg_hostfs.c:85: + int32_t st_size; /* Size of file/directory, in bytes */ CHECK: Prefer kernel type 's32' over 'int32_t' #134: FILE: drivers/rpmsg/rpmsg_hostfs.c:85: + int32_t st_size; /* Size of file/directory, in bytes */ WARNING: line over 80 characters #135: FILE: drivers/rpmsg/rpmsg_hostfs.c:86: + int16_t st_blksize; /* Blocksize used for filesystem I/O */ CHECK: Prefer kernel type 's16' over 'int16_t' #135: FILE: drivers/rpmsg/rpmsg_hostfs.c:86: + int16_t st_blksize; /* Blocksize used for filesystem I/O */ WARNING: line over 80 characters #136: FILE: drivers/rpmsg/rpmsg_hostfs.c:87: + uint32_t st_blocks; /* Number of blocks allocated */ CHECK: Prefer kernel type 'u32' over 'uint32_t' #136: FILE: drivers/rpmsg/rpmsg_hostfs.c:87: + uint32_t st_blocks; /* Number of blocks allocated */ WARNING: line over 80 characters #137: FILE: drivers/rpmsg/rpmsg_hostfs.c:88: + uint32_t st_atim; /* Time of last access */ CHECK: Prefer kernel type 'u32' over 'uint32_t' #137: FILE: drivers/rpmsg/rpmsg_hostfs.c:88: + uint32_t st_atim; /* Time of last access */ WARNING: line over 80 characters #138: FILE: drivers/rpmsg/rpmsg_hostfs.c:89: + uint32_t st_mtim; /* Time of last modification */ CHECK: Prefer kernel type 'u32' over 'uint32_t' #138: FILE: drivers/rpmsg/rpmsg_hostfs.c:89: + uint32_t st_mtim; /* Time of last modification */ WARNING: line over 80 characters #139: FILE: drivers/rpmsg/rpmsg_hostfs.c:90: + uint32_t st_ctim; /* Time of last status change */ CHECK: Prefer kernel type 'u32' over 'uint32_t' #139: FILE: drivers/rpmsg/rpmsg_hostfs.c:90: + uint32_t st_ctim; /* Time of last status change */ WARNING: line over 80 characters #142: FILE: drivers/rpmsg/rpmsg_hostfs.c:93: +/* These must exactly match the definition from REMOTE fs/hostfs/hostfs_rpmsg.h: */ CHECK: Prefer kernel type 'u32' over 'uint32_t' #145: FILE: drivers/rpmsg/rpmsg_hostfs.c:96: + uint32_t command; CHECK: Prefer kernel type 's32' over 'int32_t' #146: FILE: drivers/rpmsg/rpmsg_hostfs.c:97: + int32_t result; CHECK: Prefer kernel type 'u64' over 'uint64_t' #147: FILE: drivers/rpmsg/rpmsg_hostfs.c:98: + uint64_t cookie; CHECK: Prefer kernel type 's32' over 'int32_t' #152: FILE: drivers/rpmsg/rpmsg_hostfs.c:103: + int32_t flags; CHECK: Prefer kernel type 's32' over 'int32_t' #153: FILE: drivers/rpmsg/rpmsg_hostfs.c:104: + int32_t mode; CHECK: Prefer kernel type 's32' over 'int32_t' #159: FILE: drivers/rpmsg/rpmsg_hostfs.c:110: + int32_t fd; CHECK: Prefer kernel type 's32' over 'int32_t' #164: FILE: drivers/rpmsg/rpmsg_hostfs.c:115: + int32_t fd; CHECK: Prefer kernel type 'u32' over 'uint32_t' #165: FILE: drivers/rpmsg/rpmsg_hostfs.c:116: + uint32_t count; CHECK: Prefer kernel type 's32' over 'int32_t' #173: FILE: drivers/rpmsg/rpmsg_hostfs.c:124: + int32_t fd; CHECK: Prefer kernel type 's32' over 'int32_t' #174: FILE: drivers/rpmsg/rpmsg_hostfs.c:125: + int32_t whence; CHECK: Prefer kernel type 's32' over 'int32_t' #175: FILE: drivers/rpmsg/rpmsg_hostfs.c:126: + int32_t offset; CHECK: Prefer kernel type 's32' over 'int32_t' #180: FILE: drivers/rpmsg/rpmsg_hostfs.c:131: + int32_t fd; CHECK: Prefer kernel type 's32' over 'int32_t' #181: FILE: drivers/rpmsg/rpmsg_hostfs.c:132: + int32_t request; CHECK: Prefer kernel type 's32' over 'int32_t' #182: FILE: drivers/rpmsg/rpmsg_hostfs.c:133: + int32_t arg; CHECK: Prefer kernel type 's32' over 'int32_t' #190: FILE: drivers/rpmsg/rpmsg_hostfs.c:141: + int32_t fd; CHECK: Prefer kernel type 'u32' over 'uint32_t' #191: FILE: drivers/rpmsg/rpmsg_hostfs.c:142: + uint32_t reserved; CHECK: Prefer kernel type 's32' over 'int32_t' #197: FILE: drivers/rpmsg/rpmsg_hostfs.c:148: + int32_t fd; CHECK: Prefer kernel type 's32' over 'int32_t' #198: FILE: drivers/rpmsg/rpmsg_hostfs.c:149: + int32_t length; CHECK: Prefer kernel type 's32' over 'int32_t' #208: FILE: drivers/rpmsg/rpmsg_hostfs.c:159: + int32_t fd; CHECK: Prefer kernel type 'u32' over 'uint32_t' #209: FILE: drivers/rpmsg/rpmsg_hostfs.c:160: + uint32_t type; CHECK: Prefer kernel type 'u32' over 'uint32_t' #220: FILE: drivers/rpmsg/rpmsg_hostfs.c:171: + uint32_t reserved[16]; CHECK: Prefer kernel type 's32' over 'int32_t' #229: FILE: drivers/rpmsg/rpmsg_hostfs.c:180: + int32_t mode; CHECK: Prefer kernel type 'u32' over 'uint32_t' #230: FILE: drivers/rpmsg/rpmsg_hostfs.c:181: + uint32_t reserved; CHECK: Prefer kernel type 'u32' over 'uint32_t' #241: FILE: drivers/rpmsg/rpmsg_hostfs.c:192: + uint32_t reserved[16]; CHECK: Prefer kernel type 'u32' over 'uint32_t' #249: FILE: drivers/rpmsg/rpmsg_hostfs.c:200: + uint32_t namelen; CHECK: Prefer kernel type 'u32' over 'uint32_t' #250: FILE: drivers/rpmsg/rpmsg_hostfs.c:201: + uint32_t space; CHECK: Alignment should match open parenthesis #259: FILE: drivers/rpmsg/rpmsg_hostfs.c:210: +static int rpmsg_hostfs_open_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv_, u32 src) CHECK: braces {} should be used on all arms of this statement #282: FILE: drivers/rpmsg/rpmsg_hostfs.c:233: + if (IS_ERR(filp)) [...] + else { [...] CHECK: Unbalanced braces around else statement #284: FILE: drivers/rpmsg/rpmsg_hostfs.c:235: + else { CHECK: Alignment should match open parenthesis #286: FILE: drivers/rpmsg/rpmsg_hostfs.c:237: + id = idr_alloc(&priv->files, filp, + RPMSG_HOSTFS_ID_START, 0, GFP_KERNEL); CHECK: Alignment should match open parenthesis #297: FILE: drivers/rpmsg/rpmsg_hostfs.c:248: +static int rpmsg_hostfs_close_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv_, u32 src) CHECK: Alignment should match open parenthesis #315: FILE: drivers/rpmsg/rpmsg_hostfs.c:266: +static int rpmsg_hostfs_read_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv_, u32 src) CHECK: Prefer kernel type 'u32' over 'uint32_t' #321: FILE: drivers/rpmsg/rpmsg_hostfs.c:272: + uint32_t space; WARNING: line over 80 characters #352: FILE: drivers/rpmsg/rpmsg_hostfs.c:303: + return rpmsg_send_nocopy(rpdev->ept, rsp, (ret < 0 ? 0 : ret) + sizeof(*rsp)); CHECK: Alignment should match open parenthesis #356: FILE: drivers/rpmsg/rpmsg_hostfs.c:307: +static int rpmsg_hostfs_write_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv_, u32 src) CHECK: braces {} should be used on all arms of this statement #368: FILE: drivers/rpmsg/rpmsg_hostfs.c:319: + if (priv->need_copy) { [...] + } else [...] CHECK: Unbalanced braces around else statement #371: FILE: drivers/rpmsg/rpmsg_hostfs.c:322: + } else CHECK: Alignment should match open parenthesis #382: FILE: drivers/rpmsg/rpmsg_hostfs.c:333: +static int rpmsg_hostfs_lseek_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv_, u32 src) CHECK: Alignment should match open parenthesis #398: FILE: drivers/rpmsg/rpmsg_hostfs.c:349: +static int rpmsg_hostfs_ioctl_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv_, u32 src) CHECK: Alignment should match open parenthesis #414: FILE: drivers/rpmsg/rpmsg_hostfs.c:365: +static int rpmsg_hostfs_sync_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv_, u32 src) CHECK: Alignment should match open parenthesis #430: FILE: drivers/rpmsg/rpmsg_hostfs.c:381: +static int rpmsg_hostfs_dup_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv_, u32 src) CHECK: Alignment should match open parenthesis #442: FILE: drivers/rpmsg/rpmsg_hostfs.c:393: + id = idr_alloc(&priv->files, new_filp, + RPMSG_HOSTFS_ID_START, 0, GFP_KERNEL); WARNING: line over 80 characters #454: FILE: drivers/rpmsg/rpmsg_hostfs.c:405: +static void rpmsg_hostfs_stat_convert(struct kstat *hostbuf, struct hostfs_stat *buf) CHECK: Alignment should match open parenthesis #484: FILE: drivers/rpmsg/rpmsg_hostfs.c:435: +static int rpmsg_hostfs_fstat_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv_, u32 src) CHECK: Alignment should match open parenthesis #505: FILE: drivers/rpmsg/rpmsg_hostfs.c:456: +static int rpmsg_hostfs_ftruncate_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv_, u32 src) CHECK: Alignment should match open parenthesis #521: FILE: drivers/rpmsg/rpmsg_hostfs.c:472: +static int rpmsg_hostfs_opendir_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv_, u32 src) CHECK: spaces preferred around that '|' (ctx:VxV) #528: FILE: drivers/rpmsg/rpmsg_hostfs.c:479: + filp = filp_open(msg->pathname, O_RDONLY|O_DIRECTORY, 0); ^ CHECK: braces {} should be used on all arms of this statement #529: FILE: drivers/rpmsg/rpmsg_hostfs.c:480: + if (IS_ERR(filp)) [...] + else { [...] CHECK: Unbalanced braces around else statement #531: FILE: drivers/rpmsg/rpmsg_hostfs.c:482: + else { CHECK: Alignment should match open parenthesis #533: FILE: drivers/rpmsg/rpmsg_hostfs.c:484: + id = idr_alloc(&priv->files, filp, + RPMSG_HOSTFS_ID_START, 0, GFP_KERNEL); CHECK: Alignment should match open parenthesis #544: FILE: drivers/rpmsg/rpmsg_hostfs.c:495: +static int rpmsg_hostfs_filldir(struct dir_context *ctx, const char *name, + int namelen, loff_t offset, u64 ino, unsigned int d_type) CHECK: Alignment should match open parenthesis #577: FILE: drivers/rpmsg/rpmsg_hostfs.c:528: +static int rpmsg_hostfs_readdir_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv_, u32 src) WARNING: line over 80 characters #603: FILE: drivers/rpmsg/rpmsg_hostfs.c:554: + return rpmsg_send_nocopy(rpdev->ept, cb.rsp, sizeof(*cb.rsp) + cb.namelen); CHECK: Alignment should match open parenthesis #607: FILE: drivers/rpmsg/rpmsg_hostfs.c:558: +static int rpmsg_hostfs_rewinddir_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv_, u32 src) CHECK: Alignment should match open parenthesis #627: FILE: drivers/rpmsg/rpmsg_hostfs.c:578: +static int rpmsg_hostfs_closedir_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv_, u32 src) CHECK: Alignment should match open parenthesis #645: FILE: drivers/rpmsg/rpmsg_hostfs.c:596: +static int rpmsg_hostfs_statfs_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv, u32 src) CHECK: braces {} should be used on all arms of this statement #654: FILE: drivers/rpmsg/rpmsg_hostfs.c:605: + if (IS_ERR(filp)) [...] + else { [...] CHECK: Unbalanced braces around else statement #656: FILE: drivers/rpmsg/rpmsg_hostfs.c:607: + else { CHECK: Alignment should match open parenthesis #676: FILE: drivers/rpmsg/rpmsg_hostfs.c:627: +static int rpmsg_hostfs_unlink_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv, u32 src) CHECK: braces {} should be used on all arms of this statement #684: FILE: drivers/rpmsg/rpmsg_hostfs.c:635: + if (IS_ERR(dentry)) [...] + else { [...] CHECK: Unbalanced braces around else statement #686: FILE: drivers/rpmsg/rpmsg_hostfs.c:637: + else { CHECK: Alignment should match open parenthesis #702: FILE: drivers/rpmsg/rpmsg_hostfs.c:653: +static int rpmsg_hostfs_mkdir_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv, u32 src) WARNING: line over 80 characters #709: FILE: drivers/rpmsg/rpmsg_hostfs.c:660: + dentry = kern_path_create(AT_FDCWD, msg->pathname, &path, LOOKUP_DIRECTORY); CHECK: braces {} should be used on all arms of this statement #710: FILE: drivers/rpmsg/rpmsg_hostfs.c:661: + if (IS_ERR(dentry)) [...] + else { [...] CHECK: Unbalanced braces around else statement #712: FILE: drivers/rpmsg/rpmsg_hostfs.c:663: + else { CHECK: Alignment should match open parenthesis #722: FILE: drivers/rpmsg/rpmsg_hostfs.c:673: +static int rpmsg_hostfs_rmdir_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv, u32 src) CHECK: braces {} should be used on all arms of this statement #730: FILE: drivers/rpmsg/rpmsg_hostfs.c:681: + if (IS_ERR(dentry)) [...] + else { [...] CHECK: Unbalanced braces around else statement #732: FILE: drivers/rpmsg/rpmsg_hostfs.c:683: + else { CHECK: Alignment should match open parenthesis #748: FILE: drivers/rpmsg/rpmsg_hostfs.c:699: +static int rpmsg_hostfs_rename_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv, u32 src) CHECK: Alignment should match open parenthesis #776: FILE: drivers/rpmsg/rpmsg_hostfs.c:727: + ret = vfs_rename(oldpath.dentry->d_parent->d_inode, + oldpath.dentry, CHECK: Alignment should match open parenthesis #793: FILE: drivers/rpmsg/rpmsg_hostfs.c:744: +static int rpmsg_hostfs_stat_handler(struct rpmsg_device *rpdev, + void *data, int len, void *priv, u32 src) CHECK: braces {} should be used on all arms of this statement #801: FILE: drivers/rpmsg/rpmsg_hostfs.c:752: + if (IS_ERR(filp)) [...] + else { [...] CHECK: Unbalanced braces around else statement #803: FILE: drivers/rpmsg/rpmsg_hostfs.c:754: + else { CHECK: Alignment should match open parenthesis #840: FILE: drivers/rpmsg/rpmsg_hostfs.c:791: +static int rpmsg_hostfs_callback(struct rpmsg_device *rpdev, + void *data, int len, void *priv, u32 src) CHECK: Prefer kernel type 'u32' over 'uint32_t' #843: FILE: drivers/rpmsg/rpmsg_hostfs.c:794: + uint32_t command = header->command; WARNING: line over 80 characters #847: FILE: drivers/rpmsg/rpmsg_hostfs.c:798: + ret = rpmsg_hostfs_handler[command](rpdev, data, len, priv, src); WARNING: line over 80 characters #849: FILE: drivers/rpmsg/rpmsg_hostfs.c:800: + dev_err(&rpdev->dev, "command handle error %d\n", command); WARNING: line over 80 characters #868: FILE: drivers/rpmsg/rpmsg_hostfs.c:819: + priv->need_copy = of_property_read_bool(np, "memory-aligned-access"); CHECK: Alignment should match open parenthesis #909: FILE: drivers/rpmsg/rpmsg_hostfs.c:860: +module_driver(rpmsg_hostfs_driver, + register_rpmsg_driver, total: 0 errors, 33 warnings, 98 checks, 886 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. ./linux-sof-driver/0001-rpmsg-Add-fs-API-redirection-driver.patch has style problems, please review. NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.