On Wed, May 07, 2014 at 07:12:30PM +0200, Boris BREZILLON wrote: > > On 29/04/2014 01:40, Maxime Ripard wrote: > > On Mon, Apr 28, 2014 at 04:58:48PM +0200, Boris BREZILLON wrote: > >> The PRCM (Power/Reset/Clock Management) unit provides several clock > >> devices: > >> - AR100 clk: used to clock the Power Management co-processor > >> - AHB0 clk: used to clock the AHB0 bus > >> - APB0 clk and gates: used to clk > > Used to clk? > "Used to clk peripherals connected on the APB0 bus" > > I'll add the missing words in the next version :-). > > > > [...] > > Ditto. > > > > And you'll probably want to use devm_ioremap_resource when you'll have > > a single clock for the AR100. > > Absolutely. > > > > >> + > >> + clk_parent = of_clk_get_parent_name(np, 0); > >> + if (!clk_parent) > >> + return -EINVAL; > [...] > >> + > >> +static struct platform_driver sun6i_a31_prcm_clk_driver = { > >> + .driver = { > >> + .name = "sun6i-a31-prcm-clk", > >> + .owner = THIS_MODULE, > >> + .of_match_table = sun6i_a31_prcm_clk_dt_ids, > >> + }, > >> + .probe = sun6i_a31_prcm_clk_probe, > > You're not calling the of_clk_del_provider, and you should probably > > unregister your clocks too. > > This driver cannot be compiled as a module, and as a result the probed > clks will never be removed. > > Do you really want to support clk removal for this HW block ? Hmm, no, then it's fine. Thanks! Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
Attachment:
signature.asc
Description: Digital signature