On Tue, Jan 07, 2014 at 10:18:15AM +0000, Balaji T K wrote:
> On Monday 06 January 2014 11:49 PM, Mark Rutland wrote:
> > On Fri, Dec 20, 2013 at 05:35:53PM +0000, Balaji T K wrote:
> >> Add pbias regulator node as a child of system control
> >> module - syscon.
> >>
> >> Signed-off-by: Balaji T K <[email protected]>
> >> ---
> >> arch/arm/boot/dts/dra7.dtsi | 18 ++++++++++++++++++
> >> arch/arm/boot/dts/omap2430.dtsi | 18 ++++++++++++++++++
> >> arch/arm/boot/dts/omap3.dtsi | 18 ++++++++++++++++++
> >> arch/arm/boot/dts/omap4.dtsi | 18 ++++++++++++++++++
> >> arch/arm/boot/dts/omap5.dtsi | 18 ++++++++++++++++++
> >> 5 files changed, 90 insertions(+), 0 deletions(-)
> >>
> >> diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
> >> index d0df4c4..4e68df1 100644
> >> --- a/arch/arm/boot/dts/dra7.dtsi
> >> +++ b/arch/arm/boot/dts/dra7.dtsi
> >> @@ -110,6 +110,23 @@
> >> ti,hwmods = "counter_32k";
> >> };
> >>
> >> + dra7_ctrl_general: tisyscon@4a002e00 {
> >> + compatible = "ti,control-syscon", "syscon",
> >> "simple-bus";
> >
> > Please, don't use simple-bus like that. The components below this node
> > depend on it. It is _NOT_ a simple bus. Make the ti,control-syscon
> > driver probe it's children.
>
> Hi Mark,
>
> Actually ti,control-syscon driver does not exist, so I can remove it,
> and simple-bus is needed for child creation.
This still shows up as a syscon node, with a reg property, and syscon is
not an extension of simple-bus.
There are properties in the parent node that children depend on, and
that makes me wary of describing it as a simple bus. I'd expect to be
able to move child nodes out of a simple-bus if ranges provided an
idmap, and I can't do that here.
Thanks,
Mark.
>
> >
> >> + reg = <0x4a002e00 0x7c>;
> >> + #address-cells = <1>;
> >> + #size-cells = <1>;
> >> + ranges;
> >> + pbias_regulator: pbias_regulator {
> >> + compatible = "ti,pbias-omap";
> >> + reg = <0 0x4>;
> >> + pbias_mmc_reg: pbias_mmc_omap5 {
> >> + regulator-name = "pbias_mmc_omap5";
> >> + regulator-min-microvolt = <1800000>;
> >> + regulator-max-microvolt = <3000000>;
> >> + };
> >> + };
> >> + };
> >> +
> >
> > Thanks,
> > Mark.
> >
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
