2023-01-25 09:08:58

by Krzysztof Kozlowski

Subject: [PATCH] dt-bindings: interconnect: samsung,exynos-bus: allow opp-table

The opp-table can be located in the exynos-bus node which uses it, so
allow such child node.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
.../interconnect/samsung,exynos-bus.yaml | 27 +++++++++++++++++++
1 file changed, 27 insertions(+)

diff --git a/Documentation/devicetree/bindings/interconnect/samsung,exynos-bus.yaml b/Documentation/devicetree/bindings/interconnect/samsung,exynos-bus.yaml
index ad9ed596dfef..5e26e48c7217 100644
--- a/Documentation/devicetree/bindings/interconnect/samsung,exynos-bus.yaml
+++ b/Documentation/devicetree/bindings/interconnect/samsung,exynos-bus.yaml
@@ -196,6 +196,8 @@ properties:
maxItems: 2

operating-points-v2: true
+ opp-table:
+ type: object

samsung,data-clock-ratio:
$ref: /schemas/types.yaml#/definitions/uint32
@@ -227,6 +229,31 @@ examples:
operating-points-v2 = <&bus_dmc_opp_table>;
devfreq-events = <&ppmu_dmc0_3>, <&ppmu_dmc1_3>;
vdd-supply = <&buck1_reg>;
+
+ bus_dmc_opp_table: opp-table {
+ compatible = "operating-points-v2";
+
+ opp-50000000 {
+ opp-hz = /bits/ 64 <50000000>;
+ opp-microvolt = <800000>;
+ };
+ opp-100000000 {
+ opp-hz = /bits/ 64 <100000000>;
+ opp-microvolt = <800000>;
+ };
+ opp-134000000 {
+ opp-hz = /bits/ 64 <134000000>;
+ opp-microvolt = <800000>;
+ };
+ opp-200000000 {
+ opp-hz = /bits/ 64 <200000000>;
+ opp-microvolt = <825000>;
+ };
+ opp-400000000 {
+ opp-hz = /bits/ 64 <400000000>;
+ opp-microvolt = <875000>;
+ };
+ };
};

ppmu_dmc0: ppmu@106a0000 {
--
2.34.1

2023-01-25 21:04:13

by Rob Herring (Arm)

Subject: Re: [PATCH] dt-bindings: interconnect: samsung,exynos-bus: allow opp-table

On Wed, 25 Jan 2023 10:08:49 +0100, Krzysztof Kozlowski wrote:
> The opp-table can be located in the exynos-bus node which uses it, so
> allow such child node.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> .../interconnect/samsung,exynos-bus.yaml | 27 +++++++++++++++++++
> 1 file changed, 27 insertions(+)
>

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

2023-01-30 14:13:44

by Chanwoo Choi

Subject: Re: [PATCH] dt-bindings: interconnect: samsung,exynos-bus: allow opp-table

On 23. 1. 25. 18:08, Krzysztof Kozlowski wrote:
> The opp-table can be located in the exynos-bus node which uses it, so
> allow such child node.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> .../interconnect/samsung,exynos-bus.yaml | 27 +++++++++++++++++++
> 1 file changed, 27 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/interconnect/samsung,exynos-bus.yaml b/Documentation/devicetree/bindings/interconnect/samsung,exynos-bus.yaml
> index ad9ed596dfef..5e26e48c7217 100644
> --- a/Documentation/devicetree/bindings/interconnect/samsung,exynos-bus.yaml
> +++ b/Documentation/devicetree/bindings/interconnect/samsung,exynos-bus.yaml
> @@ -196,6 +196,8 @@ properties:
> maxItems: 2
>
> operating-points-v2: true
> + opp-table:
> + type: object
>
> samsung,data-clock-ratio:
> $ref: /schemas/types.yaml#/definitions/uint32
> @@ -227,6 +229,31 @@ examples:
> operating-points-v2 = <&bus_dmc_opp_table>;
> devfreq-events = <&ppmu_dmc0_3>, <&ppmu_dmc1_3>;
> vdd-supply = <&buck1_reg>;
> +
> + bus_dmc_opp_table: opp-table {
> + compatible = "operating-points-v2";
> +
> + opp-50000000 {
> + opp-hz = /bits/ 64 <50000000>;
> + opp-microvolt = <800000>;
> + };
> + opp-100000000 {
> + opp-hz = /bits/ 64 <100000000>;
> + opp-microvolt = <800000>;
> + };
> + opp-134000000 {
> + opp-hz = /bits/ 64 <134000000>;
> + opp-microvolt = <800000>;
> + };
> + opp-200000000 {
> + opp-hz = /bits/ 64 <200000000>;
> + opp-microvolt = <825000>;
> + };
> + opp-400000000 {
> + opp-hz = /bits/ 64 <400000000>;
> + opp-microvolt = <875000>;
> + };
> + };
> };
>
> ppmu_dmc0: ppmu@106a0000 {

Acked-by: Chanwoo Choi <[email protected]>

--
Best Regards,
Samsung Electronics
Chanwoo Choi