Skip to contentSkip to conversation

arm: qcom: sa8775p: add socinfo support

PATCH Superseded Bartosz Golaszewski 2 participants 7 messages Feb 7, 2023

Subsystems (3)

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore
#0 lore raw

cover arm: qcom: sa8775p: add socinfo support

From: Bartosz Golaszewski <[email protected]>

This series adds support for qcom-socinfo for SA8775P SoCs.

Bartosz Golaszewski (3):
soc: qcom: smem: Update max processor count
dt-bindings: arm: add the SoC ID for qcom SA8775P
soc: qcom: socinfo: add support for SA8775P

 drivers/soc/qcom/smem.c            | 2 +-
 drivers/soc/qcom/socinfo.c         | 1 +
 include/dt-bindings/arm/qcom,ids.h | 1 +
 3 files changed, 3 insertions(+), 1 deletion(-)
Signature · 1 line
2.37.2
#1 lore raw

PATCH 1/3 soc: qcom: smem: Update max processor count

From: Bartosz Golaszewski <[email protected]>

Update max processor count to reflect the number of co-processors on SA8775P SoCs.

---
 drivers/soc/qcom/smem.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/soc/qcom/smem.c +1 -1
index 4f163d62942c..7666b3425b7c 100644
@@ -85,7 +85,7 @@
 #define SMEM_GLOBAL_HOST	0xfffe
 
 /* Max number of processors/hosts in a system */
-#define SMEM_HOST_COUNT		15
+#define SMEM_HOST_COUNT		20
 
 /**
   * struct smem_proc_comm - proc_comm communication struct (legacy)
Signature · 1 line
2.37.2
#2 lore raw

PATCH 2/3 dt-bindings: arm: add the SoC ID for qcom SA8775P

In reply to Bartosz Golaszewski

From: Bartosz Golaszewski <[email protected]>

Add the SoC ID entry for SA8775P.

---
 include/dt-bindings/arm/qcom,ids.h | 1 +
 1 file changed, 1 insertion(+)
include/dt-bindings/arm/qcom,ids.h +1 -0
index 22d7ba17804b..bfdf066cc690 100644
@@ -223,6 +223,7 @@
 #define QCOM_ID_SC7180P			495
 #define QCOM_ID_SM6375			507
 #define QCOM_ID_SM8550			519
+#define QCOM_ID_SA8775P			534
 #define QCOM_ID_QRU1000			539
 #define QCOM_ID_QDU1000			545
 #define QCOM_ID_QDU1010			587
Signature · 1 line
2.37.2
#4 lore raw

PATCH 3/3 soc: qcom: socinfo: add support for SA8775P

In reply to Bartosz Golaszewski

From: Bartosz Golaszewski <[email protected]>

Add SocInfo support for SA8775P.

---
 drivers/soc/qcom/socinfo.c | 1 +
 1 file changed, 1 insertion(+)
drivers/soc/qcom/socinfo.c +1 -0
index 3b970a80f3aa..c5208aa99125 100644
@@ -419,6 +419,7 @@ static const struct soc_id soc_id[] = {
 	{ qcom_board_id(SM8350) },
 	{ qcom_board_id(SM6115) },
 	{ qcom_board_id(SC8280XP) },
+	{ qcom_board_id(SA8775P) },
 	{ qcom_board_id(IPQ6005) },
 	{ qcom_board_id(QRB5165) },
 	{ qcom_board_id(SM8450) },
Signature · 1 line
2.37.2
#5 lore raw

Re: [PATCH 3/3] soc: qcom: socinfo: add support for SA8775P

Quoted 14 lines · Bartosz Golaszewski

From: Bartosz Golaszewski <[email protected]>

Add SocInfo support for SA8775P.

---
 drivers/soc/qcom/socinfo.c | 1 +
 1 file changed, 1 insertion(+)
drivers/soc/qcom/socinfo.c +0 -0
index 3b970a80f3aa..c5208aa99125 100644

drivers/soc/qcom/socinfo.c static const struct soc_id soc_id[] = { · line 422

Quoted 4 lines
drivers/soc/qcom/socinfo.c +1 -0
@@ -419,6 +419,7 @@ static const struct soc_id soc_id[] = {
 	{ qcom_board_id(SM8350) },
 	{ qcom_board_id(SM6115) },
 	{ qcom_board_id(SC8280XP) },
+	{ qcom_board_id(SA8775P) },

This does not look like correctly ordered.

Best regards,
Krzysztof

#6 lore raw

Re: [PATCH 3/3] soc: qcom: socinfo: add support for SA8775P

Quoted 11 lines · Krzysztof Kozlowski
Quoted 9 lines · Bartosz Golaszewski

From: Bartosz Golaszewski <[email protected]>

Add SocInfo support for SA8775P.

---
 drivers/soc/qcom/socinfo.c | 1 +
 1 file changed, 1 insertion(+)
Quoted 12 lines

drivers/soc/qcom/socinfo.c static const struct soc_id soc_id[] = { · line 422

Quoted 9 lines
drivers/soc/qcom/socinfo.c +1 -0
index 3b970a80f3aa..c5208aa99125 100644
@@ -419,6 +419,7 @@ static const struct soc_id soc_id[] = {
      { qcom_board_id(SM8350) },
      { qcom_board_id(SM6115) },
      { qcom_board_id(SC8280XP) },
+     { qcom_board_id(SA8775P) },

This does not look like correctly ordered.

Ugh, I missed the fact that these are ordered by the chip ID that's defined elsewhere. Thanks, will fix that in v2.

Bart