Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20221207211327.2848665-1-robh@kernel.org>
In-Reply-To: <20221207211327.2848665-1-robh@kernel.org>
From:   Rob Herring <robh@kernel.org>
Date:   Tue, 3 Jan 2023 13:29:08 -0600
Message-ID: <CAL_JsqKmULjtP7UCMZDxJTL3p7C_WS9qMaL5tMg5Jv74Sb0QQw@mail.gmail.com>
Subject: Re: [PATCH] arm64: dts: qcom: sc7280: Fix CPU nodes compatible string
To:     Andy Gross <agross@kernel.org>,
        Bjorn Andersson <andersson@kernel.org>,
        Konrad Dybcio <konrad.dybcio@somainline.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc:     linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Wed, Dec 7, 2022 at 3:13 PM Rob Herring <robh@kernel.org> wrote:
>
> 'arm,kryo' is not documented and is not an Arm Ltd thing either as that
> is Qualcomm branding. The correct compatible is 'qcom,kryo'.
>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
>  arch/arm64/boot/dts/qcom/sc7280.dtsi | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)

Ping!

>
> diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
> index 212580316d3e..f06cc7588acc 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
> @@ -166,7 +166,7 @@ cpus {
>
>                 CPU0: cpu@0 {
>                         device_type = "cpu";
> -                       compatible = "arm,kryo";
> +                       compatible = "qcom,kryo";
>                         reg = <0x0 0x0>;
>                         enable-method = "psci";
>                         cpu-idle-states = <&LITTLE_CPU_SLEEP_0
> @@ -189,7 +189,7 @@ L3_0: l3-cache {
>
>                 CPU1: cpu@100 {
>                         device_type = "cpu";
> -                       compatible = "arm,kryo";
> +                       compatible = "qcom,kryo";
>                         reg = <0x0 0x100>;
>                         enable-method = "psci";
>                         cpu-idle-states = <&LITTLE_CPU_SLEEP_0
> @@ -209,7 +209,7 @@ L2_100: l2-cache {
>
>                 CPU2: cpu@200 {
>                         device_type = "cpu";
> -                       compatible = "arm,kryo";
> +                       compatible = "qcom,kryo";
>                         reg = <0x0 0x200>;
>                         enable-method = "psci";
>                         cpu-idle-states = <&LITTLE_CPU_SLEEP_0
> @@ -229,7 +229,7 @@ L2_200: l2-cache {
>
>                 CPU3: cpu@300 {
>                         device_type = "cpu";
> -                       compatible = "arm,kryo";
> +                       compatible = "qcom,kryo";
>                         reg = <0x0 0x300>;
>                         enable-method = "psci";
>                         cpu-idle-states = <&LITTLE_CPU_SLEEP_0
> @@ -249,7 +249,7 @@ L2_300: l2-cache {
>
>                 CPU4: cpu@400 {
>                         device_type = "cpu";
> -                       compatible = "arm,kryo";
> +                       compatible = "qcom,kryo";
>                         reg = <0x0 0x400>;
>                         enable-method = "psci";
>                         cpu-idle-states = <&BIG_CPU_SLEEP_0
> @@ -269,7 +269,7 @@ L2_400: l2-cache {
>
>                 CPU5: cpu@500 {
>                         device_type = "cpu";
> -                       compatible = "arm,kryo";
> +                       compatible = "qcom,kryo";
>                         reg = <0x0 0x500>;
>                         enable-method = "psci";
>                         cpu-idle-states = <&BIG_CPU_SLEEP_0
> @@ -289,7 +289,7 @@ L2_500: l2-cache {
>
>                 CPU6: cpu@600 {
>                         device_type = "cpu";
> -                       compatible = "arm,kryo";
> +                       compatible = "qcom,kryo";
>                         reg = <0x0 0x600>;
>                         enable-method = "psci";
>                         cpu-idle-states = <&BIG_CPU_SLEEP_0
> @@ -309,7 +309,7 @@ L2_600: l2-cache {
>
>                 CPU7: cpu@700 {
>                         device_type = "cpu";
> -                       compatible = "arm,kryo";
> +                       compatible = "qcom,kryo";
>                         reg = <0x0 0x700>;
>                         enable-method = "psci";
>                         cpu-idle-states = <&BIG_CPU_SLEEP_0
> --
> 2.35.1
>