02 Mar, 2016
2 commits
-
drivers/clk/ti/clk-814x.c:34:12: warning: symbol 'dm814x_adpll_early_init' was not declared. Should it be static?
drivers/clk/ti/clk-814x.c:58:12: warning: symbol 'dm814x_adpll_enable_init_clocks' was not declared. Should it be static?
drivers/clk/ti/adpll.c:465 ti_adpll_recalc_rate() warn: should '__readw(d->regs + 20) << 18' be a 64 bit type?
drivers/clk/ti/adpll.c:945 ti_adpll_probe() error: we previously assumed 'd->clocks' could be null (see line 921)The last one looks like a real bug because we don't return an
error on allocation failure.Cc: Tero Kristo
Tested-by: Tony Lindgren
Signed-off-by: Stephen Boyd -
On dm814x we have 13 ADPLLs with 3 to 4 outputs on each. The
ADPLLs have several dividers and muxes controlled by a shared
control register for each PLL.Note that for the clocks to work as device drivers for booting on
dm814x, this patch depends on "ARM: OMAP2+: Change core_initcall
levels to postcore_initcall" that has already been merged.Also note that this patch does not implement clk_set_rate for the
PLL, that will be posted later on when available.Cc: Stephen Boyd
Acked-by: Tero Kristo
Signed-off-by: Tony Lindgren
Signed-off-by: Michael Turquette
