On Thu, Jan 19, 2017 at 12:31 AM, Kuninori Morimoto <kuninori.morimoto...@renesas.com> wrote: > > > > Hi Rob > > Now many driver is getting remote-endpoint by manually, > but we should use same method to get same result IMO. > Thus this patch adds of_graph_get_remote_endpoint() for this purpose. > And use it on several drivers
This looks okay, but I think we need to be improving the graph api at a higher level. Each user seems to be doing too much open coding of walking of the graph. The user typically just wants the remote node parent of a given port and endpoint number. Why does the user walk the graph to find the endpoint node, then parse remote-endpoint, then walk up parents. We have lots of other graph users and you didn't need to update them which means they are already using helpers. Maybe the drivers here need to be fixed. omapfb in particular looks like it duplicates a bunch of helpers we already have, but maybe we're close to removing that driver. Rob
