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 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.


Both lead to the same amount of pages, but I'd like the commit message
to be correct for easier double-checking :) With commit message
corrected and the Bspec: tag added (or more correct Bspec: tag
specified), this is;

Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>

Regards, Joonas

_______________________________________________
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