| Message ID | 20190604122305.07B9068B05@newverein.lst.de |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | Add anx6345 DP/eDP bridge for Olimex Teres-I | expand |
On 04.06.2019 14:23, Torsten Duwe wrote: > The anx6345 is an ultra-low power DisplayPort/eDP transmitter designed > for portable devices. > > Add a binding document for it. > > Signed-off-by: Icenowy Zheng <icenowy@aosc.io> > Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com> > Reviewed-by: Rob Herring <robh@kernel.org> > Signed-off-by: Torsten Duwe <duwe@suse.de> > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > --- > .../devicetree/bindings/display/bridge/anx6345.txt | 57 ++++++++++++++++++++++ > 1 file changed, 57 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/bridge/anx6345.txt > > diff --git a/Documentation/devicetree/bindings/display/bridge/anx6345.txt b/Documentation/devicetree/bindings/display/bridge/anx6345.txt > new file mode 100644 > index 000000000000..bd63f6ac107e > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/bridge/anx6345.txt > @@ -0,0 +1,57 @@ > +Analogix ANX6345 eDP Transmitter > +-------------------------------- > + > +The ANX6345 is an ultra-low power Full-HD eDP transmitter designed for > +portable devices. > + > +Required properties: > + > + - compatible : "analogix,anx6345" > + - reg : I2C address of the device > + - reset-gpios : Which GPIO to use for reset You have not specified it's active state, since in driver's code you named it RESETN I guess it should be active low. > + - dvdd12-supply : Regulator for 1.2V digital core power. > + - dvdd25-supply : Regulator for 2.5V digital core power. > + - Video port for LVTTL input, using the DT bindings defined in [1]. Please assign port number for input (I guess 0). > + > +Optional properties: > + > + - Video port for eDP output (panel or connector) using the DT bindings > + defined in [1]. Shouldn't it be also required? Regards Andrzej > + > +[1]: Documentation/devicetree/bindings/media/video-interfaces.txt > + > +Example: > + > +anx6345: anx6345@38 { > + compatible = "analogix,anx6345"; > + reg = <0x38>; > + reset-gpios = <&pio 3 24 GPIO_ACTIVE_LOW>; /* PD24 */ > + dvdd25-supply = <®_dldo2>; > + dvdd12-supply = <®_fldo1>; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + anx6345_in: port@0 { > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0>; > + anx6345_in_tcon0: endpoint@0 { > + reg = <0>; > + remote-endpoint = <&tcon0_out_anx6345>; > + }; > + }; > + > + anx6345_out: port@1 { > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <1>; > + > + anx6345_out_panel: endpoint@0 { > + reg = <0>; > + remote-endpoint = <&panel_in_edp>; > + }; > + }; > + }; > +};
On Wed, Jun 12, 2019 at 10:16:37AM +0200, Andrzej Hajda wrote: > > +The ANX6345 is an ultra-low power Full-HD eDP transmitter designed for > > +portable devices. > > + > > +Required properties: > > + > > + - compatible : "analogix,anx6345" > > + - reg : I2C address of the device > > + - reset-gpios : Which GPIO to use for reset > > > You have not specified it's active state, since in driver's code you > named it RESETN I guess it should be active low. Yes. The chip's reset is active low. > > > + - dvdd12-supply : Regulator for 1.2V digital core power. > > + - dvdd25-supply : Regulator for 2.5V digital core power. > > + - Video port for LVTTL input, using the DT bindings defined in [1]. > > > Please assign port number for input (I guess 0). True. > > > + > > +Optional properties: > > + > > + - Video port for eDP output (panel or connector) using the DT bindings > > + defined in [1]. > > > Shouldn't it be also required? See previous discussion. Surely there should be _something_ connected to the output side, but that something might not be relevant for the software side, so it might be omitted from the device tree. In fact, I'll submit v3 with the SPDX changes and without exactly this output port spec which had caused the heated discussion. Torsten
diff --git a/Documentation/devicetree/bindings/display/bridge/anx6345.txt b/Documentation/devicetree/bindings/display/bridge/anx6345.txt new file mode 100644 index 000000000000..bd63f6ac107e --- /dev/null +++ b/Documentation/devicetree/bindings/display/bridge/anx6345.txt @@ -0,0 +1,57 @@ +Analogix ANX6345 eDP Transmitter +-------------------------------- + +The ANX6345 is an ultra-low power Full-HD eDP transmitter designed for +portable devices. + +Required properties: + + - compatible : "analogix,anx6345" + - reg : I2C address of the device + - reset-gpios : Which GPIO to use for reset + - dvdd12-supply : Regulator for 1.2V digital core power. + - dvdd25-supply : Regulator for 2.5V digital core power. + - Video port for LVTTL input, using the DT bindings defined in [1]. + +Optional properties: + + - Video port for eDP output (panel or connector) using the DT bindings + defined in [1]. + +[1]: Documentation/devicetree/bindings/media/video-interfaces.txt + +Example: + +anx6345: anx6345@38 { + compatible = "analogix,anx6345"; + reg = <0x38>; + reset-gpios = <&pio 3 24 GPIO_ACTIVE_LOW>; /* PD24 */ + dvdd25-supply = <®_dldo2>; + dvdd12-supply = <®_fldo1>; + + ports { + #address-cells = <1>; + #size-cells = <0>; + + anx6345_in: port@0 { + #address-cells = <1>; + #size-cells = <0>; + reg = <0>; + anx6345_in_tcon0: endpoint@0 { + reg = <0>; + remote-endpoint = <&tcon0_out_anx6345>; + }; + }; + + anx6345_out: port@1 { + #address-cells = <1>; + #size-cells = <0>; + reg = <1>; + + anx6345_out_panel: endpoint@0 { + reg = <0>; + remote-endpoint = <&panel_in_edp>; + }; + }; + }; +};