Commit 215bb2ce authored by Lucas De Marchi's avatar Lucas De Marchi Committed by Rodrigo Vivi

drm/xe: Fix comment on Wa_22013088509

On i915 the "see comment about Wa_22013088509" referred to the comment
in the graphics version >= 11 branch, where there were more details
about it. From the platforms supported by xe, only PVC needs
Wa_22013088509, but as the comment says, it's simpler to do it for all
platforms as there is no downside.  Bring the missing comment over from
i915 and reword it to fit xe better.
Reviewed-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
Link: https://lore.kernel.org/r/20230504073250.1436293-3-lucas.demarchi@intel.comSigned-off-by: default avatarLucas De Marchi <lucas.demarchi@intel.com>
Signed-off-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
parent dbeb2bd2
...@@ -461,9 +461,16 @@ static u32 rw_with_mcr_steering(struct xe_gt *gt, struct xe_reg_mcr reg_mcr, ...@@ -461,9 +461,16 @@ static u32 rw_with_mcr_steering(struct xe_gt *gt, struct xe_reg_mcr reg_mcr,
} }
/* /*
* Always leave the hardware in multicast mode when doing reads * Always leave the hardware in multicast mode when doing reads and only
* (see comment about Wa_22013088509 below) and only change it * change it to unicast mode when doing writes of a specific instance.
* to unicast mode when doing writes of a specific instance. *
* The setting of the multicast/unicast bit usually wouldn't matter for
* read operations (which always return the value from a single register
* instance regardless of how that bit is set), but some platforms may
* have workarounds requiring us to remain in multicast mode for reads,
* e.g. Wa_22013088509 on PVC. There's no real downside to this, so
* we'll just go ahead and do so on all platforms; we'll only clear the
* multicast bit from the mask when explicitly doing a write operation.
* *
* No need to save old steering reg value. * No need to save old steering reg value.
*/ */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment