Hi Peter, Thank you for the patch. On Sunday, 18 March 2018 00:15:23 EET Peter Rosin wrote: > Start list of actual chips compatible with "lvds-encoder". > > Signed-off-by: Peter Rosin <peda@xxxxxxxxxx> > --- > .../devicetree/bindings/display/bridge/lvds-transmitter.txt | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git > a/Documentation/devicetree/bindings/display/bridge/lvds-transmitter.txt > b/Documentation/devicetree/bindings/display/bridge/lvds-transmitter.txt > index fd39ad34c383..9d09190d9210 100644 > --- a/Documentation/devicetree/bindings/display/bridge/lvds-transmitter.txt > +++ b/Documentation/devicetree/bindings/display/bridge/lvds-transmitter.txt > @@ -24,6 +24,11 @@ Required properties: > > - compatible: Must be "lvds-encoder" > > + Known actual chips (these should still use "lvds-encoder" as a > + fallback compatible) include: > + > + "ti,ds90c185" The wording sounds a bit strange to me. How about - compatible: Must be one or more of the following - "ti,ds90c185" for the TI DS90C185 FPD-Link Serializer - "lvds-encoder" for a generic LVDS encoder device When compatible with the generic version nodes must list the device-specific version corresponding to the device first followed by the generic version. > Required nodes: > > This device has two video ports. Their connections are modeled using the OF -- Regards, Laurent Pinchart -- 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