> -----Original Message-----
> From: Intel-gfx <[email protected]> On Behalf Of Ville 
> Syrjala
> Sent: Tuesday, 23 September 2025 20.19
> To: [email protected]
> Cc: [email protected]
> Subject: [PATCH 00/20] drm/i915/cdclk: Decouple CDCLK from state->modeset
> 
> From: Ville Syrjälä <[email protected]>
> 
> The state->modeset dependency on the cdclk stuff is annoying and confusing. 
> The cdclk code also interacts in weird ways with
> several other parts of the driver. Decouple cdclk from state->modeset and 
> attempt to make the interactions between different
> parts less confusing.

For the whole series

Reviewed-by: Mika Kahola <[email protected]>

> 
> Ville Syrjälä (20):
>   drm/i915: Introduce intel_crtc_enable_changed() and
>     intel_any_crtc_enable_changed()
>   drm/i915: Introduce intel_crtc_active_changed() and
>     intel_any_crtc_active_changed()
>   drm/i915/bw: Skip the bw_state->active_pipes update if no pipe is
>     changing its active state
>   drm/1915/bw: Drop redundant display version checks
>   drm/i915/cdclk: Extract glk_cdclk_audio_wa_needed()
>   drm/i915/cdclk: Extract dg2_power_well_count()
>   drm/i915/cdclk: Introduce intel_cdclk_modeset_checks()
>   drm/i915/cdclk: Handle the force_min_cdclk state locking in
>     intel_cdclk_atomic_check()
>   drm/i915/cdclk: Extract intel_cdclk_update_bw_min_cdclk()
>   drm/i915/cdclk: Extract intel_cdclk_update_crtc_min_cdclk()
>   drm/i915/cdclk: Rework bw_min_cdclk handling
>   drm/i915/cdclk: Do intel_cdclk_update_crtc_min_cdclk() per-pipe
>   drm/i915/cdclk: Relocate intel_plane_calc_min_cdclk() calls
>   drm/i915/cdclk: Rework crtc min_cdclk handling
>   drm/i915/cdclk: Move intel_bw_crtc_min_cdclk() handling into
>     intel_crtc_compute_min_cdclk()
>   drm/i915/cdclk: Decuple cdclk from state->modeset
>   drm/i915: Introduce intel_calc_enabled_pipes()
>   drm/i915/cdclk: Use enabled_pipes instead of active_pipes for the glk
>     audio w/a
>   drm/i915/cdclk: Hide intel_modeset_calc_cdclk()
>   drm/i915/cdclk: Move intel_cdclk_atomic_check()
> 
>  drivers/gpu/drm/i915/display/intel_bw.c      |  85 ++---
>  drivers/gpu/drm/i915/display/intel_bw.h      |   3 +-
>  drivers/gpu/drm/i915/display/intel_cdclk.c   | 312 +++++++++++++------
>  drivers/gpu/drm/i915/display/intel_cdclk.h   |   7 +-
>  drivers/gpu/drm/i915/display/intel_crtc.c    |  42 +++
>  drivers/gpu/drm/i915/display/intel_crtc.h    |   7 +
>  drivers/gpu/drm/i915/display/intel_display.c |  47 +--
>  drivers/gpu/drm/i915/display/intel_display.h |   2 +
>  drivers/gpu/drm/i915/display/intel_plane.c   |  52 +---
>  drivers/gpu/drm/i915/display/intel_plane.h   |   3 -
>  10 files changed, 335 insertions(+), 225 deletions(-)
> 
> --
> 2.49.1

Reply via email to