On Thu, Mar 17, 2011 at 09:54:35AM +0800, Shawn Guo wrote:
On Tue, Mar 15, 2011 at 01:03:42AM -0600, Grant Likely wrote:
+#ifdef CONFIG_OF +static struct clk *mx5_dt_clk_get(struct device_node *np,
const char *output_id, void *data)
+{
- return data;
+}
+static __init void mx5_dt_scan_clks(void) +{
- struct device_node *node;
- struct clk *clk;
- const char *id;
- int rc;
- for_each_compatible_node(node, NULL, "clock") {
id = of_get_property(node, "clock-outputs", NULL);
if (!id)
continue;
clk = clk_get_sys(id, NULL);
if (IS_ERR(clk))
continue;
rc = of_clk_add_provider(node, mx5_dt_clk_get, clk);
if (rc)
pr_err("error adding fixed clk %s\n", node->name);
- }
+}
+void __init mx5_clk_dt_init(void) +{
- mx5_dt_scan_clks();
+} +#endif
Nitpick: Would it make sense for these 3 functions to be in a separate .c file?
Sorry for that I'm late on this. It will not make much sense to do so if considering that dynamic dt clock codes have to be put in clock-mx51-mx53.c anyway, since they are referring to existing enable/disable/get_rate/set_rate/... functions.
Okay.
g.