Matthew Auld <matthew.william.a...@gmail.com> writes:

> [ text/plain ]
>> +       /* WaDisableDynamicCreditSharing:kbl */
>> +       if (IS_KBL_REVID(dev_priv, 0, KBL_REVID_B0))
>> +               WA_SET_BIT(GAMT_CHKN_BIT_REG, (1 << 28));
>> +
> Let's play name that bit!
>
> Otherwise the patch looks good, although slightly worrying that the
> hsd's state the WA is needed up to B0, but the WA db says up to A0...
>
> I guess we should rather trust the hsd's?

Yes I consider hsd's the authoritative source.

-Mika



>
> Reviewed-by: Matthew Auld <matthew.a...@intel.com>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to