Re: [PATCH] drm/i915: Use drm_dbg for rpm logging

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

 



On Wed, 11 May 2022 06:04:54 -0700, Anshuman Gupta wrote:
>
> RPM suspend/resume also supported on gfx platforms which doesn't have
> kms support and even on platforms without any connected display panel.
> There is no good reason to log rpm suspend/resume debug message with
> drm_dbg_kms() therefore changing it to drm_dbg().

Reviewed-by: Ashutosh Dixit <ashutosh.dixit@xxxxxxxxx>

> Signed-off-by: Anshuman Gupta <anshuman.gupta@xxxxxxxxx>
> ---
>  drivers/gpu/drm/i915/i915_driver.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_driver.c b/drivers/gpu/drm/i915/i915_driver.c
> index 90b0ce5051af..ed6028fd442d 100644
> --- a/drivers/gpu/drm/i915/i915_driver.c
> +++ b/drivers/gpu/drm/i915/i915_driver.c
> @@ -1549,7 +1549,7 @@ static int intel_runtime_suspend(struct device *kdev)
>	if (drm_WARN_ON_ONCE(&dev_priv->drm, !HAS_RUNTIME_PM(dev_priv)))
>		return -ENODEV;
>
> -	drm_dbg_kms(&dev_priv->drm, "Suspending device\n");
> +	drm_dbg(&dev_priv->drm, "Suspending device\n");
>
>	disable_rpm_wakeref_asserts(rpm);
>
> @@ -1625,7 +1625,7 @@ static int intel_runtime_suspend(struct device *kdev)
>	if (!IS_VALLEYVIEW(dev_priv) && !IS_CHERRYVIEW(dev_priv))
>		intel_hpd_poll_enable(dev_priv);
>
> -	drm_dbg_kms(&dev_priv->drm, "Device suspended\n");
> +	drm_dbg(&dev_priv->drm, "Device suspended\n");
>	return 0;
>  }
>
> @@ -1639,7 +1639,7 @@ static int intel_runtime_resume(struct device *kdev)
>	if (drm_WARN_ON_ONCE(&dev_priv->drm, !HAS_RUNTIME_PM(dev_priv)))
>		return -ENODEV;
>
> -	drm_dbg_kms(&dev_priv->drm, "Resuming device\n");
> +	drm_dbg(&dev_priv->drm, "Resuming device\n");
>
>	drm_WARN_ON_ONCE(&dev_priv->drm, atomic_read(&rpm->wakeref_count));
>	disable_rpm_wakeref_asserts(rpm);
> @@ -1683,7 +1683,7 @@ static int intel_runtime_resume(struct device *kdev)
>		drm_err(&dev_priv->drm,
>			"Runtime resume failed, disabling it (%d)\n", ret);
>	else
> -		drm_dbg_kms(&dev_priv->drm, "Device resumed\n");
> +		drm_dbg(&dev_priv->drm, "Device resumed\n");
>
>	return ret;
>  }
> --
> 2.26.2
>



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

  Powered by Linux