Re: [PATCH v11 1/3] dt-bindings: display: Add powertip,{st7272|hx8238a} as DT Schema description

Lukasz Majewski
2024-12-20T10:30:03Z

[-- Attachment #1: Type: text/plain, Size: 4051 bytes --]

Dear Community,

> Dear Community,
>
> > This patch provides the DT Schema description of:
> > - powertip,st7272 320 x 240 LCD display
> > - powertip,hx8238a 320 x 240 LCD display
> >
> > Used with the different HW revisions of btt3 devices.
> >
> > Signed-off-by: Lukasz Majewski <lukma@denx.de>
> > Reviewed-by: Rob Herring <robh@kernel.org>
>
> Are there any more comments for this particular patch?
>
> If not - can it be applied as it prevents dts patchs to be pull by
> Shawn.

Gentle ping on this patch ...

>
> > ---
> >
> > Changes for v9:
> > - New patch
> >
> > Changes for v10:
> > - None
> >
> > Changes for v11:
> > - Combine both separate dt-bindings patches for powertip,st7272 and
> > powertip,hx8238a into one
> > - Drop the quotes for in "title" entry of powertip*.yaml files
> > ---
> > .../display/panel/powertip,hx8238a.yaml | 29
> > +++++++++++++++++++ .../display/panel/powertip,st7272.yaml |
> > 29 +++++++++++++++++++ 2 files changed, 58 insertions(+)
> > create mode 100644
> > Documentation/devicetree/bindings/display/panel/powertip,hx8238a.yaml
> > create mode 100644
> > Documentation/devicetree/bindings/display/panel/powertip,st7272.yaml
> >
> > diff --git
> > a/Documentation/devicetree/bindings/display/panel/powertip,hx8238a.yaml
> > b/Documentation/devicetree/bindings/display/panel/powertip,hx8238a.yaml
> > new file mode 100644 index 000000000000..b7d74faeb5d5 --- /dev/null
> > +++
> > b/Documentation/devicetree/bindings/display/panel/powertip,hx8238a.yaml
> > @@ -0,0 +1,29 @@ +# SPDX-License-Identifier: GPL-2.0-only OR
> > BSD-2-Clause +%YAML 1.2
> > +---
> > +$id:
> > http://devicetree.org/schemas/display/panel/powertip,hx8238a.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml# +
> > +title: Powertip Electronic Technology Co. 320 x 240 LCD panel
> > +
> > +maintainers:
> > + - Lukasz Majewski <lukma@denx.de>
> > +
> > +allOf:
> > + - $ref: panel-dpi.yaml#
> > +
> > +properties:
> > + compatible:
> > + items:
> > + - const: powertip,hx8238a
> > + - {} # panel-dpi, but not listed here to avoid false select
> > +
> > + height-mm: true
> > + panel-timing: true
> > + port: true
> > + power-supply: true
> > + width-mm: true
> > +
> > +additionalProperties: false
> > +
> > +...
> > diff --git
> > a/Documentation/devicetree/bindings/display/panel/powertip,st7272.yaml
> > b/Documentation/devicetree/bindings/display/panel/powertip,st7272.yaml
> > new file mode 100644 index 000000000000..f3622800f13f --- /dev/null
> > +++
> > b/Documentation/devicetree/bindings/display/panel/powertip,st7272.yaml
> > @@ -0,0 +1,29 @@ +# SPDX-License-Identifier: GPL-2.0-only OR
> > BSD-2-Clause +%YAML 1.2
> > +---
> > +$id:
> > http://devicetree.org/schemas/display/panel/powertip,st7272.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml# +
> > +title: Powertip Electronic Technology Co. 320 x 240 LCD panel
> > +
> > +maintainers:
> > + - Lukasz Majewski <lukma@denx.de>
> > +
> > +allOf:
> > + - $ref: panel-dpi.yaml#
> > +
> > +properties:
> > + compatible:
> > + items:
> > + - const: powertip,st7272
> > + - {} # panel-dpi, but not listed here to avoid false select
> > +
> > + height-mm: true
> > + panel-timing: true
> > + port: true
> > + power-supply: true
> > + width-mm: true
> > +
> > +additionalProperties: false
> > +
> > +...
>
>
>
>
> Best regards,
>
> Lukasz Majewski
>
> --
>
> DENX Software Engineering GmbH, Managing Director: Erika Unter
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email:
> lukma@denx.de