On 02/04/15 15:33, Ray Jui wrote: > > On 2/4/2015 3:13 PM, Stephen Boyd wrote: >> On 02/03/15 10:33, Ray Jui wrote: >>> +/* >>> + * Get the clock rate based on name >>> + */ >>> +static unsigned long __get_rate(const char *clk_name) >>> +{ >>> + struct clk *clk; >>> + >>> + clk = __clk_lookup(clk_name); >>> + if (!clk) { >>> + pr_err("%s: unable to find clock by name: %s\n", __func__, >>> + clk_name); >>> + return 0; >>> + } >>> + >>> + return clk_get_rate(clk); >>> +} >>> + >> This looks like something we should be providing in the core framework. >> Care to make it into an of_clk_get_parent_rate() API? >> > I would love to! Note __get_rate here is really getting the clock rate > with a name provided. Can you be more specific on what you want? If I'm > not mistaken, what you really want is this? > > unsigned long of_clk_get_parent_rate(struct device_node *np, int index); Yes that looks good. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html