Hi Andrzej,
On Monday 10 March 2014 09:58:07 Andrzej Hajda wrote:
> On 03/08/2014 04:54 PM, Laurent Pinchart wrote:
> > On Saturday 08 March 2014 13:07:23 Philipp Zabel wrote:
> >> On Fri, Mar 7, 2014 at 6:18 PM, Grant Likely wrote:
> >>> On Wed, 26 Feb 2014 16:24:57 +0100, Philipp Zabel wrote:
> >>>> The 'ports' node is optional. It is only needed if the parent node has
> >>>> its own #address-cells and #size-cells properties. If the ports are
> >>>> direct children of the device node, there might be other nodes than
> >>>>
> >>>> ports:
> >>>> device {
> >>>> #address-cells = <1>;
> >>>> #size-cells = <0>;
> >>>>
> >>>> port@0 {
> >>>> endpoint { ... };
> >>>> };
> >>>> port@1 {
> >>>> endpoint { ... };
> >>>> };
> >>>>
> >>>> some-other-child { ... };
> >>>> };
> >>>>
> >>>> device {
> >>>> #address-cells = <x>;
> >>>> #size-cells = <y>;
> >>>>
> >>>> ports {
> >>>> #address-cells = <1>;
> >>>> #size-cells = <0>;
> >>>>
> >>>> port@0 {
> >>>> endpoint { ... };
> >>>> };
> >>>> port@1 {
> >>>> endpoint { ... };
> >>>> };
> >>>> };
> >>>>
> >>>> some-other-child { ... };
> >>>> };
> >>>
> >>> From a pattern perspective I have no problem with that.... From an
> >>> individual driver binding perspective that is just dumb! It's fine for
> >>> the ports node to be optional, but an individual driver using the
> >>> binding should be explicit about which it will accept. Please use either
> >>> a flag or a separate wrapper so that the driver can select the
> >>> behaviour.
> >>
> >> If the generic binding exists in both forms, most drivers should be
> >> able to cope with both. Maybe it should be mentioned in the bindings
> >> that the short form without ports node should be used where possible
> >> (i.e. for devices that don't already have #address,size-cells != 1,0).
> >>
> >> Having a separate wrapper to enforce the ports node for devices that
> >> need it might be useful.
> >>
> >>>> The helper should find the two endpoints in both cases.
> >>>>
> >>>> Tomi suggests an even more compact form for devices with just one port:
> >>>>
> >>>> device {
> >>>> endpoint { ... };
> >>>>
> >>>> some-other-child { ... };
> >>>> };
> >>>
> >>> That's fine. In that case the driver would specifically require the
> >>> endpoint to be that one node.... although the above looks a little weird
> >>> to me. I would recommend that if there are other non-port child nodes
> >>> then the ports should still be encapsulated by a ports node. The device
> >>> binding should not be ambiguous about which nodes are ports.
> >>
> >> Sylwester suggested as an alternative, if I understood correctly, to
> >> drop the endpoint node and instead keep the port:
> >>
> >> device-a {
> >> implicit_output_ep: port {
> >> remote-endpoint = <&explicit_input_ep>;
> >> };
> >> };
> >>
> >> device-b {
> >> port {
> >> explicit_input_ep: endpoint {
> >> remote-endpoint = <&implicit_output_ep>;
> >> };
> >> };
> >> };
> >>
> >> This would have the advantage to reduce verbosity for devices with
> >> multiple ports that are only connected via one endport each, and you'd
> >> always have the connected ports in the device tree as 'port' nodes.
> >
> > I like that idea. I would prefer making the 'port' nodes mandatory and the
> > 'ports' and 'endpoint' nodes optional. Leaving the 'port' node out
> > slightly decreases readability in my opinion, but making the 'endpoint'
> > node optional increases it. That's just my point of view though.
>
> I want to propose another solution to simplify bindings, in fact I have
> few ideas to consider:
>
> 1. Use named ports instead of address-cells/regs. Ie instead of
> port@number schema, use port-function. This will allow to avoid ports
> node and #address-cells, #size-cells, reg properties.
> Additionally it should increase readability of the bindings.
>
> device {
> port-dsi {
> endpoint { ... };
> };
> port-rgb {
> endpoint { ... };
> };
> };
>
> It is little bit like with gpios vs reset-gpios properties.
> Another advantage I see we do not need do mappings of port numbers
> to functions between dts, drivers and documentation.
The problem with this approach is that ports are identified by a number inside
the kernel, so we would still need to define name to number mappings, or
switch to port names internally first.
> 2. Similar approach can be taken to endpoint nodes, in fact
> as endpoints are children of port node and as I understand port node
> have no other children we can use any name instead of endpoint@number,
> of course some convention can be helpful.
>
> device {
> port-dsi {
> ep-soc1 { ... };
> ep-soc2 { ... };
> };
> port-rgb {
> ep-panel { ... };
> };
> };
I see less issues here, as we don't need to number endpoints if I'm not
mistaken.
> I would like to add that those ideas would work nicely with Sylwester's
> proposition of skipping endpoints nodes in case there is only one
> endpoint - the most common cases are devices with one or two ports, each
> port having only one remote endpoint.
> The complete graph for DSI/LVDS bridge I work recently will look like:
>
> dsim {
> dsim_ep: port-dsi {
> remote-endpoint = <&bridge_dsi_ep>;
> };
> };
>
> bridge {
> bridge_dsi_ep: port-dsi {
> remote-endpoint = <&dsim_ep>;
> };
> bridge_lvds_ep: port-lvds {
> remote-endpoint = <&panel_ep>;
> };
> };
>
> panel {
> port-lvds {
> remote-endpoint <&bridge_lvds_ep>;
> };
> };
--
Regards,
Laurent Pinchart
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
