On Tue, Feb 25, 2025 at 02:17:42PM +0200, Jani Nikula wrote:
> The i915 core debugfs has no business looking at power domain guts for
> runtime power status. Move the info to the more appropriate place.
>
> Cc: Imre Deak <imre.deak@xxxxxxxxx>
> Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx>
Reviewed-by: Imre Deak <imre.deak@xxxxxxxxx>
> ---
> drivers/gpu/drm/i915/display/intel_display_power.c | 3 +++
> drivers/gpu/drm/i915/i915_debugfs.c | 3 ---
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_display_power.c b/drivers/gpu/drm/i915/display/intel_display_power.c
> index 396930937d98..96346b33d7e9 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_power.c
> +++ b/drivers/gpu/drm/i915/display/intel_display_power.c
> @@ -2317,6 +2317,9 @@ void intel_display_power_debug(struct intel_display *display, struct seq_file *m
>
> mutex_lock(&power_domains->lock);
>
> + seq_printf(m, "Runtime power status: %s\n",
> + str_enabled_disabled(!power_domains->init_wakeref));
> +
> seq_printf(m, "%-25s %s\n", "Power well/domain", "Use count");
> for (i = 0; i < power_domains->power_well_count; i++) {
> struct i915_power_well *power_well;
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index 1c2a97f593c7..0d9e263913ff 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -411,9 +411,6 @@ static int i915_runtime_pm_status(struct seq_file *m, void *unused)
> if (!HAS_RUNTIME_PM(dev_priv))
> seq_puts(m, "Runtime power management not supported\n");
>
> - seq_printf(m, "Runtime power status: %s\n",
> - str_enabled_disabled(!dev_priv->display.power.domains.init_wakeref));
> -
> seq_printf(m, "GPU idle: %s\n", str_yes_no(!to_gt(dev_priv)->awake));
> seq_printf(m, "IRQs disabled: %s\n",
> str_yes_no(!intel_irqs_enabled(dev_priv)));
> --
> 2.39.5
>
