The problem is that the bitwise OR operation has higher precedence than the ternary expression. The existing code will either set HPO_I, VPO_I, or "mode->clock >> 16" but not a combination of the three which is what we want. Fixes: e7f12054a1b9 ("drm/bridge: chrontel-ch7033: Add a new driver") Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> ---