From: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>
Cc: Baruch Siach <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org>,
Sergei Shtylyov
<sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>,
Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
Florian Fainelli
<f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
"David S. Miller" <davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>,
Russell King <rmk+kernel-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH v2 2/3] dt-binding: net: sfp binding documentation
Date: Fri, 1 Sep 2017 12:12:37 -0500 [thread overview]
Message-ID: <20170901171237.ge7dconrtrog2tah@rob-hp-laptop> (raw)
In-Reply-To: <20170830145829.GA22289-g2DYL2Zd6BY@public.gmane.org>
On Wed, Aug 30, 2017 at 04:58:29PM +0200, Andrew Lunn wrote:
> > > > > Your example shows there's GPIO phandle *and* specifier.
> > > >
> > > > Would "GPIO specifier" be enough here?
> > >
> > > No, specifier is the cells following GPIO (or any other) phandle.
> >
> > So this should be "GPIO phandle and specifier of ...", is that correct?
> >
> > I have found very few (< 4) occurrences of this language in (lots of) '-gpios'
> > property descriptions under Documentation/devicetree/bindings/. Is this a new
> > requirement?
>
> Sometimes it is just easier to refer to another document:
>
> GPIO, as defined in Documentation/devicetree/binding/gpio/gpio.txt
Yes, and what I care about here is how many GPIOs, direction and active
state. IOW, worry about the information necessary to validate a specific
instance is correct. And hopefully someday we'll have a format parseable
to do that checking, and all the free form text will be gone.
Rob
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2017-09-01 17:12 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-30 9:51 [PATCH v2 1/3] dt-bindings: add SFF vendor prefix Baruch Siach
[not found] ` <d1e64c5752ef0dd5c5b543c2d4c3ef1783318173.1504086672.git.baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org>
2017-08-30 9:51 ` [PATCH v2 2/3] dt-binding: net: sfp binding documentation Baruch Siach
[not found] ` <7ae5d883fd038cf518949b0aa9cbe5369e2a1851.1504086672.git.baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org>
2017-08-30 11:04 ` Sergei Shtylyov
2017-08-30 11:11 ` Baruch Siach
2017-08-30 11:15 ` Sergei Shtylyov
[not found] ` <c3e8af44-bf58-81d7-bd51-d2538dcd41d9-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
2017-08-30 11:25 ` Baruch Siach
2017-08-30 12:13 ` Sergei Shtylyov
2017-08-30 14:58 ` Andrew Lunn
[not found] ` <20170830145829.GA22289-g2DYL2Zd6BY@public.gmane.org>
2017-09-01 17:12 ` Rob Herring [this message]
2017-08-30 9:51 ` [PATCH v2 3/3] net: phy: sfp: rename the rate-select property Baruch Siach
2017-09-01 17:13 ` [PATCH v2 1/3] dt-bindings: add SFF vendor prefix Rob Herring
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=20170901171237.ge7dconrtrog2tah@rob-hp-laptop \
--to=robh-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
--cc=andrew-g2DYL2Zd6BY@public.gmane.org \
--cc=baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org \
--cc=davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
--cc=netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=rmk+kernel-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
--cc=sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.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