On Sat, Dec 29, 2012 at 10:04 PM, Ezequiel Garcia <[email protected]> wrote: > These regulators are common to igep0020 and igep0030 boards > and are needed by smsc911x controller. > > Cc: Benoît Cousson <[email protected]> > Cc: Tony Lindgren <[email protected]> > Cc: Enric Balletbo i Serra <[email protected]> > Cc: Javier Martinez Canillas <[email protected]> > Signed-off-by: Ezequiel Garcia <[email protected]> > --- > arch/arm/boot/dts/omap3-igep.dtsi | 12 ++++++++++++ > 1 files changed, 12 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/boot/dts/omap3-igep.dtsi > b/arch/arm/boot/dts/omap3-igep.dtsi > index dd01c46..91a42ce 100644 > --- a/arch/arm/boot/dts/omap3-igep.dtsi > +++ b/arch/arm/boot/dts/omap3-igep.dtsi > @@ -24,6 +24,18 @@ > ti,mcbsp = <&mcbsp2>; > ti,codec = <&twl_audio>; > }; > + > + vddvario: regulator-vddvario { > + compatible = "regulator-fixed"; > + regulator-name = "vddvario"; > + regulator-always-on; > + }; > + > + vdd33a: regulator-vdd33a { > + compatible = "regulator-fixed"; > + regulator-name = "vdd33a"; > + regulator-always-on; > + }; > }; > > &omap3_pmx_core {
Hi Ezequiel, This shouldn't be defined on the generic omap3-igep.dtsi but on omap3-igep0020.dts since the IGEP COM Module doesn't have the LAN SMSC9221 chip which is the one using these regulators. Otherwise, the patch looks good to me and I think it has to be part of the series that add SMSC9221 support for IGEPv2. Thanks a lot and best regards, Javier _______________________________________________ devicetree-discuss mailing list [email protected] https://lists.ozlabs.org/listinfo/devicetree-discuss
