2023-01-17 06:27:17

by Oleksij Rempel

Subject: [PATCH v2 14/19] clk: imx: add imx_obtain_fixed_of_clock()

Add imx_obtain_fixed_of_clock() to optionally add clock not configured in
the devicetree.

Signed-off-by: Oleksij Rempel <[email protected]>
---
drivers/clk/imx/clk.c | 14 ++++++++++++++
drivers/clk/imx/clk.h | 3 +++
2 files changed, 17 insertions(+)

diff --git a/drivers/clk/imx/clk.c b/drivers/clk/imx/clk.c
index b636cc099d96..5f1f729008ee 100644
--- a/drivers/clk/imx/clk.c
+++ b/drivers/clk/imx/clk.c
@@ -110,6 +110,20 @@ struct clk_hw *imx_obtain_fixed_clock_hw(
return __clk_get_hw(clk);
}

+struct clk_hw *imx_obtain_fixed_of_clock(struct device_node *np,
+ const char *name, unsigned long rate)
+{
+ struct clk *clk = of_clk_get_by_name(np, name);
+ struct clk_hw *hw;
+
+ if (IS_ERR(clk))
+ hw = imx_obtain_fixed_clock_hw(name, rate);
+ else
+ hw = __clk_get_hw(clk);
+
+ return hw;
+}
+
struct clk_hw *imx_get_clk_hw_by_name(struct device_node *np, const char *name)
{
struct clk *clk;
diff --git a/drivers/clk/imx/clk.h b/drivers/clk/imx/clk.h
index 801213109697..f0a24cd54d1c 100644
--- a/drivers/clk/imx/clk.h
+++ b/drivers/clk/imx/clk.h
@@ -288,6 +288,9 @@ struct clk * imx_obtain_fixed_clock(
struct clk_hw *imx_obtain_fixed_clock_hw(
const char *name, unsigned long rate);

+struct clk_hw *imx_obtain_fixed_of_clock(struct device_node *np,
+ const char *name, unsigned long rate);
+
struct clk_hw *imx_get_clk_hw_by_name(struct device_node *np, const char *name);

struct clk_hw *imx_clk_hw_gate_exclusive(const char *name, const char *parent,
--
2.30.2

2023-01-29 17:33:32

by Abel Vesa

Subject: Re: [PATCH v2 14/19] clk: imx: add imx_obtain_fixed_of_clock()

On 23-01-17 07:14:48, Oleksij Rempel wrote:
> Add imx_obtain_fixed_of_clock() to optionally add clock not configured in
> the devicetree.
>
> Signed-off-by: Oleksij Rempel <[email protected]>

Reviewed-by: Abel Vesa <[email protected]>

> ---
> drivers/clk/imx/clk.c | 14 ++++++++++++++
> drivers/clk/imx/clk.h | 3 +++
> 2 files changed, 17 insertions(+)
>
> diff --git a/drivers/clk/imx/clk.c b/drivers/clk/imx/clk.c
> index b636cc099d96..5f1f729008ee 100644
> --- a/drivers/clk/imx/clk.c
> +++ b/drivers/clk/imx/clk.c
> @@ -110,6 +110,20 @@ struct clk_hw *imx_obtain_fixed_clock_hw(
> return __clk_get_hw(clk);
> }
>
> +struct clk_hw *imx_obtain_fixed_of_clock(struct device_node *np,
> + const char *name, unsigned long rate)
> +{
> + struct clk *clk = of_clk_get_by_name(np, name);
> + struct clk_hw *hw;
> +
> + if (IS_ERR(clk))
> + hw = imx_obtain_fixed_clock_hw(name, rate);
> + else
> + hw = __clk_get_hw(clk);
> +
> + return hw;
> +}
> +
> struct clk_hw *imx_get_clk_hw_by_name(struct device_node *np, const char *name)
> {
> struct clk *clk;
> diff --git a/drivers/clk/imx/clk.h b/drivers/clk/imx/clk.h
> index 801213109697..f0a24cd54d1c 100644
> --- a/drivers/clk/imx/clk.h
> +++ b/drivers/clk/imx/clk.h
> @@ -288,6 +288,9 @@ struct clk * imx_obtain_fixed_clock(
> struct clk_hw *imx_obtain_fixed_clock_hw(
> const char *name, unsigned long rate);
>
> +struct clk_hw *imx_obtain_fixed_of_clock(struct device_node *np,
> + const char *name, unsigned long rate);
> +
> struct clk_hw *imx_get_clk_hw_by_name(struct device_node *np, const char *name);
>
> struct clk_hw *imx_clk_hw_gate_exclusive(const char *name, const char *parent,
> --
> 2.30.2
>