On Sat, Sep 20, 2014 at 8:57 PM, Javier Martinez Canillas <javier@xxxxxxxxxxxx> wrote: > [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. Kukjin, Can you do something about this? Or, I shall make video-port changes on top of whatever gets merged? Ajay > 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 > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |