On 11/06, Tero Kristo wrote: > On 03/11/17 17:27, Stephen Boyd wrote: > >On 10/30, Tero Kristo wrote: > >>+ } > >>+ > >>+ if (num_args && clkctrl_nodes_missing) > >>+ continue; > >>+ > >>+ node = of_find_node_by_name(NULL, buf); > >>+ if (num_args) > >>+ node = of_find_node_by_name(node, "clk"); > >> clkspec.np = node; > >>+ clkspec.args_count = num_args; > >>+ for (i = 0; i < num_args; i++) { > >>+ ret = kstrtoint(tags[i], i ? 10 : 16, clkspec.args + i); > > > >Is this some sort of DT checker? > > Not really, it just parses a string of format: > > <clkctrl_provider>:<hex-offset>:<bit-offset>. > > ... so that it can find a matching clock. Needed temporarily for > omap_hwmod backwards compatibility. This piece can be removed once > hwmod is gone. > > I can add some comment along these lines to the code if you want. > No need. Thanks. -- 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 linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html