Re: [PATCH v7 5/7] ARM: dts: add pbias dt node

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 <balajitk@xxxxxx>
> >> ---
> >>   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 linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux