[PATCH RFC 2/3] ARM: dts: omap4 clock data

Tony Lindgren
2013-06-04T19:37:45Z

* Mike Turquette <mturquette@linaro.org> [130604 11:40]:
> Quoting Tony Lindgren (2013-06-04 07:55:43)
> > * Mike Turquette <mturquette@linaro.org> [130603 23:45]:
> > > This is a first pass at creating a unique node for each clock in the
> > > OMAP4 power, reset and & clock manager (PRCM). So far I have only
> > > converted mux clocks & fixed-rate clocks, which coexist with the current
> > > clock data in the kernel. The rest needs to be done but better to
> > > publish early and often to see what others think of this approach.
> >
> > > +/* FIXME need to print the address directly */
> > > +/*
> > > +#include "../../mach-omap2/prm44xx.h"
> > > +#include "../../mach-omap2/cm2_44xx.h"
> > > +#include "../../mach-omap2/cm1_44xx.h"
> > > +*/
> >
> > I don't think you're using the above includes any longer
> > in this file?
> >
>
> Correct. I actually spotted this before emailing the patches out at
> midnight, but by then I didn't care to fix it. The benefits of marking
> a patch as "RFC" ;-)

:)

> I had a mind to use the new preprocessor capabilities of dtc for the
> PRCM bit masks and shift values, but instead I ended up using the raw
> hex values. I've actually come to prefer using the raw hex values for
> DT, which I think makes more sense for a description of the hardware
> which is not tied to any Linux implementation.

Agreed, especially if the value is only used once.

Regards,

Tony