On Fri, Sep 13, 2024 at 06:07:47PM +0300, Dzmitry Sankouski wrote: > Add binding for the Samsung s6e3ha8 panel found in the Samsung S9. > > Signed-off-by: Dzmitry Sankouski <dsankouski@xxxxxxxxx> > > --- > Changes in v4: > - change dts example intendation from tabs > to spaces > - remove reset-gpios description > --- > .../bindings/display/panel/samsung,s6e3ha8.yaml | 75 ++++++++++++++++++++++ > MAINTAINERS | 5 ++ > 2 files changed, 80 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/panel/samsung,s6e3ha8.yaml b/Documentation/devicetree/bindings/display/panel/samsung,s6e3ha8.yaml > new file mode 100644 > index 000000000000..94c812e07571 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/panel/samsung,s6e3ha8.yaml > @@ -0,0 +1,75 @@ > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/display/panel/samsung,s6e3ha8.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Samsung s6e3ha8 AMOLED DSI panel > + > +description: The s6e3ha8 is a 1440x2960 DPI display panel from Samsung Mobile > + Displays (SMD). > + > +maintainers: > + - Dzmitry Sankouski <dsankouski@xxxxxxxxx> > + > +allOf: > + - $ref: panel-common.yaml# > + > +properties: > + compatible: > + const: samsung,s6e3ha8 > + > + reg: > + maxItems: 1 > + > + reset-gpios: true > + > + port: true > + > + vdd3-supply: > + description: VDD regulator > + > + vci-supply: > + description: VCI regulator > + > + vddr-supply: > + description: VDDR regulator > + > +required: > + - compatible > + - reset-gpios > + - vdd3-supply > + - vddr-supply > + - vci-supply If there is going to be resend, then keep the same order as in properties: block. > + > +unevaluatedProperties: false > + > +examples: > + - | > + #include <dt-bindings/gpio/gpio.h> > + > + dsi { > + #address-cells = <1>; > + #size-cells = <0>; > + > + panel@0 { > + compatible = "samsung,s6e3ha8"; > + reg = <0>; > + vci-supply = <&s2dos05_ldo4>; > + vddr-supply = <&s2dos05_buck1>; > + vdd3-supply = <&s2dos05_ldo1>; > + te-gpios = <&tlmm 10 GPIO_ACTIVE_HIGH>; > + reset-gpios = <&tlmm 6 GPIO_ACTIVE_HIGH>; > + pinctrl-0 = <&sde_dsi_active &sde_te_active_sleep>; > + pinctrl-1 = <&sde_dsi_suspend &sde_te_active_sleep>; > + pinctrl-names = "default", "sleep"; > + > + port { > + panel_in: endpoint { > + remote-endpoint = <&mdss_dsi0_out>; > + }; > + }; > + }; Indentation is still mixed up. > + }; > + > +... Best regards, Krzysztof
