On Sat, Mar 19, 2011 at 02:24:32AM +0800, Shawn Guo wrote:
> With the platform clock support, the 'struct clk' should have been
> associated with device_node->data. So the use of function
> __of_clk_get_from_provider can be eliminated.
>
> Signed-off-by: Shawn Guo <shawn....@linaro.org>
Not really true since a device node describes a clock provider which
may be able to provide more than one clock output.
g.
> ---
> drivers/of/clock.c | 23 ++---------------------
> 1 files changed, 2 insertions(+), 21 deletions(-)
>
> diff --git a/drivers/of/clock.c b/drivers/of/clock.c
> index 7b5ea67..1b34937 100644
> --- a/drivers/of/clock.c
> +++ b/drivers/of/clock.c
> @@ -71,24 +71,6 @@ void of_clk_del_provider(struct device_node *np,
> mutex_unlock(&of_clk_lock);
> }
>
> -static struct clk *__of_clk_get_from_provider(struct device_node *np, const
> char *clk_output)
> -{
> - struct of_clk_provider *provider;
> - struct clk *clk = NULL;
> -
> - /* Check if we have such a provider in our array */
> - mutex_lock(&of_clk_lock);
> - list_for_each_entry(provider, &of_clk_providers, link) {
> - if (provider->node == np)
> - clk = provider->get(np, clk_output, provider->data);
> - if (clk)
> - break;
> - }
> - mutex_unlock(&of_clk_lock);
> -
> - return clk;
> -}
> -
> struct clk *of_clk_get(struct device *dev, const char *id)
> {
> struct device_node *provnode;
> @@ -123,9 +105,8 @@ struct clk *of_clk_get(struct device *dev, const char *id)
> __func__, prop_name, dev->of_node->full_name);
> return NULL;
> }
> - clk = __of_clk_get_from_provider(provnode, prop);
> - if (clk)
> - dev_dbg(dev, "Using clock from %s\n", provnode->full_name);
> +
> + clk = provnode->clk;
>
> of_node_put(provnode);
>
> --
> 1.7.1
>
_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev
