Thanks, Wei.

On Fri, Jun 16, 2017 at 3:55 AM, Wei Xu [off-list ref] wrote:

Hi Guodong,

On 2017/6/8 23:08, Guodong Xu wrote:

quoted

From: Wang Xiaoyin <redacted>

add device node for hi6421 pmic core and hi6421v530
voltage regulator,include LDO(1,3,9,11,15,16)

Signed-off-by: Wang Xiaoyin <redacted>
Signed-off-by: Guodong Xu <redacted>
Acked-by: Arnd Bergmann <arnd@arndb.de>
---
 arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts | 46 +++++++++++++++++++++++
 1 file changed, 46 insertions(+)

diff --git a/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts b/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts
index ca448f0..e579333 100644
--- a/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts
+++ b/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts

@@ -97,6 +97,52 @@
                      default-state = "off";
              };
      };
+
+     pmic: pmic at fff34000 {
+             compatible = "hisilicon,hi6421v530-pmic";
+             reg = <0x0 0xfff34000 0x0 0x1000>;
+             interrupt-controller;
+             #interrupt-cells = <2>;
+
+             regulators {
+                     ldo3: LDO3 { /* HDMI */
+                             regulator-name = "VOUT3_1V85";
+                             regulator-min-microvolt = <1800000>;
+                             regulator-max-microvolt = <2200000>;
+                             regulator-enable-ramp-delay = <120>;
+                     };
+
+                     ldo9: LDO9 { /* SDCARD I/O */
+                             regulator-name = "VOUT9_1V8_2V95";
+                             regulator-min-microvolt = <1750000>;
+                             regulator-max-microvolt = <3300000>;
+                             regulator-enable-ramp-delay = <240>;
+                     };
+
+                     ldo11: LDO11 { /* Low Speed Connector */
+                             regulator-name = "VOUT11_1V8_2V95";
+                             regulator-min-microvolt = <1750000>;
+                             regulator-max-microvolt = <3300000>;
+                             regulator-enable-ramp-delay = <240>;
+                     };
+
+                     ldo15: LDO15 { /* UFS VCC */
+                             regulator-name = "VOUT15_3V0";
+                             regulator-min-microvolt = <1750000>;
+                             regulator-max-microvolt = <3000000>;
+                             regulator-boot-on;
+                             regulator-always-on;
+                             regulator-enable-ramp-delay = <120>;
+                     };
+
+                     ldo16: LDO16 { /* SD VDD */
+                             regulator-name = "VOUT16_2V95";
+                             regulator-min-microvolt = <1750000>;
+                             regulator-max-microvolt = <3000000>;
+                             regulator-enable-ramp-delay = <360>;
+                     };
+             };
+     };
 };

 &i2c0 {

Since you put the dts part into another patch set[1] and the driver part has been applied.
I will pick up [1].
Thanks!

[1]: https://lkml.org/lkml/2017/6/14/1052

BR,
Wei