Quoting Antony Pavlov (2016-02-09 00:13:47) > +static void __init ar9130_init(struct device_node *np) > +{ > + int retval; > + struct ath79_cblk *cblk; > + > + cblk = ath79_cblk_new(ar9331_clocks, ARRAY_SIZE(ar9331_clocks), np); > + if (!cblk) { > + pr_err("%s: failed to initialise clk block\n", __func__); > + return; > + } > + > + retval = ath79_cblk_register_clocks(cblk); > + if (retval) > + pr_err("%s: failed to register clocks\n", __func__); > +} > +CLK_OF_DECLARE(ar933x_clk, "qca,ar9330-pll", ar9130_init); Is there any reason this isn't a platform_driver? Thanks, Mike