[adding Kukjin as cc] Hello Ajay, On Sat, Sep 20, 2014 at 1:22 PM, Ajay kumar <ajaynumb@xxxxxxxxx> wrote: >> Generally speaking, I sense that we have different views of how display >> devices and drivers are structured. You say "If some XYZ platform wishes >> to pick the DT node via a different method, they are always welcome to >> do it.". This sounds to me that you see the connections between display >> devices as something handled by a platform specific driver. >> I, on the other hand, see connections between display devices as common >> properties. > Well, I am okay with using video ports to describe the relationship > between the encoder, bridge and the panel. > But, its just that I need to make use of 2 functions when phandle > does it using just one function ;) > - panel_node = of_parse_phandle(dev->of_node, "panel", 0) > + endpoint = of_graph_get_next_endpoint(dev->of_node, NULL); > + if (!endpoint) > + return -EPROBE_DEFER; > + > + panel_node = of_graph_get_remote_port_parent(endpoint); > + if (!panel_node) > + return -EPROBE_DEFER; > > > If nobody else has objections over using of_graph functions instead > of phandles, I can respin this patchset by making use of video ports. I certainly have no objections about re-using the video ports/endpoints DT bindings for the bridges but just wanted to point out that Kukjin has already merged on his tree the DTS changes for display on Snow and Peach Pit using the current binding and also sent the pull request [0] to ARM SoC maintainers since he probably was expecting this series to make ti for 3.18. So that should be handled somehow. Tomi, I see that Documentation/devicetree/bindings/video/ti,omap-dss.txt mentions that the Video Ports binding documentation is in Documentation/devicetree/bindings/video/video-ports.txt but I don't see that this file exists in the kernel [1]. I see though that is was included on your series adding DSS DT support [2]. Do you know what happened with this file? Best regards, Javier [0]: https://www.mail-archive.com/linux-samsung-soc@xxxxxxxxxxxxxxx/msg36681.html [1]: https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/tree/Documentation/devicetree/bindings/video [2]: http://lists.infradead.org/pipermail/linux-arm-kernel/2014-January/227088.html _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
