Re: [PATCH] PM / Domains: Add perf_state attribute to genpd debugfs

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

 



On 30 May 2018 at 11:45, Rajendra Nayak <rnayak@xxxxxxxxxxxxxx> wrote:
> Now that genpd supports performance states, add this additional
> attribute as part of the power domains debugfs entry, to display
> the current performance state for the Power domain.
>
> Suggested-by: David Collins <collinsd@xxxxxxxxxxxxxx>
> Signed-off-by: Rajendra Nayak <rnayak@xxxxxxxxxxxxxx>

Acked-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx>

Kind regards
Uffe

> ---
>  drivers/base/power/domain.c | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>
> diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c
> index ffeb6ea8d6eb..a02d793c8275 100644
> --- a/drivers/base/power/domain.c
> +++ b/drivers/base/power/domain.c
> @@ -2802,6 +2802,19 @@ static int genpd_devices_show(struct seq_file *s, void *data)
>         return ret;
>  }
>
> +static int genpd_perf_state_show(struct seq_file *s, void *data)
> +{
> +       struct generic_pm_domain *genpd = s->private;
> +
> +       if (genpd_lock_interruptible(genpd))
> +               return -ERESTARTSYS;
> +
> +       seq_printf(s, "%u\n", genpd->performance_state);
> +
> +       genpd_unlock(genpd);
> +       return 0;
> +}
> +
>  #define define_genpd_open_function(name) \
>  static int genpd_##name##_open(struct inode *inode, struct file *file) \
>  { \
> @@ -2815,6 +2828,7 @@ define_genpd_open_function(idle_states);
>  define_genpd_open_function(active_time);
>  define_genpd_open_function(total_idle_time);
>  define_genpd_open_function(devices);
> +define_genpd_open_function(perf_state);
>
>  #define define_genpd_debugfs_fops(name) \
>  static const struct file_operations genpd_##name##_fops = { \
> @@ -2831,6 +2845,7 @@ define_genpd_debugfs_fops(idle_states);
>  define_genpd_debugfs_fops(active_time);
>  define_genpd_debugfs_fops(total_idle_time);
>  define_genpd_debugfs_fops(devices);
> +define_genpd_debugfs_fops(perf_state);
>
>  static int __init genpd_debug_init(void)
>  {
> @@ -2864,6 +2879,9 @@ static int __init genpd_debug_init(void)
>                                 d, genpd, &genpd_total_idle_time_fops);
>                 debugfs_create_file("devices", 0444,
>                                 d, genpd, &genpd_devices_fops);
> +               if (genpd->set_performance_state)
> +                       debugfs_create_file("perf_state", 0444,
> +                                           d, genpd, &genpd_perf_state_fops);
>         }
>
>         return 0;
> --
> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
> of Code Aurora Forum, hosted by The Linux Foundation
>
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux