[PATCH 01/11] clk: Add clk_name_* functions
Sascha Hauer
s.hauer at pengutronix.de
Tue Jun 15 07:16:31 PDT 2021
At some places a clk name may be known without having a struct clk *
directly. Add some convenience functions to handle this situation and
use them in the clk commands.
Signed-off-by: Sascha Hauer <s.hauer at pengutronix.de>
---
commands/clk.c | 18 ++----------------
drivers/clk/clk.c | 22 ++++++++++++++++++++++
include/linux/clk.h | 4 ++++
3 files changed, 28 insertions(+), 16 deletions(-)
diff --git a/commands/clk.c b/commands/clk.c
index 649a0a7cb2..b63c82455e 100644
--- a/commands/clk.c
+++ b/commands/clk.c
@@ -54,19 +54,14 @@ BAREBOX_CMD_END
static int do_clk_set_rate(int argc, char *argv[])
{
- struct clk *clk;
unsigned long rate;
if (argc != 3)
return COMMAND_ERROR_USAGE;
- clk = clk_lookup(argv[1]);
- if (IS_ERR(clk))
- return PTR_ERR(clk);
-
rate = simple_strtoul(argv[2], NULL, 0);
- return clk_set_rate(clk, rate);
+ return clk_name_set_rate(argv[1], rate);
}
BAREBOX_CMD_HELP_START(clk_set_rate)
@@ -182,19 +177,10 @@ BAREBOX_CMD_END
static int do_clk_set_parent(int argc, char *argv[])
{
- struct clk *clk, *parent;
-
if (argc != 3)
return COMMAND_ERROR_USAGE;
- clk = clk_lookup(argv[1]);
- if (IS_ERR(clk))
- return PTR_ERR(clk);
- parent = clk_lookup(argv[2]);
- if (IS_ERR(parent))
- return PTR_ERR(parent);
-
- return clk_set_parent(clk, parent);
+ return clk_name_set_parent(argv[1], argv[2]);
}
BAREBOX_CMD_START(clk_set_parent)
diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c
index ba726c342c..8932cfe54b 100644
--- a/drivers/clk/clk.c
+++ b/drivers/clk/clk.c
@@ -525,6 +525,28 @@ int clk_parent_set_rate(struct clk_hw *hw, unsigned long rate,
return clk_set_rate(clk_get_parent(clk), rate);
}
+int clk_name_set_parent(const char *clkname, const char *clkparentname)
+{
+ struct clk *clk = clk_lookup(clkname);
+ struct clk *parent = clk_lookup(clkparentname);
+
+ if (IS_ERR(clk))
+ return -ENOENT;
+ if (IS_ERR(parent))
+ return -ENOENT;
+ return clk_set_parent(clk, parent);
+}
+
+int clk_name_set_rate(const char *clkname, unsigned long rate)
+{
+ struct clk *clk = clk_lookup(clkname);
+
+ if (IS_ERR(clk))
+ return -ENOENT;
+
+ return clk_set_rate(clk, rate);
+}
+
#if defined(CONFIG_COMMON_CLK_OF_PROVIDER)
/**
* struct of_clk_provider - Clock provider registration structure
diff --git a/include/linux/clk.h b/include/linux/clk.h
index e54acdd918..b7240189b3 100644
--- a/include/linux/clk.h
+++ b/include/linux/clk.h
@@ -722,6 +722,10 @@ static inline unsigned int clk_get_num_parents(const struct clk *hw)
{
return hw->num_parents;
}
+
+int clk_name_set_parent(const char *clkname, const char *clkparentname);
+int clk_name_set_rate(const char *clkname, unsigned long rate);
+
#else
--
2.29.2
More information about the barebox
mailing list