2022-12-25 12:06:10

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 1/2] dt-bindings: display: msm: drop redundant part of title

The Devicetree bindings document does not have to say in the title that
contains "DT properties", but instead just describe the hardware.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
.../devicetree/bindings/display/msm/dsi-phy-common.yaml | 7 +++----
.../devicetree/bindings/display/msm/qcom,msm8998-dpu.yaml | 2 +-
.../devicetree/bindings/display/msm/qcom,qcm2290-dpu.yaml | 2 +-
.../devicetree/bindings/display/msm/qcom,sc7180-dpu.yaml | 2 +-
.../devicetree/bindings/display/msm/qcom,sc7280-dpu.yaml | 2 +-
.../devicetree/bindings/display/msm/qcom,sdm845-dpu.yaml | 2 +-
.../devicetree/bindings/display/msm/qcom,sm6115-dpu.yaml | 2 +-
7 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/msm/dsi-phy-common.yaml b/Documentation/devicetree/bindings/display/msm/dsi-phy-common.yaml
index 76d40f7933dd..0f6f08890e7e 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-phy-common.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-phy-common.yaml
@@ -4,14 +4,13 @@
$id: http://devicetree.org/schemas/display/msm/dsi-phy-common.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

-title: Description of Qualcomm Display DSI PHY common dt properties
+title: Qualcomm Display DSI PHY Common Properties

maintainers:
- Krishna Manikandan <[email protected]>

-description: |
- This defines the DSI PHY dt properties which are common for all
- dsi phy versions.
+description:
+ Common properties for Qualcomm Display DSI PHY.

properties:
"#clock-cells":
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,msm8998-dpu.yaml b/Documentation/devicetree/bindings/display/msm/qcom,msm8998-dpu.yaml
index b02adba36e9e..727a20c4375c 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,msm8998-dpu.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,msm8998-dpu.yaml
@@ -4,7 +4,7 @@
$id: http://devicetree.org/schemas/display/msm/qcom,msm8998-dpu.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

-title: Qualcomm Display DPU dt properties for MSM8998 target
+title: Qualcomm Display DPU on MSM8998

maintainers:
- AngeloGioacchino Del Regno <[email protected]>
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-dpu.yaml b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-dpu.yaml
index a7b382f01b56..a2fd9f8e456c 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-dpu.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-dpu.yaml
@@ -4,7 +4,7 @@
$id: http://devicetree.org/schemas/display/msm/qcom,qcm2290-dpu.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

-title: Qualcomm Display DPU dt properties for QCM2290 target
+title: Qualcomm Display DPU on QCM2290

maintainers:
- Loic Poulain <[email protected]>
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sc7180-dpu.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sc7180-dpu.yaml
index bd590a6b5b96..2ade94a216a8 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,sc7180-dpu.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,sc7180-dpu.yaml
@@ -4,7 +4,7 @@
$id: http://devicetree.org/schemas/display/msm/qcom,sc7180-dpu.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

-title: Qualcomm Display DPU dt properties for SC7180 target
+title: Qualcomm Display DPU on SC7180

maintainers:
- Krishna Manikandan <[email protected]>
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sc7280-dpu.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sc7280-dpu.yaml
index 924059b387b6..9481f68d50d7 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,sc7280-dpu.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,sc7280-dpu.yaml
@@ -4,7 +4,7 @@
$id: http://devicetree.org/schemas/display/msm/qcom,sc7280-dpu.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

-title: Qualcomm Display DPU dt properties for SC7280
+title: Qualcomm Display DPU on SC7280

maintainers:
- Krishna Manikandan <[email protected]>
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sdm845-dpu.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sdm845-dpu.yaml
index 5719b45f2860..a2f305b04ee1 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,sdm845-dpu.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,sdm845-dpu.yaml
@@ -4,7 +4,7 @@
$id: http://devicetree.org/schemas/display/msm/qcom,sdm845-dpu.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

-title: Qualcomm Display DPU dt properties for SDM845 target
+title: Qualcomm Display DPU on SDM845

maintainers:
- Krishna Manikandan <[email protected]>
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-dpu.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-dpu.yaml
index 4a39a3031409..6ad828a20332 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-dpu.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-dpu.yaml
@@ -4,7 +4,7 @@
$id: http://devicetree.org/schemas/display/msm/qcom,sm6115-dpu.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

-title: Qualcomm Display DPU dt properties for SM6115 target
+title: Qualcomm Display DPU on SM6115

maintainers:
- Dmitry Baryshkov <[email protected]>
--
2.34.1


2022-12-25 12:30:56

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 2/2] dt-bindings: display: msm: drop unneeded list for single compatible

With only one compatible, there is no need to define it as list (items).

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
.../devicetree/bindings/display/msm/qcom,msm8998-dpu.yaml | 3 +--
.../devicetree/bindings/display/msm/qcom,msm8998-mdss.yaml | 3 +--
.../devicetree/bindings/display/msm/qcom,qcm2290-dpu.yaml | 3 +--
.../devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml | 3 +--
.../devicetree/bindings/display/msm/qcom,sc7180-dpu.yaml | 3 +--
.../devicetree/bindings/display/msm/qcom,sc7180-mdss.yaml | 3 +--
.../devicetree/bindings/display/msm/qcom,sdm845-dpu.yaml | 3 +--
.../devicetree/bindings/display/msm/qcom,sdm845-mdss.yaml | 3 +--
.../devicetree/bindings/display/msm/qcom,sm6115-dpu.yaml | 3 +--
.../devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml | 3 +--
.../devicetree/bindings/display/msm/qcom,sm8250-mdss.yaml | 3 +--
11 files changed, 11 insertions(+), 22 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/msm/qcom,msm8998-dpu.yaml b/Documentation/devicetree/bindings/display/msm/qcom,msm8998-dpu.yaml
index 727a20c4375c..943b6f017f7c 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,msm8998-dpu.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,msm8998-dpu.yaml
@@ -13,8 +13,7 @@ $ref: /schemas/display/msm/dpu-common.yaml#

properties:
compatible:
- items:
- - const: qcom,msm8998-dpu
+ const: qcom,msm8998-dpu

reg:
items:
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,msm8998-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,msm8998-mdss.yaml
index cf52ff77a41a..f67632a7e8fa 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,msm8998-mdss.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,msm8998-mdss.yaml
@@ -18,8 +18,7 @@ $ref: /schemas/display/msm/mdss-common.yaml#

properties:
compatible:
- items:
- - const: qcom,msm8998-mdss
+ const: qcom,msm8998-mdss

clocks:
items:
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-dpu.yaml b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-dpu.yaml
index a2fd9f8e456c..6c234b3b2765 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-dpu.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-dpu.yaml
@@ -13,8 +13,7 @@ $ref: /schemas/display/msm/dpu-common.yaml#

properties:
compatible:
- items:
- - const: qcom,qcm2290-dpu
+ const: qcom,qcm2290-dpu

reg:
items:
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml
index 4795e13c7b59..f5b24f81f382 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml
@@ -18,8 +18,7 @@ $ref: /schemas/display/msm/mdss-common.yaml#

properties:
compatible:
- items:
- - const: qcom,qcm2290-mdss
+ const: qcom,qcm2290-mdss

clocks:
items:
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sc7180-dpu.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sc7180-dpu.yaml
index 2ade94a216a8..86cfa3ddce62 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,sc7180-dpu.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,sc7180-dpu.yaml
@@ -13,8 +13,7 @@ $ref: /schemas/display/msm/dpu-common.yaml#

properties:
compatible:
- items:
- - const: qcom,sc7180-dpu
+ const: qcom,sc7180-dpu

reg:
items:
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sc7180-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sc7180-mdss.yaml
index 13e396d61a51..2a1d4510db6d 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,sc7180-mdss.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,sc7180-mdss.yaml
@@ -18,8 +18,7 @@ $ref: /schemas/display/msm/mdss-common.yaml#

properties:
compatible:
- items:
- - const: qcom,sc7180-mdss
+ const: qcom,sc7180-mdss

clocks:
items:
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sdm845-dpu.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sdm845-dpu.yaml
index a2f305b04ee1..5cabb899977d 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,sdm845-dpu.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,sdm845-dpu.yaml
@@ -13,8 +13,7 @@ $ref: /schemas/display/msm/dpu-common.yaml#

properties:
compatible:
- items:
- - const: qcom,sdm845-dpu
+ const: qcom,sdm845-dpu

reg:
items:
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sdm845-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sdm845-mdss.yaml
index 31ca6f99fc22..289d61debc38 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,sdm845-mdss.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,sdm845-mdss.yaml
@@ -18,8 +18,7 @@ $ref: /schemas/display/msm/mdss-common.yaml#

properties:
compatible:
- items:
- - const: qcom,sdm845-mdss
+ const: qcom,sdm845-mdss

clocks:
items:
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-dpu.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-dpu.yaml
index 6ad828a20332..bf62c2f5325a 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-dpu.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-dpu.yaml
@@ -13,8 +13,7 @@ $ref: /schemas/display/msm/dpu-common.yaml#

properties:
compatible:
- items:
- - const: qcom,sm6115-dpu
+ const: qcom,sm6115-dpu

reg:
items:
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml
index 886858ef6700..2491cb100b33 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml
@@ -18,8 +18,7 @@ $ref: /schemas/display/msm/mdss-common.yaml#

properties:
compatible:
- items:
- - const: qcom,sm6115-mdss
+ const: qcom,sm6115-mdss

clocks:
items:
diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm8250-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm8250-mdss.yaml
index 0d3be5386b3f..753bc99c868a 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,sm8250-mdss.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,sm8250-mdss.yaml
@@ -18,8 +18,7 @@ $ref: /schemas/display/msm/mdss-common.yaml#

properties:
compatible:
- items:
- - const: qcom,sm8250-mdss
+ const: qcom,sm8250-mdss

clocks:
items:
--
2.34.1

2023-01-08 18:33:32

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 1/2] dt-bindings: display: msm: drop redundant part of title


On Sun, 25 Dec 2022 12:59:24 +0100, Krzysztof Kozlowski wrote:
> The Devicetree bindings document does not have to say in the title that
> contains "DT properties", but instead just describe the hardware.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> .../devicetree/bindings/display/msm/dsi-phy-common.yaml | 7 +++----
> .../devicetree/bindings/display/msm/qcom,msm8998-dpu.yaml | 2 +-
> .../devicetree/bindings/display/msm/qcom,qcm2290-dpu.yaml | 2 +-
> .../devicetree/bindings/display/msm/qcom,sc7180-dpu.yaml | 2 +-
> .../devicetree/bindings/display/msm/qcom,sc7280-dpu.yaml | 2 +-
> .../devicetree/bindings/display/msm/qcom,sdm845-dpu.yaml | 2 +-
> .../devicetree/bindings/display/msm/qcom,sm6115-dpu.yaml | 2 +-
> 7 files changed, 9 insertions(+), 10 deletions(-)
>

Acked-by: Rob Herring <[email protected]>

2023-01-08 19:21:37

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 2/2] dt-bindings: display: msm: drop unneeded list for single compatible


On Sun, 25 Dec 2022 12:59:25 +0100, Krzysztof Kozlowski wrote:
> With only one compatible, there is no need to define it as list (items).
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> .../devicetree/bindings/display/msm/qcom,msm8998-dpu.yaml | 3 +--
> .../devicetree/bindings/display/msm/qcom,msm8998-mdss.yaml | 3 +--
> .../devicetree/bindings/display/msm/qcom,qcm2290-dpu.yaml | 3 +--
> .../devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml | 3 +--
> .../devicetree/bindings/display/msm/qcom,sc7180-dpu.yaml | 3 +--
> .../devicetree/bindings/display/msm/qcom,sc7180-mdss.yaml | 3 +--
> .../devicetree/bindings/display/msm/qcom,sdm845-dpu.yaml | 3 +--
> .../devicetree/bindings/display/msm/qcom,sdm845-mdss.yaml | 3 +--
> .../devicetree/bindings/display/msm/qcom,sm6115-dpu.yaml | 3 +--
> .../devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml | 3 +--
> .../devicetree/bindings/display/msm/qcom,sm8250-mdss.yaml | 3 +--
> 11 files changed, 11 insertions(+), 22 deletions(-)
>

Acked-by: Rob Herring <[email protected]>

2023-01-09 23:38:13

by Dmitry Baryshkov

[permalink] [raw]
Subject: Re: [PATCH 1/2] dt-bindings: display: msm: drop redundant part of title


On Sun, 25 Dec 2022 12:59:24 +0100, Krzysztof Kozlowski wrote:
> The Devicetree bindings document does not have to say in the title that
> contains "DT properties", but instead just describe the hardware.
>
>

Applied, thanks!

[1/2] dt-bindings: display: msm: drop redundant part of title
https://gitlab.freedesktop.org/lumag/msm/-/commit/e90c380ea283
[2/2] dt-bindings: display: msm: drop unneeded list for single compatible
https://gitlab.freedesktop.org/lumag/msm/-/commit/7ad6586652fa

Best regards,
--
Dmitry Baryshkov <[email protected]>