Hi Rob
Thank you for your review
> > +{
> > + struct device_node *node;
> > + struct device_node *endpoint;
> > + int i, id;
> > +
> > + node = of_graph_get_port_parent(ep);
> > +
> > + i = 0;
> > + id = -1;
> > + for_each_endpoint_of_node(node, endpoint) {
> > + if (endpoint == ep)
> > + id = i;
>
> I don't see how this works when you have 1 DAI controller with
> multiple endpoints versus multiple DAI controllers with a single
> endpoint each. All the IDs will be 0 in the latter case.
It support 1:1 endpoint pattern only.
Best regards
---
Kuninori Morimoto
--
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
