✓ CI.checkpatch: success for drm/xe/dg2: Fix number of mocs entries
Patchwork
patchwork at emeril.freedesktop.org
Thu Feb 22 20:59:55 UTC 2024
== Series Details ==
Series: drm/xe/dg2: Fix number of mocs entries
URL : https://patchwork.freedesktop.org/series/130286/
State : success
== Summary ==
+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
35591fb8b4d5305b37ce31483f85ac0956eaa536
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit c029dd8dcb5b34ae50fdcf1163200c13d21ddaa1
Author: Lucas De Marchi <lucas.demarchi at intel.com>
Date: Thu Feb 22 12:42:03 2024 -0800
drm/xe/dg2: Fix number of mocs entries
While investigating why xe_mocs kunit was failing for DG2, Matt Roper
noticed that LNCFCMOCS[31] is actually RO for DG2. It would be a spec
bug to document a register to be programmed that is actually RO. However
upon further inspection, it seems like we should not be programming the
other values. Early DG2 SKUs documented that the entire set of
LNCFCMOCS registers needed to be programmed, but this is not true
anymore.
Do like is done in MTL: the additional registers are there, but we are
not supposed to program them, not even with the unused_index value.
While at it, fix MTL define to follow the same alignment as others, with
tabs.
Bspec: 44988, 55267
Closes: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/1253
Closes: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/1233
Cc: Matt Roper <matthew.d.roper at intel.com>
Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
+ /mt/dim checkpatch 164222e11030db989ad66c48f764b5ce4b896181 drm-intel
c029dd8dcb5b drm/xe/dg2: Fix number of mocs entries
More information about the Intel-xe
mailing list