[PATCH] drm/i915/dmc_wl: Do not check for DMC payload

Gustavo Sousa gustavo.sousa at intel.com
Fri Jan 31 20:16:29 UTC 2025


Quoting Krzysztof Karas (2025-01-30 11:18:28-03:00)
>Hi Gustavo,
>
>[...]
>> Let's remove that check, since it is unnecessary and causes the
>> inconsistency illustrated above.
>> 
>> Signed-off-by: Gustavo Sousa <gustavo.sousa at intel.com>
>> ---
>
>Would you add:
>Fixes: c01e78a96e12 ("drm/i915/dmc_wl: Sanitize enable_dmc_wl according to hardware support")?

I believe the above commit should not be a "fixes" candidate, the check
for intel_dmc_has_payload() was already there when that commit was
applied.

Also, do we need a "Fixes:" tag here? There only platform where the DMC
Wakelock mechanism is enabled by default is PTL, which is still under
force_probe protection.

--
Gustavo Sousa

>
>after that LGTM:
>Reviewed-by: Krzysztof Karas <krzysztof.karas at intel.com>
>
>Krzysztof


More information about the Intel-xe mailing list