On 21 May 2014 01:17, Alex Deucher <alexdeucher at gmail.com> wrote: > + val = RREG32(HDMI_CONTROL + offset); > + val &= ~HDMI_DEEP_COLOR_ENABLE; > + val &= ~HDMI_DEEP_COLOR_DEPTH_MASK; > +
> + switch (bpc) { > (...) > + } > + > + WREG32(HDMI_CONTROL + offset, val); What about using helper: WREG32_P(HDMI_CONTROL + offset, val, ~(HDMI_DEEP_COLOR_ENABLE | HDMI_DEEP_COLOR_DEPTH_MASK));