[PATCH] drm/panel: simple: Add support for Seiko 43WVF1G

Fabio Estevam festevam at gmail.com
Wed Feb 8 00:48:16 UTC 2017


On Tue, Feb 7, 2017 at 9:36 PM, Rob Herring <robh at kernel.org> wrote:

> Except I have no way of knowing whether: a) you omitted a supply
> because you don't (yet) care, b) the panel has a single supply and you
> are using power-supply or c) the panel has multiple supplies and your
> binding is wrong.
>
> I can only eliminate A if you list the supplies. Just need something
> like "power-supply : see simple-panel.txt". I've still got to go read
> the panel spec if I really want to check the binding.

Just checked the panel datasheet at
http://www.glyn.de/data/glyn/media/doc/43wvf1g-0.pdf and it lists two
supplies:
DVDD (Digital power supply) and AVDD (Analog power supply).

Our dts was just providing a single 'power-supply' which referred to a
GPIO enabled regulator that drives DVDD.

So it seems we missed to pass AVDD (not software controlled in our
case, but we need to describe it in dts anyway).

Does this mean we cannot use simple-panel for this particular panel
and we should add a separate driver for it?

Thanks


More information about the dri-devel mailing list