[PATCH v2 0/3] dt-bindings: display: msm: document the SM8[345]50 displayport MDSS subnode

devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed

* [PATCH v2 0/3] dt-bindings: display: msm: document the SM8[345]50 displayport MDSS subnode
@ 2023-06-23 12:32 Neil Armstrong
  2023-06-23 12:32 ` [PATCH v2 1/3] dt-bindings: display: msm: sm8350-mdss: document displayport controller subnode Neil Armstrong
                   ` (3 more replies)
  0 siblings, 4 replies; 8+ messages in thread
From: Neil Armstrong @ 2023-06-23 12:32 UTC (permalink / raw)
  To: Rob Clark, Abhinav Kumar, Dmitry Baryshkov, Sean Paul,
	Marijn Suijten, David Airlie, Daniel Vetter, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Robert Foss
  Cc: linux-arm-msm, dri-devel, freedreno, devicetree, linux-kernel,
	Neil Armstrong, Rob Herring

Document the displayport subnode to fix the bindings check error:

arch/arm64/boot/dts/qcom/sm8550-mtp.dtb: display-subsystem@ae00000: Unevaluated properties are not allowed ('displayport-controller@ae90000' was unexpected)   
        From schema: Documentation/devicetree/bindings/display/msm/qcom,sm8550-mdss.yaml                                                                 

And same for SM8350 and SM8450.

Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
Changes in v2:
- Removed duplicate "document" in commit messages
- Added review trailers
- Link to v1: https://lore.kernel.org/r/20230621-topic-sm8x50-upstream-mdss-bindings-dp-subnode-v1-0-8bf386b373eb@linaro.org

---
Neil Armstrong (3):
      dt-bindings: display: msm: sm8350-mdss: document displayport controller subnode
      dt-bindings: display: msm: sm8450-mdss: document displayport controller subnode
      dt-bindings: display: msm: sm8550-mdss: document displayport controller subnode

 .../devicetree/bindings/display/msm/qcom,sm8350-mdss.yaml         | 6 ++++++
 .../devicetree/bindings/display/msm/qcom,sm8450-mdss.yaml         | 8 ++++++++
 .../devicetree/bindings/display/msm/qcom,sm8550-mdss.yaml         | 8 ++++++++
 3 files changed, 22 insertions(+)
---
base-commit: 15e71592dbae49a674429c618a10401d7f992ac3
change-id: 20230621-topic-sm8x50-upstream-mdss-bindings-dp-subnode-4fe367bf5cbe

Best regards,
-- 
Neil Armstrong <neil.armstrong@linaro.org>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH v2 1/3] dt-bindings: display: msm: sm8350-mdss: document displayport controller subnode
  2023-06-23 12:32 [PATCH v2 0/3] dt-bindings: display: msm: document the SM8[345]50 displayport MDSS subnode Neil Armstrong
@ 2023-06-23 12:32 ` Neil Armstrong
  2023-06-23 20:22   ` Marijn Suijten
  2023-06-23 12:32 ` [PATCH v2 2/3] dt-bindings: display: msm: sm8450-mdss: " Neil Armstrong
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 8+ messages in thread
From: Neil Armstrong @ 2023-06-23 12:32 UTC (permalink / raw)
  To: Rob Clark, Abhinav Kumar, Dmitry Baryshkov, Sean Paul,
	Marijn Suijten, David Airlie, Daniel Vetter, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Robert Foss
  Cc: linux-arm-msm, dri-devel, freedreno, devicetree, linux-kernel,
	Neil Armstrong, Rob Herring

Document the optional displayport controller subnode of the SM8350 MDSS.

Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
 Documentation/devicetree/bindings/display/msm/qcom,sm8350-mdss.yaml | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm8350-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm8350-mdss.yaml
index 79a226e4cc6a..f2cbeb435f1b 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,sm8350-mdss.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,sm8350-mdss.yaml
@@ -52,6 +52,12 @@ patternProperties:
       compatible:
         const: qcom,sm8350-dpu
 
+  "^displayport-controller@[0-9a-f]+$":
+    type: object
+    properties:
+      compatible:
+        const: qcom,sm8350-dp
+
   "^dsi@[0-9a-f]+$":
     type: object
     properties:

-- 
2.34.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH v2 1/3] dt-bindings: display: msm: sm8350-mdss: document displayport controller subnode
  2023-06-23 12:32 ` [PATCH v2 1/3] dt-bindings: display: msm: sm8350-mdss: document displayport controller subnode Neil Armstrong
@ 2023-06-23 20:22   ` Marijn Suijten
  0 siblings, 0 replies; 8+ messages in thread
From: Marijn Suijten @ 2023-06-23 20:22 UTC (permalink / raw)
  To: Neil Armstrong
  Cc: Rob Clark, Abhinav Kumar, Dmitry Baryshkov, Sean Paul,
	David Airlie, Daniel Vetter, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Robert Foss, linux-arm-msm, dri-devel, freedreno,
	devicetree, linux-kernel, Rob Herring

On 2023-06-23 14:32:53, Neil Armstrong wrote:
> Document the optional displayport controller subnode of the SM8350 MDSS.
> 
> Acked-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>

Reviewed-by: Marijn Suijten <marijn.suijten@somainline.org>

> ---
>  Documentation/devicetree/bindings/display/msm/qcom,sm8350-mdss.yaml | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm8350-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm8350-mdss.yaml
> index 79a226e4cc6a..f2cbeb435f1b 100644
> --- a/Documentation/devicetree/bindings/display/msm/qcom,sm8350-mdss.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/qcom,sm8350-mdss.yaml
> @@ -52,6 +52,12 @@ patternProperties:
>        compatible:
>          const: qcom,sm8350-dpu
>  
> +  "^displayport-controller@[0-9a-f]+$":
> +    type: object
> +    properties:
> +      compatible:
> +        const: qcom,sm8350-dp
> +
>    "^dsi@[0-9a-f]+$":
>      type: object
>      properties:
> 
> -- 
> 2.34.1
> 

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH v2 2/3] dt-bindings: display: msm: sm8450-mdss: document displayport controller subnode
  2023-06-23 12:32 [PATCH v2 0/3] dt-bindings: display: msm: document the SM8[345]50 displayport MDSS subnode Neil Armstrong
  2023-06-23 12:32 ` [PATCH v2 1/3] dt-bindings: display: msm: sm8350-mdss: document displayport controller subnode Neil Armstrong
@ 2023-06-23 12:32 ` Neil Armstrong
  2023-06-23 20:22   ` Marijn Suijten
  2023-06-23 12:32 ` [PATCH v2 3/3] dt-bindings: display: msm: sm8550-mdss: " Neil Armstrong
  2023-07-11 14:21 ` [PATCH v2 0/3] dt-bindings: display: msm: document the SM8[345]50 displayport MDSS subnode Dmitry Baryshkov
  3 siblings, 1 reply; 8+ messages in thread
From: Neil Armstrong @ 2023-06-23 12:32 UTC (permalink / raw)
  To: Rob Clark, Abhinav Kumar, Dmitry Baryshkov, Sean Paul,
	Marijn Suijten, David Airlie, Daniel Vetter, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Robert Foss
  Cc: linux-arm-msm, dri-devel, freedreno, devicetree, linux-kernel,
	Neil Armstrong, Rob Herring

Document the optional displayport controller subnode of the SM8450 MDSS.

Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
 .../devicetree/bindings/display/msm/qcom,sm8450-mdss.yaml         | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm8450-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm8450-mdss.yaml
index f26eb5643aed..494e2a080e99 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,sm8450-mdss.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,sm8450-mdss.yaml
@@ -42,6 +42,14 @@ patternProperties:
       compatible:
         const: qcom,sm8450-dpu
 
+  "^displayport-controller@[0-9a-f]+$":
+    type: object
+    properties:
+      compatible:
+        items:
+          - const: qcom,sm8450-dp
+          - const: qcom,sm8350-dp
+
   "^dsi@[0-9a-f]+$":
     type: object
     properties:

-- 
2.34.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH v2 2/3] dt-bindings: display: msm: sm8450-mdss: document displayport controller subnode
  2023-06-23 12:32 ` [PATCH v2 2/3] dt-bindings: display: msm: sm8450-mdss: " Neil Armstrong
@ 2023-06-23 20:22   ` Marijn Suijten
  0 siblings, 0 replies; 8+ messages in thread
From: Marijn Suijten @ 2023-06-23 20:22 UTC (permalink / raw)
  To: Neil Armstrong
  Cc: Rob Clark, Abhinav Kumar, Dmitry Baryshkov, Sean Paul,
	David Airlie, Daniel Vetter, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Robert Foss, linux-arm-msm, dri-devel, freedreno,
	devicetree, linux-kernel, Rob Herring

On 2023-06-23 14:32:54, Neil Armstrong wrote:
> Document the optional displayport controller subnode of the SM8450 MDSS.
> 
> Acked-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>

Reviewed-by: Marijn Suijten <marijn.suijten@somainline.org>

> ---
>  .../devicetree/bindings/display/msm/qcom,sm8450-mdss.yaml         | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm8450-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm8450-mdss.yaml
> index f26eb5643aed..494e2a080e99 100644
> --- a/Documentation/devicetree/bindings/display/msm/qcom,sm8450-mdss.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/qcom,sm8450-mdss.yaml
> @@ -42,6 +42,14 @@ patternProperties:
>        compatible:
>          const: qcom,sm8450-dpu
>  
> +  "^displayport-controller@[0-9a-f]+$":
> +    type: object
> +    properties:
> +      compatible:
> +        items:
> +          - const: qcom,sm8450-dp
> +          - const: qcom,sm8350-dp
> +
>    "^dsi@[0-9a-f]+$":
>      type: object
>      properties:
> 
> -- 
> 2.34.1
> 

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH v2 3/3] dt-bindings: display: msm: sm8550-mdss: document displayport controller subnode
  2023-06-23 12:32 [PATCH v2 0/3] dt-bindings: display: msm: document the SM8[345]50 displayport MDSS subnode Neil Armstrong
  2023-06-23 12:32 ` [PATCH v2 1/3] dt-bindings: display: msm: sm8350-mdss: document displayport controller subnode Neil Armstrong
  2023-06-23 12:32 ` [PATCH v2 2/3] dt-bindings: display: msm: sm8450-mdss: " Neil Armstrong
@ 2023-06-23 12:32 ` Neil Armstrong
  2023-06-23 20:23   ` Marijn Suijten
  2023-07-11 14:21 ` [PATCH v2 0/3] dt-bindings: display: msm: document the SM8[345]50 displayport MDSS subnode Dmitry Baryshkov
  3 siblings, 1 reply; 8+ messages in thread
From: Neil Armstrong @ 2023-06-23 12:32 UTC (permalink / raw)
  To: Rob Clark, Abhinav Kumar, Dmitry Baryshkov, Sean Paul,
	Marijn Suijten, David Airlie, Daniel Vetter, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Robert Foss
  Cc: linux-arm-msm, dri-devel, freedreno, devicetree, linux-kernel,
	Neil Armstrong, Rob Herring

Document the optional displayport controller subnode of the SM8550 MDSS.

Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
 .../devicetree/bindings/display/msm/qcom,sm8550-mdss.yaml         | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm8550-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm8550-mdss.yaml
index 887be33ba108..70ce7cb7a80d 100644
--- a/Documentation/devicetree/bindings/display/msm/qcom,sm8550-mdss.yaml
+++ b/Documentation/devicetree/bindings/display/msm/qcom,sm8550-mdss.yaml
@@ -42,6 +42,14 @@ patternProperties:
       compatible:
         const: qcom,sm8550-dpu
 
+  "^displayport-controller@[0-9a-f]+$":
+    type: object
+    properties:
+      compatible:
+        items:
+          - const: qcom,sm8550-dp
+          - const: qcom,sm8350-dp
+
   "^dsi@[0-9a-f]+$":
     type: object
     properties:

-- 
2.34.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH v2 3/3] dt-bindings: display: msm: sm8550-mdss: document displayport controller subnode
  2023-06-23 12:32 ` [PATCH v2 3/3] dt-bindings: display: msm: sm8550-mdss: " Neil Armstrong
@ 2023-06-23 20:23   ` Marijn Suijten
  0 siblings, 0 replies; 8+ messages in thread
From: Marijn Suijten @ 2023-06-23 20:23 UTC (permalink / raw)
  To: Neil Armstrong
  Cc: Rob Clark, Abhinav Kumar, Dmitry Baryshkov, Sean Paul,
	David Airlie, Daniel Vetter, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Robert Foss, linux-arm-msm, dri-devel, freedreno,
	devicetree, linux-kernel, Rob Herring

On 2023-06-23 14:32:55, Neil Armstrong wrote:
> Document the optional displayport controller subnode of the SM8550 MDSS.
> 
> Acked-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>

Reviewed-by: Marijn Suijten <marijn.suijten@somainline.org>

> ---
>  .../devicetree/bindings/display/msm/qcom,sm8550-mdss.yaml         | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm8550-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,sm8550-mdss.yaml
> index 887be33ba108..70ce7cb7a80d 100644
> --- a/Documentation/devicetree/bindings/display/msm/qcom,sm8550-mdss.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/qcom,sm8550-mdss.yaml
> @@ -42,6 +42,14 @@ patternProperties:
>        compatible:
>          const: qcom,sm8550-dpu
>  
> +  "^displayport-controller@[0-9a-f]+$":
> +    type: object
> +    properties:
> +      compatible:
> +        items:
> +          - const: qcom,sm8550-dp
> +          - const: qcom,sm8350-dp
> +
>    "^dsi@[0-9a-f]+$":
>      type: object
>      properties:
> 
> -- 
> 2.34.1
> 

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH v2 0/3] dt-bindings: display: msm: document the SM8[345]50 displayport MDSS subnode
  2023-06-23 12:32 [PATCH v2 0/3] dt-bindings: display: msm: document the SM8[345]50 displayport MDSS subnode Neil Armstrong
                   ` (2 preceding siblings ...)
  2023-06-23 12:32 ` [PATCH v2 3/3] dt-bindings: display: msm: sm8550-mdss: " Neil Armstrong
@ 2023-07-11 14:21 ` Dmitry Baryshkov
  3 siblings, 0 replies; 8+ messages in thread
From: Dmitry Baryshkov @ 2023-07-11 14:21 UTC (permalink / raw)
  To: Rob Clark, Abhinav Kumar, Sean Paul, Marijn Suijten, David Airlie,
	Daniel Vetter, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Robert Foss, Neil Armstrong
  Cc: linux-arm-msm, dri-devel, freedreno, devicetree, linux-kernel,
	Rob Herring


On Fri, 23 Jun 2023 14:32:52 +0200, Neil Armstrong wrote:
> Document the displayport subnode to fix the bindings check error:
> 
> arch/arm64/boot/dts/qcom/sm8550-mtp.dtb: display-subsystem@ae00000: Unevaluated properties are not allowed ('displayport-controller@ae90000' was unexpected)
>         From schema: Documentation/devicetree/bindings/display/msm/qcom,sm8550-mdss.yaml
> 
> And same for SM8350 and SM8450.
> 
> [...]

Applied, thanks!

[1/3] dt-bindings: display: msm: sm8350-mdss: document displayport controller subnode
      https://gitlab.freedesktop.org/lumag/msm/-/commit/c474443b723a
[2/3] dt-bindings: display: msm: sm8450-mdss: document displayport controller subnode
      https://gitlab.freedesktop.org/lumag/msm/-/commit/8d3850881242
[3/3] dt-bindings: display: msm: sm8550-mdss: document displayport controller subnode
      https://gitlab.freedesktop.org/lumag/msm/-/commit/1a1506d2fd7a

Best regards,
-- 
Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2023-07-11 14:23 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-23 12:32 [PATCH v2 0/3] dt-bindings: display: msm: document the SM8[345]50 displayport MDSS subnode Neil Armstrong
2023-06-23 12:32 ` [PATCH v2 1/3] dt-bindings: display: msm: sm8350-mdss: document displayport controller subnode Neil Armstrong
2023-06-23 20:22   ` Marijn Suijten
2023-06-23 12:32 ` [PATCH v2 2/3] dt-bindings: display: msm: sm8450-mdss: " Neil Armstrong
2023-06-23 20:22   ` Marijn Suijten
2023-06-23 12:32 ` [PATCH v2 3/3] dt-bindings: display: msm: sm8550-mdss: " Neil Armstrong
2023-06-23 20:23   ` Marijn Suijten
2023-07-11 14:21 ` [PATCH v2 0/3] dt-bindings: display: msm: document the SM8[345]50 displayport MDSS subnode Dmitry Baryshkov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).