CHECK: Macro argument '_field' may be better as '(_field)' to avoid precedence issues #677: FILE: drivers/dpll/zl3073x/out.c:109: +#define ZL3073X_OUT_SYNC_ONE(_dev, _dst, _src, _type, _field, _reg) \ + ((_dst)->_field != (_src)->_field ? \ + zl3073x_write_##_type(_dev, _reg, (_src)->_field) : 0) total: 0 errors, 0 warnings, 1 checks, 689 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 fff582e39b92 ("dpll: zl3073x: Cache all output properties in zl3073x_out") has style problems, please review. NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE FILE_PATH_CHANGES 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, 689 lines checked