From: Eric Hyeung Dong Jeong <eric.jeong.opensource@diasemi.com>
To: Rob Herring <robh@kernel.org>,
Eric Hyeung Dong Jeong <eric.jeong.opensource@diasemi.com>
Cc: DEVICETREE <devicetree@vger.kernel.org>,
LINUX-KERNEL <linux-kernel@vger.kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
Alexandre Courbot <gnurou@gmail.com>,
LINUX-GPIO <linux-gpio@vger.kernel.org>,
Lee Jones <lee.jones@linaro.org>,
Liam Girdwood <lgirdwood@gmail.com>,
Linus Walleij <linus.walleij@linaro.org>,
Mark Brown <broonie@kernel.org>,
Support Opensource <Support.Opensource@diasemi.com>
Subject: RE: [PATCH V3 1/4] Documentation: pv88080: Move binding document
Date: Tue, 22 Nov 2016 09:20:40 +0000 [thread overview]
Message-ID: <8FA4409277D6E54680546B8811D8541002A087DE90@NB-EX-MBX01.diasemi.com> (raw)
In-Reply-To: <20161118153842.bbcmfczxvrwamlqs@rob-hp-laptop>
On Saturday, November 19, 2016 12:39 AM, Rob Herring wrote:
> On Fri, Nov 18, 2016 at 09:35:46AM +0900, Eric Jeong wrote:
> >
> > From: Eric Jeong <eric.jeong.opensource@diasemi.com>
> >
> > The change is to move pv88080 binding document from the regulator
> > directory to mfd binding directory for PV88080 PMIC MFD support.
> > And, GPIO properties are added.
> >
> >
> > Signed-off-by: Eric Jeong <eric.jeong.opensource@diasemi.com>
> >
> > ---
> > This patch applies against linux-next and next-20161117
> >
> > Hi,
> >
> > Change since PATCH V2
> > - Added property and description for gpio
> >
> > Change since PATCH V1
> > - Patch separated from PATCH V1
> >
> > Regards,
> > Eric Jeong, Dialog Semiconductor Ltd.
> >
> >
> > .../bindings/{regulator => mfd}/pv88080.txt | 16 +++++++++++++++-
> > 1 file changed, 15 insertions(+), 1 deletion(-) rename
> > Documentation/devicetree/bindings/{regulator => mfd}/pv88080.txt (79%)
> >
> > diff --git a/Documentation/devicetree/bindings/regulator/pv88080.txt
> > b/Documentation/devicetree/bindings/mfd/pv88080.txt
> > similarity index 79%
> > rename from Documentation/devicetree/bindings/regulator/pv88080.txt
> > rename to Documentation/devicetree/bindings/mfd/pv88080.txt
> > index e6e4b9c8..7e24f95 100644
> > --- a/Documentation/devicetree/bindings/regulator/pv88080.txt
> > +++ b/Documentation/devicetree/bindings/mfd/pv88080.txt
> > @@ -1,4 +1,4 @@
> > -* Powerventure Semiconductor PV88080 Voltage Regulator
> > +* Powerventure Semiconductor PV88080 PMIC
> >
> > Required properties:
> > - compatible: Must be one of the following, depending on the @@ -16,8
> > +16,15 @@ Required properties:
> > standard binding for regulators; see regulator.txt.
> > BUCK1, BUCK2, BUCK3 and HVBUCK.
> >
> > +- gpio-controller: Marks the device node as a GPIO controller.
> > +- #gpio-cells: Should be 2. See gpio.txt in this directory
> > + for a description of the cells format.
> > +
> > Optional properties:
> > +- ngpios : Number of in-use slots of available slots for GPIO.
> > + Maximum is 2.
> > - Any optional property defined in regulator.txt
> > + and gpio.txt for more information.
> >
> > Example:
> >
> > @@ -27,6 +34,13 @@ Example:
> > interrupt-parent = <&gpio>;
> > interrupts = <24 24>;
> >
> > + gpioex {
>
> gpio-controller {
>
> > + compatible = "pvs,pv88080-gpio";
>
> Is this documented?
No, It's not documented. I will update this document with the compatible string.
>
> > + gpio-controller;
> > + #gpio-cells = <2>;
> > + ngpios = <2>;
> > + };
> > +
> > regulators {
> > BUCK1 {
> > regulator-name = "buck1";
> > --
> > end-of-patch for PATCH V3
> >
next prev parent reply other threads:[~2016-11-22 9:21 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-18 0:35 [PATCH V3 0/4] pv88080: PV88080 driver submission Eric Jeong
2016-11-18 0:35 ` [PATCH V3 2/4] mfd: pv88080: MFD core support Eric Jeong
2016-11-21 13:09 ` Lee Jones
2016-11-25 6:03 ` Eric Hyeung Dong Jeong
2016-11-25 8:34 ` Lee Jones
2016-11-18 0:35 ` [PATCH V3 3/4] regulator: pv88080: Update Regulator driver for MFD support Eric Jeong
2016-11-18 12:00 ` Mark Brown
2016-11-18 0:35 ` [PATCH V3 1/4] Documentation: pv88080: Move binding document Eric Jeong
2016-11-18 15:38 ` Rob Herring
2016-11-22 9:20 ` Eric Hyeung Dong Jeong [this message]
2016-11-18 0:35 ` [PATCH V3 4/4] gpio: pv88080: Add GPIO function support Eric Jeong
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=8FA4409277D6E54680546B8811D8541002A087DE90@NB-EX-MBX01.diasemi.com \
--to=eric.jeong.opensource@diasemi.com \
--cc=Support.Opensource@diasemi.com \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=gnurou@gmail.com \
--cc=lee.jones@linaro.org \
--cc=lgirdwood@gmail.com \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=robh@kernel.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).