On Fri, Oct 30, 2015 at 12:37 PM, Thierry Reding <thierry.reding@xxxxxxxxx> wrote: > On Fri, Oct 30, 2015 at 11:40:03AM -0500, Rob Herring wrote: >> On Fri, Oct 30, 2015 at 9:20 AM, Maxime Ripard > [...] >> > +Optional properties: >> > + - allwinner,tv-encoder: phandle to the TV Encoder in our pipeline >> > + - allwinner,panel: phandle to the panel used in our RGB interface >> >> Use of-graph please. > > Why? Panels are a really simple resource and a simple phandle is fully > capable of describing the relationship. So we can have some consistency in bindings and common parsing code. It is simple when it is just a panel, but when it can be tv-encoder or other board level downstream bridge devices it doesn't stay so simple. Also, we already have a simple way to do panels which is as a child node. Let's not have a third way. Rob -- 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