[PATCH 0/2] dt-bindings: opp: operating-points-v2-ti-cpu: Describe opp-supported-hw

[PATCH 0/2] dt-bindings: opp: operating-points-v2-ti-cpu: Describe opp-supported-hw

[Dhruva Gole]

dt-bindings: opp: operating-points-v2-ti-cpu: Describe opp-supported-hw

This series tries to bring back the description about what goes into
opp-supported-hw from the older binding.

While at it, also updating the maintainers entry.

Signed-off-by: Dhruva Gole <d-gole@ti.com>
---
Dhruva Gole (2):
      dt-bindings: opp: operating-points-v2-ti-cpu: Describe opp-supported-hw
      dt-bindings: opp: operating-points-v2-ti-cpu: Update maintainers

 .../devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml       | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)
---
base-commit: ecc768a84f0b8e631986f9ade3118fa37852fef0
change-id: 20240903-b4-opp-dt-binding-fix-73c6829751d2

Best regards,
-- 
Dhruva Gole <d-gole@ti.com>

[PATCH 1/2] dt-bindings: opp: operating-points-v2-ti-cpu: Describe opp-supported-hw

[Dhruva Gole]

It seems like we missed migrating the complete information from the old
DT binding where we had described what the opp-supported-hw is supposed
to describe. Hence, bring back the exact description from the previous
binding to the current one.

Fixes: e576a9a8603f ("dt-bindings: cpufreq: Convert ti-cpufreq to json schema")
Signed-off-by: Dhruva Gole <d-gole@ti.com>
---
 .../devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml         | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml b/Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml
index 02d1d2c17129..90855009cb81 100644
--- a/Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml
+++ b/Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml
@@ -45,7 +45,11 @@ patternProperties:
       clock-latency-ns: true
       opp-hz: true
       opp-microvolt: true
-      opp-supported-hw: true
+      opp-supported-hw:
+        description: |
+          Two bitfields indicating:
+            1. Which revision of the SoC the OPP is supported by
+            2. Which eFuse bits indicate this OPP is available
       opp-suspend: true
       turbo-mode: true
 

-- 
2.34.1

[PATCH 2/2] dt-bindings: opp: operating-points-v2-ti-cpu: Update maintainers

[Dhruva Gole]

Update the maintainers entry since I will be taking over this file from
now.

Signed-off-by: Dhruva Gole <d-gole@ti.com>
---
 Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml b/Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml
index 90855009cb81..a857ca34c98d 100644
--- a/Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml
+++ b/Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml
@@ -19,7 +19,7 @@ description:
   the hardware description for the scheme mentioned above.
 
 maintainers:
-  - Nishanth Menon <nm@ti.com>
+  - Dhruva Gole <d-gole@ti.com>
 
 allOf:
   - $ref: opp-v2-base.yaml#

-- 
2.34.1

Re: [PATCH 1/2] dt-bindings: opp: operating-points-v2-ti-cpu: Describe opp-supported-hw

[Rob Herring]

On Tue, Sep 03, 2024 at 04:00:07PM +0530, Dhruva Gole wrote:
> It seems like we missed migrating the complete information from the old
> DT binding where we had described what the opp-supported-hw is supposed
> to describe. Hence, bring back the exact description from the previous
> binding to the current one.
> 
> Fixes: e576a9a8603f ("dt-bindings: cpufreq: Convert ti-cpufreq to json schema")
> Signed-off-by: Dhruva Gole <d-gole@ti.com>
> ---
>  .../devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml         | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml b/Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml
> index 02d1d2c17129..90855009cb81 100644
> --- a/Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml
> +++ b/Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml
> @@ -45,7 +45,11 @@ patternProperties:
>        clock-latency-ns: true
>        opp-hz: true
>        opp-microvolt: true
> -      opp-supported-hw: true
> +      opp-supported-hw:
> +        description: |
> +          Two bitfields indicating:
> +            1. Which revision of the SoC the OPP is supported by
> +            2. Which eFuse bits indicate this OPP is available

This does nothing to tell me how to populate the property.

>        opp-suspend: true
>        turbo-mode: true
>  
> 
> -- 
> 2.34.1
> 

Re: [PATCH 2/2] dt-bindings: opp: operating-points-v2-ti-cpu: Update maintainers

[Rob Herring (Arm)]

On Tue, 03 Sep 2024 16:00:08 +0530, Dhruva Gole wrote:
> Update the maintainers entry since I will be taking over this file from
> now.
> 
> Signed-off-by: Dhruva Gole <d-gole@ti.com>
> ---
>  Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>

Re: [PATCH 2/2] dt-bindings: opp: operating-points-v2-ti-cpu: Update maintainers

[Viresh Kumar]

On 03-09-24, 16:00, Dhruva Gole wrote:
> Update the maintainers entry since I will be taking over this file from
> now.
> 
> Signed-off-by: Dhruva Gole <d-gole@ti.com>
> ---
>  Documentation/devicetree/bindings/opp/operating-points-v2-ti-cpu.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied. Thanks.

-- 
viresh