Hi Rajendra, On Tue, Jul 23, 2013 at 1:24 AM, Rajendra Nayak <rnayak@xxxxxx> wrote: [..] > diff --git a/arch/arm/mach-omap2/omap_hwmod.c b/arch/arm/mach-omap2/omap_hwmod.c > index 12fa589..e5c804b 100644 > --- a/arch/arm/mach-omap2/omap_hwmod.c > +++ b/arch/arm/mach-omap2/omap_hwmod.c > @@ -805,6 +805,65 @@ static int _init_interface_clks(struct omap_hwmod *oh) > return ret; > } > > +static const char **_parse_opt_clks_dt(struct omap_hwmod *oh, > + struct device_node *np, > + int *opt_clks_cnt) > +{ > + int i, clks_cnt; > + const char *clk_name; > + const char **opt_clk_names; > + > + clks_cnt = of_property_count_strings(np, "clock-names"); > + if (!clks_cnt) > + return NULL; > + > + opt_clk_names = kzalloc(sizeof(char *)*clks_cnt, GFP_KERNEL); > + if (!opt_clk_names) > + return NULL; > + > + for (i = 0; i < clks_cnt; i++) { > + of_property_read_string_index(np, "clock-names", i, &clk_name); > + if (!strcmp(clk_name, "fck")) Could we instead parse for names that are "optional,role_name" instead of assuming anything other than fck is optional clocks? > + continue; > + opt_clks_cnt++; > + opt_clk_names[i] = clk_name; > + } > + return opt_clk_names; > +} > + [...] -- Regards, Nishanth Menon -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html