Message-ID: <cfe73771-33dd-e285-ed84-d620ee1edef0@gmail.com>
Date:   Mon, 30 Jan 2023 23:13:25 +0900
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.4.2
Subject: Re: [PATCH] dt-bindings: interconnect: samsung,exynos-bus: allow
 opp-table
Content-Language: en-US
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Chanwoo Choi <cw00.choi@samsung.com>,
        Georgi Djakov <djakov@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Alim Akhtar <alim.akhtar@samsung.com>,
        linux-pm@vger.kernel.org, linux-samsung-soc@vger.kernel.org,
        devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org
References: <20230125090849.122189-1-krzysztof.kozlowski@linaro.org>
From:   Chanwoo Choi <cwchoi00@gmail.com>
In-Reply-To: <20230125090849.122189-1-krzysztof.kozlowski@linaro.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

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 <krzysztof.kozlowski@linaro.org>
> ---
>  .../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 <cw00.choi@samsung.com>

-- 
Best Regards,
Samsung Electronics
Chanwoo Choi