On Fri, 2018-12-21 at 19:14 +0200, Ville Syrjala wrote: > From: Ville Syrjälä <[email protected]> > > The spec used to say "8bpp" which someone took to mean 8 bytes per > pixel when in fact it was supposed to be 8 bits per pixel. The > spec has been updated to make it more clear now. Fix the code > to match. > > Signed-off-by: Ville Syrjälä <[email protected]> > --- > drivers/gpu/drm/i915/intel_pm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_pm.c > b/drivers/gpu/drm/i915/intel_pm.c > index 0aac7e7b660f..55a1c577f060 100644 > --- a/drivers/gpu/drm/i915/intel_pm.c > +++ b/drivers/gpu/drm/i915/intel_pm.c > @@ -4618,7 +4618,7 @@ skl_compute_plane_wm_params(const struct > intel_crtc_state *cstate, > intel_p > state); > > if (INTEL_GEN(dev_priv) >= 11 && > - fb->modifier == I915_FORMAT_MOD_Yf_TILED && wp->cpp == > 8) > + fb->modifier == I915_FORMAT_MOD_Yf_TILED && wp->cpp == > 1) > wp->dbuf_block_size = 256; > else > wp->dbuf_block_size = 512;
Interesting how this small misunderstanding could affect our bugs previously. Reviewed-by: Stanislav Lisovskiy <[email protected]> -- Best Regards, Lisovskiy Stanislav _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
