linux-mediatek.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] dt-bindings: interconnect: restrict opp-table to objects
@ 2022-08-18  6:16 Krzysztof Kozlowski
  2022-08-22 18:03 ` Rob Herring
  0 siblings, 1 reply; 2+ messages in thread
From: Krzysztof Kozlowski @ 2022-08-18  6:16 UTC (permalink / raw)
  To: Georgi Djakov, Rob Herring, Krzysztof Kozlowski, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, MyungJoo Ham, Kyungmin Park, Chanwoo Choi,
	Andy Gross, Bjorn Andersson, Konrad Dybcio, Matthias Brugger,
	Peng Fan, Jia-Wei Chang, Johnson Wang, linux-pm, devicetree,
	linux-arm-kernel, linux-kernel, linux-arm-msm, linux-mediatek
  Cc: Krzysztof Kozlowski

Simple 'opp-table:true' accepts a boolean property as opp-table, so
restrict it to object to properly enforce real OPP table nodes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

---

Changes since v1:
1. Correct typo in msg.
---
 .../devicetree/bindings/interconnect/fsl,imx8m-noc.yaml        | 3 ++-
 .../devicetree/bindings/interconnect/mediatek,cci.yaml         | 3 ++-
 .../devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml   | 3 ++-
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/interconnect/fsl,imx8m-noc.yaml b/Documentation/devicetree/bindings/interconnect/fsl,imx8m-noc.yaml
index fa4f7685ab2b..f7a5e31c506e 100644
--- a/Documentation/devicetree/bindings/interconnect/fsl,imx8m-noc.yaml
+++ b/Documentation/devicetree/bindings/interconnect/fsl,imx8m-noc.yaml
@@ -47,7 +47,8 @@ properties:
     maxItems: 1
 
   operating-points-v2: true
-  opp-table: true
+  opp-table:
+    type: object
 
   fsl,ddrc:
     $ref: "/schemas/types.yaml#/definitions/phandle"
diff --git a/Documentation/devicetree/bindings/interconnect/mediatek,cci.yaml b/Documentation/devicetree/bindings/interconnect/mediatek,cci.yaml
index 449c7c988229..58611ba2a0f4 100644
--- a/Documentation/devicetree/bindings/interconnect/mediatek,cci.yaml
+++ b/Documentation/devicetree/bindings/interconnect/mediatek,cci.yaml
@@ -36,7 +36,8 @@ properties:
       - const: intermediate
 
   operating-points-v2: true
-  opp-table: true
+  opp-table:
+    type: object
 
   proc-supply:
     description:
diff --git a/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml b/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
index c2e697f6e6cf..a4743386c4c1 100644
--- a/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
+++ b/Documentation/devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml
@@ -35,7 +35,8 @@ properties:
     maxItems: 1
 
   operating-points-v2: true
-  opp-table: true
+  opp-table:
+    type: object
 
   reg:
     # BWMON v4 (currently described) and BWMON v5 use one register address
-- 
2.34.1



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

* Re: [PATCH v2] dt-bindings: interconnect: restrict opp-table to objects
  2022-08-18  6:16 [PATCH v2] dt-bindings: interconnect: restrict opp-table to objects Krzysztof Kozlowski
@ 2022-08-22 18:03 ` Rob Herring
  0 siblings, 0 replies; 2+ messages in thread
From: Rob Herring @ 2022-08-22 18:03 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Kyungmin Park, Georgi Djakov, Matthias Brugger, linux-kernel,
	Johnson Wang, Krzysztof Kozlowski, Sascha Hauer, NXP Linux Team,
	Andy Gross, Konrad Dybcio, Jia-Wei Chang, Chanwoo Choi,
	devicetree, linux-arm-msm, Fabio Estevam, linux-pm,
	Pengutronix Kernel Team, Bjorn Andersson, Rob Herring,
	linux-arm-kernel, MyungJoo Ham, Peng Fan, linux-mediatek,
	Shawn Guo

On Thu, 18 Aug 2022 09:16:53 +0300, Krzysztof Kozlowski wrote:
> Simple 'opp-table:true' accepts a boolean property as opp-table, so
> restrict it to object to properly enforce real OPP table nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
> 
> Changes since v1:
> 1. Correct typo in msg.
> ---
>  .../devicetree/bindings/interconnect/fsl,imx8m-noc.yaml        | 3 ++-
>  .../devicetree/bindings/interconnect/mediatek,cci.yaml         | 3 ++-
>  .../devicetree/bindings/interconnect/qcom,msm8998-bwmon.yaml   | 3 ++-
>  3 files changed, 6 insertions(+), 3 deletions(-)
> 

Applied, thanks!


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

end of thread, other threads:[~2022-08-22 18:04 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-18  6:16 [PATCH v2] dt-bindings: interconnect: restrict opp-table to objects Krzysztof Kozlowski
2022-08-22 18:03 ` Rob Herring

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).