From: Guodong Xu <guodong.xu@linaro.org>
To: lee.jones@linaro.org, robh+dt@kernel.org, mark.rutland@arm.com,
xuwei5@hisilicon.com, catalin.marinas@arm.com,
will.deacon@arm.com, lgirdwood@gmail.com, broonie@kernel.org,
khilman@baylibre.com, arnd@arndb.de,
gregory.clement@free-electrons.com, horms+renesas@verge.net.au,
olof@lixom.net, thomas.petazzoni@free-electrons.com,
yamada.masahiro@socionext.com, riku.voipio@linaro.org,
treding@nvidia.com, krzk@kernel.org, eric@anholt.net,
damm+renesas@opensource.se, ard.biesheuvel@linaro.org,
linus.walleij@linaro.org, geert+renesas@glider.be
Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
hw.wangxiaoyin@hisilicon.com, Guodong Xu <guodong.xu@linaro.org>
Subject: [PATCH v6 0/8] MFD: add driver for HiSilicon Hi6421v530 PMIC
Date: Fri, 9 Jun 2017 06:08:33 +0800 [thread overview]
Message-ID: <20170608220841.26531-1-guodong.xu@linaro.org> (raw)
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.
[1], http://www.spinics.net/lists/devicetree/msg178303.html
Major changes in v6:
- Patch 5, solve review comments from Mark Brown, add hi6421v530 regulator
driver to module device table.
- Add Acked-by from Arnd Bergmann
Major changes in v5:
- Patch 3, solve review comments from Lee Johes
Major changes in v4:
- put hi6421-pmic cleanup in separate patches.
- solve review comments from Lee Johes.
- regulator-name should not have '/' character. Otherwise it "Failed to
create debugfs directory"
Major changes in v3:
- in hi6421-pmic-core.c
* use shorter license script.
* arrange #include in alphabetical order.
* using recommended error log messages from Lee Jones.
- in hi6421v530-regulator.c
* remove unused #include files
* arrange remaining ones in alphabetical order.
Major changes in v2:
- instead of writing a new driver, extend hi6421-pmic-core.c
to support its v530 revision
- update hi6421v530-regulator.c to use modern regulator driver
design logics.
Guodong Xu (6):
mfd: hi6421-pmic: cleanup: change license text to shorter form
mfd: hi6421-pmic: cleanup: update dev_err messages
dt-bindings: mfd: hi6421: Add hi6421v530 compatible string
mfd: hi6421-pmic: add support for HiSilicon Hi6421v530
regulator: hi6421: Describe consumed platform device
arm64: defconfig: enable support hi6421v530 PMIC
Wang Xiaoyin (2):
regulator: hi6421v530: add driver for hi6421v530 voltage regulator
arm64: dts: hikey960: add device node for pmic and regulators
Documentation/devicetree/bindings/mfd/hi6421.txt | 4 +-
arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts | 46 +++++
arch/arm64/configs/defconfig | 2 +
drivers/mfd/hi6421-pmic-core.c | 89 +++++----
drivers/regulator/Kconfig | 10 +
drivers/regulator/Makefile | 1 +
drivers/regulator/hi6421-regulator.c | 7 +
drivers/regulator/hi6421v530-regulator.c | 214 ++++++++++++++++++++++
include/linux/mfd/hi6421-pmic.h | 5 +
9 files changed, 345 insertions(+), 33 deletions(-)
create mode 100644 drivers/regulator/hi6421v530-regulator.c
--
2.10.2
next reply other threads:[~2017-06-08 22:08 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-08 22:08 Guodong Xu [this message]
2017-06-08 22:08 ` [PATCH v6 2/8] mfd: hi6421-pmic: cleanup: update dev_err messages Guodong Xu
2017-06-08 22:08 ` [PATCH v6 5/8] regulator: hi6421v530: add driver for hi6421v530 voltage regulator Guodong Xu
[not found] ` <20170608220841.26531-6-guodong.xu-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2017-06-13 20:35 ` Mark Brown
[not found] ` <20170613203527.ecykjzzpvkrzl7zz-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2017-06-14 8:13 ` Guodong Xu
[not found] ` <20170608220841.26531-1-guodong.xu-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2017-06-08 22:08 ` [PATCH v6 1/8] mfd: hi6421-pmic: cleanup: change license text to shorter form Guodong Xu
2017-06-08 22:08 ` [PATCH v6 3/8] dt-bindings: mfd: hi6421: Add hi6421v530 compatible string Guodong Xu
2017-06-08 22:08 ` [PATCH v6 4/8] mfd: hi6421-pmic: add support for HiSilicon Hi6421v530 Guodong Xu
[not found] ` <20170608220841.26531-5-guodong.xu-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2017-07-03 12:00 ` Lee Jones
2017-07-03 12:01 ` Lee Jones
2017-06-08 22:08 ` [PATCH v6 6/8] regulator: hi6421: Describe consumed platform device Guodong Xu
2017-06-08 22:08 ` [PATCH v6 8/8] arm64: defconfig: enable support hi6421v530 PMIC Guodong Xu
2017-06-08 22:08 ` [PATCH v6 7/8] arm64: dts: hikey960: add device node for pmic and regulators Guodong Xu
[not found] ` <20170608220841.26531-8-guodong.xu-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2017-06-15 15:01 ` Wei Xu
2017-06-15 19:55 ` Wei Xu
[not found] ` <5942E633.9040908-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org>
2017-06-16 3:47 ` Guodong Xu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170608220841.26531-1-guodong.xu@linaro.org \
--to=guodong.xu@linaro.org \
--cc=ard.biesheuvel@linaro.org \
--cc=arnd@arndb.de \
--cc=broonie@kernel.org \
--cc=catalin.marinas@arm.com \
--cc=damm+renesas@opensource.se \
--cc=devicetree@vger.kernel.org \
--cc=eric@anholt.net \
--cc=geert+renesas@glider.be \
--cc=gregory.clement@free-electrons.com \
--cc=horms+renesas@verge.net.au \
--cc=hw.wangxiaoyin@hisilicon.com \
--cc=khilman@baylibre.com \
--cc=krzk@kernel.org \
--cc=lee.jones@linaro.org \
--cc=lgirdwood@gmail.com \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=olof@lixom.net \
--cc=riku.voipio@linaro.org \
--cc=robh+dt@kernel.org \
--cc=thomas.petazzoni@free-electrons.com \
--cc=treding@nvidia.com \
--cc=will.deacon@arm.com \
--cc=xuwei5@hisilicon.com \
--cc=yamada.masahiro@socionext.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).