linux-kernel - RE: [PATCH V2 6/6] ARM: davinci: da850: add tps6507x regulator DT data

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <A4887BF146CD57468F700B415D2F4701583323@DBDE01.ent.ti.com>
Date:	Thu, 7 Feb 2013 10:22:19 +0000
From:	"Vishwanathrao Badarkhe, Manish" <manishv.b@...com>
To:	"Vishwanathrao Badarkhe, Manish" <manishv.b@...com>,
	"devicetree-discuss@...ts.ozlabs.org" 
	<devicetree-discuss@...ts.ozlabs.org>,
	"linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"davinci-linux-open-source@...ux.davincidsp.com" 
	<davinci-linux-open-source@...ux.davincidsp.com>
CC:	"AnilKumar, Chimata" <anilkumar@...com>,
	"tony@...mide.com" <tony@...mide.com>, "hs@...x.de" <hs@...x.de>,
	"broonie@...nsource.wolfsonmicro.com" 
	<broonie@...nsource.wolfsonmicro.com>,
	"sameo@...ux.intel.com" <sameo@...ux.intel.com>,
	"khilman@...prootsystems.com" <khilman@...prootsystems.com>,
	"Nori, Sekhar" <nsekhar@...com>,
	"linus.walleij@...aro.org" <linus.walleij@...aro.org>,
	"linux@....linux.org.uk" <linux@....linux.org.uk>,
	"rob@...dley.net" <rob@...dley.net>,
	"rob.herring@...xeda.com" <rob.herring@...xeda.com>,
	"grant.likely@...retlab.ca" <grant.likely@...retlab.ca>
Subject: RE: [PATCH V2 6/6] ARM: davinci: da850: add tps6507x regulator DT
 data

Hi Mark,

On Tue, Jan 29, 2013 at 13:08:54, Vishwanathrao Badarkhe, Manish wrote:
> Add tps6507x regulator device tree data to da850-evm by adding regulator consumers with tightened constraints and regulator-name.TPS6507x regulator handle can be obtained by using this regulator name.
> Regulator constraints are added as per da850 board file.
> 
> Regulator names are given as per maximum output voltage the regulator is capable to provide.
> for e.g. regulator name for dcdc1 is "VDCDC1_3.3V".
> Also, add tps6507x PMIC I2C slave device under I2C0 node.
> 
> Tested on da850-evm.
> 
> Signed-off-by: Vishwanathrao Badarkhe, Manish <manishv.b@...com>
> ---
> Changes since V1:
>  - No change
> 
> :100644 100644 c9ed683... 58e6961... M	arch/arm/boot/dts/da850-evm.dts
>  arch/arm/boot/dts/da850-evm.dts |   63 +++++++++++++++++++++++++++++++++++++++
>  1 files changed, 63 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts index c9ed683..58e6961 100644
> --- a/arch/arm/boot/dts/da850-evm.dts
> +++ b/arch/arm/boot/dts/da850-evm.dts
> @@ -31,6 +31,10 @@
>  			status = "okay";
>  			pinctrl-names = "default";
>  			pinctrl-0 = <&i2c0_pins>;
> +
> +			tps: tps@48 {
> +				reg = <0x48>;
> +			};
>  		};
>  	};
>  	nand_cs3@...00000 {
> @@ -38,4 +42,63 @@
>  		pinctrl-names = "default";
>  		pinctrl-0 = <&nand_cs3_pins>;
>  	};
> +	vbat: fixedregulator@0 {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vbat";
> +		regulator-min-microvolt = <5000000>;
> +		regulator-max-microvolt = <5000000>;
> +		regulator-boot-on;
> +	};
> +};
> +
> +/include/ "tps6507x.dtsi"
> +
> +&tps {
> +        vdcdc1_2-supply = <&vbat>;
> +        vdcdc3-supply = <&vbat>;
> +        vldo1_2-supply = <&vbat>;
> +
> +        regulators {
> +                vdcdc1_reg: regulator@0 {
> +                        regulator-name = "VDCDC1_3.3V";
> +                        regulator-min-microvolt = <3150000>;
> +                        regulator-max-microvolt = <3450000>;
> +                        regulator-always-on;
> +                        regulator-boot-on;
> +                };
> +
> +                vdcdc2_reg: regulator@1 {
> +                        regulator-name = "VDCDC2_3.3V";
> +                        regulator-min-microvolt = <1710000>;
> +                        regulator-max-microvolt = <3450000>;
> +                        regulator-always-on;
> +                        regulator-boot-on;
> +                        ti,defdcdc_default = <1>;
> +                };
> +
> +                vdcdc3_reg: regulator@2 {
> +                        regulator-name = "VDCDC3_1.2V";
> +                        regulator-min-microvolt = <950000>;
> +                        regulator-max-microvolt = <1350000>;
> +                        regulator-always-on;
> +                        regulator-boot-on;
> +                        ti,defdcdc_default = <1>;
> +                };
> +
> +                ldo1_reg: regulator@3 {
> +                        regulator-name = "LDO1_1.8V";
> +                        regulator-min-microvolt = <1710000>;
> +                        regulator-max-microvolt = <1890000>;
> +                        regulator-always-on;
> +                        regulator-boot-on;
> +                };
> +
> +                ldo2_reg: regulator@4 {
> +                        regulator-name = "LDO2_1.2V";
> +                        regulator-min-microvolt = <1140000>;
> +                        regulator-max-microvolt = <1320000>;
> +                        regulator-always-on;
> +                        regulator-boot-on;
> +                };
> +        };
>  };
> --
> 1.7.4.1
> 
> 

If there are not any comments on this patch, Could you please accept 
this patch?

Regards, 
Manish
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/