[PATCH] soc: qcom: socinfo: Add support to retrieve APPSBL build details

[PATCH] soc: qcom: socinfo: Add support to retrieve APPSBL build details

[Kathiravan Thirumoorthy]

Add support to retrieve APPS (Application Processor Subsystem) Bootloader
image details from SMEM.

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

diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c
index 391380820f082d8daa5c764d5f3c44e6240e18a2..dc8796ce5fe252cd03e98cd420c72760bb707197 100644
--- a/drivers/soc/qcom/socinfo.c
+++ b/drivers/soc/qcom/socinfo.c
@@ -38,6 +38,7 @@
 #define SMEM_IMAGE_TABLE_BOOT_INDEX     0
 #define SMEM_IMAGE_TABLE_TZ_INDEX       1
 #define SMEM_IMAGE_TABLE_RPM_INDEX      3
+#define SMEM_IMAGE_TABLE_APPSBL_INDEX	9
 #define SMEM_IMAGE_TABLE_APPS_INDEX     10
 #define SMEM_IMAGE_TABLE_MPSS_INDEX     11
 #define SMEM_IMAGE_TABLE_ADSP_INDEX     12
@@ -56,6 +57,7 @@
  */
 static const char *const socinfo_image_names[] = {
 	[SMEM_IMAGE_TABLE_ADSP_INDEX] = "adsp",
+	[SMEM_IMAGE_TABLE_APPSBL_INDEX] = "appsbl",
 	[SMEM_IMAGE_TABLE_APPS_INDEX] = "apps",
 	[SMEM_IMAGE_TABLE_BOOT_INDEX] = "boot",
 	[SMEM_IMAGE_TABLE_CNSS_INDEX] = "cnss",

---
base-commit: a62b7a37e6fcf4a675b1548e7c168b96ec836442
change-id: 20250711-appsbl_crm_version-6d304c851a8f

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

Re: [PATCH] soc: qcom: socinfo: Add support to retrieve APPSBL build details

[Konrad Dybcio]

On 7/11/25 1:03 PM, Kathiravan Thirumoorthy wrote:
> Add support to retrieve APPS (Application Processor Subsystem) Bootloader
> image details from SMEM.
> 
> Signed-off-by: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com>
> ---

Can we expand the driver with all the known IDs at once instead?

Konrad

Re: [PATCH] soc: qcom: socinfo: Add support to retrieve APPSBL build details

[Kathiravan Thirumoorthy]

On 7/11/2025 5:41 PM, Konrad Dybcio wrote:
> On 7/11/25 1:03 PM, Kathiravan Thirumoorthy wrote:
>> Add support to retrieve APPS (Application Processor Subsystem) Bootloader
>> image details from SMEM.
>>
>> Signed-off-by: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com>
>> ---
> Can we expand the driver with all the known IDs at once instead?

With respect to the IPQ SoCs only APPSBL image detail is missing, which 
this change address it

I can expand the list, but unfortunately don't have ways to validate 
them. I can check internally with folks to test it or even send an RTF. 
Till then, can this patch go independently?

>
> Konrad

Re: [PATCH] soc: qcom: socinfo: Add support to retrieve APPSBL build details

[Konrad Dybcio]

On 7/11/25 3:24 PM, Kathiravan Thirumoorthy wrote:
> 
> On 7/11/2025 5:41 PM, Konrad Dybcio wrote:
>> On 7/11/25 1:03 PM, Kathiravan Thirumoorthy wrote:
>>> Add support to retrieve APPS (Application Processor Subsystem) Bootloader
>>> image details from SMEM.
>>>
>>> Signed-off-by: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com>
>>> ---
>> Can we expand the driver with all the known IDs at once instead?
> 
> With respect to the IPQ SoCs only APPSBL image detail is missing, which this change address it

Next week someone else will come around and say the same thing
for e.g. automotive platforms

> 
> I can expand the list, but unfortunately don't have ways to validate them. I can check internally with folks to test it or even send an RTF. Till then, can this patch go independently?

There is nothing to validate here really, the bootloader either
populates the data or not

For now, this one alone is good to go, but please look into the
broader scope

Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>

Konrad

Re: [PATCH] soc: qcom: socinfo: Add support to retrieve APPSBL build details

[Kathiravan Thirumoorthy]

On 7/11/2025 7:00 PM, Konrad Dybcio wrote:
> On 7/11/25 3:24 PM, Kathiravan Thirumoorthy wrote:
>> On 7/11/2025 5:41 PM, Konrad Dybcio wrote:
>>> On 7/11/25 1:03 PM, Kathiravan Thirumoorthy wrote:
>>>> Add support to retrieve APPS (Application Processor Subsystem) Bootloader
>>>> image details from SMEM.
>>>>
>>>> Signed-off-by: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com>
>>>> ---
>>> Can we expand the driver with all the known IDs at once instead?
>> With respect to the IPQ SoCs only APPSBL image detail is missing, which this change address it
> Next week someone else will come around and say the same thing
> for e.g. automotive platforms
>
>> I can expand the list, but unfortunately don't have ways to validate them. I can check internally with folks to test it or even send an RTF. Till then, can this patch go independently?
> There is nothing to validate here really, the bootloader either
> populates the data or not
>
> For now, this one alone is good to go, but please look into the
> broader scope
Got it! I can send the patch with the remaining bits for the 
completeness in a week or two.
>
> Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
>
> Konrad

Thanks!

Re: [PATCH] soc: qcom: socinfo: Add support to retrieve APPSBL build details

[Bjorn Andersson]

On Fri, 11 Jul 2025 16:33:06 +0530, Kathiravan Thirumoorthy wrote:
> Add support to retrieve APPS (Application Processor Subsystem) Bootloader
> image details from SMEM.
> 
> 

Applied, thanks!

[1/1] soc: qcom: socinfo: Add support to retrieve APPSBL build details
      commit: 955a41218d2bd2ffadd0406b14a4b4efb67b056c

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