| Message ID | 1446430166-7378-1-git-send-email-jitao.shi@mediatek.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Mon, 2015-11-02 at 10:09 +0800, Jitao Shi wrote: > Add documentation for DT properties supported by > ps8640 DSI-eDP converter. > > Signed-off-by: Jitao Shi <jitao.shi@mediatek.com> --- changes since v1: 1. changes power-gpios to sleep-gpios. 2. drop "ps8640-" in properties. 3. add ports property. > --- > .../devicetree/bindings/display/bridge/ps8640.txt | 43 ++++++++++++++++++++ > 1 file changed, 43 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/bridge/ps8640.txt > > diff --git a/Documentation/devicetree/bindings/display/bridge/ps8640.txt b/Documentation/devicetree/bindings/display/bridge/ps8640.txt > new file mode 100644 > index 0000000..7edc547 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/bridge/ps8640.txt > @@ -0,0 +1,43 @@ > +ps8640-bridge bindings > + > +Required properties: > + - compatible: "parade,ps8640" > + - reg: first page address of the bridge. > + - sleep-gpios: OF device-tree gpio specification for PD_ pin. > + - reset-gpios: OF device-tree gpio specification for reset pin. > + - mode-sel-gpios: OF device-tree gpio specification for mode-sel pin. > + - vdd12-supply: OF device-tree regulator specification for 1.2V power. > + - vdd33-supply: OF device-tree regulator specification for 3.3V power. > + - ports: The device node can contain video interface port nodes per > + the video-interfaces bind[1]. For port@0,set the reg = <0> as > + ps8640 dsi in and port@1,set the reg = <1> as ps8640 eDP out. > + > +[1]: Documentation/devicetree/bindings/media/video-interfaces.txt > + > +Example: > + edp-bridge@18 { > + compatible = "parade,ps8640"; > + reg = <0x18>; > + sleep-gpios = <&pio 116 GPIO_ACTIVE_HIGH>; > + reset-gpios = <&pio 115 GPIO_ACTIVE_HIGH>; > + mode-sel-gpios = <&pio 92 GPIO_ACTIVE_HIGH>; > + ps8640-1v2-supply = <&ps8640_fixed_1v2>; > + ps8640-3v3-supply = <&mt6397_vgp2_reg>; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + port@0 { > + reg = <0>; > + ps8640_in: endpoint { > + remote-endpoint = <&dsi0_out>; > + }; > + }; > + port@1 { > + reg = <1>; > + ps8640_out: endpoint { > + remote-endpoint = <&panel_in>; > + }; > + }; > + }; > + };
Hi Jitao, Am Montag, den 02.11.2015, 11:53 +0800 schrieb jitao shi: [...] > > +Example: > > + edp-bridge@18 { > > + compatible = "parade,ps8640"; > > + reg = <0x18>; > > + sleep-gpios = <&pio 116 GPIO_ACTIVE_HIGH>; > > + reset-gpios = <&pio 115 GPIO_ACTIVE_HIGH>; > > + mode-sel-gpios = <&pio 92 GPIO_ACTIVE_HIGH>; > > + ps8640-1v2-supply = <&ps8640_fixed_1v2>; This should be vdd12-supply now. > > + ps8640-3v3-supply = <&mt6397_vgp2_reg>; Should be vdd33-supply now. regards Philipp
Hi Jitao, Am Montag, den 02.11.2015, 10:09 +0800 schrieb Jitao Shi: > Add documentation for DT properties supported by > ps8640 DSI-eDP converter. > > Signed-off-by: Jitao Shi <jitao.shi@mediatek.com> > --- > .../devicetree/bindings/display/bridge/ps8640.txt | 43 ++++++++++++++++++++ > 1 file changed, 43 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/bridge/ps8640.txt > > diff --git a/Documentation/devicetree/bindings/display/bridge/ps8640.txt b/Documentation/devicetree/bindings/display/bridge/ps8640.txt > new file mode 100644 > index 0000000..7edc547 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/bridge/ps8640.txt > @@ -0,0 +1,43 @@ > +ps8640-bridge bindings > + > +Required properties: > + - compatible: "parade,ps8640" > + - reg: first page address of the bridge. > + - sleep-gpios: OF device-tree gpio specification for PD_ pin. > + - reset-gpios: OF device-tree gpio specification for reset pin. > + - mode-sel-gpios: OF device-tree gpio specification for mode-sel pin. > + - vdd12-supply: OF device-tree regulator specification for 1.2V power. > + - vdd33-supply: OF device-tree regulator specification for 3.3V power. > + - ports: The device node can contain video interface port nodes per > + the video-interfaces bind[1]. For port@0,set the reg = <0> as > + ps8640 dsi in and port@1,set the reg = <1> as ps8640 eDP out. > + > +[1]: Documentation/devicetree/bindings/media/video-interfaces.txt > + > +Example: > + edp-bridge@18 { > + compatible = "parade,ps8640"; > + reg = <0x18>; > + sleep-gpios = <&pio 116 GPIO_ACTIVE_HIGH>; > + reset-gpios = <&pio 115 GPIO_ACTIVE_HIGH>; > + mode-sel-gpios = <&pio 92 GPIO_ACTIVE_HIGH>; > + ps8640-1v2-supply = <&ps8640_fixed_1v2>; > + ps8640-3v3-supply = <&mt6397_vgp2_reg>; These two should be updated to the new regulator property names. regards Philipp
On Mon, 2015-11-02 at 12:09 +0100, Philipp Zabel wrote: > Hi Jitao, > > Am Montag, den 02.11.2015, 11:53 +0800 schrieb jitao shi: > [...] > > > +Example: > > > + edp-bridge@18 { > > > + compatible = "parade,ps8640"; > > > + reg = <0x18>; > > > + sleep-gpios = <&pio 116 GPIO_ACTIVE_HIGH>; > > > + reset-gpios = <&pio 115 GPIO_ACTIVE_HIGH>; > > > + mode-sel-gpios = <&pio 92 GPIO_ACTIVE_HIGH>; > > > + ps8640-1v2-supply = <&ps8640_fixed_1v2>; > > This should be vdd12-supply now. Thanks a lot. I'll fix it on V3. > > > > + ps8640-3v3-supply = <&mt6397_vgp2_reg>; > > Should be vdd33-supply now. I'll fix it on V3. > > regards > Philipp >
On Mon, Nov 02, 2015 at 10:09:25AM +0800, Jitao Shi wrote: > Add documentation for DT properties supported by > ps8640 DSI-eDP converter. > > Signed-off-by: Jitao Shi <jitao.shi@mediatek.com> Acked-by: Rob Herring <robh@kernel.org> > --- > .../devicetree/bindings/display/bridge/ps8640.txt | 43 ++++++++++++++++++++ > 1 file changed, 43 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/bridge/ps8640.txt > > diff --git a/Documentation/devicetree/bindings/display/bridge/ps8640.txt b/Documentation/devicetree/bindings/display/bridge/ps8640.txt > new file mode 100644 > index 0000000..7edc547 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/bridge/ps8640.txt > @@ -0,0 +1,43 @@ > +ps8640-bridge bindings > + > +Required properties: > + - compatible: "parade,ps8640" > + - reg: first page address of the bridge. > + - sleep-gpios: OF device-tree gpio specification for PD_ pin. > + - reset-gpios: OF device-tree gpio specification for reset pin. > + - mode-sel-gpios: OF device-tree gpio specification for mode-sel pin. > + - vdd12-supply: OF device-tree regulator specification for 1.2V power. > + - vdd33-supply: OF device-tree regulator specification for 3.3V power. > + - ports: The device node can contain video interface port nodes per > + the video-interfaces bind[1]. For port@0,set the reg = <0> as > + ps8640 dsi in and port@1,set the reg = <1> as ps8640 eDP out. > + > +[1]: Documentation/devicetree/bindings/media/video-interfaces.txt > + > +Example: > + edp-bridge@18 { > + compatible = "parade,ps8640"; > + reg = <0x18>; > + sleep-gpios = <&pio 116 GPIO_ACTIVE_HIGH>; > + reset-gpios = <&pio 115 GPIO_ACTIVE_HIGH>; > + mode-sel-gpios = <&pio 92 GPIO_ACTIVE_HIGH>; > + ps8640-1v2-supply = <&ps8640_fixed_1v2>; > + ps8640-3v3-supply = <&mt6397_vgp2_reg>; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + port@0 { > + reg = <0>; > + ps8640_in: endpoint { > + remote-endpoint = <&dsi0_out>; > + }; > + }; > + port@1 { > + reg = <1>; > + ps8640_out: endpoint { > + remote-endpoint = <&panel_in>; > + }; > + }; > + }; > + }; > -- > 1.7.9.5 >
diff --git a/Documentation/devicetree/bindings/display/bridge/ps8640.txt b/Documentation/devicetree/bindings/display/bridge/ps8640.txt new file mode 100644 index 0000000..7edc547 --- /dev/null +++ b/Documentation/devicetree/bindings/display/bridge/ps8640.txt @@ -0,0 +1,43 @@ +ps8640-bridge bindings + +Required properties: + - compatible: "parade,ps8640" + - reg: first page address of the bridge. + - sleep-gpios: OF device-tree gpio specification for PD_ pin. + - reset-gpios: OF device-tree gpio specification for reset pin. + - mode-sel-gpios: OF device-tree gpio specification for mode-sel pin. + - vdd12-supply: OF device-tree regulator specification for 1.2V power. + - vdd33-supply: OF device-tree regulator specification for 3.3V power. + - ports: The device node can contain video interface port nodes per + the video-interfaces bind[1]. For port@0,set the reg = <0> as + ps8640 dsi in and port@1,set the reg = <1> as ps8640 eDP out. + +[1]: Documentation/devicetree/bindings/media/video-interfaces.txt + +Example: + edp-bridge@18 { + compatible = "parade,ps8640"; + reg = <0x18>; + sleep-gpios = <&pio 116 GPIO_ACTIVE_HIGH>; + reset-gpios = <&pio 115 GPIO_ACTIVE_HIGH>; + mode-sel-gpios = <&pio 92 GPIO_ACTIVE_HIGH>; + ps8640-1v2-supply = <&ps8640_fixed_1v2>; + ps8640-3v3-supply = <&mt6397_vgp2_reg>; + + ports { + #address-cells = <1>; + #size-cells = <0>; + port@0 { + reg = <0>; + ps8640_in: endpoint { + remote-endpoint = <&dsi0_out>; + }; + }; + port@1 { + reg = <1>; + ps8640_out: endpoint { + remote-endpoint = <&panel_in>; + }; + }; + }; + };
Add documentation for DT properties supported by ps8640 DSI-eDP converter. Signed-off-by: Jitao Shi <jitao.shi@mediatek.com> --- .../devicetree/bindings/display/bridge/ps8640.txt | 43 ++++++++++++++++++++ 1 file changed, 43 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/bridge/ps8640.txt