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.
>
> Then the endpoint id is always 0 and this function is pointless.
Sorry, I checked my patch-list, and I noticed that
this function will be expand to use callback function in next
patch-set (= HDMI support).
Thus, inded current function is pointless at this point.
I will merge this expansion patch in v5
--
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
