On Wed, Feb 03, 2016 at 02:19:31PM -0600, Rob Herring wrote: > >> > +Required properties: > >> > + - compatible: value must be one of: > >> > + * allwinner,sun5i-a13-display-engine > >> > + - allwinner,pipelines: list of phandle to the entry points of the > >> > + pipelines (either to the frontend or backend) > >> > + > >> > +Example: > >> > + > >> > +panel: panel { > >> > + compatible = "olimex,lcd-olinuxino-43-ts"; > >> > >> This should either be a child of what it is attached to or use of-graph. > > > > You had a similar discussion with Thierry last time I sent it, and it > > seemed to settle on keeping that property: > > > > http://lists.infradead.org/pipermail/linux-arm-kernel/2015-October/382115.html > > > > Has that changed? > > No, I still disagree. We already have 2 ways to describe panels: as a > child node of its controller or with of-graph. We don't need a 3rd way > even if it is simple. The problem with of-graph today is too much of > the parsing is left to the individual drivers. We need to fix that and > make it be handled by common code. Ok. I'm guessing from your answer you prefer of-graph over a child node ? Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
Attachment:
signature.asc
Description: Digital signature