linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: bjorn.andersson@sonymobile.com (Bjorn Andersson)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v7 1/3] mfd: devicetree: bindings: Add Qualcomm RPM DT binding
Date: Wed, 24 Sep 2014 08:53:20 -0700	[thread overview]
Message-ID: <20140924155319.GA28481@sonymobile.com> (raw)
In-Reply-To: <20140924082228.GF17801@lee--X1>

On Wed 24 Sep 01:22 PDT 2014, Lee Jones wrote:

> On Mon, 22 Sep 2014, Bjorn Andersson wrote:
> 
> > Add binding for the Qualcomm Resource Power Manager (RPM) found in 8660,
> > 8960 and 8064 based devices. The binding currently describes the rpm
> > itself and the regulator subnodes.
> > 
> > Signed-off-by: Bjorn Andersson <bjorn.andersson@sonymobile.com>
> > ---
> >  Documentation/devicetree/bindings/mfd/qcom-rpm.txt |  246 ++++++++++++++++++++
> 
> Please adhere to: Documentation/devicetree/bindings/submitting-patches.txt
> 

I assume you mean this part:

  1) The Documentation/ portion of the patch should be a separate patch.

But I doubt this document was written with include/dt-bindings in mind, because
in my eyes they are part of "device tree bindings".

> >  include/dt-bindings/mfd/qcom-rpm.h                 |  154 ++++++++++++
> >  2 files changed, 400 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/mfd/qcom-rpm.txt
> >  create mode 100644 include/dt-bindings/mfd/qcom-rpm.h
> > 
> > diff --git a/Documentation/devicetree/bindings/mfd/qcom-rpm.txt b/Documentation/devicetree/bindings/mfd/qcom-rpm.txt
> > new file mode 100644
> > index 0000000..4264021
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/mfd/qcom-rpm.txt
> 
> This requires a DT Ack.
> 
> [...]
> 
> > +++ b/include/dt-bindings/mfd/qcom-rpm.h
> 
> Please split this into a separate patch.
> 

But this is part of the device tree binding and the binding "depends" on it.

> When you do, for this file only, please apply my:
> 
>   Acked-by: Lee Jones <lee.jones@linaro.org>
> 

Thanks,
Bjorn

  reply	other threads:[~2014-09-24 15:53 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-22 23:25 [PATCH v7 0/3] Qualcomm Resource Power Manager driver Bjorn Andersson
2014-09-22 23:25 ` [PATCH v7 1/3] mfd: devicetree: bindings: Add Qualcomm RPM DT binding Bjorn Andersson
2014-09-24  8:22   ` Lee Jones
2014-09-24 15:53     ` Bjorn Andersson [this message]
2014-09-24 16:21       ` Lee Jones
2014-09-24 16:39   ` Kumar Gala
2014-09-30 15:28     ` Bjorn Andersson
2014-09-30 16:02       ` Kumar Gala
2014-09-30 16:25         ` Bjorn Andersson
2014-09-22 23:25 ` [PATCH v7 2/3] mfd: qcom-rpm: Driver for the Qualcomm RPM Bjorn Andersson
2014-09-24  8:23   ` Lee Jones
2014-09-22 23:25 ` [PATCH v7 3/3] regulator: qcom-rpm: Regulator driver " Bjorn Andersson
2014-09-24  8:35   ` Mark Brown
2014-09-24 13:50     ` Kumar Gala
2014-09-24 15:45       ` Mark Brown
2014-09-30 15:33       ` Bjorn Andersson
2014-09-23  8:17 ` [PATCH v7 0/3] Qualcomm Resource Power Manager driver Srinivas Kandagatla
2014-09-24  4:27   ` Bjorn Andersson
2014-09-24  7:55     ` Lee Jones
2014-09-24  8:11       ` Lee Jones
2014-09-24 13:46     ` Kumar Gala

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=20140924155319.GA28481@sonymobile.com \
    --to=bjorn.andersson@sonymobile.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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).