All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christian Daudt <bcm@fixthebug.org>
To: Olof Johansson <olof@lixom.net>
Cc: "linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	Arnd Bergmann <arnd@arndb.de>, Kevin Hilman <khilman@linaro.org>,
	Matt Porter <mporter@linaro.org>,
	Linus Walleij <linus.walleij@linaro.org>,
	Sherman Yin <syin@broadcom.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	arm@kernel.org
Subject: Re: [GIT PULL] bcm pinctrl rename
Date: Mon, 24 Feb 2014 21:01:37 -0800	[thread overview]
Message-ID: <CAEPRUYqUNxOT=KnhSmbXbL2e2tCs=jAhQqKLiVVc13okpBxHnw@mail.gmail.com> (raw)
In-Reply-To: <20140221222947.GB13233@quad.lixom.net>

On Fri, Feb 21, 2014 at 2:29 PM, Olof Johansson <olof@lixom.net> wrote:
> (As per irc discussion). The rename isn't the right thing to do at this
> time in the release, but the binding needs to be tweaked before it ships
> in a full release, or we'll always need to live with it.
>
> So, I'll be happy to take the bindings piece and the dtsi piece (and the
> small edit to the driver to change the compatible string match table),
> but the file rename and Kconfig entry edit is 3.15 material.
>
> Please send new pull request with just that (or, if you prefer, just send the
> patches and we can apply them too).
>

Can you please pull the patches below. Note that if you wanto to
absolutely reduce this to the bone, you can drop the last patch in the
series titled "pinctrl: Rename Broadcom Capri pinctrl binding". That
is the documentation fixup, so the binding doc will be wrong until
3.15. But that will reduce the change to just 2 lines :)
 thanks,
   csd


The following changes since commit cfbf8d4857c26a8a307fb7cd258074c9dcd8c691:

  Linux 3.14-rc4 (2014-02-23 17:40:03 -0800)

are available in the git repository at:

  git://github.com/broadcom/bcm11351.git
tags/bcm-for-3.14-pinctrl-reduced-rename

for you to fetch changes up to 735ea23c4868bf3123a4c79184e9206e0cc60211:

  pinctrl: Rename Broadcom Capri pinctrl binding (2014-02-24 20:25:54 -0800)

----------------------------------------------------------------
Rename pinctrl dt binding to restore consistency with
other bcm mobile bindings.

----------------------------------------------------------------
Christian Daudt (1):
      pinctrl: refer to updated dt binding string.

Sherman Yin (2):
      Update dtsi with new pinctrl compatible string
      pinctrl: Rename Broadcom Capri pinctrl binding

 Documentation/devicetree/bindings/pinctrl/{brcm,capri-pinctrl.txt =>
brcm,bcm11351-pinctrl.txt} | 8 ++++----
 arch/arm/boot/dts/bcm11351.dtsi
                          | 2 +-
 drivers/pinctrl/pinctrl-capri.c
                          | 2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)
 rename Documentation/devicetree/bindings/pinctrl/{brcm,capri-pinctrl.txt
=> brcm,bcm11351-pinctrl.txt} (98%)

WARNING: multiple messages have this Message-ID (diff)
From: bcm@fixthebug.org (Christian Daudt)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] bcm pinctrl rename
Date: Mon, 24 Feb 2014 21:01:37 -0800	[thread overview]
Message-ID: <CAEPRUYqUNxOT=KnhSmbXbL2e2tCs=jAhQqKLiVVc13okpBxHnw@mail.gmail.com> (raw)
In-Reply-To: <20140221222947.GB13233@quad.lixom.net>

On Fri, Feb 21, 2014 at 2:29 PM, Olof Johansson <olof@lixom.net> wrote:
> (As per irc discussion). The rename isn't the right thing to do at this
> time in the release, but the binding needs to be tweaked before it ships
> in a full release, or we'll always need to live with it.
>
> So, I'll be happy to take the bindings piece and the dtsi piece (and the
> small edit to the driver to change the compatible string match table),
> but the file rename and Kconfig entry edit is 3.15 material.
>
> Please send new pull request with just that (or, if you prefer, just send the
> patches and we can apply them too).
>

Can you please pull the patches below. Note that if you wanto to
absolutely reduce this to the bone, you can drop the last patch in the
series titled "pinctrl: Rename Broadcom Capri pinctrl binding". That
is the documentation fixup, so the binding doc will be wrong until
3.15. But that will reduce the change to just 2 lines :)
 thanks,
   csd


The following changes since commit cfbf8d4857c26a8a307fb7cd258074c9dcd8c691:

  Linux 3.14-rc4 (2014-02-23 17:40:03 -0800)

are available in the git repository at:

  git://github.com/broadcom/bcm11351.git
tags/bcm-for-3.14-pinctrl-reduced-rename

for you to fetch changes up to 735ea23c4868bf3123a4c79184e9206e0cc60211:

  pinctrl: Rename Broadcom Capri pinctrl binding (2014-02-24 20:25:54 -0800)

----------------------------------------------------------------
Rename pinctrl dt binding to restore consistency with
other bcm mobile bindings.

----------------------------------------------------------------
Christian Daudt (1):
      pinctrl: refer to updated dt binding string.

Sherman Yin (2):
      Update dtsi with new pinctrl compatible string
      pinctrl: Rename Broadcom Capri pinctrl binding

 Documentation/devicetree/bindings/pinctrl/{brcm,capri-pinctrl.txt =>
brcm,bcm11351-pinctrl.txt} | 8 ++++----
 arch/arm/boot/dts/bcm11351.dtsi
                          | 2 +-
 drivers/pinctrl/pinctrl-capri.c
                          | 2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)
 rename Documentation/devicetree/bindings/pinctrl/{brcm,capri-pinctrl.txt
=> brcm,bcm11351-pinctrl.txt} (98%)

  parent reply	other threads:[~2014-02-25  5:07 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-12 17:20 [GIT PULL] bcm pinctrl rename Christian Daudt
2014-02-12 17:20 ` Christian Daudt
2014-02-21 22:29 ` Olof Johansson
2014-02-21 22:29   ` Olof Johansson
2014-02-24 17:32   ` Christian Daudt
2014-02-24 17:32     ` Christian Daudt
2014-02-24 18:29     ` Matt Porter
2014-02-24 18:29       ` Matt Porter
2014-02-25  5:01   ` Christian Daudt [this message]
2014-02-25  5:01     ` Christian Daudt
2014-02-25 19:18     ` Arnd Bergmann
2014-02-25 19:18       ` Arnd Bergmann
2014-03-09  6:14       ` Olof Johansson
2014-03-09  6:14         ` Olof Johansson

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='CAEPRUYqUNxOT=KnhSmbXbL2e2tCs=jAhQqKLiVVc13okpBxHnw@mail.gmail.com' \
    --to=bcm@fixthebug.org \
    --cc=arm@kernel.org \
    --cc=arnd@arndb.de \
    --cc=khilman@linaro.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mporter@linaro.org \
    --cc=olof@lixom.net \
    --cc=syin@broadcom.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.