[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