[PATCH 2/5] drm/xe/xelp: Move Wa_16011163337 from tunings to workarounds
Lucas De Marchi
lucas.demarchi at intel.com
Sat Mar 1 00:55:32 UTC 2025
On Fri, Feb 28, 2025 at 08:08:27AM +0000, Tvrtko Ursulin wrote:
>
>On 27/02/2025 20:58, Lucas De Marchi wrote:
>>On Thu, Feb 27, 2025 at 10:13:01AM +0000, Tvrtko Ursulin wrote:
>>>Workaround database specifies 16011163337 as a workaround so lets move it
>>>there.
>>>
>>>Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin at igalia.com>
>>>Cc: Lucas De Marchi <lucas.demarchi at intel.com>
>>>Cc: Matt Roper <matthew.d.roper at intel.com>
>>>Cc: Gustavo Sousa <gustavo.sousa at intel.com>
>>>---
>>>drivers/gpu/drm/xe/xe_tuning.c | 8 --------
>>>drivers/gpu/drm/xe/xe_wa.c | 7 +++++++
>>>2 files changed, 7 insertions(+), 8 deletions(-)
>>>
>>>diff --git a/drivers/gpu/drm/xe/xe_tuning.c b/drivers/gpu/drm/xe/
>>>xe_tuning.c
>>>index d449de0fb6ec..3c78f3d71559 100644
>>>--- a/drivers/gpu/drm/xe/xe_tuning.c
>>>+++ b/drivers/gpu/drm/xe/xe_tuning.c
>>>@@ -97,14 +97,6 @@ static const struct xe_rtp_entry_sr
>>>engine_tunings[] = {
>>>};
>>>
>>>static const struct xe_rtp_entry_sr lrc_tunings[] = {
>>>- { XE_RTP_NAME("Tuning: ganged timer, also known as 16011163337"),
>>
>>we now lose the infor that we have this tuning implemented, but I'm ok
>>with keeping it only on the WA side.... for newer platforms it's there
>>so this is mostly making the older ones work.
>
>Good point, and I am happy to respin with a reversed description like
>"16011163337, also known as Ganged timer tuning" or something.
I prefer this version better, thanks
Lucas De Marchi
More information about the Intel-xe
mailing list