For patches 6-8: Reviewed-by: Lyude <[email protected]>
On Tue, 2016-09-06 at 21:52 -0300, Paulo Zanoni wrote: > Hi > > This series is the result of me comparing the code against BSpec. It > doesn't solve any particular problem I was seeing, but there's enough > changes that this code could potentially change the watermarks values > for most machines. Maybe this will fix somebody's underrun? Even if > it > doesn't, we should still have a code that matches the specification > unless there's a big justification. > > I still didn't finish reviewing everything and there's at least one > workaround that we seem to be missing, but let's try to get these > patches merged first. > > Thanks, > Paulo > > Cc: Lyude <[email protected]> > Cc: Matt Roper <[email protected]> > Cc: Mahesh Kumar <[email protected]> > > Paulo Zanoni (8): > drm/i915: SAGV is not SKL-only, so rename a few things > drm/i915: introduce intel_has_sagv() > drm/i915/kbl: KBL also needs to run the SAGV code > drm/i915/gen9: fix the WaWmMemoryReadLatency implementation > drm/i915/gen9: minimum scanlines for Y tile is not always 4 > drm/i915/gen9: fix plane_blocks_per_line on watermarks calculations > drm/i915/gen9: fix the watermark res_blocks value > drm/i915/gen9: implement missing case for SKL watermarks > calculation > > drivers/gpu/drm/i915/i915_drv.h | 10 +- > drivers/gpu/drm/i915/intel_display.c | 9 +- > drivers/gpu/drm/i915/intel_drv.h | 6 +- > drivers/gpu/drm/i915/intel_pm.c | 177 ++++++++++++++++++++----- > ---------- > 4 files changed, 111 insertions(+), 91 deletions(-) > -- Cheers, Lyude _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
