Re: [PATCH v4 0/8] MFD: add driver for HiSilicon Hi6421v530 PMIC

From: Arnd Bergmann
Date: Tue Jun 06 2017 - 08:54:25 EST


On Tue, Jun 6, 2017 at 10:51 AM, Guodong Xu <guodong.xu@xxxxxxxxxx> wrote:
> This patchset adds driver for HiSilicon Hi6421v530 PMIC.
>
> Mainline kernel already has driver support to a similar chip, Hi6421.
> Hi6421 and Hi6421v530 are both from the same vendor, HiSilicon, but
> they are at different revisions. They both use the same Memory-mapped
> I/O method to communicate with Main SoC. However, they differ quite a
> lot in their regulator designs. Eg. they have completely different LDO
> voltage points.
>
> Patch 1 and 2 are hi6421-pmic cleaning up.
> Patch 3 and 4 extends hi6421-pmic-core.c to support Hi6421v530 revision.
> Patch 5 add hi6421v530-regulator.c driver for LDO regulators.
> Patch 6 fixes an issue for hi6421 regulator, which is not related to v530
> but it's found in this review.
> Patch 7 is dts change, it depends on and can be applied to hi3660/hikey960
> patchset [1].
> Patch 8 enables the relevant config items.

Looks good to me,

Arnd