Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6954d432 authored by Geert Uytterhoeven's avatar Geert Uytterhoeven Committed by Rafael J. Wysocki
Browse files

PM / Domains: Restore alignment of slaves in debugfs output 



The slave domains are no longer aligned with the table header in the
/sys/kernel/debug/pm_genpd/pm_genpd_summary output. Worse, the alignment
differs depending on the actual name of the state.

Format the state name and index into a buffer, and print that like
before to restore alignment.

Use "%u" for unsigned int while we're at it.

Fixes: fc5cbf0c (PM / Domains: Support for multiple states)
Signed-off-by: default avatarGeert Uytterhoeven <geert+renesas@glider.be>
Acked-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
Tested-by: default avatarKevin Hilman <khilman@baylibre.com>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent 90e63452

drivers/base/power/domain.c

+6 −4
Original line number Diff line number Diff line
@@ -1882,6 +1882,7 @@ static int pm_genpd_summary_one(struct seq_file *s, 
	struct pm_domain_data *pm_data;

	const char *kobj_path;

	struct gpd_link *link;

	char state[16];

	int ret;



	ret = mutex_lock_interruptible(&genpd->lock);
@@ -1890,12 +1891,13 @@ static int pm_genpd_summary_one(struct seq_file *s, 


	if (WARN_ON(genpd->status >= ARRAY_SIZE(status_lookup)))

		goto exit;

	seq_printf(s, "%-30s  %s", genpd->name, status_lookup[genpd->status]);



	if (genpd->status == GPD_STATE_POWER_OFF)

		seq_printf(s, " %-13d ", genpd->state_idx);

		snprintf(state, sizeof(state), "%s %u",

			 status_lookup[genpd->status], genpd->state_idx);

	else

		seq_printf(s, " %-15s ", "");

		snprintf(state, sizeof(state), "%s",

			 status_lookup[genpd->status]);

	seq_printf(s, "%-30s  %-15s ", genpd->name, state);



	/*

	 * Modifications on the list require holding locks on both