> -----Original Message-----
> From: Manna, Animesh <[email protected]>
> Sent: Monday, November 18, 2024 10:53 AM
> To: [email protected]
> Cc: Kandpal, Suraj <[email protected]>; Manna, Animesh
> <[email protected]>
> Subject: [PATCH] drm/i915/display: Adjust Added Wake Time with
> PKG_C_LATENCY
>
> The PKG_C_LATENCY Added Wake Time field is not working.
> When added wake time is needed, such as for flip queue DSB execution,
> increase the PKG_C_LATENCY Pkg C Latency field by the added wake time.
>
> HSD: 14023564296
No need to add the HSD like this the WA no should be enough
Also I would like for you to hold on this series since there is another WA I
have sent which clashes with this can you build it over that
It has quite a lot of refactors so it would be better to rebase them on that
here is a reference to that series.
https://patchwork.freedesktop.org/series/141412/
Regards,
Suraj Kandpal
> WA: 22020432604
>
> Signed-off-by: Animesh Manna <[email protected]>
> ---
> drivers/gpu/drm/i915/display/skl_watermark.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/display/skl_watermark.c
> b/drivers/gpu/drm/i915/display/skl_watermark.c
> index 1a4c1fa24820..9be38ca5346d 100644
> --- a/drivers/gpu/drm/i915/display/skl_watermark.c
> +++ b/drivers/gpu/drm/i915/display/skl_watermark.c
> @@ -2875,6 +2875,12 @@ skl_program_dpkgc_latency(struct
> drm_i915_private *i915, bool enable_dpkgc)
> added_wake_time = 0;
> }
>
> + /* Wa_22020432604 */
> + if (DISPLAY_VER(i915) == 30) {
> + max_latency += added_wake_time;
> + added_wake_time = 0;
> + }
> +
> clear |= LNL_ADDED_WAKE_TIME_MASK |
> LNL_PKG_C_LATENCY_MASK;
> val |= REG_FIELD_PREP(LNL_PKG_C_LATENCY_MASK, max_latency);
> val |= REG_FIELD_PREP(LNL_ADDED_WAKE_TIME_MASK,
> added_wake_time);
> --
> 2.29.0
