Hello Lee jones, > -----Original Message----- > From: Lee Jones [mailto:lee.jones@xxxxxxxxxx] > Sent: Monday, June 10, 2013 3:36 PM > To: J, KEERTHY > Cc: Cousson, Benoit; devicetree-discuss@xxxxxxxxxxxxxxxx; linux- > omap@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; > ldewangan@xxxxxxxxxx; grant.likely@xxxxxxxxxxxx; swarren@xxxxxxxxxxxxx; > swarren@xxxxxxxxxx; sameo@xxxxxxxxxxxxxxx; gg@xxxxxxxxxxxxxxx > Subject: Re: [PATCH v2 2/2] ARM: dts: OMAP5: add palmas node and omap > specific palmas regulator properties > > On Mon, 10 Jun 2013, J Keerthy wrote: > > > Add palmas node and omap specific palmas regulator properties. > > > > Signed-off-by: J Keerthy <j-keerthy@xxxxxx> > > --- > > arch/arm/boot/dts/omap5-uevm.dts | 147 > > ++++++++++++++++++++++++++++++++++++++ > > 1 files changed, 147 insertions(+), 0 deletions(-) > > > > diff --git a/arch/arm/boot/dts/omap5-uevm.dts > > b/arch/arm/boot/dts/omap5-uevm.dts > > index 927db1e..ffbcc3f 100644 > > --- a/arch/arm/boot/dts/omap5-uevm.dts > > +++ b/arch/arm/boot/dts/omap5-uevm.dts > > @@ -8,6 +8,8 @@ > > /dts-v1/; > > > > #include "omap5.dtsi" > > +#include <dt-bindings/interrupt-controller/irq.h> > > +#include <dt-bindings/interrupt-controller/arm-gic.h> > > > > / { > > model = "TI OMAP5 uEVM board"; > > @@ -254,6 +256,151 @@ > > pinctrl-0 = <&i2c1_pins>; > > > > clock-frequency = <400000>; > > + > > + palmas: palmas@48 { > > + reg = <0x48>; > > + /* SPI = 0, IRQ# = 7, active high level-sensitive */ > > I still think this is superfluous, especially now you're using the > defines which basically say the same thing. I retained the whole comment as it was needed to specify IRQ# as 7 And thought it completely described the below interrupt property. I can knock it off if it is totally unnecessary. If there are no further comments. Can I add a Reviewed-by? > > > + interrupts = <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>; /* IRQ_SYS_1N > */ > > + interrupt-parent = <&gic>; > > + }; > > + > > +}; > > + > > +#include "palmas.dtsi" > > + > > +&palmas { > > + palmas_pmic { > > + ti,ldo6-vibrator; > > + > > + regulators { > > + smps123_reg: smps123 { > > + regulator-min-microvolt = < 600000>; > > + regulator-max-microvolt = <1500000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + smps45_reg: smps45 { > > + regulator-min-microvolt = < 600000>; > > + regulator-max-microvolt = <1310000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + smps6_reg: smps6 { > > + regulator-min-microvolt = <1200000>; > > + regulator-max-microvolt = <1200000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + smps7_reg: smps7 { > > + regulator-min-microvolt = <1800000>; > > + regulator-max-microvolt = <1800000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + smps8_reg: smps8 { > > + regulator-min-microvolt = < 600000>; > > + regulator-max-microvolt = <1310000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + smps9_reg: smps9 { > > + regulator-min-microvolt = <2100000>; > > + regulator-max-microvolt = <2100000>; > > + regulator-always-on; > > + regulator-boot-on; > > + ti,smps-range = <0x80>; > > + }; > > + > > + smps10_reg: smps10 { > > + regulator-min-microvolt = <5000000>; > > + regulator-max-microvolt = <5000000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + ldo1_reg: ldo1 { > > + regulator-min-microvolt = <2800000>; > > + regulator-max-microvolt = <2800000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + ldo2_reg: ldo2 { > > + regulator-min-microvolt = <2900000>; > > + regulator-max-microvolt = <2900000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + ldo3_reg: ldo3 { > > + regulator-min-microvolt = <3000000>; > > + regulator-max-microvolt = <3000000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + ldo4_reg: ldo4 { > > + regulator-min-microvolt = <2200000>; > > + regulator-max-microvolt = <2200000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + ldo5_reg: ldo5 { > > + regulator-min-microvolt = <1800000>; > > + regulator-max-microvolt = <1800000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + ldo6_reg: ldo6 { > > + regulator-min-microvolt = <1500000>; > > + regulator-max-microvolt = <1500000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + ldo7_reg: ldo7 { > > + regulator-min-microvolt = <1500000>; > > + regulator-max-microvolt = <1500000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + ldo8_reg: ldo8 { > > + regulator-min-microvolt = <1500000>; > > + regulator-max-microvolt = <1500000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + ldo9_reg: ldo9 { > > + regulator-min-microvolt = <1800000>; > > + regulator-max-microvolt = <3300000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + ldoln_reg: ldoln { > > + regulator-min-microvolt = <1800000>; > > + regulator-max-microvolt = <1800000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + ldousb_reg: ldousb { > > + regulator-min-microvolt = <3250000>; > > + regulator-max-microvolt = <3250000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + }; > > + }; > > }; > > > > &i2c5 { > > -- > Lee Jones > Linaro ST-Ericsson Landing Team Lead > Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook > | Twitter | Blog Regards, Keerthy ��.n��������+%������w��{.n�����{�������ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f