On Fri, 10 Feb 2023, Ville Syrjälä <[email protected]> wrote: > On Thu, Feb 09, 2023 at 05:40:14PM +0200, Jani Nikula wrote: >> On Thu, 09 Feb 2023, Ville Syrjala <[email protected]> wrote: >> > @@ -3229,7 +3216,7 @@ static void ilk_wm_merge(struct drm_i915_private >> > *dev_priv, >> > merged->fbc_wm_enabled = DISPLAY_VER(dev_priv) >= 6; >> > >> > /* merge each WM1+ level */ >> > - for (level = 1; level <= max_level; level++) { >> > + for (level = 1; level < num_levels - 1; level++) { >> >> That's off by one, right? > > Whoops. Good eyes.
Thanks for v2, I went ahead and pushed these to drm-intel-next so I can rebase my wm stuff on top. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center
