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