Re: [PATCH v3] drm/i915/lnl: Remove watchdog timers for PSR

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2023-10-10 at 12:52 +0300, Mika Kahola wrote:
> Watchdorg timers for Lunarlake HW were removed for PSR/PSR2
> The patch removes the use of these timers from the driver code.
> 
> BSpec: 69895
> 
> v2: Reword commit message (Ville)
>     Drop HPD mask from LNL (Ville)
>     Revise masking logic (Jouni)
> v3: Revise commit message (Ville)
>     Revert HPD mask removal as irrelevant for this patch (Ville)
> 
> Signed-off-by: Mika Kahola <mika.kahola@xxxxxxxxx>

Consider fixing typo above (watchdorg) before merging:

Reviewed-by: Jouni Högander <jouni.hogander@xxxxxxxxx>

> ---
>  drivers/gpu/drm/i915/display/intel_psr.c | 10 +++++++---
>  1 file changed, 7 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_psr.c
> b/drivers/gpu/drm/i915/display/intel_psr.c
> index bb65881e87cc..4f1f31fc9529 100644
> --- a/drivers/gpu/drm/i915/display/intel_psr.c
> +++ b/drivers/gpu/drm/i915/display/intel_psr.c
> @@ -676,7 +676,9 @@ static void hsw_activate_psr1(struct intel_dp
> *intel_dp)
>  
>         val |=
> EDP_PSR_IDLE_FRAMES(psr_compute_idle_frames(intel_dp));
>  
> -       val |= EDP_PSR_MAX_SLEEP_TIME(max_sleep_time);
> +       if (DISPLAY_VER(dev_priv) < 20)
> +               val |= EDP_PSR_MAX_SLEEP_TIME(max_sleep_time);
> +
>         if (IS_HASWELL(dev_priv))
>                 val |= EDP_PSR_MIN_LINK_ENTRY_TIME_8_LINES;
>  
> @@ -1400,8 +1402,10 @@ static void intel_psr_enable_source(struct
> intel_dp *intel_dp,
>          */
>         mask = EDP_PSR_DEBUG_MASK_MEMUP |
>                EDP_PSR_DEBUG_MASK_HPD |
> -              EDP_PSR_DEBUG_MASK_LPSP |
> -              EDP_PSR_DEBUG_MASK_MAX_SLEEP;
> +              EDP_PSR_DEBUG_MASK_LPSP;
> +
> +       if (DISPLAY_VER(dev_priv) < 20)
> +               mask |= EDP_PSR_DEBUG_MASK_MAX_SLEEP;
>  
>         /*
>          * No separate pipe reg write mask on hsw/bdw, so have to
> unmask all





[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux