"ext Peter 'p2' De Schrijver" <peter.de-schrijver@xxxxxxxxx> writes:
> This patch provides the debugfs entries and a function which will be called
> by the PM code to register the time spent per domain per state.
>
> Signed-off-by: Peter 'p2' De Schrijver <peter.de-schrijver@xxxxxxxxx>
> ---
> arch/arm/mach-omap2/pm-debug.c | 175 +++++++++++++++++++++++++
> arch/arm/mach-omap2/pm.h | 2 +
> 2 files changed, 177 insertions(+)
>
> diff --git a/arch/arm/mach-omap2/pm-debug.c b/arch/arm/mach-omap2/pm-debug.c
> index 0b5c044..4ba6cec 100644
> --- a/arch/arm/mach-omap2/pm-debug.c
> +++ b/arch/arm/mach-omap2/pm-debug.c
> @@ -29,6 +29,8 @@
>
> #include <mach/clock.h>
> #include <mach/board.h>
> +#include <mach/powerdomain.h>
> +#include <mach/clockdomain.h>
>
> #include "prm.h"
> #include "cm.h"
> @@ -288,4 +290,177 @@ void omap2_pm_dump(int mode, int resume, unsigned int us)
> printk("%-20s: 0x%08x\n", regs[i].name, regs[i].val);
> }
>
> +#ifdef CONFIG_DEBUG_FS
> +#include <linux/debugfs.h>
> +#include <linux/seq_file.h>
> +
> +struct dentry *pm_dbg_dir;
> +
> +static int pm_dbg_init_done;
> +
> +enum {
> + DEBUG_FILE_COUNTERS = 0,
> + DEBUG_FILE_TIMERS,
> +};
> +
> +static const char pwrdm_state_names[][4] = {
> + "OFF",
> + "RET",
> + "INA",
> + "ON"
> +};
> +
> +void pm_dbg_update_time(struct powerdomain *pwrdm, int prev)
> +{
> + s64 t;
> + struct timespec now;
> +
> + if (!pm_dbg_init_done)
> + return ;
> +
> + /* Update timer for previous state */
> + getnstimeofday(&now);
> + t = timespec_to_ns(&now);
> +
> + pwrdm->state_timer[prev] += t - pwrdm->timer;
> +
> + pwrdm->timer = t;
> +}
> +
> +static int clkdm_dbg_show_counter(struct clockdomain *clkdm, void *user)
> +{
> + struct seq_file *s = (struct seq_file *)user;
> +
> + if (strcmp(clkdm->name, "emu_clkdm") == 0 ||
> + strcmp(clkdm->name, "wkup_clkdm") == 0)
> + return 0;
Why emu and wkup are not taken into account? If wkup is closed out
here, I think also prm_clkdm and cm_clkdm should be.
> +
> + seq_printf(s, "%s->%s (%d)", clkdm->name,
> + clkdm->pwrdm.ptr->name,
> + atomic_read(&clkdm->usecount));
> + seq_printf(s, "\n");
> +
> + return 0;
> +}
> +
> +static int pwrdm_dbg_show_counter(struct powerdomain *pwrdm, void *user)
> +{
> + struct seq_file *s = (struct seq_file *)user;
> + int i;
> +
> + if (strcmp(pwrdm->name, "emu_pwrdm") == 0 ||
> + strcmp(pwrdm->name, "wkup_pwrdm") == 0)
> + return 0;
Why emu is closed out here? It has pwstst register. I think also
dpll1..5_pwrdm should be closed out here. I'm not sure why those are
even modelled in a clocktree.
> +
> + if (pwrdm->state != pwrdm_read_pwrst(pwrdm))
> + printk(KERN_ERR "pwrdm state mismatch(%s) %d != %d\n",
> + pwrdm->name, pwrdm->state, pwrdm_read_pwrst(pwrdm));
> +
> + seq_printf(s, "%s (%s)", pwrdm->name,
> + pwrdm_state_names[pwrdm->state]);
> + for (i = 0; i < 4; i++)
> + seq_printf(s, ",%s:%d", pwrdm_state_names[i],
> + pwrdm->state_counter[i]);
> +
> + seq_printf(s, "\n");
> +
> + return 0;
> +}
> +
> +static int pwrdm_dbg_show_timer(struct powerdomain *pwrdm, void *user)
> +{
> + struct seq_file *s = (struct seq_file *)user;
> + int i;
> +
> + if (strcmp(pwrdm->name, "emu_pwrdm") == 0 ||
> + strcmp(pwrdm->name, "wkup_pwrdm") == 0)
> + return 0;
Same comment as above.
> +
> + pwrdm_state_switch(pwrdm);
> +
> + seq_printf(s, "%s (%s)", pwrdm->name,
> + pwrdm_state_names[pwrdm->state]);
> +
> + for (i = 0; i < 4; i++)
> + seq_printf(s, ",%s:%lld", pwrdm_state_names[i],
> + pwrdm->state_timer[i]);
> +
> + seq_printf(s, "\n");
> +
> + return 0;
> +}
> +
> +static void pm_dbg_show_counters(struct seq_file *s, void *unused)
> +{
> + pwrdm_for_each(pwrdm_dbg_show_counter, s);
> + clkdm_for_each(clkdm_dbg_show_counter, s);
> +}
> +
> +static void pm_dbg_show_timers(struct seq_file *s, void *unused)
> +{
> + pwrdm_for_each(pwrdm_dbg_show_timer, s);
> +}
> +
> +static int pm_dbg_open(struct inode *inode, struct file *file)
> +{
> + switch ((int)inode->i_private) {
> + case DEBUG_FILE_COUNTERS:
> + return single_open(file, pm_dbg_show_counters,
> + &inode->i_private);
> + case DEBUG_FILE_TIMERS:
> + default:
> + return single_open(file, pm_dbg_show_timers,
> + &inode->i_private);
> + };
> +}
> +
> +static const struct file_operations debug_fops = {
> + .open = pm_dbg_open,
> + .read = seq_read,
> + .llseek = seq_lseek,
> + .release = single_release,
> +};
> +
> +static int __init pwrdms_setup(struct powerdomain *pwrdm, void *unused)
> +{
> + int i;
> + s64 t;
> + struct timespec now;
> +
> + getnstimeofday(&now);
> + t = timespec_to_ns(&now);
> +
> + for (i = 0; i < 4; i++)
> + pwrdm->state_timer[i] = 0;
> +
> + pwrdm->timer = t;
> +
> + return 0;
> +}
> +
> +static int __init pm_dbg_init(void)
> +{
> + struct dentry *d;
> +
> + printk(KERN_INFO "pm_dbg_init()\n");
> +
> + d = debugfs_create_dir("pm_debug", NULL);
> + if (IS_ERR(d))
> + return PTR_ERR(d);
> +
> + (void) debugfs_create_file("count", S_IRUGO,
> + d, (void *)DEBUG_FILE_COUNTERS, &debug_fops);
> + (void) debugfs_create_file("time", S_IRUGO,
> + d, (void *)DEBUG_FILE_TIMERS, &debug_fops);
> +
> + pwrdm_for_each(pwrdms_setup, NULL);
> +
> + pm_dbg_init_done = 1;
> +
> + return 0;
> +}
> +late_initcall(pm_dbg_init);
> +
> +#endif
> +
> #endif
> diff --git a/arch/arm/mach-omap2/pm.h b/arch/arm/mach-omap2/pm.h
> index 68c9278..fb6693b 100644
> --- a/arch/arm/mach-omap2/pm.h
> +++ b/arch/arm/mach-omap2/pm.h
> @@ -31,6 +31,7 @@ extern void serial_console_fclk_mask(u32 *f1, u32 *f2);
> extern void pm_init_serial_console(void);
> extern void serial_console_sleep(int enable);
> extern int omap2_pm_debug;
> +extern void pm_dbg_update_time(struct powerdomain *pwrdm, int prev);
> #else
> #define omap2_read_32k_sync_counter() 0
> #define serial_console_sleep(enable) do {} while (0);
> @@ -38,5 +39,6 @@ extern int omap2_pm_debug;
> #define omap2_pm_dump(mode, resume, us) do {} while (0);
> #define serial_console_fclk_mask(f1, f2) do {} while (0);
> #define omap2_pm_debug 0
> +#define pm_dbg_update_time(pwrdm, prev) do {} while (0);
> #endif /* CONFIG_PM_DEBUG */
> #endif
> --
> 1.5.6.3
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>
--
Jouni Högander
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
