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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Tue, Feb 7, 2017 at 12:55 PM, Fabio Estevam <festevam@xxxxxxxxx> wrote:
> Hi Rob,
>
> On Tue, Feb 7, 2017 at 4:49 PM, Rob Herring <robh@xxxxxxxxxx> 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
--
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



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux