Hi Laurent, On Fri, Apr 06, 2018 at 04:15:35PM +0300, Laurent Pinchart wrote: > Hi Jacopo, > > Thank you for the patch. > > On Friday, 6 April 2018 15:41:56 EEST Jacopo Mondi wrote: > > Document Thine THC63LVD1024 LVDS decoder device tree bindings. > > > > Signed-off-by: Jacopo Mondi <jacopo+rene...@jmondi.org> > > Reviewed-by: Andrzej Hajda <a.ha...@samsung.com> > > Reviewed-by: Niklas Söderlund <niklas.soderlund+rene...@ragnatech.se> > > Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com> > > --- > > .../bindings/display/bridge/thine,thc63lvd1024.txt | 60 +++++++++++++++++++ > > 1 file changed, 60 insertions(+) > > create mode 100644 > > Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.txt > > > > diff --git > > a/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.txt > > b/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.txt > > new file mode 100644 > > index 0000000..1191f17 > > --- /dev/null > > +++ > > b/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.txt > > @@ -0,0 +1,60 @@ > > +Thine Electronics THC63LVD1024 LVDS decoder > > +------------------------------------------- > > + > > +The THC63LVD1024 is a dual link LVDS receiver designed to convert LVDS > > streams > > +to parallel data outputs. The chip supports single/dual input/output modes, > > +handling up to two two input LVDS stream and up to two digital CMOS/TTL > > s/to two two/to two/ > s/stream/streams/ > > > outputs. > > + > > +Single or dual operation modes, output data mapping and DDR output modes > > are > > +configured through input signals and the chip does not expose any control > > bus. > > + > > +Required properties: > > +- compatible: Shall be "thine,thc63lvd1024" > > + > > +Optional properties: > > +- vcc-supply: Power supply for TTL output, TTL CLOCKOUT signal, LVDS input, > > + PPL and digital circuitry > > +- powerdown-gpios: Power down GPIO signal, pin name "/PDWN". Active low > > +- enable-gpios: Output enable GPIO signal, pin name "OE". Active high > > As Rob mentioned in a reply to v6, we currently use "enable" as the inverse of > "powerdown". I would call this one oe-gpios instead. Quoting Rob: > > "Debating "oe" vs. "output-enable" is bikeshedding IMO. Anyone familiar with > h/w design should recognize OE." >
I got a different understanding of what Rob meant. I thought "anyone familiar with h/w design should recognize OE" as that nobody would get confused if a pin named OE in the chip manual is descibed by an 'enable' property. But as discussed offline, enable has probably to be used as the opposite of powerdown for complete chip sleep, not just for output pad. Anyway, we spent enough time on naming issues, starting from my first stupid 'pdwn' permutations then on this semi-standard names. I'll send next version with 'powerdown-gpios' and 'oe-gpios' properties hoping that would be finally accepted by everyone. Same on the mandatory/optional VCC supply thing. Let's try to make next version the final one. If the optional property with the dummy regulator doesn't satisfy you and it is preferred to have a fixed-regulator anyhow in DT I'll do in next version, othewise let's try not to change it again. I'll just remark here that in the current Eagle design vcc is connected to a power rail with no regulator at all :) Thanks j > > + > > +The THC63LVD1024 video port connections are modeled according > > +to OF graph bindings specified by Documentation/devicetree/bindings/ > > graph.txt > > + > > +Required video port nodes: > > +- port@0: First LVDS input port > > +- port@2: First digital CMOS/TTL parallel output > > + > > +Optional video port nodes: > > +- port@1: Second LVDS input port > > +- port@3: Second digital CMOS/TTL parallel output > > + > > +Example: > > +-------- > > + > > + thc63lvd1024: lvds-decoder { > > + compatible = "thine,thc63lvd1024"; > > + > > + vcc-supply = <®_lvds_vcc>; > > + powerdown-gpios = <&gpio4 15 GPIO_ACTIVE_LOW>; > > + > > + ports { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + port@0 { > > + reg = <0>; > > + > > + lvds_dec_in_0: endpoint { > > + remote-endpoint = <&lvds_out>; > > + }; > > + }; > > + > > + port@2{ > > + reg = <2>; > > + > > + lvds_dec_out_2: endpoint { > > + remote-endpoint = <&adv7511_in>; > > + }; > > + }; > > + }; > > + }; > > -- > Regards, > > Laurent Pinchart > > >
signature.asc
Description: PGP signature
_______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel