Messages in this thread | | | From | Grant Likely <> | Subject | Re: [PATCH v4 3/3] Documentation: of: Document graph bindings | Date | Sat, 08 Mar 2014 01:20:57 +0800 |
| |
On Wed, 26 Feb 2014 15:14:17 +0200, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote: > On 25/02/14 16:58, Philipp Zabel wrote: > > > +Optional endpoint properties > > +---------------------------- > > + > > +- remote-endpoint: phandle to an 'endpoint' subnode of a remote device node. > > Why is that optional? What use is an endpoint, if it's not connected to > something? > > Also, if this is being worked on, I'd like to propose the addition of > simpler single-endpoint cases which I've been using with OMAP DSS. So if > there's just a single endpoint for the device, which is very common, you > can have just: > > device { > ... > endpoint { ... }; > }; > > However, I guess that the patch just keeps growing and growing, so maybe > it's better to add such things later =).
It's good to talk about it now while boiling down the core behaviour into a useful pattern. That said, I think if the stuff I've commented on already is addressed then it will probably be sufficient for me to ack or merge the change.
g.
| |