Subject: [PATCH RFC v1 1/3] clk: Add function to parse an arbitrary clocks list property

From: [email-masked] (Mike Turquette)
Date: Sun, 23 Feb 2014 16:43:35 -0800

Quoting Sylwester Nawrocki (2014-02-19 08:58:42)
> The of_clk_get_list_entry() function is like of_clk_get() except it allows
> to pass name of a DT property containing list of phandles and clock specifiers.
> For of_clk_get() it has been hard coded to "clocks".
> 
> Signed-off-by: Sylwester Nawrocki <[email-masked]>
> Acked-by: Kyungmin Park <[email-masked]>
> ---
>  drivers/clk/clk.h    |    3 +++
>  drivers/clk/clkdev.c |   25 +++++++++++++++++++++----
>  2 files changed, 24 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/clk/clk.h b/drivers/clk/clk.h
> index 795cc9f..dd9def1 100644
> --- a/drivers/clk/clk.h
> +++ b/drivers/clk/clk.h
> @@ -10,7 +10,10 @@
>   */
> 
>  #if defined(CONFIG_OF) && defined(CONFIG_COMMON_CLK)
> +
>  struct clk *__of_clk_get_from_provider(struct of_phandle_args *clkspec);
>  void of_clk_lock(void);
>  void of_clk_unlock(void);
> +struct clk *of_clk_get_list_entry(struct device_node *np,
> +                                 const char *list_name, int index);
>  #endif
> diff --git a/drivers/clk/clkdev.c b/drivers/clk/clkdev.c
> index a360b2e..465662e 100644
> --- a/drivers/clk/clkdev.c
> +++ b/drivers/clk/clkdev.c
> @@ -27,17 +27,28 @@ static LIST_HEAD(clocks);
>  static DEFINE_MUTEX(clocks_mutex);
> 
>  #if defined(CONFIG_OF) && defined(CONFIG_COMMON_CLK)
> -struct clk *of_clk_get(struct device_node *np, int index)
> +/**
> + * of_clk_get_list_entry() - Parse and lookup a clock referenced by a device node
> + * @np: pointer to clock consumer node
> + * @list_name: name of the clock list property
> + * @index: index to the clock list
> + *
> + * This function parses the @list_name property and together with @index
> + * value indicating an entry of the list uses it to look up the struct clk
> + * from the registered list of clock providers.
> + */
> +struct clk *of_clk_get_list_entry(struct device_node *np,
> +                                 const char *list_name, int index)

Bikeshed alert: how about of_clk_get_by_property or of_clk_get_by_prop?

No strong opinion on the topic though.

Regards,
Mike

>  {
>         struct of_phandle_args clkspec;
>         struct clk *clk;
>         int rc;
> 
> -       if (index < 0)
> +       if (index < 0 || !list_name)
>                 return ERR_PTR(-EINVAL);
> 
> -       rc = of_parse_phandle_with_args(np, "clocks", "#clock-cells", index,
> -                                       &clkspec);
> +       rc = of_parse_phandle_with_args(np, list_name, "#clock-cells",
> +                                       index, &clkspec);
>         if (rc)
>                 return ERR_PTR(rc);
> 
> @@ -51,6 +62,12 @@ struct clk *of_clk_get(struct device_node *np, int index)
>         of_node_put(clkspec.np);
>         return clk;
>  }
> +EXPORT_SYMBOL(of_clk_get_list_entry);
> +
> +struct clk *of_clk_get(struct device_node *np, int index)
> +{
> +       return of_clk_get_list_entry(np, "clocks", index);
> +}
>  EXPORT_SYMBOL(of_clk_get);
> 
>  /**
> --
> 1.7.9.5
>