[U-Boot] [PATCH v3 3/6] arm64: mvebu: pinctrl: Add pin control driver for A8K family

Kostya Porotchkin kostap at marvell.com
Wed Dec 7 09:03:14 CET 2016


Hello, Simon,

Thank you very much for your review.
Yes, this patch was only adding some help to Kconfig (as you requested), no other changes from the previous version.
I have to re-send the entire series one more time for moving the change log from commit messages to comment sections.
Some changes will be done on other patches, but this one will remain the same as v3.

Best Regards
Kosta
________________________________________
From: sjg at google.com <sjg at google.com> on behalf of Simon Glass <sjg at chromium.org>
Sent: Wednesday, December 7, 2016 05:47
To: Kostya Porotchkin
Cc: U-Boot Mailing List; Stefan Roese; Nadav Haklai; Neta Zur Hershkovits; Omri Itach; Igal Liberman; Haim Boot; Hanna Hawa
Subject: Re: [PATCH v3 3/6] arm64: mvebu: pinctrl: Add pin control driver for A8K family



----------------------------------------------------------------------
On 5 December 2016 at 05:16,  <kostap at marvell.com> wrote:
> From: Konstantin Porotchkin <kostap at marvell.com>
>
> Add a DM port of Marvell pin control driver.
> The A8K SoC family contains several silicone dies interconnected
> in a single package. Every die is normally equipped with its own
> pin controller unit.
> There are 2 pin controllers in A70x0 SoC and 3 in A80x0 SoC.
>
> Changes for v3:
> - Extend the Kconfig help for pin control driver
>
> Signed-off-by: Konstantin Porotchkin <kostap at marvell.com>
> Cc: Simon Glass <sjg at chromium.org>
> Cc: Stefan Roese <sr at denx.de>
> Cc: Nadav Haklai <nadavh at marvell.com>
> Cc: Neta Zur Hershkovits <neta at marvell.com>
> Cc: Omri Itach <omrii at marvell.com>
> Cc: Igal Liberman <igall at marvell.com>
> Cc: Haim Boot <hayim at marvell.com>
> Cc: Hanna Hawa <hannah at marvell.com>
> ---
>  arch/arm/include/asm/arch-armada8k/soc-info.h      |  17 ++
>  .../pinctrl/marvell,armada-apn806-pinctrl.txt      |  25 ++
>  .../pinctrl/marvell,armada-cp110-pinctrl.txt       | 270 +++++++++++++++++++++
>  .../pinctrl/marvell,mvebu-pinctrl.txt              | 113 +++++++++
>  drivers/pinctrl/Kconfig                            |   1 +
>  drivers/pinctrl/Makefile                           |   1 +
>  drivers/pinctrl/mvebu/Kconfig                      |  16 ++
>  drivers/pinctrl/mvebu/Makefile                     |   7 +
>  drivers/pinctrl/mvebu/pinctrl-mvebu.c              | 179 ++++++++++++++
>  drivers/pinctrl/mvebu/pinctrl-mvebu.h              |  31 +++
>  10 files changed, 660 insertions(+)
>  create mode 100644 arch/arm/include/asm/arch-armada8k/soc-info.h
>  create mode 100644 doc/device-tree-bindings/pinctrl/marvell,armada-apn806-pinctrl.txt
>  create mode 100644 doc/device-tree-bindings/pinctrl/marvell,armada-cp110-pinctrl.txt
>  create mode 100644 doc/device-tree-bindings/pinctrl/marvell,mvebu-pinctrl.txt
>  create mode 100644 drivers/pinctrl/mvebu/Kconfig
>  create mode 100644 drivers/pinctrl/mvebu/Makefile
>  create mode 100644 drivers/pinctrl/mvebu/pinctrl-mvebu.c
>  create mode 100644 drivers/pinctrl/mvebu/pinctrl-mvebu.h
>

I think I already reviewed this, but just in case:

Reviewed-by: Simon Glass <sjg at chromium.org>


More information about the U-Boot mailing list