Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20210820081458.83406-1-krzysztof.kozlowski@canonical.com>
In-Reply-To: <20210820081458.83406-1-krzysztof.kozlowski@canonical.com>
From:   Alim Akhtar <alim.akhtar@gmail.com>
Date:   Fri, 20 Aug 2021 20:14:50 +0530
Message-ID: <CAGOxZ53oTWuSLupGKWd+XWPXJZQfZ28-Vo=wWT1exty0LpCfOA@mail.gmail.com>
Subject: Re: [PATCH 1/2] arm64: dts: exynos: align operating-points table name
 with dtschema in Exynos5433
To:     Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Cc:     Rob Herring <robh+dt@kernel.org>, devicetree@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org,
        linux-samsung-soc@vger.kernel.org,
        open list <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Fri, Aug 20, 2021 at 1:51 PM Krzysztof Kozlowski
<krzysztof.kozlowski@canonical.com> wrote:
>
> Align the name of operating-points node to dtschema to fix warnings like:
>
Did you miss adding warnings here?

> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> ---
Rest looks good.
Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com>

>  arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi | 10 +++++-----
>  arch/arm64/boot/dts/exynos/exynos5433.dtsi     |  4 ++--
>  2 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi b/arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi
> index 8997f8f2b96c..72ccf18eb9d1 100644
> --- a/arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi
> +++ b/arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi
> @@ -87,7 +87,7 @@ bus_noc2: bus9 {
>                 status = "disabled";
>         };
>
> -       bus_g2d_400_opp_table: opp-table2 {
> +       bus_g2d_400_opp_table: opp-table-2 {
>                 compatible = "operating-points-v2";
>                 opp-shared;
>
> @@ -117,7 +117,7 @@ opp-100000000 {
>                 };
>         };
>
> -       bus_g2d_266_opp_table: opp-table3 {
> +       bus_g2d_266_opp_table: opp-table-3 {
>                 compatible = "operating-points-v2";
>
>                 opp-267000000 {
> @@ -137,7 +137,7 @@ opp-100000000 {
>                 };
>         };
>
> -       bus_gscl_opp_table: opp-table4 {
> +       bus_gscl_opp_table: opp-table-4 {
>                 compatible = "operating-points-v2";
>
>                 opp-333000000 {
> @@ -151,7 +151,7 @@ opp-166500000 {
>                 };
>         };
>
> -       bus_hevc_opp_table: opp-table5 {
> +       bus_hevc_opp_table: opp-table-5 {
>                 compatible = "operating-points-v2";
>                 opp-shared;
>
> @@ -175,7 +175,7 @@ opp-100000000 {
>                 };
>         };
>
> -       bus_noc2_opp_table: opp-table6 {
> +       bus_noc2_opp_table: opp-table-6 {
>                 compatible = "operating-points-v2";
>
>                 opp-400000000 {
> diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> index 6a6f7dd1d65c..ebd5091d68b4 100644
> --- a/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> +++ b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> @@ -239,7 +239,7 @@ cluster_a53_l2: l2-cache1 {
>                 };
>         };
>
> -       cluster_a53_opp_table: opp-table0 {
> +       cluster_a53_opp_table: opp-table-0 {
>                 compatible = "operating-points-v2";
>                 opp-shared;
>
> @@ -285,7 +285,7 @@ opp-1300000000 {
>                 };
>         };
>
> -       cluster_a57_opp_table: opp-table1 {
> +       cluster_a57_opp_table: opp-table-1 {
>                 compatible = "operating-points-v2";
>                 opp-shared;
>
> --
> 2.30.2
>


-- 
Regards,
Alim