On Thu, 2025-08-28 at 15:21 +0300, Jani Nikula wrote:
> Unify on using poll_timeout_us() throughout instead of mixing with
> readx_poll_timeout().
> 
> Signed-off-by: Jani Nikula <jani.nik...@intel.com>

Reviewed-by: Jouni Högander <jouni.hogan...@intel.com>

> ---
>  drivers/gpu/drm/i915/display/intel_pps.c | 8 +++-----
>  1 file changed, 3 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_pps.c
> b/drivers/gpu/drm/i915/display/intel_pps.c
> index b84eb43bd2d0..327e0de86f1e 100644
> --- a/drivers/gpu/drm/i915/display/intel_pps.c
> +++ b/drivers/gpu/drm/i915/display/intel_pps.c
> @@ -627,11 +627,9 @@ static void wait_panel_status(struct intel_dp
> *intel_dp,
>                   intel_de_read(display, pp_stat_reg),
>                   intel_de_read(display, pp_ctrl_reg));
>  
> -     ret = read_poll_timeout(intel_de_read, val,
> -                             (val & mask) == value,
> -                             10 * 1000, 5000 * 1000, true,
> -                             display, pp_stat_reg);
> -
> +     ret = poll_timeout_us(val = intel_de_read(display,
> pp_stat_reg),
> +                           (val & mask) == value,
> +                           10 * 1000, 5000 * 1000, true);
>       if (ret) {
>               drm_err(display->drm,
>                       "[ENCODER:%d:%s] %s panel status timeout:
> PP_STATUS: 0x%08x PP_CONTROL: 0x%08x\n",

Reply via email to