| Message ID | 20250109154149.1212631-1-lukma@denx.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [RESEND,v11,1/3] dt-bindings: display: Add powertip,{st7272|hx8238a} as DT Schema description | expand |
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. > Gentle ping on this patch... > Signed-off-by: Lukasz Majewski <lukma@denx.de> > Reviewed-by: Rob Herring <robh@kernel.org> > --- > > 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
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. > > > > Gentle ping on this patch... > Gentle ping on this patch... This patch is blocking adding DTS description for BTT3 device - as Shawn shared that he will pull other patches from this patch set when this one is in. Thanks in advance for help. > > Signed-off-by: Lukasz Majewski <lukma@denx.de> > > Reviewed-by: Rob Herring <robh@kernel.org> > > --- > > > > 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 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
On 27/01/2025 09:19, Lukasz Majewski wrote: > 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. >>> >> >> Gentle ping on this patch... Merge window, not need to ping anyone, unless these are fixes. Are these? Do not look like. >> > > Gentle ping on this patch... Still merge window, really no need to ping twice. Best regards, Krzysztof
Hi Krzysztof, > On 27/01/2025 09:19, Lukasz Majewski wrote: > > 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. > >>> > >> > >> Gentle ping on this patch... > > > Merge window, not need to ping anyone, unless these are fixes. Are > these? Do not look like. Please correct me if I'm wring (or something has changed recently), but wasn't there a policy that DTS bindings are allowed to be pulled no matter if merge window is open or not? > > >> > > > > Gentle ping on this patch... > > Still merge window, really no need to ping twice. > > Best regards, > Krzysztof 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
On 27/01/2025 09:46, Lukasz Majewski wrote: > Hi Krzysztof, > >> On 27/01/2025 09:19, Lukasz Majewski wrote: >>> 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. >>>>> >>>> >>>> Gentle ping on this patch... >> >> >> Merge window, not need to ping anyone, unless these are fixes. Are >> these? Do not look like. > > Please correct me if I'm wring (or something has changed recently), but > wasn't there a policy that DTS bindings are allowed to be pulled no > matter if merge window is open or not? Bindings? No such rule, no difference between bindings and any other thing. No non-fix patches are taken by maintainers during merge window, with some exceptions. There is a difference in DRM, though, because it follows a bit different cycle schedule, but you pinged us all, not only DRM folks. So was the ping towards DRM? Is their tree now open? Best regards, Krzysztof
Hi Krzysztof, > On 27/01/2025 09:46, Lukasz Majewski wrote: > > Hi Krzysztof, > > > >> On 27/01/2025 09:19, Lukasz Majewski wrote: > >>> 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. > >>>>> > >>>> > >>>> Gentle ping on this patch... > >> > >> > >> Merge window, not need to ping anyone, unless these are fixes. Are > >> these? Do not look like. > > > > Please correct me if I'm wring (or something has changed recently), > > but wasn't there a policy that DTS bindings are allowed to be > > pulled no matter if merge window is open or not? > > > Bindings? No such rule, no difference between bindings and any other > thing. No non-fix patches are taken by maintainers during merge > window, with some exceptions. There is a difference in DRM, though, > because it follows a bit different cycle schedule, but you pinged us > all, not only DRM folks. Thanks for explanation - I've been mistaken... > > So was the ping towards DRM? Is their tree now open? > You mean https://cgit.freedesktop.org/drm/drm ? Some work (i.e. code is pull) is done there. > Best regards, > Krzysztof 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
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. > Gentle ping on this window... I hope that it will make it into upstream during this merge window. Thanks in advance. > Signed-off-by: Lukasz Majewski <lukma@denx.de> > Reviewed-by: Rob Herring <robh@kernel.org> > --- > > 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
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. > > > > Gentle ping on this window... I hope that it will make it into Not "ping on this window" :-) but on this patch :-) > upstream during this merge window. > > Thanks in advance. > > > Signed-off-by: Lukasz Majewski <lukma@denx.de> > > Reviewed-by: Rob Herring <robh@kernel.org> > > --- > > > > 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 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
On Thu, 09 Jan 2025 16:41:49 +0100, Lukasz Majewski wrote: > 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> > --- > > 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 > Applied, thanks!
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 + +...