On 9/27/2023 9:04 AM, Andi Shyti wrote:
Hi Nirmoy,

On Tue, Sep 26, 2023 at 11:58:02PM +0200, Nirmoy Das wrote:
On MTL GEN12_RING_FAULT_REG is not replicated so don't
do mcr based operation for this register.

v2: use MEDIA_VER() instead of GRAPHICS_VER()(Matt).

Signed-off-by: Nirmoy Das <[email protected]>
This looks very good!


My tired brain took it as end of mail.


-       if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 50)) {
+       /*
+        * for media tile this ring fault register is not replicated
+        * so skip doing mcr ops on it.
Can we spare a few more words on "doing mcr ops on it"?
Sent out https://patchwork.freedesktop.org/patch/559759/?series=124325&rev=1

to improve this.


Thanks,

Nirmoy


Andi

Reply via email to