== 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 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx