Hi Kieran, Building kernel with clang gives me: drivers/media/i2c/adv748x/adv748x-afe.c:452:12: warning: operator '?:' has lower precedence than '|'; '|' will be evaluated first [-Wbitwise-conditional-parentheses] ret = sdp_clrset(state, ADV748X_SDP_FRP, ADV748X_SDP_FRP_MASK, enable ? ctrl->val - 1 : 0); #define sdp_clrset(s, r, m, v) sdp_write(s, r, (sdp_read(s, r) & ~m) | v) This is on linux-next commit a722f75b2923b4fd44c17e7255e822ac48fe85f0 Date: Tue Oct 22 16:17:17 2019 +1100 Add linux-next specific files for 20191022 It seems sdp_clrset needs some more parens.