[PATCH v3 2/4] ARM: imx: add initial support for MVF600

Shawn Guo shawn.guo at linaro.org
Fri May 17 09:06:05 EDT 2013


On Fri, May 17, 2013 at 02:29:52PM +0200, Arnd Bergmann wrote:
> > > Actually I think you should move that driver to drivers/clk and use
> > > of_clk_init(NULL) to initialize it.
> > > 
> > The mvf600 clock driver uses a lot of base clk support from mach-imx,
> > and can not be moved into drivers/clk as a single driver.  Right now, in
> > IMX clock drivers, we call of_clk_init() to only register fixed rate
> > clocks, since all the other clocks are not represented in device tree.
> 
> What are your plans for this in the long run?
> 
We can move all the IMX clock drivers into drivers/clk at some point
when necessary.  But I do not have a plan to register all the clocks
by merely calling of_clk_init(), because doing that would mean we have
to represent all these clocks in device tree.  For imx6q example, it's
about 200 ~ 300 nodes addition to DTB.  Device tree maintainers are
against to the idea.  They are perfectly fine with having clock driver
in kernel to represent/register these SoC internal clocks to clk
framework.

Shawn




More information about the linux-arm-kernel mailing list