mbox series

[0/3] drm/i915: intel_color_check() cleanup

Message ID 20240523182818.15382-1-ville.syrjala@linux.intel.com (mailing list archive)
Headers show
Series drm/i915: intel_color_check() cleanup | expand

Message

Ville Syrjala May 23, 2024, 6:28 p.m. UTC 
From: Ville Syrjälä <ville.syrjala@linux.intel.com>

Eliminate the crtc_state->state footgun from intel_color_check(),
and hide some mundane C8 plane details inside it.

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(-)

Comments

Jani Nikula May 24, 2024, 9:17 a.m. UTC | #1 
On Thu, 23 May 2024, Ville Syrjala <ville.syrjala@linux.intel.com> wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> 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@intel.com>


>
> 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(-)