Re: [U-Boot] [PATCH 2/3] arm: dts: s5pc1xx-goni: add the pmic node for using DM

[Simon Glass]

On 15 December 2016 at 02:21, Jaehoon Chung  wrote:
> To use driver-model adds the pmic node for max8998.
> This is used as kerel device-tree in Linux.
>
> Signed-off-by: Jaehoon Chung 
> ---
>  arch/arm/dts/s5pc1xx-goni.dts | 165 
> ++
>  1 file changed, 165 insertions(+)

Reviewed-by: Simon Glass 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/3] arm: dts: s5pc1xx-goni: add the pmic node for using DM

[Minkyu Kang]

Hi,

On 15 December 2016 at 18:21, Jaehoon Chung  wrote:

> To use driver-model adds the pmic node for max8998.
> This is used as kerel device-tree in Linux.
>
> Signed-off-by: Jaehoon Chung 
> ---
>  arch/arm/dts/s5pc1xx-goni.dts | 165 ++
> 
>  1 file changed, 165 insertions(+)
>
> diff --git a/arch/arm/dts/s5pc1xx-goni.dts b/arch/arm/dts/s5pc1xx-goni.dts
> index 7bbfe59..e80132d 100644
> --- a/arch/arm/dts/s5pc1xx-goni.dts
> +++ b/arch/arm/dts/s5pc1xx-goni.dts
> @@ -19,6 +19,7 @@
> serial2 = "/serial@e2900800";
> console = "/serial@e2900800";
> pinctrl0 = 
> +   i2c3 = _pmic;
> };
>
> pinctrl0: pinctrl@e020 {
> @@ -32,4 +33,168 @@
> id = <2>;
> };
>
> +   i2c_pmic: i2c-pmic {
> +   compatible = "i2c-gpio";
> +   gpios = < 0 0>, /* sda */
> +   < 3 0>; /* scl */
> +   i2c-gpio,delay-us = <2>;/* ~100 kHz */
> +   #address-cells = <1>;
> +   #size-cells = <0>;
> +   status = "okay";
> +
> +   pmic@66 {
> +   compatible = "maxim,max8998";
> +   reg = <0x66 0 0>;
> +
> +   voltage-regulators {
> +   ldo2_reg: LDO2 {
> +   regulator-compatible = "LDO2";
> +   regulator-name = "VALIVE_1.1V";
> +   regulator-min-microvolt =
> <110>;
> +   regulator-max-microvolt =
> <110>;
> +   regulator-always-on;
> +   };
> +
> +   ldo3_reg: LDO3 {
> +   regulator-compatible = "LDO3";
> +   regulator-name = "VUSB+MIPI_1.1V";
> +   regulator-min-microvolt =
> <110>;
> +   regulator-max-microvolt =
> <110>;
> +   regulator-always-on;
> +   };
> +
> +   ldo4_reg: LDO4 {
> +   regulator-compatible = "LDO4";
> +   regulator-name = "VADC_3.3V";
> +   regulator-min-microvolt =
> <330>;
> +   regulator-max-microvolt =
> <330>;
> +   };
> +
> +   ldo5_reg: LDO5 {
> +   regulator-compatible = "LDO5";
> +   regulator-name = "VTF_2.8V";
> +   regulator-min-microvolt =
> <280>;
> +   regulator-max-microvolt =
> <280>;
> +   };
> +
> +   ldo6_reg: LDO6 {
> +   regulator-compatible = "LDO6";
> +   regulator-name = "VCC_3.3V";
> +   regulator-min-microvolt =
> <330>;
> +   regulator-max-microvolt =
> <330>;
> +   };
> +
> +   ldo7_reg: LDO7 {
> +   regulator-compatible = "LDO7";
> +   regulator-name = "VLCD_1.8V";
> +   regulator-min-microvolt =
> <180>;
> +   regulator-max-microvolt =
> <180>;
> +   regulator-always-on;
> +   };
> +
> +   ldo8_reg: LDO8 {
> +   regulator-compatible = "LDO8";
> +   regulator-name = "VUSB+VDAC_3.3V";
> +   regulator-min-microvolt =
> <330>;
> +   regulator-max-microvolt =
> <330>;
> +   };
> +
> +   ldo9_reg: LDO9 {
> +   regulator-compatible = "LDO9";
> +   regulator-name = "VCC+VCAM_2.8V";
> +   regulator-min-microvolt =
> <280>;
> +   regulator-max-microvolt =
> <280>;
> +   };
> +
> +   ldo10_reg: LDO10 {
> +   regulator-compatible = "LDO10";
> +   regulator-name = "VPLL_1.1V";
> +

[U-Boot] [PATCH 2/3] arm: dts: s5pc1xx-goni: add the pmic node for using DM

[Jaehoon Chung]

To use driver-model adds the pmic node for max8998.
This is used as kerel device-tree in Linux.

Signed-off-by: Jaehoon Chung 
---
 arch/arm/dts/s5pc1xx-goni.dts | 165 ++
 1 file changed, 165 insertions(+)

diff --git a/arch/arm/dts/s5pc1xx-goni.dts b/arch/arm/dts/s5pc1xx-goni.dts
index 7bbfe59..e80132d 100644
--- a/arch/arm/dts/s5pc1xx-goni.dts
+++ b/arch/arm/dts/s5pc1xx-goni.dts
@@ -19,6 +19,7 @@
serial2 = "/serial@e2900800";
console = "/serial@e2900800";
pinctrl0 = 
+   i2c3 = _pmic;
};
 
pinctrl0: pinctrl@e020 {
@@ -32,4 +33,168 @@
id = <2>;
};
 
+   i2c_pmic: i2c-pmic {
+   compatible = "i2c-gpio";
+   gpios = < 0 0>, /* sda */
+   < 3 0>; /* scl */
+   i2c-gpio,delay-us = <2>;/* ~100 kHz */
+   #address-cells = <1>;
+   #size-cells = <0>;
+   status = "okay";
+
+   pmic@66 {
+   compatible = "maxim,max8998";
+   reg = <0x66 0 0>;
+
+   voltage-regulators {
+   ldo2_reg: LDO2 {
+   regulator-compatible = "LDO2";
+   regulator-name = "VALIVE_1.1V";
+   regulator-min-microvolt = <110>;
+   regulator-max-microvolt = <110>;
+   regulator-always-on;
+   };
+
+   ldo3_reg: LDO3 {
+   regulator-compatible = "LDO3";
+   regulator-name = "VUSB+MIPI_1.1V";
+   regulator-min-microvolt = <110>;
+   regulator-max-microvolt = <110>;
+   regulator-always-on;
+   };
+
+   ldo4_reg: LDO4 {
+   regulator-compatible = "LDO4";
+   regulator-name = "VADC_3.3V";
+   regulator-min-microvolt = <330>;
+   regulator-max-microvolt = <330>;
+   };
+
+   ldo5_reg: LDO5 {
+   regulator-compatible = "LDO5";
+   regulator-name = "VTF_2.8V";
+   regulator-min-microvolt = <280>;
+   regulator-max-microvolt = <280>;
+   };
+
+   ldo6_reg: LDO6 {
+   regulator-compatible = "LDO6";
+   regulator-name = "VCC_3.3V";
+   regulator-min-microvolt = <330>;
+   regulator-max-microvolt = <330>;
+   };
+
+   ldo7_reg: LDO7 {
+   regulator-compatible = "LDO7";
+   regulator-name = "VLCD_1.8V";
+   regulator-min-microvolt = <180>;
+   regulator-max-microvolt = <180>;
+   regulator-always-on;
+   };
+
+   ldo8_reg: LDO8 {
+   regulator-compatible = "LDO8";
+   regulator-name = "VUSB+VDAC_3.3V";
+   regulator-min-microvolt = <330>;
+   regulator-max-microvolt = <330>;
+   };
+
+   ldo9_reg: LDO9 {
+   regulator-compatible = "LDO9";
+   regulator-name = "VCC+VCAM_2.8V";
+   regulator-min-microvolt = <280>;
+   regulator-max-microvolt = <280>;
+   };
+
+   ldo10_reg: LDO10 {
+   regulator-compatible = "LDO10";
+   regulator-name = "VPLL_1.1V";
+   regulator-min-microvolt = <110>;
+   regulator-max-microvolt = <110>;
+   regulator-boot-on;
+   };
+
+   ldo11_reg: LDO11 {
+