* [PATCH v3] clk: Add clk_set_parent debugfs node
@ 2021-10-07 13:56 Sam Protsenko
2021-10-07 14:04 ` Sam Protsenko
2021-10-07 14:17 ` Fabio Estevam
0 siblings, 2 replies; 4+ messages in thread
From: Sam Protsenko @ 2021-10-07 13:56 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd, Russell King
Cc: Chanwoo Choi, Sylwester Nawrocki, Krzysztof Kozlowski,
Mike Tipton, Andy Shevchenko, Andy Shevchenko,
Geert Uytterhoeven, linux-clk, linux-kernel
Useful for testing mux clocks. One can write the index of the parent to
set into clk_parent node, starting from 0. Example
# cd /sys/kernel/debug/clk/mout_peri_bus
# cat clk_possible_parrents
dout_shared0_div4 dout_shared1_div4
# cat clk_parent
dout_shared0_div4
# echo 1 > clk_parent
# cat clk_parent
dout_shared1_div4
One needs to define CLOCK_ALLOW_WRITE_DEBUGFS in drivers/clk/clk.c in
order to use this feature.
Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>
--
Changes in v3:
- Removed unwanted changes added by mistake
Changes in v2:
- Merged write() function into existing 'clk_parent' file
- Removed 'if (val >= core->num_parents)' check
- Removed incorrect usage of IS_ERR_OR_NULL()
drivers/clk/clk.c | 53 ++++++++++++++++++++++++++++++++++++++++++++++-
1 file changed, 52 insertions(+), 1 deletion(-)
diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c
index 806c55f0991b..f57e33efda9b 100644
--- a/drivers/clk/clk.c
+++ b/drivers/clk/clk.c
@@ -3224,6 +3224,46 @@ static int current_parent_show(struct seq_file *s, void *data)
}
DEFINE_SHOW_ATTRIBUTE(current_parent);
+#ifdef CLOCK_ALLOW_WRITE_DEBUGFS
+static ssize_t current_parent_write(struct file *file, const char __user *ubuf,
+ size_t count, loff_t *ppos)
+{
+ struct seq_file *s = file->private_data;
+ struct clk_core *core = s->private;
+ struct clk_core *parent;
+ char buf[4] = { 0 };
+ u8 idx;
+ int err;
+
+ if (copy_from_user(&buf, ubuf, min_t(size_t, sizeof(buf) - 1, count)))
+ return -EFAULT;
+
+ err = kstrtou8(buf, 0, &idx);
+ if (err)
+ return err;
+
+ parent = clk_core_get_parent_by_index(core, idx);
+ if (!parent)
+ return -ENOENT;
+
+ clk_prepare_lock();
+ err = clk_core_set_parent_nolock(core, parent);
+ clk_prepare_unlock();
+ if (err)
+ return err;
+
+ return count;
+}
+
+static const struct file_operations current_parent_rw_fops = {
+ .open = current_parent_open,
+ .write = current_parent_write,
+ .read = seq_read,
+ .llseek = seq_lseek,
+ .release = single_release,
+};
+#endif
+
static int clk_duty_cycle_show(struct seq_file *s, void *data)
{
struct clk_core *core = s->private;
@@ -3291,9 +3331,20 @@ static void clk_debug_create_one(struct clk_core *core, struct dentry *pdentry)
&clk_prepare_enable_fops);
#endif
- if (core->num_parents > 0)
+ if (core->num_parents > 0) {
+#ifdef CLOCK_ALLOW_WRITE_DEBUGFS
+ if (core->num_parents > 1) {
+ debugfs_create_file("clk_parent", 0644, root, core,
+ ¤t_parent_rw_fops);
+ } else {
+ debugfs_create_file("clk_parent", 0444, root, core,
+ ¤t_parent_fops);
+ }
+#else
debugfs_create_file("clk_parent", 0444, root, core,
¤t_parent_fops);
+#endif
+ }
if (core->num_parents > 1)
debugfs_create_file("clk_possible_parents", 0444, root, core,
--
2.30.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v3] clk: Add clk_set_parent debugfs node
2021-10-07 13:56 [PATCH v3] clk: Add clk_set_parent debugfs node Sam Protsenko
@ 2021-10-07 14:04 ` Sam Protsenko
2021-10-07 14:17 ` Fabio Estevam
1 sibling, 0 replies; 4+ messages in thread
From: Sam Protsenko @ 2021-10-07 14:04 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd, Russell King
Cc: Chanwoo Choi, Sylwester Nawrocki, Krzysztof Kozlowski,
Mike Tipton, Andy Shevchenko, Andy Shevchenko,
Geert Uytterhoeven, linux-clk, Linux Kernel Mailing List
On Thu, 7 Oct 2021 at 16:56, Sam Protsenko <semen.protsenko@linaro.org> wrote:
>
> Useful for testing mux clocks. One can write the index of the parent to
> set into clk_parent node, starting from 0. Example
>
> # cd /sys/kernel/debug/clk/mout_peri_bus
> # cat clk_possible_parrents
> dout_shared0_div4 dout_shared1_div4
> # cat clk_parent
> dout_shared0_div4
> # echo 1 > clk_parent
> # cat clk_parent
> dout_shared1_div4
>
> One needs to define CLOCK_ALLOW_WRITE_DEBUGFS in drivers/clk/clk.c in
> order to use this feature.
>
> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>
> --
Bummer... Now the title is obviously wrong. Will send v4 soon.
> Changes in v3:
> - Removed unwanted changes added by mistake
>
> Changes in v2:
> - Merged write() function into existing 'clk_parent' file
> - Removed 'if (val >= core->num_parents)' check
> - Removed incorrect usage of IS_ERR_OR_NULL()
>
> drivers/clk/clk.c | 53 ++++++++++++++++++++++++++++++++++++++++++++++-
> 1 file changed, 52 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c
> index 806c55f0991b..f57e33efda9b 100644
> --- a/drivers/clk/clk.c
> +++ b/drivers/clk/clk.c
> @@ -3224,6 +3224,46 @@ static int current_parent_show(struct seq_file *s, void *data)
> }
> DEFINE_SHOW_ATTRIBUTE(current_parent);
>
> +#ifdef CLOCK_ALLOW_WRITE_DEBUGFS
> +static ssize_t current_parent_write(struct file *file, const char __user *ubuf,
> + size_t count, loff_t *ppos)
> +{
> + struct seq_file *s = file->private_data;
> + struct clk_core *core = s->private;
> + struct clk_core *parent;
> + char buf[4] = { 0 };
> + u8 idx;
> + int err;
> +
> + if (copy_from_user(&buf, ubuf, min_t(size_t, sizeof(buf) - 1, count)))
> + return -EFAULT;
> +
> + err = kstrtou8(buf, 0, &idx);
> + if (err)
> + return err;
> +
> + parent = clk_core_get_parent_by_index(core, idx);
> + if (!parent)
> + return -ENOENT;
> +
> + clk_prepare_lock();
> + err = clk_core_set_parent_nolock(core, parent);
> + clk_prepare_unlock();
> + if (err)
> + return err;
> +
> + return count;
> +}
> +
> +static const struct file_operations current_parent_rw_fops = {
> + .open = current_parent_open,
> + .write = current_parent_write,
> + .read = seq_read,
> + .llseek = seq_lseek,
> + .release = single_release,
> +};
> +#endif
> +
> static int clk_duty_cycle_show(struct seq_file *s, void *data)
> {
> struct clk_core *core = s->private;
> @@ -3291,9 +3331,20 @@ static void clk_debug_create_one(struct clk_core *core, struct dentry *pdentry)
> &clk_prepare_enable_fops);
> #endif
>
> - if (core->num_parents > 0)
> + if (core->num_parents > 0) {
> +#ifdef CLOCK_ALLOW_WRITE_DEBUGFS
> + if (core->num_parents > 1) {
> + debugfs_create_file("clk_parent", 0644, root, core,
> + ¤t_parent_rw_fops);
> + } else {
> + debugfs_create_file("clk_parent", 0444, root, core,
> + ¤t_parent_fops);
> + }
> +#else
> debugfs_create_file("clk_parent", 0444, root, core,
> ¤t_parent_fops);
> +#endif
> + }
>
> if (core->num_parents > 1)
> debugfs_create_file("clk_possible_parents", 0444, root, core,
> --
> 2.30.2
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v3] clk: Add clk_set_parent debugfs node
2021-10-07 13:56 [PATCH v3] clk: Add clk_set_parent debugfs node Sam Protsenko
2021-10-07 14:04 ` Sam Protsenko
@ 2021-10-07 14:17 ` Fabio Estevam
2021-10-07 15:31 ` Sam Protsenko
1 sibling, 1 reply; 4+ messages in thread
From: Fabio Estevam @ 2021-10-07 14:17 UTC (permalink / raw)
To: Sam Protsenko
Cc: Michael Turquette, Stephen Boyd, Russell King, Chanwoo Choi,
Sylwester Nawrocki, Krzysztof Kozlowski, Mike Tipton,
Andy Shevchenko, Andy Shevchenko, Geert Uytterhoeven, linux-clk,
linux-kernel
Hi Sam,
On Thu, Oct 7, 2021 at 10:57 AM Sam Protsenko
<semen.protsenko@linaro.org> wrote:
>
> Useful for testing mux clocks. One can write the index of the parent to
> set into clk_parent node, starting from 0. Example
>
> # cd /sys/kernel/debug/clk/mout_peri_bus
> # cat clk_possible_parrents
s/clk_possible_parrents/clk_possible_parents
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v3] clk: Add clk_set_parent debugfs node
2021-10-07 14:17 ` Fabio Estevam
@ 2021-10-07 15:31 ` Sam Protsenko
0 siblings, 0 replies; 4+ messages in thread
From: Sam Protsenko @ 2021-10-07 15:31 UTC (permalink / raw)
To: Fabio Estevam
Cc: Michael Turquette, Stephen Boyd, Russell King, Chanwoo Choi,
Sylwester Nawrocki, Krzysztof Kozlowski, Mike Tipton,
Andy Shevchenko, Andy Shevchenko, Geert Uytterhoeven, linux-clk,
linux-kernel
On Thu, 7 Oct 2021 at 17:17, Fabio Estevam <festevam@gmail.com> wrote:
>
> Hi Sam,
>
> On Thu, Oct 7, 2021 at 10:57 AM Sam Protsenko
> <semen.protsenko@linaro.org> wrote:
> >
> > Useful for testing mux clocks. One can write the index of the parent to
> > set into clk_parent node, starting from 0. Example
> >
> > # cd /sys/kernel/debug/clk/mout_peri_bus
> > # cat clk_possible_parrents
>
> s/clk_possible_parrents/clk_possible_parents
Thanks, will be fixed.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-10-07 15:31 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-07 13:56 [PATCH v3] clk: Add clk_set_parent debugfs node Sam Protsenko
2021-10-07 14:04 ` Sam Protsenko
2021-10-07 14:17 ` Fabio Estevam
2021-10-07 15:31 ` Sam Protsenko
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.