On 10/05/2019 11:23:40+0000, Claudiu.Beznea@xxxxxxxxxxxxx wrote: > From: Claudiu Beznea <claudiu.beznea@xxxxxxxxxxxxx> > > Add support for SAM9X60's slow clock. > > Signed-off-by: Claudiu Beznea <claudiu.beznea@xxxxxxxxxxxxx> Acked-by: Alexandre Belloni <alexandre.belloni@xxxxxxxxxxx> > --- > drivers/clk/at91/sckc.c | 74 +++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 74 insertions(+) > > diff --git a/drivers/clk/at91/sckc.c b/drivers/clk/at91/sckc.c > index 2a4ac548de80..2c410f41b413 100644 > --- a/drivers/clk/at91/sckc.c > +++ b/drivers/clk/at91/sckc.c > @@ -415,6 +415,80 @@ static void __init of_sama5d3_sckc_setup(struct device_node *np) > CLK_OF_DECLARE(sama5d3_clk_sckc, "atmel,sama5d3-sckc", > of_sama5d3_sckc_setup); > > +static const struct clk_slow_bits at91sam9x60_bits = { > + .cr_osc32en = BIT(1), > + .cr_osc32byp = BIT(2), > + .cr_oscsel = BIT(24), > +}; > + > +static void __init of_sam9x60_sckc_setup(struct device_node *np) > +{ > + void __iomem *regbase = of_iomap(np, 0); > + struct clk_hw_onecell_data *clk_data; > + struct clk_hw *slow_rc, *slow_osc; > + const char *xtal_name; > + const char *parent_names[2] = { "slow_rc_osc", "slow_osc" }; > + bool bypass; > + int ret; > + > + if (!regbase) > + return; > + > + slow_rc = clk_hw_register_fixed_rate(NULL, parent_names[0], NULL, 0, > + 32768); > + if (IS_ERR(slow_rc)) > + return; > + > + xtal_name = of_clk_get_parent_name(np, 0); > + if (!xtal_name) > + goto unregister_slow_rc; > + > + bypass = of_property_read_bool(np, "atmel,osc-bypass"); > + slow_osc = at91_clk_register_slow_osc(regbase, parent_names[1], > + xtal_name, 5000000, bypass, > + &at91sam9x60_bits); > + if (IS_ERR(slow_osc)) > + goto unregister_slow_rc; > + > + clk_data = kzalloc(sizeof(*clk_data) + (2 * sizeof(struct clk_hw *)), > + GFP_KERNEL); > + if (!clk_data) > + goto unregister_slow_osc; > + > + /* MD_SLCK and TD_SLCK. */ > + clk_data->num = 2; > + clk_data->hws[0] = clk_hw_register_fixed_rate(NULL, "md_slck", > + parent_names[0], > + 0, 32768); > + if (IS_ERR(clk_data->hws[0])) > + goto clk_data_free; > + > + clk_data->hws[1] = at91_clk_register_sam9x5_slow(regbase, "td_slck", > + parent_names, 2, > + &at91sam9x60_bits); > + if (IS_ERR(clk_data->hws[1])) > + goto unregister_md_slck; > + > + ret = of_clk_add_hw_provider(np, of_clk_hw_onecell_get, clk_data); > + if (WARN_ON(ret)) > + goto unregister_td_slck; > + > + return; > + > +unregister_td_slck: > + clk_hw_unregister(clk_data->hws[1]); > +unregister_md_slck: > + clk_hw_unregister(clk_data->hws[0]); > +clk_data_free: > + kfree(clk_data); > +unregister_slow_osc: > + clk_hw_unregister(slow_osc); > +unregister_slow_rc: > + clk_hw_unregister(slow_rc); > +} > +CLK_OF_DECLARE(sam9x60_clk_sckc, "microchip,sam9x60-sckc", > + of_sam9x60_sckc_setup); > + > static int clk_sama5d4_slow_osc_prepare(struct clk_hw *hw) > { > struct clk_sama5d4_slow_osc *osc = to_clk_sama5d4_slow_osc(hw); > -- > 2.7.4 > -- Alexandre Belloni, Bootlin Embedded Linux and Kernel engineering https://bootlin.com