devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Lee Jones <lee.jones@linaro.org>
To: Rob Herring <robherring2@gmail.com>
Cc: Peter Griffin <peter.griffin@linaro.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Srinivas Kandagatla <srinivas.kandagatla@gmail.com>,
	Maxime Coquelin <maxime.coquelin@st.com>,
	Patrice Chotard <patrice.chotard@st.com>,
	Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	hugues.fruchet@st.com,
	"linux-media@vger.kernel.org" <linux-media@vger.kernel.org>,
	valentinrothberg@gmail.com
Subject: Re: [PATCH v3 4/6] [media] c8sectpfe: Update binding to reset-gpios
Date: Tue, 1 Sep 2015 08:58:11 +0100	[thread overview]
Message-ID: <20150901075811.GK4796@x1> (raw)
In-Reply-To: <CAL_JsqLYes5L2Bg0R45ui24yYWgHzFSLgBPajieSLgLz09=1aw@mail.gmail.com>

On Mon, 31 Aug 2015, Rob Herring wrote:

> On Fri, Aug 28, 2015 at 12:52 PM, Peter Griffin
> <peter.griffin@linaro.org> wrote:
> > gpio.txt documents that GPIO properties should be named
> > "[<name>-]gpios", with <name> being the purpose of this
> > GPIO for the device.
> >
> > This change has been done as one atomic commit.
> 
> dtb and kernel updates are not necessarily atomic, so you are breaking
> compatibility with older dtbs. You should certainly highlight that in
> the commit message.

Good idea.

> I only point this out. I'll leave it to platform
> maintainers whether or not this breakage is acceptable.

This driver is new.  The 'original' bindings are in next.  So this
binding is not even close to being ABI.

[...]

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

  reply	other threads:[~2015-09-01  7:58 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-28 17:52 [PATCH v3 0/6] [media] c8sectpfe: Various fixups Peter Griffin
     [not found] ` <1440784362-31217-1-git-send-email-peter.griffin-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2015-08-28 17:52   ` [PATCH v3 1/6] ARM: DT: STi: stihxxx-b2120: Add pulse-width properties to ssc2 & ssc3 Peter Griffin
2015-08-28 17:52 ` [PATCH v3 2/6] ARM: DT: STi: STiH407: Add c8sectpfe LinuxDVB DT node Peter Griffin
2015-09-01  7:59   ` Lee Jones
2015-08-28 17:52 ` [PATCH v3 3/6] [media] c8sectpfe: Remove select on undefined LIBELF_32 Peter Griffin
2015-08-28 17:52 ` [PATCH v3 4/6] [media] c8sectpfe: Update binding to reset-gpios Peter Griffin
2015-08-31 15:25   ` Rob Herring
2015-09-01  7:58     ` Lee Jones [this message]
2015-09-01 10:12     ` Peter Griffin
2015-09-01  8:32   ` Javier Martinez Canillas
2015-09-01  9:09     ` Lee Jones
2015-09-01  9:16       ` Javier Martinez Canillas
2015-09-01  9:25         ` Lee Jones
     [not found]         ` <CABxcv==iC+mkVGKi_9LSidPjFa2WTooARMCOcWW1Nyr80UYJgw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-09-01 10:34           ` Peter Griffin
2015-09-01 12:09             ` Lee Jones
2015-09-01 11:54     ` Rob Herring
2015-09-01 12:30       ` Javier Martinez Canillas
2015-09-02 15:03       ` Peter Griffin
2015-08-28 17:52 ` [PATCH v3 5/6] [media] c8sectpfe: Update DT binding doc with some minor fixes Peter Griffin
2015-08-28 17:52 ` [PATCH v3 6/6] [media] c8sectpfe: Simplify for loop in load_slim_core_fw Peter Griffin

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=20150901075811.GK4796@x1 \
    --to=lee.jones@linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=hugues.fruchet@st.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=maxime.coquelin@st.com \
    --cc=mchehab@osg.samsung.com \
    --cc=patrice.chotard@st.com \
    --cc=peter.griffin@linaro.org \
    --cc=robherring2@gmail.com \
    --cc=srinivas.kandagatla@gmail.com \
    --cc=valentinrothberg@gmail.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).