| Message ID | 1604993797-14240-1-git-send-email-victor.liu@nxp.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v2] dt-bindings: display: panel: one file of all simple LVDS panels with dual ports | expand |
On Wed, 2020-11-11 at 16:55 -0600, Rob Herring wrote: > On Tue, Nov 10, 2020 at 03:36:37PM +0800, Liu Ying wrote: > > To complement panel-simple.yaml, create panel-simple-lvds-dual- > > ports.yaml. > > panel-simple-lvds-dual-ports.yaml is for all simple LVDS panels > > that > > has dual LVDS ports and requires only a single power-supply. > > One LVDS port receives even pixels, and the other receives odd > > pixels. > > Optionally, a backlight and an enable GPIO can be specified as > > properties. > > > > Migrate 'auo,g133han01', 'auo,g185han01', 'auo,g190ean01', > > 'koe,tx26d202vm0bwa' and 'nlt,nl192108ac18-02d' over to the new > > file. > > > > The objectives with one file for all the simple LVDS panels with > > dual ports are: > > - Make it simpler to add bindings for this kind of LVDS panels > > - Keep the number of bindings file lower > > - Keep the binding documentation for this kind of LVDS panels more > > consistent > > - Make it possible for drivers to get pixel order via > > drm_of_lvds_get_dual_link_pixel_order(), as the optional 'ports' > > property is > > allowed > > > > Suggested-by: Sam Ravnborg <sam@ravnborg.org> > > Cc: Thierry Reding <thierry.reding@gmail.com> > > Cc: Sam Ravnborg <sam@ravnborg.org> > > Cc: David Airlie <airlied@linux.ie> > > Cc: Daniel Vetter <daniel@ffwll.ch> > > Cc: Rob Herring <robh+dt@kernel.org> > > Cc: Lucas Stach <l.stach@pengutronix.de> > > Cc: Sebastian Reichel <sebastian.reichel@collabora.com> > > Signed-off-by: Liu Ying <victor.liu@nxp.com> > > --- > > v1->v2: > > * Correct pixel order in example LVDS panel node. > > > > .../panel/panel-simple-lvds-dual-ports.yaml | 85 > > ++++++++++++++++++++++ > > .../bindings/display/panel/panel-simple.yaml | 10 --- > > 2 files changed, 85 insertions(+), 10 deletions(-) > > create mode 100644 > > Documentation/devicetree/bindings/display/panel/panel-simple-lvds- > > dual-ports.yaml > > > > diff --git a/Documentation/devicetree/bindings/display/panel/panel- > > simple-lvds-dual-ports.yaml > > b/Documentation/devicetree/bindings/display/panel/panel-simple- > > lvds-dual-ports.yaml > > new file mode 100644 > > index 00000000..58774ed > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/display/panel/panel-simple- > > lvds-dual-ports.yaml > > @@ -0,0 +1,85 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: > > http://devicetree.org/schemas/display/panel/panel-simple-lvds-dual-ports.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Simple LVDS panels with one power supply and dual LVDS > > ports > > + > > +maintainers: > > + - Liu Ying <victor.liu@nxp.com> > > + - Thierry Reding <thierry.reding@gmail.com> > > + - Sam Ravnborg <sam@ravnborg.org> > > + > > +description: | > > + This binding file is a collection of the LVDS panels that > > + has dual LVDS ports and requires only a single power-supply. > > + One LVDS port receives even pixels, and the other receives odd > > pixels. > > You have to define which port number is which in the schema. Ok. It looks that most planes receive odd pixels via the first port and even pixels via the second port, at least all the planes to be migrated do, so I'll specify that in v3. > > > + There are optionally a backlight and an enable GPIO. > > + The panel may use an OF graph binding for the association to the > > display, > > + or it may be a direct child node of the display. > > + > > + If the panel is more advanced a dedicated binding file is > > required. > > + > > +allOf: > > + - $ref: panel-common.yaml# > > + > > +properties: > > + > > + compatible: > > + enum: > > + # compatible must be listed in alphabetical order, ordered by > > compatible. > > + # The description in the comment is mandatory for each > > compatible. > > + > > + # AU Optronics Corporation 13.3" FHD (1920x1080) TFT LCD > > panel > > + - auo,g133han01 > > + # AU Optronics Corporation 18.5" FHD (1920x1080) TFT LCD > > panel > > + - auo,g185han01 > > + # AU Optronics Corporation 19.0" (1280x1024) TFT LCD panel > > + - auo,g190ean01 > > + # Kaohsiung Opto-Electronics Inc. 10.1" WUXGA (1920 x > > 1200) LVDS TFT LCD panel > > + - koe,tx26d202vm0bwa > > + # NLT Technologies, Ltd. 15.6" FHD (1920x1080) LVDS TFT > > LCD panel > > + - nlt,nl192108ac18-02d > > + > > + backlight: true > > + enable-gpios: true > > + port: true > > A single port shouldn't be valid. This is inherited from panel-simple.yaml, but I assume it can dropped in v3 because it looks that the panels to be migrated are NOT referenced in any in-tree devicetree. > > > + ports: true > > + power-supply: true > > + > > +additionalProperties: false > > + > > +required: > > + - compatible > > + - power-supply > > + > > +examples: > > + - | > > + panel: panel-lvds { > > + compatible = "koe,tx26d202vm0bwa"; > > + power-supply = <&vdd_lcd_reg>; > > + > > + ports { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + port@0 { > > + dual-lvds-odd-pixels; > > This needs to be documented. Though really, this property seems > pointless if the port numbering is fixed (though 0 for even and 1 > for > odd would be a bit more logical). Will do in v3. Not sure if there is any panel swaps the pixel order or not. If there is, a dedicated binding is needed perhaps. > > > + reg = <0>; > > + > > + panel_lvds0_in: endpoint { > > + remote-endpoint = <&lvds0_out>; > > + }; > > + }; > > + > > + port@1 { > > + dual-lvds-even-pixels; > > + reg = <1>; > > + > > + panel_lvds1_in: endpoint { > > + remote-endpoint = <&lvds1_out>; > > + }; > > + }; > > + }; > > + }; > > diff --git a/Documentation/devicetree/bindings/display/panel/panel- > > simple.yaml > > b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml > > index f9750b0..62618e4 100644 > > --- a/Documentation/devicetree/bindings/display/panel/panel- > > simple.yaml > > +++ b/Documentation/devicetree/bindings/display/panel/panel- > > simple.yaml > > @@ -57,14 +57,8 @@ properties: > > - auo,g104sn02 > > # AU Optronics Corporation 12.1" (1280x800) TFT LCD panel > > - auo,g121ean01 > > - # AU Optronics Corporation 13.3" FHD (1920x1080) TFT LCD > > panel > > - - auo,g133han01 > > # AU Optronics Corporation 15.6" (1366x768) TFT LCD panel > > - auo,g156xtn01 > > - # AU Optronics Corporation 18.5" FHD (1920x1080) TFT LCD > > panel > > - - auo,g185han01 > > - # AU Optronics Corporation 19.0" (1280x1024) TFT LCD panel > > - - auo,g190ean01 > > # AU Optronics Corporation 31.5" FHD (1920x1080) TFT LCD > > panel > > - auo,p320hvn03 > > # AU Optronics Corporation 21.5" FHD (1920x1080) color TFT > > LCD panel > > @@ -167,8 +161,6 @@ properties: > > - kingdisplay,kd116n21-30nv-a010 > > # Kaohsiung Opto-Electronics Inc. 5.7" QVGA (320 x 240) > > TFT LCD panel > > - koe,tx14d24vm1bpa > > - # Kaohsiung Opto-Electronics Inc. 10.1" WUXGA (1920 x > > 1200) LVDS TFT LCD panel > > - - koe,tx26d202vm0bwa > > # Kaohsiung Opto-Electronics. TX31D200VM0BAA 12.3" HSXGA > > LVDS panel > > - koe,tx31d200vm0baa > > # Kyocera Corporation 12.1" XGA (1024x768) TFT LCD panel > > @@ -205,8 +197,6 @@ properties: > > - neweast,wjfh116008a > > # Newhaven Display International 480 x 272 TFT LCD panel > > - newhaven,nhd-4.3-480272ef-atxl > > - # NLT Technologies, Ltd. 15.6" FHD (1920x1080) LVDS TFT > > LCD panel > > - - nlt,nl192108ac18-02d > > # New Vision Display 7.0" 800 RGB x 480 TFT LCD panel > > - nvd,9128 > > # OKAYA Electric America, Inc. RS800480T-7X0GP 7" WVGA LCD > > panel > > -- > > 2.7.4 > > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Liu Ying
On Wed, 2020-11-11 at 16:57 -0600, Rob Herring wrote: > On Tue, Nov 10, 2020 at 03:36:37PM +0800, Liu Ying wrote: > > To complement panel-simple.yaml, create panel-simple-lvds-dual- > > ports.yaml. > > panel-simple-lvds-dual-ports.yaml is for all simple LVDS panels > > that > > has dual LVDS ports and requires only a single power-supply. > > One LVDS port receives even pixels, and the other receives odd > > pixels. > > Optionally, a backlight and an enable GPIO can be specified as > > properties. > > > > Migrate 'auo,g133han01', 'auo,g185han01', 'auo,g190ean01', > > 'koe,tx26d202vm0bwa' and 'nlt,nl192108ac18-02d' over to the new > > file. > > Also, what about advantech,idk-2121wr? It won't be migrated, because it is compatible with 'panel- lvds'(covered by lvds.yaml) and it requires some properties which are NOT allowed in this binding doc, like width-mm, data-mapping, panel- timing. > > > The objectives with one file for all the simple LVDS panels with > > dual ports are: > > - Make it simpler to add bindings for this kind of LVDS panels > > - Keep the number of bindings file lower > > - Keep the binding documentation for this kind of LVDS panels more > > consistent > > - Make it possible for drivers to get pixel order via > > drm_of_lvds_get_dual_link_pixel_order(), as the optional 'ports' > > property is > > allowed > > > > Suggested-by: Sam Ravnborg <sam@ravnborg.org> > > Cc: Thierry Reding <thierry.reding@gmail.com> > > Cc: Sam Ravnborg <sam@ravnborg.org> > > Cc: David Airlie <airlied@linux.ie> > > Cc: Daniel Vetter <daniel@ffwll.ch> > > Cc: Rob Herring <robh+dt@kernel.org> > > Cc: Lucas Stach <l.stach@pengutronix.de> > > Cc: Sebastian Reichel <sebastian.reichel@collabora.com> > > Signed-off-by: Liu Ying <victor.liu@nxp.com> > > --- > > v1->v2: > > * Correct pixel order in example LVDS panel node. > > > > .../panel/panel-simple-lvds-dual-ports.yaml | 85 > > ++++++++++++++++++++++ > > .../bindings/display/panel/panel-simple.yaml | 10 --- > > 2 files changed, 85 insertions(+), 10 deletions(-) > > create mode 100644 > > Documentation/devicetree/bindings/display/panel/panel-simple-lvds- > > dual-ports.yaml > > > > diff --git a/Documentation/devicetree/bindings/display/panel/panel- > > simple-lvds-dual-ports.yaml > > b/Documentation/devicetree/bindings/display/panel/panel-simple- > > lvds-dual-ports.yaml > > new file mode 100644 > > index 00000000..58774ed > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/display/panel/panel-simple- > > lvds-dual-ports.yaml > > @@ -0,0 +1,85 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: > > http://devicetree.org/schemas/display/panel/panel-simple-lvds-dual-ports.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Simple LVDS panels with one power supply and dual LVDS > > ports > > + > > +maintainers: > > + - Liu Ying <victor.liu@nxp.com> > > + - Thierry Reding <thierry.reding@gmail.com> > > + - Sam Ravnborg <sam@ravnborg.org> > > + > > +description: | > > + This binding file is a collection of the LVDS panels that > > + has dual LVDS ports and requires only a single power-supply. > > + One LVDS port receives even pixels, and the other receives odd > > pixels. > > + There are optionally a backlight and an enable GPIO. > > + The panel may use an OF graph binding for the association to the > > display, > > + or it may be a direct child node of the display. > > + > > + If the panel is more advanced a dedicated binding file is > > required. > > + > > +allOf: > > + - $ref: panel-common.yaml# > > + > > +properties: > > + > > + compatible: > > + enum: > > + # compatible must be listed in alphabetical order, ordered by > > compatible. > > + # The description in the comment is mandatory for each > > compatible. > > + > > + # AU Optronics Corporation 13.3" FHD (1920x1080) TFT LCD > > panel > > + - auo,g133han01 > > + # AU Optronics Corporation 18.5" FHD (1920x1080) TFT LCD > > panel > > + - auo,g185han01 > > + # AU Optronics Corporation 19.0" (1280x1024) TFT LCD panel > > + - auo,g190ean01 > > + # Kaohsiung Opto-Electronics Inc. 10.1" WUXGA (1920 x > > 1200) LVDS TFT LCD panel > > + - koe,tx26d202vm0bwa > > + # NLT Technologies, Ltd. 15.6" FHD (1920x1080) LVDS TFT > > LCD panel > > + - nlt,nl192108ac18-02d > > + > > + backlight: true > > + enable-gpios: true > > + port: true > > + ports: true > > + power-supply: true > > + > > +additionalProperties: false > > + > > +required: > > + - compatible > > + - power-supply > > + > > +examples: > > + - | > > + panel: panel-lvds { > > + compatible = "koe,tx26d202vm0bwa"; > > + power-supply = <&vdd_lcd_reg>; > > + > > + ports { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + port@0 { > > + dual-lvds-odd-pixels; > > + reg = <0>; > > + > > + panel_lvds0_in: endpoint { > > + remote-endpoint = <&lvds0_out>; > > + }; > > + }; > > + > > + port@1 { > > + dual-lvds-even-pixels; > > + reg = <1>; > > + > > + panel_lvds1_in: endpoint { > > + remote-endpoint = <&lvds1_out>; > > + }; > > + }; > > + }; > > + }; > > diff --git a/Documentation/devicetree/bindings/display/panel/panel- > > simple.yaml > > b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml > > index f9750b0..62618e4 100644 > > --- a/Documentation/devicetree/bindings/display/panel/panel- > > simple.yaml > > +++ b/Documentation/devicetree/bindings/display/panel/panel- > > simple.yaml > > @@ -57,14 +57,8 @@ properties: > > - auo,g104sn02 > > # AU Optronics Corporation 12.1" (1280x800) TFT LCD panel > > - auo,g121ean01 > > - # AU Optronics Corporation 13.3" FHD (1920x1080) TFT LCD > > panel > > - - auo,g133han01 > > # AU Optronics Corporation 15.6" (1366x768) TFT LCD panel > > - auo,g156xtn01 > > - # AU Optronics Corporation 18.5" FHD (1920x1080) TFT LCD > > panel > > - - auo,g185han01 > > - # AU Optronics Corporation 19.0" (1280x1024) TFT LCD panel > > - - auo,g190ean01 > > # AU Optronics Corporation 31.5" FHD (1920x1080) TFT LCD > > panel > > - auo,p320hvn03 > > # AU Optronics Corporation 21.5" FHD (1920x1080) color TFT > > LCD panel > > @@ -167,8 +161,6 @@ properties: > > - kingdisplay,kd116n21-30nv-a010 > > # Kaohsiung Opto-Electronics Inc. 5.7" QVGA (320 x 240) > > TFT LCD panel > > - koe,tx14d24vm1bpa > > - # Kaohsiung Opto-Electronics Inc. 10.1" WUXGA (1920 x > > 1200) LVDS TFT LCD panel > > - - koe,tx26d202vm0bwa > > # Kaohsiung Opto-Electronics. TX31D200VM0BAA 12.3" HSXGA > > LVDS panel > > - koe,tx31d200vm0baa > > # Kyocera Corporation 12.1" XGA (1024x768) TFT LCD panel > > @@ -205,8 +197,6 @@ properties: > > - neweast,wjfh116008a > > # Newhaven Display International 480 x 272 TFT LCD panel > > - newhaven,nhd-4.3-480272ef-atxl > > - # NLT Technologies, Ltd. 15.6" FHD (1920x1080) LVDS TFT > > LCD panel > > - - nlt,nl192108ac18-02d > > # New Vision Display 7.0" 800 RGB x 480 TFT LCD panel > > - nvd,9128 > > # OKAYA Electric America, Inc. RS800480T-7X0GP 7" WVGA LCD > > panel > > -- > > 2.7.4 > > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Liu Ying
diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple-lvds-dual-ports.yaml b/Documentation/devicetree/bindings/display/panel/panel-simple-lvds-dual-ports.yaml new file mode 100644 index 00000000..58774ed --- /dev/null +++ b/Documentation/devicetree/bindings/display/panel/panel-simple-lvds-dual-ports.yaml @@ -0,0 +1,85 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/display/panel/panel-simple-lvds-dual-ports.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Simple LVDS panels with one power supply and dual LVDS ports + +maintainers: + - Liu Ying <victor.liu@nxp.com> + - Thierry Reding <thierry.reding@gmail.com> + - Sam Ravnborg <sam@ravnborg.org> + +description: | + This binding file is a collection of the LVDS panels that + has dual LVDS ports and requires only a single power-supply. + One LVDS port receives even pixels, and the other receives odd pixels. + There are optionally a backlight and an enable GPIO. + The panel may use an OF graph binding for the association to the display, + or it may be a direct child node of the display. + + If the panel is more advanced a dedicated binding file is required. + +allOf: + - $ref: panel-common.yaml# + +properties: + + compatible: + enum: + # compatible must be listed in alphabetical order, ordered by compatible. + # The description in the comment is mandatory for each compatible. + + # AU Optronics Corporation 13.3" FHD (1920x1080) TFT LCD panel + - auo,g133han01 + # AU Optronics Corporation 18.5" FHD (1920x1080) TFT LCD panel + - auo,g185han01 + # AU Optronics Corporation 19.0" (1280x1024) TFT LCD panel + - auo,g190ean01 + # Kaohsiung Opto-Electronics Inc. 10.1" WUXGA (1920 x 1200) LVDS TFT LCD panel + - koe,tx26d202vm0bwa + # NLT Technologies, Ltd. 15.6" FHD (1920x1080) LVDS TFT LCD panel + - nlt,nl192108ac18-02d + + backlight: true + enable-gpios: true + port: true + ports: true + power-supply: true + +additionalProperties: false + +required: + - compatible + - power-supply + +examples: + - | + panel: panel-lvds { + compatible = "koe,tx26d202vm0bwa"; + power-supply = <&vdd_lcd_reg>; + + ports { + #address-cells = <1>; + #size-cells = <0>; + + port@0 { + dual-lvds-odd-pixels; + reg = <0>; + + panel_lvds0_in: endpoint { + remote-endpoint = <&lvds0_out>; + }; + }; + + port@1 { + dual-lvds-even-pixels; + reg = <1>; + + panel_lvds1_in: endpoint { + remote-endpoint = <&lvds1_out>; + }; + }; + }; + }; diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml index f9750b0..62618e4 100644 --- a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml +++ b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml @@ -57,14 +57,8 @@ properties: - auo,g104sn02 # AU Optronics Corporation 12.1" (1280x800) TFT LCD panel - auo,g121ean01 - # AU Optronics Corporation 13.3" FHD (1920x1080) TFT LCD panel - - auo,g133han01 # AU Optronics Corporation 15.6" (1366x768) TFT LCD panel - auo,g156xtn01 - # AU Optronics Corporation 18.5" FHD (1920x1080) TFT LCD panel - - auo,g185han01 - # AU Optronics Corporation 19.0" (1280x1024) TFT LCD panel - - auo,g190ean01 # AU Optronics Corporation 31.5" FHD (1920x1080) TFT LCD panel - auo,p320hvn03 # AU Optronics Corporation 21.5" FHD (1920x1080) color TFT LCD panel @@ -167,8 +161,6 @@ properties: - kingdisplay,kd116n21-30nv-a010 # Kaohsiung Opto-Electronics Inc. 5.7" QVGA (320 x 240) TFT LCD panel - koe,tx14d24vm1bpa - # Kaohsiung Opto-Electronics Inc. 10.1" WUXGA (1920 x 1200) LVDS TFT LCD panel - - koe,tx26d202vm0bwa # Kaohsiung Opto-Electronics. TX31D200VM0BAA 12.3" HSXGA LVDS panel - koe,tx31d200vm0baa # Kyocera Corporation 12.1" XGA (1024x768) TFT LCD panel @@ -205,8 +197,6 @@ properties: - neweast,wjfh116008a # Newhaven Display International 480 x 272 TFT LCD panel - newhaven,nhd-4.3-480272ef-atxl - # NLT Technologies, Ltd. 15.6" FHD (1920x1080) LVDS TFT LCD panel - - nlt,nl192108ac18-02d # New Vision Display 7.0" 800 RGB x 480 TFT LCD panel - nvd,9128 # OKAYA Electric America, Inc. RS800480T-7X0GP 7" WVGA LCD panel
To complement panel-simple.yaml, create panel-simple-lvds-dual-ports.yaml. panel-simple-lvds-dual-ports.yaml is for all simple LVDS panels that has dual LVDS ports and requires only a single power-supply. One LVDS port receives even pixels, and the other receives odd pixels. Optionally, a backlight and an enable GPIO can be specified as properties. Migrate 'auo,g133han01', 'auo,g185han01', 'auo,g190ean01', 'koe,tx26d202vm0bwa' and 'nlt,nl192108ac18-02d' over to the new file. The objectives with one file for all the simple LVDS panels with dual ports are: - Make it simpler to add bindings for this kind of LVDS panels - Keep the number of bindings file lower - Keep the binding documentation for this kind of LVDS panels more consistent - Make it possible for drivers to get pixel order via drm_of_lvds_get_dual_link_pixel_order(), as the optional 'ports' property is allowed Suggested-by: Sam Ravnborg <sam@ravnborg.org> Cc: Thierry Reding <thierry.reding@gmail.com> Cc: Sam Ravnborg <sam@ravnborg.org> Cc: David Airlie <airlied@linux.ie> Cc: Daniel Vetter <daniel@ffwll.ch> Cc: Rob Herring <robh+dt@kernel.org> Cc: Lucas Stach <l.stach@pengutronix.de> Cc: Sebastian Reichel <sebastian.reichel@collabora.com> Signed-off-by: Liu Ying <victor.liu@nxp.com> --- v1->v2: * Correct pixel order in example LVDS panel node. .../panel/panel-simple-lvds-dual-ports.yaml | 85 ++++++++++++++++++++++ .../bindings/display/panel/panel-simple.yaml | 10 --- 2 files changed, 85 insertions(+), 10 deletions(-) create mode 100644 Documentation/devicetree/bindings/display/panel/panel-simple-lvds-dual-ports.yaml