On Fri, Feb 10, 2023 at 12:37:21PM +0200, Jani Nikula wrote:
> On Fri, 10 Feb 2023, Ville Syrjälä <ville.syrj...@linux.intel.com> wrote:
> > On Thu, Feb 09, 2023 at 05:40:14PM +0200, Jani Nikula wrote:
> >> On Thu, 09 Feb 2023, Ville Syrjala <ville.syrj...@linux.intel.com> 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.
Thanks.
--
Ville Syrjälä
Intel
