From: Steffen Trumtrar <s.trumtrar@pengutronix.de>
To: Philipp Zabel <p.zabel@pengutronix.de>
Cc: Alessandro Zummo <a.zummo@towertech.it>,
Alexandre Belloni <alexandre.belloni@free-electrons.com>,
Mark Rutland <mark.rutland@arm.com>,
devicetree@vger.kernel.org, rtc-linux@googlegroups.com,
Pawel Moll <pawel.moll@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Rob Herring <robh+dt@kernel.org>,
kernel@pengutronix.de, Kumar Gala <galak@codeaurora.org>
Subject: [rtc-linux] Re: [PATCH v3 1/2] Documentation: devicetree: add epson rx6110 binding
Date: Thu, 10 Dec 2015 11:56:59 +0100 [thread overview]
Message-ID: <20151210105659.GC20212@pengutronix.de> (raw)
In-Reply-To: <1449570566.3691.31.camel@pengutronix.de>
Hi!
On Tue, Dec 08, 2015 at 11:29:26AM +0100, Philipp Zabel wrote:
> Am Dienstag, den 08.12.2015, 10:54 +0100 schrieb Steffen Trumtrar:
> > Add the binding documentation for the Epson RX6110 RTC.
> >
> > Acked-by: Rob Herring <robh@kernel.org>
> > Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
>
> Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de>
>
> although I'd like a small change to be made below:
>
> > ---
> > Changes since v2:
> > - removed the size field in i2c binding
> >
> > .../devicetree/bindings/rtc/epson,rx6110.txt | 39 ++++++++++++++++++++++
> > 1 file changed, 39 insertions(+)
> > create mode 100644 Documentation/devicetree/bindings/rtc/epson,rx6110.txt
> >
> > diff --git a/Documentation/devicetree/bindings/rtc/epson,rx6110.txt b/Documentation/devicetree/bindings/rtc/epson,rx6110.txt
> > new file mode 100644
> > index 000000000000..71353542a59d
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/rtc/epson,rx6110.txt
> > @@ -0,0 +1,39 @@
> > +Epson RX6110 Real Time Clock
> > +============================
> > +
> > +The Epson RX6110 can be used with SPI or I2C busses. The kind of
> > +bus depends on the SPISEL pin and can not be configured via software.
> > +
> > +I2C mode
> > +--------
> > +
> > +Required properties:
> > + - compatible: should be: "epson,rx6110"
> > + - reg : offset of the register set of the device
>
> That should be the I2C address of the device, as in:
> git grep "reg.*I2C address" Documentation/
>
Sounds reasonable. Changed.
Thanks,
Steffen Trumtrar
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
--
--
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
---
You received this message because you are subscribed to the Google Groups "rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
next prev parent reply other threads:[~2015-12-10 10:57 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-08 9:54 [rtc-linux] [PATCH v3 1/2] Documentation: devicetree: add epson rx6110 binding Steffen Trumtrar
2015-12-08 9:54 ` [rtc-linux] [PATCH v3 2/2] rtc: add driver for RX6110SA real time clock Steffen Trumtrar
2015-12-08 10:30 ` [rtc-linux] " Philipp Zabel
2015-12-10 11:01 ` Steffen Trumtrar
2015-12-08 10:29 ` [rtc-linux] Re: [PATCH v3 1/2] Documentation: devicetree: add epson rx6110 binding Philipp Zabel
2015-12-10 10:56 ` Steffen Trumtrar [this message]
2015-12-08 10:29 ` Philipp Zabel
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=20151210105659.GC20212@pengutronix.de \
--to=s.trumtrar@pengutronix.de \
--cc=a.zummo@towertech.it \
--cc=alexandre.belloni@free-electrons.com \
--cc=devicetree@vger.kernel.org \
--cc=galak@codeaurora.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=kernel@pengutronix.de \
--cc=mark.rutland@arm.com \
--cc=p.zabel@pengutronix.de \
--cc=pawel.moll@arm.com \
--cc=robh+dt@kernel.org \
--cc=rtc-linux@googlegroups.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