diff mbox

[v2,14/16] arm64: dts: hikey: Add hi655x pmic dts node

Message ID 1459589383-16914-15-git-send-email-guodong.xu@linaro.org
State Superseded
Headers show

Commit Message

Guodong Xu April 2, 2016, 9:29 a.m. UTC 
From: Chen Feng <puck.chen@hisilicon.com>


Add the mfd hi655x dts node and regulator support on
hi6220 platform.

Signed-off-by: Chen Feng <puck.chen@hisilicon.com>

Signed-off-by: Fei Wang <w.f@huawei.com>

Signed-off-by: Xinwei Kong <kong.kongxinwei@hisilicon.com>

Reviewed-by: Haojian Zhuang <haojian.zhuang@linaro.org>

Reviewed-by: Guodong Xu <guodong.xu@linaro.org>

---
 arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts | 87 ++++++++++++++++++++++++++
 1 file changed, 87 insertions(+)

-- 
1.9.1

Comments

Rob Herring (Arm) April 4, 2016, 5:16 a.m. UTC | #1 
On Sat, Apr 02, 2016 at 05:29:41PM +0800, Guodong Xu wrote:
> From: Chen Feng <puck.chen@hisilicon.com>

> 

> Add the mfd hi655x dts node and regulator support on

> hi6220 platform.

> 

> Signed-off-by: Chen Feng <puck.chen@hisilicon.com>

> Signed-off-by: Fei Wang <w.f@huawei.com>

> Signed-off-by: Xinwei Kong <kong.kongxinwei@hisilicon.com>

> Reviewed-by: Haojian Zhuang <haojian.zhuang@linaro.org>

> Reviewed-by: Guodong Xu <guodong.xu@linaro.org>

> ---

>  arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts | 87 ++++++++++++++++++++++++++

>  1 file changed, 87 insertions(+)

> 

> diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts

> index 28bc646..a6dffa8 100644

> --- a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts

> +++ b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts

> @@ -6,6 +6,7 @@

>   */

>  

>  /dts-v1/;

> +#include <dt-bindings/gpio/gpio.h>

>  

>  #include "hi6220.dtsi"

>  #include "hikey-gpio.dtsi"

> @@ -104,6 +105,92 @@

>  			default-state = "off";

>  		};

>  	};

> +

> +	pmic: pmic@f8000000 {

> +		compatible = "hisilicon,hi655x-pmic";

> +		reg = <0x0 0xf8000000 0x0 0x1000>;

> +		interrupt-controller;

> +		#interrupt-cells = <2>;

> +		pmic-gpios = <&gpio1 2 GPIO_ACTIVE_HIGH>;

> +		status = "okay";

> +

> +		regulators {

> +			ldo2: LDO2@a21 {


@a21 is not a unit address. Please drop these.

> +				regulator-name = "LDO2_2V8";

> +				regulator-min-microvolt = <2500000>;

> +				regulator-max-microvolt = <3200000>;

> +				regulator-enable-ramp-delay = <120>;

> +			};

> +

> +			ldo7: LDO7@a26 {
diff mbox

Patch

diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
index 28bc646..a6dffa8 100644
--- a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
+++ b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
@@ -6,6 +6,7 @@ 
  */
 
 /dts-v1/;
+#include <dt-bindings/gpio/gpio.h>
 
 #include "hi6220.dtsi"
 #include "hikey-gpio.dtsi"
@@ -104,6 +105,92 @@ 
 			default-state = "off";
 		};
 	};
+
+	pmic: pmic@f8000000 {
+		compatible = "hisilicon,hi655x-pmic";
+		reg = <0x0 0xf8000000 0x0 0x1000>;
+		interrupt-controller;
+		#interrupt-cells = <2>;
+		pmic-gpios = <&gpio1 2 GPIO_ACTIVE_HIGH>;
+		status = "okay";
+
+		regulators {
+			ldo2: LDO2@a21 {
+				regulator-name = "LDO2_2V8";
+				regulator-min-microvolt = <2500000>;
+				regulator-max-microvolt = <3200000>;
+				regulator-enable-ramp-delay = <120>;
+			};
+
+			ldo7: LDO7@a26 {
+				regulator-name = "LDO7_SDIO";
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-enable-ramp-delay = <120>;
+			};
+
+			ldo10: LDO10@a29 {
+				regulator-name = "LDO10_2V85";
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <3000000>;
+				regulator-enable-ramp-delay = <360>;
+			};
+
+			ldo13: LDO13@a32 {
+				regulator-name = "LDO13_1V8";
+				regulator-min-microvolt = <1600000>;
+				regulator-max-microvolt = <1950000>;
+				regulator-enable-ramp-delay = <120>;
+			};
+
+			ldo14: LDO14@a33 {
+				regulator-name = "LDO14_2V8";
+				regulator-min-microvolt = <2500000>;
+				regulator-max-microvolt = <3200000>;
+				regulator-enable-ramp-delay = <120>;
+			};
+
+			ldo15: LDO15@a34 {
+				regulator-name = "LDO15_1V8";
+				regulator-min-microvolt = <1600000>;
+				regulator-max-microvolt = <1950000>;
+				regulator-boot-on;
+				regulator-always-on;
+				regulator-enable-ramp-delay = <120>;
+			};
+
+			ldo17: LDO17@a36 {
+				regulator-name = "LDO17_2V5";
+				regulator-min-microvolt = <2500000>;
+				regulator-max-microvolt = <3200000>;
+				regulator-enable-ramp-delay = <120>;
+			};
+
+			ldo19: LDO19@a38 {
+				regulator-name = "LDO19_3V0";
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <3000000>;
+				regulator-enable-ramp-delay = <360>;
+			};
+
+			ldo21: LDO21@a40 {
+				regulator-name = "LDO21_1V8";
+				regulator-min-microvolt = <1650000>;
+				regulator-max-microvolt = <2000000>;
+				regulator-always-on;
+				regulator-enable-ramp-delay = <120>;
+			};
+
+			ldo22: LDO22@a41 {
+				regulator-name = "LDO22_1V2";
+				regulator-min-microvolt = <900000>;
+				regulator-max-microvolt = <1200000>;
+				regulator-boot-on;
+				regulator-always-on;
+				regulator-enable-ramp-delay = <120>;
+			};
+		};
+	};
 };
 
 &uart2 {