[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] include: linux: Regularise the use of FIELD_SIZEOF macro
Patchwork
patchwork at emeril.freedesktop.org
Mon Apr 15 17:02:31 UTC 2019
== Series Details ==
Series: series starting with [1/2] include: linux: Regularise the use of FIELD_SIZEOF macro
URL : https://patchwork.freedesktop.org/series/59514/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
0595fc32fba7 include: linux: Regularise the use of FIELD_SIZEOF macro
-:96: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#96: FILE: fs/befs/linuxvfs.c:447:
+ FIELD_SIZEOF(struct befs_inode_info,
i_data.symlink),
-:135: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#135: FILE: fs/freevxfs/vxfs_super.c:340:
+ FIELD_SIZEOF(struct vxfs_inode_info,
vii_immed.vi_immed),
-:148: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#148: FILE: fs/orangefs/super.c:627:
+ FIELD_SIZEOF(struct orangefs_inode_s,
link_target),
-:161: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#161: FILE: fs/ufs/super.c:1478:
+ FIELD_SIZEOF(struct ufs_inode_info,
i_u1.i_symlink),
-:212: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 't' may be better as '(t)' to avoid precedence issues
#212: FILE: include/linux/stddef.h:29:
+#define FIELD_SIZEOF(t, f) (sizeof(((t *)0)->f))
-:212: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'f' may be better as '(f)' to avoid precedence issues
#212: FILE: include/linux/stddef.h:29:
+#define FIELD_SIZEOF(t, f) (sizeof(((t *)0)->f))
-:339: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 't' may be better as '(t)' to avoid precedence issues
#339: FILE: tools/testing/selftests/bpf/bpf_util.h:68:
+#define FIELD_SIZEOF(t, f) (sizeof(((t *)0)->f))
-:339: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'f' may be better as '(f)' to avoid precedence issues
#339: FILE: tools/testing/selftests/bpf/bpf_util.h:68:
+#define FIELD_SIZEOF(t, f) (sizeof(((t *)0)->f))
total: 0 errors, 0 warnings, 8 checks, 236 lines checked
b774e8d9acd2 include: linux: Remove unused macros and their defination
More information about the Intel-gfx
mailing list