Commit 6bc07d6a authored by Todd Poynor's avatar Todd Poynor Committed by Paul Walmsley

ARM: OMAP: PM: Lock clocks list while generating summary

Commit a5302572 (OMAP: Add debugfs
node to show the summary of all clocks) introduced clock summary,
however, we are interested in seeing snapshot of the clock state, not
in dynamically changing clock configurations as the data provided by
clock summary will then be useless for debugging configuration
issues. So, hold the common lock when dumping the clock summary.

Cc: Paul Walmsley <paul@pwsan.com>
Cc: Tony Lindgren <tony@atomide.com>
Signed-off-by: default avatarTodd Poynor <toddpoynor@google.com>
[nm@ti.com: added commit message]
Signed-off-by: default avatarNishanth Menon <nm@ti.com>
[paul@pwsan.com: minor edits to commit message]
Signed-off-by: default avatarPaul Walmsley <paul@pwsan.com>
parent 485802a6
...@@ -461,6 +461,7 @@ static int clk_dbg_show_summary(struct seq_file *s, void *unused) ...@@ -461,6 +461,7 @@ static int clk_dbg_show_summary(struct seq_file *s, void *unused)
struct clk *c; struct clk *c;
struct clk *pa; struct clk *pa;
mutex_lock(&clocks_mutex);
seq_printf(s, "%-30s %-30s %-10s %s\n", seq_printf(s, "%-30s %-30s %-10s %s\n",
"clock-name", "parent-name", "rate", "use-count"); "clock-name", "parent-name", "rate", "use-count");
...@@ -469,6 +470,7 @@ static int clk_dbg_show_summary(struct seq_file *s, void *unused) ...@@ -469,6 +470,7 @@ static int clk_dbg_show_summary(struct seq_file *s, void *unused)
seq_printf(s, "%-30s %-30s %-10lu %d\n", seq_printf(s, "%-30s %-30s %-10lu %d\n",
c->name, pa ? pa->name : "none", c->rate, c->usecount); c->name, pa ? pa->name : "none", c->rate, c->usecount);
} }
mutex_unlock(&clocks_mutex);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment