On Thu, 23 May 2024, Ville Syrjala <ville.syrjala@xxxxxxxxxxxxxxx> wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Eliminate the crtc_state->state footgun from intel_color_check(), > and hide some mundane C8 plane details inside it. On the series, Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx> > > Ville Syrjälä (3): > drm/i915: Plumb the entire atomic state into intel_color_check() > drm/i915: Hide the intel_crtc_needs_color_update() inside > intel_color_check() > drm/i915: Bury c8_planes_changed() in intel_color_check() > > drivers/gpu/drm/i915/display/intel_color.c | 125 ++++++++++++------- > drivers/gpu/drm/i915/display/intel_color.h | 4 +- > drivers/gpu/drm/i915/display/intel_display.c | 26 +--- > 3 files changed, 85 insertions(+), 70 deletions(-) -- Jani Nikula, Intel