Re: [PATCH] drm/i915/cnl: Do not add an extra page for precaution in the Gen10 LRC size

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2017-10-03 at 08:46 -0700, Oscar Mateo wrote:
> 
> On 10/03/2017 05:25 AM, Joonas Lahtinen wrote:
> > On Mon, 2017-10-02 at 13:31 -0700, Oscar Mateo wrote:
> > > BSpec indicates exactly 16750 DWORDs (17 pages), plus one page for PPHWSP.
> > 
> > Assuming this is;
> > 
> > Bspec: 1383
> > 
> > I'm actually getting 16801 DWORDs by copy-pasting to spreadsheet and
> > summing up. Are we looking from different places?
> 
> No, we are looking at the same BSpec page, but you need to filter by 
> "CNL" (that will remove some fields). Even then, I'm now getting 16752 
> instead of 16750 (maybe they have updated some tags?).
> I'll resend with the BSpec tag, the new count and your RB, if that's OK.

Yep, that's fine. Do include a note in the commit message that there
are exclusions of rows in the BSpec table, so when looking back, one is
reminded that copy and paste is not enough :)

Regards, Joonas
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux