[PATCH] drm/panel: simple: Add support for Seiko 43WVF1G
Rob Herring
robh at kernel.org
Tue Feb 7 23:36:38 UTC 2017
On Tue, Feb 7, 2017 at 12:55 PM, Fabio Estevam <festevam at gmail.com> wrote:
> Hi Rob,
>
> On Tue, Feb 7, 2017 at 4:49 PM, Rob Herring <robh at kernel.org> wrote:
>
>> No power supply(ies) for this panel?
>
> power-supply is mentioned in simple-panel.txt.
>
>>> +This binding is compatible with the simple-panel binding, which is specified
>>> +in simple-panel.txt in this directory.
>
> and this doc refers to simple-panel.txt. Isn't that enough?
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.
> Seems like a common pattern for the other simple panel binding doc.
We've gotten stricter about this.
Rob
More information about the dri-devel
mailing list