| Message ID | 20230217-topic-lenovo-panel-v1-1-9d7ee1602089@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Add support for Lenovo NT36523W BOE panel | expand |
On Fri, Feb 17, 2023 at 12:29:07PM +0100, Konrad Dybcio wrote: > Add bindings for the 2000x1200px IPS panel found on Lenovo Tab P11/ > XiaoXin Pad devices. > > Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org> > --- > .../display/panel/lenovo,nt36523w-boe-j606.yaml | 60 ++++++++++++++++++++++ > 1 file changed, 60 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/panel/lenovo,nt36523w-boe-j606.yaml b/Documentation/devicetree/bindings/display/panel/lenovo,nt36523w-boe-j606.yaml > new file mode 100644 > index 000000000000..43dcbe3f9f30 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/panel/lenovo,nt36523w-boe-j606.yaml > @@ -0,0 +1,60 @@ > +# SPDX-License-Identifier: GPL-2.0-only or BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/display/panel/lenovo,nt36523w-boe-j606.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: NT36523W BOE panel found on Lenovo J606 devices > + > +maintainers: > + - Konrad Dybcio <konrad.dybcio@linaro.org> > + > +allOf: > + - $ref: panel-common.yaml# > + > +properties: > + compatible: > + const: lenovo,nt36523w-boe-j606 > + > + reg: > + maxItems: 1 > + description: DSI virtual channel > + > + vddio-supply: true If only one supply, why not use panel-simple-dsi.yaml? Though probably there's more than just an IO supply? Rob
On 21.02.2023 03:08, Rob Herring wrote: > On Fri, Feb 17, 2023 at 12:29:07PM +0100, Konrad Dybcio wrote: >> Add bindings for the 2000x1200px IPS panel found on Lenovo Tab P11/ >> XiaoXin Pad devices. >> >> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org> >> --- >> .../display/panel/lenovo,nt36523w-boe-j606.yaml | 60 ++++++++++++++++++++++ >> 1 file changed, 60 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/display/panel/lenovo,nt36523w-boe-j606.yaml b/Documentation/devicetree/bindings/display/panel/lenovo,nt36523w-boe-j606.yaml >> new file mode 100644 >> index 000000000000..43dcbe3f9f30 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/display/panel/lenovo,nt36523w-boe-j606.yaml >> @@ -0,0 +1,60 @@ >> +# SPDX-License-Identifier: GPL-2.0-only or BSD-2-Clause >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/display/panel/lenovo,nt36523w-boe-j606.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: NT36523W BOE panel found on Lenovo J606 devices >> + >> +maintainers: >> + - Konrad Dybcio <konrad.dybcio@linaro.org> >> + >> +allOf: >> + - $ref: panel-common.yaml# >> + >> +properties: >> + compatible: >> + const: lenovo,nt36523w-boe-j606 >> + >> + reg: >> + maxItems: 1 >> + description: DSI virtual channel >> + >> + vddio-supply: true > > If only one supply, why not use panel-simple-dsi.yaml? Though probably > there's more than just an IO supply? Display regulators (AB/IBB/AMOLEDB, not to be confused with LAB/IBB) on modern Qualcomm platforms are controlled by the secure firmware layer (yes, you read that correctly). Some panels require additional supplies, but this one just has one positive, one negative and one I/O regulator. Konrad > > Rob
On Fri, 17 Feb 2023 12:29:07 +0100, Konrad Dybcio wrote: > Add bindings for the 2000x1200px IPS panel found on Lenovo Tab P11/ > XiaoXin Pad devices. > > Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org> > --- > .../display/panel/lenovo,nt36523w-boe-j606.yaml | 60 ++++++++++++++++++++++ > 1 file changed, 60 insertions(+) > Reviewed-by: Rob Herring <robh@kernel.org>
diff --git a/Documentation/devicetree/bindings/display/panel/lenovo,nt36523w-boe-j606.yaml b/Documentation/devicetree/bindings/display/panel/lenovo,nt36523w-boe-j606.yaml new file mode 100644 index 000000000000..43dcbe3f9f30 --- /dev/null +++ b/Documentation/devicetree/bindings/display/panel/lenovo,nt36523w-boe-j606.yaml @@ -0,0 +1,60 @@ +# SPDX-License-Identifier: GPL-2.0-only or BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/display/panel/lenovo,nt36523w-boe-j606.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: NT36523W BOE panel found on Lenovo J606 devices + +maintainers: + - Konrad Dybcio <konrad.dybcio@linaro.org> + +allOf: + - $ref: panel-common.yaml# + +properties: + compatible: + const: lenovo,nt36523w-boe-j606 + + reg: + maxItems: 1 + description: DSI virtual channel + + vddio-supply: true + reset-gpios: true + rotation: true + port: true + +additionalProperties: false + +required: + - compatible + - reg + - vddio-supply + - reset-gpios + - port + +examples: + - | + #include <dt-bindings/gpio/gpio.h> + dsi { + #address-cells = <1>; + #size-cells = <0>; + + panel@0 { + compatible = "lenovo,nt36523w-boe-j606"; + reg = <0>; + + reset-gpios = <&tlmm 82 GPIO_ACTIVE_LOW>; + vddio-supply = <&pm6125_l9>; + + rotation = <180>; + + port { + panel0_in: endpoint { + remote-endpoint = <&mdss_dsi0_out>; + }; + }; + }; + }; +...
Add bindings for the 2000x1200px IPS panel found on Lenovo Tab P11/ XiaoXin Pad devices. Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org> --- .../display/panel/lenovo,nt36523w-boe-j606.yaml | 60 ++++++++++++++++++++++ 1 file changed, 60 insertions(+)