[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