[PATCH 0/2] Add the missing entries in the SMEM image table

mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed

* [PATCH 0/2] Add the missing entries in the SMEM image table
@ 2025-09-29 14:17 Kathiravan Thirumoorthy
  2025-09-29 14:17 ` [PATCH 1/2] soc: qcom: socinfo: arrange the socinfo_image_names array in alphabetical order Kathiravan Thirumoorthy
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Kathiravan Thirumoorthy @ 2025-09-29 14:17 UTC (permalink / raw)
  To: Bjorn Andersson, Konrad Dybcio
  Cc: linux-arm-msm, linux-kernel, Kathiravan Thirumoorthy

Rather than adding support for one each image at a time, populate all
the image details at once for the completeness. Additionally sort the
socinfo_image_names array in alphabetical order.

Signed-off-by: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com>
---
Kathiravan Thirumoorthy (2):
      soc: qcom: socinfo: arrange the socinfo_image_names array in alphabetical order
      soc: qcom: socinfo: add the missing entries to the smem image table

 drivers/soc/qcom/socinfo.c | 46 ++++++++++++++++++++++++++++++++++++++++------
 1 file changed, 40 insertions(+), 6 deletions(-)
---
base-commit: 262858079afde6d367ce3db183c74d8a43a0e83f
change-id: 20250714-image_crm-272b5ca25a2b

Best regards,
-- 
Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com>


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/2] soc: qcom: socinfo: arrange the socinfo_image_names array in alphabetical order
  2025-09-29 14:17 [PATCH 0/2] Add the missing entries in the SMEM image table Kathiravan Thirumoorthy
@ 2025-09-29 14:17 ` Kathiravan Thirumoorthy
  2025-09-30  2:14   ` Dmitry Baryshkov
  2025-09-29 14:17 ` [PATCH 2/2] soc: qcom: socinfo: add the missing entries to the smem image table Kathiravan Thirumoorthy
  2025-10-27 14:09 ` [PATCH 0/2] Add the missing entries in the SMEM " Bjorn Andersson
  2 siblings, 1 reply; 6+ messages in thread
From: Kathiravan Thirumoorthy @ 2025-09-29 14:17 UTC (permalink / raw)
  To: Bjorn Andersson, Konrad Dybcio
  Cc: linux-arm-msm, linux-kernel, Kathiravan Thirumoorthy

The socinfo_image_names array is currently neither arranged alphabetically
nor by image index values, making it harder to maintain. Reorder the array
alphabetically to improve readability and simplify the addition of new
entries.

Signed-off-by: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com>
---
 drivers/soc/qcom/socinfo.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c
index 963772f45489d4d0e15d4893ac40d64502fe90bd..8d25da4c7018452f463c6ff1e7ce605d5d0c6aab 100644
--- a/drivers/soc/qcom/socinfo.c
+++ b/drivers/soc/qcom/socinfo.c
@@ -60,17 +60,17 @@ static const char *const socinfo_image_names[] = {
 	[SMEM_IMAGE_TABLE_APPSBL_INDEX] = "appsbl",
 	[SMEM_IMAGE_TABLE_APPS_INDEX] = "apps",
 	[SMEM_IMAGE_TABLE_BOOT_INDEX] = "boot",
+	[SMEM_IMAGE_TABLE_CDSP1_INDEX] = "cdsp1",
+	[SMEM_IMAGE_TABLE_CDSP_INDEX] = "cdsp",
 	[SMEM_IMAGE_TABLE_CNSS_INDEX] = "cnss",
+	[SMEM_IMAGE_TABLE_DSPS_INDEX] = "dsps",
+	[SMEM_IMAGE_TABLE_GPDSP1_INDEX] = "gpdsp1",
+	[SMEM_IMAGE_TABLE_GPDSP_INDEX] = "gpdsp",
 	[SMEM_IMAGE_TABLE_MPSS_INDEX] = "mpss",
 	[SMEM_IMAGE_TABLE_RPM_INDEX] = "rpm",
+	[SMEM_IMAGE_TABLE_TME_INDEX] = "tme",
 	[SMEM_IMAGE_TABLE_TZ_INDEX] = "tz",
 	[SMEM_IMAGE_TABLE_VIDEO_INDEX] = "video",
-	[SMEM_IMAGE_TABLE_DSPS_INDEX] = "dsps",
-	[SMEM_IMAGE_TABLE_CDSP_INDEX] = "cdsp",
-	[SMEM_IMAGE_TABLE_CDSP1_INDEX] = "cdsp1",
-	[SMEM_IMAGE_TABLE_GPDSP_INDEX] = "gpdsp",
-	[SMEM_IMAGE_TABLE_GPDSP1_INDEX] = "gpdsp1",
-	[SMEM_IMAGE_TABLE_TME_INDEX] = "tme",
 };
 
 static const char *const pmic_models[] = {

-- 
2.34.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] soc: qcom: socinfo: arrange the socinfo_image_names array in alphabetical order
  2025-09-29 14:17 ` [PATCH 1/2] soc: qcom: socinfo: arrange the socinfo_image_names array in alphabetical order Kathiravan Thirumoorthy
@ 2025-09-30  2:14   ` Dmitry Baryshkov
  0 siblings, 0 replies; 6+ messages in thread
From: Dmitry Baryshkov @ 2025-09-30  2:14 UTC (permalink / raw)
  To: Kathiravan Thirumoorthy
  Cc: Bjorn Andersson, Konrad Dybcio, linux-arm-msm, linux-kernel

On Mon, Sep 29, 2025 at 07:47:07PM +0530, Kathiravan Thirumoorthy wrote:
> The socinfo_image_names array is currently neither arranged alphabetically
> nor by image index values, making it harder to maintain. Reorder the array
> alphabetically to improve readability and simplify the addition of new
> entries.
> 
> Signed-off-by: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com>
> ---
>  drivers/soc/qcom/socinfo.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>

-- 
With best wishes
Dmitry

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 2/2] soc: qcom: socinfo: add the missing entries to the smem image table
  2025-09-29 14:17 [PATCH 0/2] Add the missing entries in the SMEM image table Kathiravan Thirumoorthy
  2025-09-29 14:17 ` [PATCH 1/2] soc: qcom: socinfo: arrange the socinfo_image_names array in alphabetical order Kathiravan Thirumoorthy
@ 2025-09-29 14:17 ` Kathiravan Thirumoorthy
  2025-09-30  2:15   ` Dmitry Baryshkov
  2025-10-27 14:09 ` [PATCH 0/2] Add the missing entries in the SMEM " Bjorn Andersson
  2 siblings, 1 reply; 6+ messages in thread
From: Kathiravan Thirumoorthy @ 2025-09-29 14:17 UTC (permalink / raw)
  To: Bjorn Andersson, Konrad Dybcio
  Cc: linux-arm-msm, linux-kernel, Kathiravan Thirumoorthy

Add the missing entries to the SMEM image table to ensure completeness,
rather than adding support for one image at a time.

Signed-off-by: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com>
---
 drivers/soc/qcom/socinfo.c | 34 ++++++++++++++++++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c
index 8d25da4c7018452f463c6ff1e7ce605d5d0c6aab..5627fb6ffbdb23338ff5e26a05964a07439aeb56 100644
--- a/drivers/soc/qcom/socinfo.c
+++ b/drivers/soc/qcom/socinfo.c
@@ -37,7 +37,13 @@
  */
 #define SMEM_IMAGE_TABLE_BOOT_INDEX     0
 #define SMEM_IMAGE_TABLE_TZ_INDEX       1
+#define SMEM_IMAGE_TABLE_TZSECAPP_INDEX	2
 #define SMEM_IMAGE_TABLE_RPM_INDEX      3
+#define SMEM_IMAGE_TABLE_SDI_INDEX      4
+#define SMEM_IMAGE_TABLE_HYP_INDEX      5
+#define SMEM_IMAGE_TABLE_ADSP1_INDEX	6
+#define SMEM_IMAGE_TABLE_ADSP2_INDEX	7
+#define SMEM_IMAGE_TABLE_CDSP2_INDEX	8
 #define SMEM_IMAGE_TABLE_APPSBL_INDEX	9
 #define SMEM_IMAGE_TABLE_APPS_INDEX     10
 #define SMEM_IMAGE_TABLE_MPSS_INDEX     11
@@ -46,31 +52,59 @@
 #define SMEM_IMAGE_TABLE_VIDEO_INDEX    14
 #define SMEM_IMAGE_TABLE_DSPS_INDEX     15
 #define SMEM_IMAGE_TABLE_CDSP_INDEX     16
+#define SMEM_IMAGE_TABLE_NPU_INDEX	17
+#define SMEM_IMAGE_TABLE_WPSS_INDEX     18
 #define SMEM_IMAGE_TABLE_CDSP1_INDEX    19
 #define SMEM_IMAGE_TABLE_GPDSP_INDEX    20
 #define SMEM_IMAGE_TABLE_GPDSP1_INDEX   21
+#define SMEM_IMAGE_TABLE_SENSORPD_INDEX	22
+#define SMEM_IMAGE_TABLE_AUDIOPD_INDEX	23
+#define SMEM_IMAGE_TABLE_OEMPD_INDEX	24
+#define SMEM_IMAGE_TABLE_CHARGERPD_INDEX	25
+#define SMEM_IMAGE_TABLE_OISPD_INDEX	26
+#define SMEM_IMAGE_TABLE_SOCCP_INDEX	27
 #define SMEM_IMAGE_TABLE_TME_INDEX	28
+#define SMEM_IMAGE_TABLE_GEARVM_INDEX	29
+#define SMEM_IMAGE_TABLE_UEFI_INDEX	30
+#define SMEM_IMAGE_TABLE_CDSP3_INDEX	31
 #define SMEM_IMAGE_VERSION_TABLE       469
 
 /*
  * SMEM Image table names
  */
 static const char *const socinfo_image_names[] = {
+	[SMEM_IMAGE_TABLE_ADSP1_INDEX] = "adsp1",
+	[SMEM_IMAGE_TABLE_ADSP2_INDEX] = "adsp2",
 	[SMEM_IMAGE_TABLE_ADSP_INDEX] = "adsp",
 	[SMEM_IMAGE_TABLE_APPSBL_INDEX] = "appsbl",
 	[SMEM_IMAGE_TABLE_APPS_INDEX] = "apps",
+	[SMEM_IMAGE_TABLE_AUDIOPD_INDEX] = "audiopd",
 	[SMEM_IMAGE_TABLE_BOOT_INDEX] = "boot",
 	[SMEM_IMAGE_TABLE_CDSP1_INDEX] = "cdsp1",
+	[SMEM_IMAGE_TABLE_CDSP2_INDEX] = "cdsp2",
+	[SMEM_IMAGE_TABLE_CDSP3_INDEX] = "cdsp3",
 	[SMEM_IMAGE_TABLE_CDSP_INDEX] = "cdsp",
+	[SMEM_IMAGE_TABLE_CHARGERPD_INDEX] = "chargerpd",
 	[SMEM_IMAGE_TABLE_CNSS_INDEX] = "cnss",
 	[SMEM_IMAGE_TABLE_DSPS_INDEX] = "dsps",
+	[SMEM_IMAGE_TABLE_GEARVM_INDEX] = "gearvm",
 	[SMEM_IMAGE_TABLE_GPDSP1_INDEX] = "gpdsp1",
 	[SMEM_IMAGE_TABLE_GPDSP_INDEX] = "gpdsp",
+	[SMEM_IMAGE_TABLE_HYP_INDEX] = "hyp",
 	[SMEM_IMAGE_TABLE_MPSS_INDEX] = "mpss",
+	[SMEM_IMAGE_TABLE_NPU_INDEX] = "npu",
+	[SMEM_IMAGE_TABLE_OEMPD_INDEX] = "oempd",
+	[SMEM_IMAGE_TABLE_OISPD_INDEX] = "oispd",
 	[SMEM_IMAGE_TABLE_RPM_INDEX] = "rpm",
+	[SMEM_IMAGE_TABLE_SDI_INDEX] = "sdi",
+	[SMEM_IMAGE_TABLE_SENSORPD_INDEX] = "sensorpd",
+	[SMEM_IMAGE_TABLE_SOCCP_INDEX] = "soccp",
 	[SMEM_IMAGE_TABLE_TME_INDEX] = "tme",
 	[SMEM_IMAGE_TABLE_TZ_INDEX] = "tz",
+	[SMEM_IMAGE_TABLE_TZSECAPP_INDEX] = "tzsecapp",
+	[SMEM_IMAGE_TABLE_UEFI_INDEX] = "uefi",
 	[SMEM_IMAGE_TABLE_VIDEO_INDEX] = "video",
+	[SMEM_IMAGE_TABLE_WPSS_INDEX] = "wpss",
 };
 
 static const char *const pmic_models[] = {

-- 
2.34.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 2/2] soc: qcom: socinfo: add the missing entries to the smem image table
  2025-09-29 14:17 ` [PATCH 2/2] soc: qcom: socinfo: add the missing entries to the smem image table Kathiravan Thirumoorthy
@ 2025-09-30  2:15   ` Dmitry Baryshkov
  0 siblings, 0 replies; 6+ messages in thread
From: Dmitry Baryshkov @ 2025-09-30  2:15 UTC (permalink / raw)
  To: Kathiravan Thirumoorthy
  Cc: Bjorn Andersson, Konrad Dybcio, linux-arm-msm, linux-kernel

On Mon, Sep 29, 2025 at 07:47:08PM +0530, Kathiravan Thirumoorthy wrote:
> Add the missing entries to the SMEM image table to ensure completeness,
> rather than adding support for one image at a time.
> 
> Signed-off-by: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com>
> ---
>  drivers/soc/qcom/socinfo.c | 34 ++++++++++++++++++++++++++++++++++
>  1 file changed, 34 insertions(+)
> 

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>


-- 
With best wishes
Dmitry

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/2] Add the missing entries in the SMEM image table
  2025-09-29 14:17 [PATCH 0/2] Add the missing entries in the SMEM image table Kathiravan Thirumoorthy
  2025-09-29 14:17 ` [PATCH 1/2] soc: qcom: socinfo: arrange the socinfo_image_names array in alphabetical order Kathiravan Thirumoorthy
  2025-09-29 14:17 ` [PATCH 2/2] soc: qcom: socinfo: add the missing entries to the smem image table Kathiravan Thirumoorthy
@ 2025-10-27 14:09 ` Bjorn Andersson
  2 siblings, 0 replies; 6+ messages in thread
From: Bjorn Andersson @ 2025-10-27 14:09 UTC (permalink / raw)
  To: Konrad Dybcio, Kathiravan Thirumoorthy; +Cc: linux-arm-msm, linux-kernel


On Mon, 29 Sep 2025 19:47:06 +0530, Kathiravan Thirumoorthy wrote:
> Rather than adding support for one each image at a time, populate all
> the image details at once for the completeness. Additionally sort the
> socinfo_image_names array in alphabetical order.
> 
> 

Applied, thanks!

[1/2] soc: qcom: socinfo: arrange the socinfo_image_names array in alphabetical order
      commit: abac241e1d5da20332160b29b23f357016abe3e1
[2/2] soc: qcom: socinfo: add the missing entries to the smem image table
      commit: dcbce2c23e5edcba2ac538e7a53d0e3d1a21f975

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2025-10-27 14:06 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-09-29 14:17 [PATCH 0/2] Add the missing entries in the SMEM image table Kathiravan Thirumoorthy
2025-09-29 14:17 ` [PATCH 1/2] soc: qcom: socinfo: arrange the socinfo_image_names array in alphabetical order Kathiravan Thirumoorthy
2025-09-30  2:14   ` Dmitry Baryshkov
2025-09-29 14:17 ` [PATCH 2/2] soc: qcom: socinfo: add the missing entries to the smem image table Kathiravan Thirumoorthy
2025-09-30  2:15   ` Dmitry Baryshkov
2025-10-27 14:09 ` [PATCH 0/2] Add the missing entries in the SMEM " Bjorn Andersson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®