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: s...@google.com <s...@google.com> on behalf of Simon Glass 
<s...@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,  <kos...@marvell.com> wrote:
> From: Konstantin Porotchkin <kos...@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 <kos...@marvell.com>
> Cc: Simon Glass <s...@chromium.org>
> Cc: Stefan Roese <s...@denx.de>
> Cc: Nadav Haklai <nad...@marvell.com>
> Cc: Neta Zur Hershkovits <n...@marvell.com>
> Cc: Omri Itach <om...@marvell.com>
> Cc: Igal Liberman <ig...@marvell.com>
> Cc: Haim Boot <ha...@marvell.com>
> Cc: Hanna Hawa <han...@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 <s...@chromium.org>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to