| Message ID | 20231101-gdsc-hwctrl-v3-2-0740ae6b2b04@linaro.org (mailing list archive) |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | PM: domains: Add control for switching back and forth to HW control | expand |
On Wed, 1 Nov 2023 at 11:06, Abel Vesa <abel.vesa@linaro.org> wrote: > > Now that genpd supports dynamically switching the control for an > attached device between hardware- and software-mode, let's add this > information to the genpd summary in debugfs. > > Suggested-by: Taniya Das <quic_tdas@quicinc.com> > Signed-off-by: Abel Vesa <abel.vesa@linaro.org> > Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org> > --- > drivers/base/power/domain.c | 15 +++++++++++++-- > 1 file changed, 13 insertions(+), 2 deletions(-) Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> -- With best wishes Dmitry
On Wed, Nov 01, 2023 at 11:04:08AM +0200, Abel Vesa wrote: > Now that genpd supports dynamically switching the control for an > attached device between hardware- and software-mode, let's add this > information to the genpd summary in debugfs. > > Suggested-by: Taniya Das <quic_tdas@quicinc.com> > Signed-off-by: Abel Vesa <abel.vesa@linaro.org> > Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org> Reviewed-by: Bjorn Andersson <andersson@kernel.org> Regards, Bjorn > --- > drivers/base/power/domain.c | 15 +++++++++++++-- > 1 file changed, 13 insertions(+), 2 deletions(-) > > diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c > index 3fb1a234c7f2..7044271ec93b 100644 > --- a/drivers/base/power/domain.c > +++ b/drivers/base/power/domain.c > @@ -3181,6 +3181,15 @@ static void rtpm_status_str(struct seq_file *s, struct device *dev) > seq_printf(s, "%-25s ", p); > } > > +static void mode_status_str(struct seq_file *s, struct device *dev) > +{ > + struct generic_pm_domain_data *gpd_data; > + > + gpd_data = to_gpd_data(dev->power.subsys_data->domain_data); > + > + seq_printf(s, "%20s", gpd_data->hw_mode ? "HW" : "SW"); > +} > + > static void perf_status_str(struct seq_file *s, struct device *dev) > { > struct generic_pm_domain_data *gpd_data; > @@ -3239,6 +3248,7 @@ static int genpd_summary_one(struct seq_file *s, > seq_printf(s, "\n %-50s ", kobj_path); > rtpm_status_str(s, pm_data->dev); > perf_status_str(s, pm_data->dev); > + mode_status_str(s, pm_data->dev); > kfree(kobj_path); > } > > @@ -3255,8 +3265,9 @@ static int summary_show(struct seq_file *s, void *data) > int ret = 0; > > seq_puts(s, "domain status children performance\n"); > - seq_puts(s, " /device runtime status\n"); > - seq_puts(s, "----------------------------------------------------------------------------------------------\n"); > + seq_puts(s, " /device runtime status managed by\n"); > + seq_puts(s, "------------------------------------------------------------------------------------------------------------\n"); > + > > ret = mutex_lock_interruptible(&gpd_list_lock); > if (ret) > > -- > 2.34.1 >
diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c index 3fb1a234c7f2..7044271ec93b 100644 --- a/drivers/base/power/domain.c +++ b/drivers/base/power/domain.c @@ -3181,6 +3181,15 @@ static void rtpm_status_str(struct seq_file *s, struct device *dev) seq_printf(s, "%-25s ", p); } +static void mode_status_str(struct seq_file *s, struct device *dev) +{ + struct generic_pm_domain_data *gpd_data; + + gpd_data = to_gpd_data(dev->power.subsys_data->domain_data); + + seq_printf(s, "%20s", gpd_data->hw_mode ? "HW" : "SW"); +} + static void perf_status_str(struct seq_file *s, struct device *dev) { struct generic_pm_domain_data *gpd_data; @@ -3239,6 +3248,7 @@ static int genpd_summary_one(struct seq_file *s, seq_printf(s, "\n %-50s ", kobj_path); rtpm_status_str(s, pm_data->dev); perf_status_str(s, pm_data->dev); + mode_status_str(s, pm_data->dev); kfree(kobj_path); } @@ -3255,8 +3265,9 @@ static int summary_show(struct seq_file *s, void *data) int ret = 0; seq_puts(s, "domain status children performance\n"); - seq_puts(s, " /device runtime status\n"); - seq_puts(s, "----------------------------------------------------------------------------------------------\n"); + seq_puts(s, " /device runtime status managed by\n"); + seq_puts(s, "------------------------------------------------------------------------------------------------------------\n"); + ret = mutex_lock_interruptible(&gpd_list_lock); if (ret)