From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> While pokingaround the watermarks/etc. I noticed our SAGV code has a bunch of bugs. Let's try to fix it. Pushed a few patches from v1 already. And based on the discussion with Stan I added a few extra refactoring patches to the end in the hopes of making the logic less confusing. I still prefer the oneliner for the immediate bugfix in patch 2 so that we don't have to backport the refactoring as well. Cc: Stanislav Lisovskiy <stanislav.lisovskiy@xxxxxxxxx> Ville Syrjälä (6): drm/i915: Correctly populate use_sagv_wm for all pipes drm/i915: Fix bw atomic check when switching between SAGV vs. no SAGV drm/i915: Split pre-icl vs. icl+ SAGV hooks apart drm/i915: Pimp icl+ sagv pre/post update drm/i915: Extract icl_qgv_points_mask() drm/i915: Extract intel_bw_check_data_rate() drivers/gpu/drm/i915/display/intel_bw.c | 71 +++++++--- drivers/gpu/drm/i915/intel_pm.c | 177 ++++++++++++++---------- 2 files changed, 154 insertions(+), 94 deletions(-) -- 2.34.1