<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jan 15, 2016 at 10:32 AM, Kumar, Shobhit <span dir="ltr"><<a href="mailto:shobhit.kumar@linux.intel.com" target="_blank">shobhit.kumar@linux.intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 01/15/2016 07:18 AM, Matt Roper wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Thu, Jan 14, 2016 at 05:32:41PM +0530, Shobhit Kumar wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
This series add a set of updates to the WM calculation and also enables<br>
arbitrated display bandwidth based WA. Some of these patches do overlap<br>
with Matts work but we wanted to send them out as we have them in our<br>
internal testing for early review. Most likley some of them can be<br>
superceded by patches from Matt, or can be re-used if deemed necessary.<br>
</blockquote>
<br>
I've left some feedback on your patches, but it was all pretty minor;<br>
they look pretty good overall.  So I think we should polish up this set<br>
and merge it; I've got more SKL WM work coming (more atomic-y watermark<br>
calculation and programming) but I'll hold off on that until your series<br>
lands so that I don't cause too much thrash.<br>
</blockquote>
<br></span>
Thanks Matt for the quick review. I will get the comments addressed and final patches uploaded quickly.<br></blockquote><div><br></div><div>We have the updated patches ready. Just completing some testing. Should be posting sometime this week.<br><br></div><div>Regards<br></div><div>Shobhit<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Regards<span class="HOEnZb"><font color="#888888"><br>
Shobhit</font></span><span class="im HOEnZb"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks.<br>
<br>
<br>
Matt<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Especially "drm/i915/skl+: Use fb size for relative data rate calculation"<br>
this already addresses some of Ville's comment on similar patch from Matt.<br>
<br>
Regards<br>
Shobhit<br>
<br>
Kumar, Mahesh (6):<br>
   drm/i915/skl+: Use proper bytes_per_pixel during WM calculation<br>
   drm/i915/skl+: Use fb size for relative data rate calculation<br>
   drm/i915/skl+: calculate ddb minimum allocation<br>
   drm/i915/skl+: calculate plane pixel rate.<br>
   drm/i915/skl+: Use scaling amount for plane data rate calculation<br>
   drm/i915/skl: WA for watermark calculation based on Arbitrated Display<br>
     BW<br>
<br>
Shobhit Kumar (1):<br>
   drm/i915: Add support to parse DMI table and get platform memory info<br>
<br>
  drivers/gpu/drm/i915/i915_dma.c  |  19 +++<br>
  drivers/gpu/drm/i915/i915_drv.h  |  15 ++<br>
  drivers/gpu/drm/i915/intel_drv.h |   2 +<br>
  drivers/gpu/drm/i915/intel_pm.c  | 294 +++++++++++++++++++++++++++++++++++++--<br>
  4 files changed, 316 insertions(+), 14 deletions(-)<br>
<br>
--<br>
2.4.3<br>
<br>
</blockquote>
<br>
</blockquote></span><div class="HOEnZb"><div class="h5">
_______________________________________________<br>
Intel-gfx mailing list<br>
<a href="mailto:Intel-gfx@lists.freedesktop.org" target="_blank">Intel-gfx@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/intel-gfx" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/intel-gfx</a><br>
</div></div></blockquote></div><br></div></div>