All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jisheng Zhang <jszhang@marvell.com>
To: Antoine Tenart <antoine.tenart@free-electrons.com>
Cc: sebastian.hesselbarth@gmail.com, linus.walleij@linaro.org,
	catalin.marinas@arm.com, will.deacon@arm.com,
	mark.rutland@arm.com, robh+dt@kernel.org, pawel.moll@arm.com,
	ijc+devicetree@hellion.org.uk, galak@codeaurora.org,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v5 0/7] arm64: berlin: add pinctrl support
Date: Mon, 12 Oct 2015 10:39:05 +0800	[thread overview]
Message-ID: <20151012103905.4ff69c90@xhacker> (raw)
In-Reply-To: <20151010223636.GH2842@kwain>

On Sun, 11 Oct 2015 00:36:36 +0200
Antoine Tenart <antoine.tenart@free-electrons.com> wrote:

> Hi Jisheng,
> 
> Please keep Acked-by tags when you submit a new version of a series.

Got it. I dunno this before, will take care in the future.

Thanks a lot for your review,
Jisheng

> 
> Thanks,
> 
> Antoine
> 
> On Fri, Oct 09, 2015 at 07:56:43PM +0800, Jisheng Zhang wrote:
> > This series adds support for Marvell berlin4ct pin-controller, allowing
> > to configure the pin muxing from the device tree.
> > 
> > Since v4:
> >  - drop ARCH_BERLIN dependency for it has been met
> >  - drop COMPILE_TEST dependency but make berlin pinctrl driver visible
> >    if COMPILE_TEST=y, and let this change be a seperate commit.
> > 
> > Since v3:
> >  - fix stupid build error: I missed local changes to berlin-bg4ct.c
> > 
> > Since v2:
> >  - introduce new berlin_pinctrl_probe_regmap() to avoid duplicated code
> >  - renaming such as s/urt/uart etc. but keep the group name as URT* to keep
> >    consistent with HW/ASIC
> > 
> > Since v1:
> >  - use generic name for pin functions
> >  - a new commit to add pinmux for uart0
> >  - correct pinctrl usage in dts
> > 
> > 
> > Jisheng Zhang (7):
> >   pinctrl: berlin: introduce berlin_pinctrl_probe_regmap()
> >   pinctrl: berlin: Make berlin pinctrl driver visible if COMPILE_TEST=y
> >   pinctrl: berlin: add the berlin4ct pinctrl driver
> >   arm64: berlin: add the pinctrl dependency for Marvell Berlin SoCs
> >   pinctrl: dt-binding: document berlin4ct SoC pinctrl
> >   arm64: dts: berlin4ct: add the pinctrl node
> >   arm64: dts: berlin4ct: add default pinmux for uart0
> > 
> >  .../devicetree/bindings/pinctrl/berlin,pinctrl.txt |   5 +-
> >  arch/arm64/Kconfig.platforms                       |   1 +
> >  arch/arm64/boot/dts/marvell/berlin4ct.dtsi         |  22 +
> >  drivers/pinctrl/berlin/Kconfig                     |   7 +-
> >  drivers/pinctrl/berlin/Makefile                    |   1 +
> >  drivers/pinctrl/berlin/berlin-bg4ct.c              | 503 +++++++++++++++++++++
> >  drivers/pinctrl/berlin/berlin.c                    |  26 +-
> >  drivers/pinctrl/berlin/berlin.h                    |   4 +
> >  8 files changed, 558 insertions(+), 11 deletions(-)
> >  create mode 100644 drivers/pinctrl/berlin/berlin-bg4ct.c
> > 
> > -- 
> > 2.6.1
> > 
> 


WARNING: multiple messages have this Message-ID (diff)
From: jszhang@marvell.com (Jisheng Zhang)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v5 0/7] arm64: berlin: add pinctrl support
Date: Mon, 12 Oct 2015 10:39:05 +0800	[thread overview]
Message-ID: <20151012103905.4ff69c90@xhacker> (raw)
In-Reply-To: <20151010223636.GH2842@kwain>

On Sun, 11 Oct 2015 00:36:36 +0200
Antoine Tenart <antoine.tenart@free-electrons.com> wrote:

> Hi Jisheng,
> 
> Please keep Acked-by tags when you submit a new version of a series.

Got it. I dunno this before, will take care in the future.

Thanks a lot for your review,
Jisheng

> 
> Thanks,
> 
> Antoine
> 
> On Fri, Oct 09, 2015 at 07:56:43PM +0800, Jisheng Zhang wrote:
> > This series adds support for Marvell berlin4ct pin-controller, allowing
> > to configure the pin muxing from the device tree.
> > 
> > Since v4:
> >  - drop ARCH_BERLIN dependency for it has been met
> >  - drop COMPILE_TEST dependency but make berlin pinctrl driver visible
> >    if COMPILE_TEST=y, and let this change be a seperate commit.
> > 
> > Since v3:
> >  - fix stupid build error: I missed local changes to berlin-bg4ct.c
> > 
> > Since v2:
> >  - introduce new berlin_pinctrl_probe_regmap() to avoid duplicated code
> >  - renaming such as s/urt/uart etc. but keep the group name as URT* to keep
> >    consistent with HW/ASIC
> > 
> > Since v1:
> >  - use generic name for pin functions
> >  - a new commit to add pinmux for uart0
> >  - correct pinctrl usage in dts
> > 
> > 
> > Jisheng Zhang (7):
> >   pinctrl: berlin: introduce berlin_pinctrl_probe_regmap()
> >   pinctrl: berlin: Make berlin pinctrl driver visible if COMPILE_TEST=y
> >   pinctrl: berlin: add the berlin4ct pinctrl driver
> >   arm64: berlin: add the pinctrl dependency for Marvell Berlin SoCs
> >   pinctrl: dt-binding: document berlin4ct SoC pinctrl
> >   arm64: dts: berlin4ct: add the pinctrl node
> >   arm64: dts: berlin4ct: add default pinmux for uart0
> > 
> >  .../devicetree/bindings/pinctrl/berlin,pinctrl.txt |   5 +-
> >  arch/arm64/Kconfig.platforms                       |   1 +
> >  arch/arm64/boot/dts/marvell/berlin4ct.dtsi         |  22 +
> >  drivers/pinctrl/berlin/Kconfig                     |   7 +-
> >  drivers/pinctrl/berlin/Makefile                    |   1 +
> >  drivers/pinctrl/berlin/berlin-bg4ct.c              | 503 +++++++++++++++++++++
> >  drivers/pinctrl/berlin/berlin.c                    |  26 +-
> >  drivers/pinctrl/berlin/berlin.h                    |   4 +
> >  8 files changed, 558 insertions(+), 11 deletions(-)
> >  create mode 100644 drivers/pinctrl/berlin/berlin-bg4ct.c
> > 
> > -- 
> > 2.6.1
> > 
> 

WARNING: multiple messages have this Message-ID (diff)
From: Jisheng Zhang <jszhang@marvell.com>
To: Antoine Tenart <antoine.tenart@free-electrons.com>
Cc: <sebastian.hesselbarth@gmail.com>, <linus.walleij@linaro.org>,
	<catalin.marinas@arm.com>, <will.deacon@arm.com>,
	<mark.rutland@arm.com>, <robh+dt@kernel.org>,
	<pawel.moll@arm.com>, <ijc+devicetree@hellion.org.uk>,
	<galak@codeaurora.org>, <devicetree@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-gpio@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v5 0/7] arm64: berlin: add pinctrl support
Date: Mon, 12 Oct 2015 10:39:05 +0800	[thread overview]
Message-ID: <20151012103905.4ff69c90@xhacker> (raw)
In-Reply-To: <20151010223636.GH2842@kwain>

On Sun, 11 Oct 2015 00:36:36 +0200
Antoine Tenart <antoine.tenart@free-electrons.com> wrote:

> Hi Jisheng,
> 
> Please keep Acked-by tags when you submit a new version of a series.

Got it. I dunno this before, will take care in the future.

Thanks a lot for your review,
Jisheng

> 
> Thanks,
> 
> Antoine
> 
> On Fri, Oct 09, 2015 at 07:56:43PM +0800, Jisheng Zhang wrote:
> > This series adds support for Marvell berlin4ct pin-controller, allowing
> > to configure the pin muxing from the device tree.
> > 
> > Since v4:
> >  - drop ARCH_BERLIN dependency for it has been met
> >  - drop COMPILE_TEST dependency but make berlin pinctrl driver visible
> >    if COMPILE_TEST=y, and let this change be a seperate commit.
> > 
> > Since v3:
> >  - fix stupid build error: I missed local changes to berlin-bg4ct.c
> > 
> > Since v2:
> >  - introduce new berlin_pinctrl_probe_regmap() to avoid duplicated code
> >  - renaming such as s/urt/uart etc. but keep the group name as URT* to keep
> >    consistent with HW/ASIC
> > 
> > Since v1:
> >  - use generic name for pin functions
> >  - a new commit to add pinmux for uart0
> >  - correct pinctrl usage in dts
> > 
> > 
> > Jisheng Zhang (7):
> >   pinctrl: berlin: introduce berlin_pinctrl_probe_regmap()
> >   pinctrl: berlin: Make berlin pinctrl driver visible if COMPILE_TEST=y
> >   pinctrl: berlin: add the berlin4ct pinctrl driver
> >   arm64: berlin: add the pinctrl dependency for Marvell Berlin SoCs
> >   pinctrl: dt-binding: document berlin4ct SoC pinctrl
> >   arm64: dts: berlin4ct: add the pinctrl node
> >   arm64: dts: berlin4ct: add default pinmux for uart0
> > 
> >  .../devicetree/bindings/pinctrl/berlin,pinctrl.txt |   5 +-
> >  arch/arm64/Kconfig.platforms                       |   1 +
> >  arch/arm64/boot/dts/marvell/berlin4ct.dtsi         |  22 +
> >  drivers/pinctrl/berlin/Kconfig                     |   7 +-
> >  drivers/pinctrl/berlin/Makefile                    |   1 +
> >  drivers/pinctrl/berlin/berlin-bg4ct.c              | 503 +++++++++++++++++++++
> >  drivers/pinctrl/berlin/berlin.c                    |  26 +-
> >  drivers/pinctrl/berlin/berlin.h                    |   4 +
> >  8 files changed, 558 insertions(+), 11 deletions(-)
> >  create mode 100644 drivers/pinctrl/berlin/berlin-bg4ct.c
> > 
> > -- 
> > 2.6.1
> > 
> 


  reply	other threads:[~2015-10-12  2:43 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-09 11:56 [PATCH v5 0/7] arm64: berlin: add pinctrl support Jisheng Zhang
2015-10-09 11:56 ` Jisheng Zhang
2015-10-09 11:56 ` Jisheng Zhang
2015-10-09 11:56 ` [PATCH v5 1/7] pinctrl: berlin: introduce berlin_pinctrl_probe_regmap() Jisheng Zhang
2015-10-09 11:56   ` Jisheng Zhang
2015-10-09 11:56   ` Jisheng Zhang
     [not found] ` <1444391810-5045-1-git-send-email-jszhang-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org>
2015-10-09 11:56   ` [PATCH v5 2/7] pinctrl: berlin: Make berlin pinctrl driver visible if COMPILE_TEST=y Jisheng Zhang
2015-10-09 11:56     ` Jisheng Zhang
2015-10-09 11:56     ` Jisheng Zhang
2015-10-09 11:56 ` [PATCH v5 3/7] pinctrl: berlin: add the berlin4ct pinctrl driver Jisheng Zhang
2015-10-09 11:56   ` Jisheng Zhang
2015-10-09 11:56   ` Jisheng Zhang
2015-10-15 21:52   ` Sebastian Hesselbarth
2015-10-15 21:52     ` Sebastian Hesselbarth
2015-10-09 11:56 ` [PATCH v5 4/7] arm64: berlin: add the pinctrl dependency for Marvell Berlin SoCs Jisheng Zhang
2015-10-09 11:56   ` Jisheng Zhang
2015-10-09 11:56   ` Jisheng Zhang
2015-10-09 11:56 ` [PATCH v5 5/7] pinctrl: dt-binding: document berlin4ct SoC pinctrl Jisheng Zhang
2015-10-09 11:56   ` Jisheng Zhang
2015-10-09 11:56   ` Jisheng Zhang
2015-10-09 11:56 ` [PATCH v5 6/7] arm64: dts: berlin4ct: add the pinctrl node Jisheng Zhang
2015-10-09 11:56   ` Jisheng Zhang
2015-10-09 11:56   ` Jisheng Zhang
2015-10-09 11:56 ` [PATCH v5 7/7] arm64: dts: berlin4ct: add default pinmux for uart0 Jisheng Zhang
2015-10-09 11:56   ` Jisheng Zhang
2015-10-09 11:56   ` Jisheng Zhang
2015-10-10 22:36 ` [PATCH v5 0/7] arm64: berlin: add pinctrl support Antoine Tenart
2015-10-10 22:36   ` Antoine Tenart
2015-10-12  2:39   ` Jisheng Zhang [this message]
2015-10-12  2:39     ` Jisheng Zhang
2015-10-12  2:39     ` Jisheng Zhang
2015-10-10 22:40 ` Antoine Tenart
2015-10-10 22:40   ` Antoine Tenart
2015-10-10 22:40   ` Antoine Tenart
2015-10-15 21:55   ` Sebastian Hesselbarth
2015-10-15 21:55     ` Sebastian Hesselbarth

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=20151012103905.4ff69c90@xhacker \
    --to=jszhang@marvell.com \
    --cc=antoine.tenart@free-electrons.com \
    --cc=catalin.marinas@arm.com \
    --cc=devicetree@vger.kernel.org \
    --cc=galak@codeaurora.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=pawel.moll@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=sebastian.hesselbarth@gmail.com \
    --cc=will.deacon@arm.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.