On Wednesday 14 August 2013 06:18 PM, Nishanth Menon wrote: > Hi Rajendra, > > On Tue, Jul 23, 2013 at 1:24 AM, Rajendra Nayak <[email protected]> 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?
you mean look for anything with optional,*? because the role names would change. > >> + 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
