[PATCH RFC 0/3] first pass converting omap4 clock data to DT

Tero Kristo
2013-06-04T13:28:54Z

Hi Mike,

On Mon, 2013-06-03 at 23:39 -0700, Mike Turquette wrote:
> This is a very incomplete conversion of a handful of OMAP4 PRCM clocks
> from the statically defined clock data in
> arch/arm/mach-omap2/cclock44xx_data.c to a new dts file in
> arch/arm/boot/dts/omap4-clocks.dtsi.

I gave a quick try for this set and it seems to be working nicely, so
for at least me this approach is fine. Feel free to add tested-by me if
you like (for the basic clock bindings also.) Can't comment much on the
DT stuff though as my experience is rather limited on it.

> I am not a DT expert so many of the choices here may be quite disgusting
> to look at, or may be on the right path. In particular I simply include
> the new omap4-clocks.dtsi from omap4.dtsi, which feels a bit kludgey.
> Also this series depends on the basic clock bindings RFC I posted
> earlier today[1].
>
> I actually have an omap4-clock.dtsi file with many more clocks converted
> to the mux-clock, divider-clock, fixed-clock, fixed-factor and
> (unpublished) gate-clock bindings in my local repo, but it is not
> currently booting. I wanted to get this early preview out regardless.

Any way I can get access to your local tree? Can you push a branch
somewhere? I could provide you debugging support on this at least, as I
spent quite a bit of time earlier moving the clock data under
drivers/clk... I guess you are probably seeing similar issues I faced.

-Tero

>
> [1] http://article.gmane.org/gmane.linux.kernel/1501216
>
> Mike Turquette (3):
> clk: omap: introduce clock driver
> ARM: dts: omap4 clock data
> ARM: omap4: register DT clocks & remove old data
>
> arch/arm/boot/dts/omap4-clocks.dtsi | 128 ++++++++++++++++++++++++++++++++++
> arch/arm/boot/dts/omap4.dtsi | 2 +
> arch/arm/mach-omap2/cclock44xx_data.c | 54 ++------------
> drivers/clk/Makefile | 1 +
> drivers/clk/omap/Makefile | 1 +
> drivers/clk/omap/clk.c | 55 +++++++++++++++
> include/linux/clk/omap.h | 24 +++++++
> 7 files changed, 217 insertions(+), 48 deletions(-)
> create mode 100644 arch/arm/boot/dts/omap4-clocks.dtsi
> create mode 100644 drivers/clk/omap/Makefile
> create mode 100644 drivers/clk/omap/clk.c
> create mode 100644 include/linux/clk/omap.h
>