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

[Oscar Mateo]

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 at linux.intel.com>
>
> Regards, Joonas