[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for include: linux: Regularise the use of FIELD_SIZEOF macro
Patchwork
patchwork at emeril.freedesktop.org
Thu Jun 13 00:31:53 UTC 2019
== Series Details ==
Series: include: linux: Regularise the use of FIELD_SIZEOF macro
URL : https://patchwork.freedesktop.org/series/61982/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
9c1c7908df88 include: linux: Regularise the use of FIELD_SIZEOF macro
-:116: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#116: FILE: fs/befs/linuxvfs.c:442:
+ FIELD_SIZEOF(struct befs_inode_info,
i_data.symlink),
-:155: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#155: FILE: fs/freevxfs/vxfs_super.c:333:
+ FIELD_SIZEOF(struct vxfs_inode_info,
vii_immed.vi_immed),
-:168: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#168: FILE: fs/orangefs/super.c:650:
+ FIELD_SIZEOF(struct orangefs_inode_s,
link_target),
-:181: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#181: FILE: fs/ufs/super.c:1473:
+ FIELD_SIZEOF(struct ufs_inode_info,
i_u1.i_symlink),
-:232: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 't' may be better as '(t)' to avoid precedence issues
#232: FILE: include/linux/stddef.h:29:
+#define FIELD_SIZEOF(t, f) (sizeof(((t *)0)->f))
-:232: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'f' may be better as '(f)' to avoid precedence issues
#232: FILE: include/linux/stddef.h:29:
+#define FIELD_SIZEOF(t, f) (sizeof(((t *)0)->f))
-:372: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 't' may be better as '(t)' to avoid precedence issues
#372: FILE: tools/testing/selftests/bpf/bpf_util.h:69:
+#define FIELD_SIZEOF(t, f) (sizeof(((t *)0)->f))
-:372: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'f' may be better as '(f)' to avoid precedence issues
#372: FILE: tools/testing/selftests/bpf/bpf_util.h:69:
+#define FIELD_SIZEOF(t, f) (sizeof(((t *)0)->f))
total: 0 errors, 0 warnings, 8 checks, 268 lines checked
More information about the Intel-gfx
mailing list