Re: [PATCH v6 7/8] arm64: dts: hikey960: add device node for pmic and regulators

[Guodong Xu]

Thanks, Wei.

On Fri, Jun 16, 2017 at 3:55 AM, Wei Xu <xuwei5@xxxxxxxxxxxxx> wrote:
> Hi Guodong,
>
> On 2017/6/8 23:08, Guodong Xu wrote:
>> From: Wang Xiaoyin <hw.wangxiaoyin@xxxxxxxxxxxxx>
>>
>> add device node for hi6421 pmic core and hi6421v530
>> voltage regulator,include LDO(1,3,9,11,15,16)
>>
>> Signed-off-by: Wang Xiaoyin <hw.wangxiaoyin@xxxxxxxxxxxxx>
>> Signed-off-by: Guodong Xu <guodong.xu@xxxxxxxxxx>
>> Acked-by: Arnd Bergmann <arnd@xxxxxxxx>
>> ---
>>  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@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
>