Skip to contentSkip to conversation

arm64: dts: qcom: sc7280: Fix CPU nodes compatible string

PATCH Applied Rob Herring 2 participants 3 messages Dec 7, 2022 – Jan 11, 2023

Subsystems (4)
lore
Share & tools
lore

Rob Herring

author maintainer
#0 lore raw

PATCH 1/1 arm64: dts: qcom: sc7280: Fix CPU nodes compatible string

'arm,kryo' is not documented and is not an Arm Ltd thing either as that is Qualcomm branding. The correct compatible is 'qcom,kryo'.

---
 arch/arm64/boot/dts/qcom/sc7280.dtsi | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)
arch/arm64/boot/dts/qcom/sc7280.dtsi +8 -8
index 212580316d3e..f06cc7588acc 100644
@@ -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
Signature · 1 line
2.35.1
author maintainer
#1 lore raw
Quoted 8 lines · Rob Herring

'arm,kryo' is not documented and is not an Arm Ltd thing either as that is Qualcomm branding. The correct compatible is 'qcom,kryo'.

---
 arch/arm64/boot/dts/qcom/sc7280.dtsi | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

Ping!

Quoted 80 lines
arch/arm64/boot/dts/qcom/sc7280.dtsi +8 -9
index 212580316d3e..f06cc7588acc 100644
@@ -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

maintainer Applied
#2 lore raw
In reply to Rob Herring
Quoted 4 lines · Rob Herring

'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,

Signature · 1 line
Bjorn Andersson <[email protected]>