* [PATCH] arm64: dts: qcom: sc7280: Fix CPU nodes compatible string
@ 2022-12-07 21:13 Rob Herring
2023-01-03 19:29 ` Rob Herring
2023-01-11 5:09 ` Bjorn Andersson
0 siblings, 2 replies; 3+ messages in thread
From: Rob Herring @ 2022-12-07 21:13 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Krzysztof Kozlowski
Cc: linux-arm-msm, devicetree, linux-kernel
'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(-)
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
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] arm64: dts: qcom: sc7280: Fix CPU nodes compatible string
2022-12-07 21:13 [PATCH] arm64: dts: qcom: sc7280: Fix CPU nodes compatible string Rob Herring
@ 2023-01-03 19:29 ` Rob Herring
2023-01-11 5:09 ` Bjorn Andersson
1 sibling, 0 replies; 3+ messages in thread
From: Rob Herring @ 2023-01-03 19:29 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Krzysztof Kozlowski
Cc: linux-arm-msm, devicetree, linux-kernel
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
>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] arm64: dts: qcom: sc7280: Fix CPU nodes compatible string
2022-12-07 21:13 [PATCH] arm64: dts: qcom: sc7280: Fix CPU nodes compatible string Rob Herring
2023-01-03 19:29 ` Rob Herring
@ 2023-01-11 5:09 ` Bjorn Andersson
1 sibling, 0 replies; 3+ messages in thread
From: Bjorn Andersson @ 2023-01-11 5:09 UTC (permalink / raw)
To: krzysztof.kozlowski+dt, agross, robh, konrad.dybcio
Cc: linux-kernel, linux-arm-msm, devicetree
On Wed, 7 Dec 2022 15:13:27 -0600, Rob Herring 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'.
>
>
Applied, thanks!
[1/1] arm64: dts: qcom: sc7280: Fix CPU nodes compatible string
commit: 9293c3e85a200d3a453ca208548d5dfc9d1af70a
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-01-11 5:19 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-12-07 21:13 [PATCH] arm64: dts: qcom: sc7280: Fix CPU nodes compatible string Rob Herring
2023-01-03 19:29 ` Rob Herring
2023-01-11 5:09 ` Bjorn Andersson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).