CHECK: Macro argument 'val8' may be better as '(val8)' to avoid precedence issues #200: FILE: include/linux/mfd/idt8a340_reg.h:861: +#define SET_U16_MSB(orig, val8) (orig = (0x00ff & (orig)) | (val8 << 8)) total: 0 errors, 0 warnings, 1 checks, 194 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. Commit 4ebbfb2338fe ("ptp: clockmatrix: move register and firmware related definition to idt8a340_reg.h") has style problems, please review. NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. total: 0 errors, 0 warnings, 1 checks, 194 lines checked