[PATCH 0/3] soc: qcom: socinfo: Add QCS8275/QCS8300 SoC ID

[PATCH 0/3] soc: qcom: socinfo: Add QCS8275/QCS8300 SoC ID

[Jingyi Wang]

Add support for socinfo for qualcomm QCS8275/QCS8300 SoC. QCS8300
is an Industrial Safe version SoC while QCS8275 is an Industrial
Non-Safe version.

Signed-off-by: Jingyi Wang <quic_jingyw@quicinc.com>
---
Jingyi Wang (3):
  dt-bindings: arm: qcom: document QCS8275/QCS8300 SoC and reference
    board
  dt-bindings: arm: qcom,ids: add SoC ID for QCS8275/QCS8300
  soc: qcom: socinfo: add QCS8275/QCS8300 SoC ID

 Documentation/devicetree/bindings/arm/qcom.yaml | 8 ++++++++
 drivers/soc/qcom/socinfo.c                      | 2 ++
 include/dt-bindings/arm/qcom,ids.h              | 2 ++
 3 files changed, 12 insertions(+)

-- 
base-commit: 320eb81df4f6c1a1814fd02ebb4ba41eb80a3c7e
2.25.1

[PATCH 1/3] dt-bindings: arm: qcom: document QCS8275/QCS8300 SoC and reference board

[Jingyi Wang]

Document the QCS8275/QCS8300 SoC and its reference board QCS8300 RIDE.
QCS8300 is an Industrial Safe SoC, while QCS8275 is the Industrial
Non-Safe version which can share the same SoC dtsi and board DTS.

Signed-off-by: Jingyi Wang <quic_jingyw@quicinc.com>
---
 Documentation/devicetree/bindings/arm/qcom.yaml | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index f08e13b61172..3952e1579767 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -42,6 +42,8 @@ description: |
         msm8996
         msm8998
         qcs404
+        qcs8275
+        qcs8300
         qcs8550
         qcm2290
         qcm6490
@@ -884,6 +886,12 @@ properties:
           - const: qcom,qcs404-evb
           - const: qcom,qcs404
 
+      - items:
+          - enum:
+              - qcom,qcs8300-ride
+          - const: qcom,qcs8275
+          - const: qcom,qcs8300
+
       - items:
           - enum:
               - qcom,sa8155p-adp
-- 
2.25.1

Re: [PATCH 1/3] dt-bindings: arm: qcom: document QCS8275/QCS8300 SoC and reference board

[Krzysztof Kozlowski]

On 14/08/2024 09:28, Jingyi Wang wrote:
> Document the QCS8275/QCS8300 SoC and its reference board QCS8300 RIDE.
> QCS8300 is an Industrial Safe SoC, while QCS8275 is the Industrial
> Non-Safe version which can share the same SoC dtsi and board DTS.
> 
> Signed-off-by: Jingyi Wang <quic_jingyw@quicinc.com>
> ---
>  Documentation/devicetree/bindings/arm/qcom.yaml | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
> index f08e13b61172..3952e1579767 100644
> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
> @@ -42,6 +42,8 @@ description: |
>          msm8996
>          msm8998
>          qcs404
> +        qcs8275
> +        qcs8300
>          qcs8550
>          qcm2290
>          qcm6490
> @@ -884,6 +886,12 @@ properties:
>            - const: qcom,qcs404-evb
>            - const: qcom,qcs404
>  
> +      - items:
> +          - enum:
> +              - qcom,qcs8300-ride

This is not used. You miss DTS patch. Look how people upstream things:
such binding *never* goes separate from the DTS.

Best regards,
Krzysztof

Re: [PATCH 1/3] dt-bindings: arm: qcom: document QCS8275/QCS8300 SoC and reference board

[Jingyi Wang]

On 8/14/2024 4:56 PM, Krzysztof Kozlowski wrote:
> On 14/08/2024 09:28, Jingyi Wang wrote:
>> Document the QCS8275/QCS8300 SoC and its reference board QCS8300 RIDE.
>> QCS8300 is an Industrial Safe SoC, while QCS8275 is the Industrial
>> Non-Safe version which can share the same SoC dtsi and board DTS.
>>
>> Signed-off-by: Jingyi Wang <quic_jingyw@quicinc.com>
>> ---
>>  Documentation/devicetree/bindings/arm/qcom.yaml | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
>> index f08e13b61172..3952e1579767 100644
>> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
>> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
>> @@ -42,6 +42,8 @@ description: |
>>          msm8996
>>          msm8998
>>          qcs404
>> +        qcs8275
>> +        qcs8300
>>          qcs8550
>>          qcm2290
>>          qcm6490
>> @@ -884,6 +886,12 @@ properties:
>>            - const: qcom,qcs404-evb
>>            - const: qcom,qcs404
>>  
>> +      - items:
>> +          - enum:
>> +              - qcom,qcs8300-ride
> 
> This is not used. You miss DTS patch. Look how people upstream things:
> such binding *never* goes separate from the DTS.
> 
> Best regards,
> Krzysztof
Thanks，I will remove that.

Thanks,
Jingyi

[PATCH 2/3] dt-bindings: arm: qcom,ids: add SoC ID for QCS8275/QCS8300

[Jingyi Wang]

Add the ID for Qualcomm QCS8275/QCS8300 SoC.

Signed-off-by: Jingyi Wang <quic_jingyw@quicinc.com>
---
 include/dt-bindings/arm/qcom,ids.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/include/dt-bindings/arm/qcom,ids.h b/include/dt-bindings/arm/qcom,ids.h
index d6c9e9472121..a213f6244c5d 100644
--- a/include/dt-bindings/arm/qcom,ids.h
+++ b/include/dt-bindings/arm/qcom,ids.h
@@ -274,6 +274,8 @@
 #define QCOM_ID_QCM8550			604
 #define QCOM_ID_IPQ5300			624
 #define QCOM_ID_IPQ5321			650
+#define QCOM_ID_QCS8300			674
+#define QCOM_ID_QCS8275			675
 
 /*
  * The board type and revision information, used by Qualcomm bootloaders and
-- 
2.25.1

Re: [PATCH 2/3] dt-bindings: arm: qcom,ids: add SoC ID for QCS8275/QCS8300

[Krzysztof Kozlowski]

On 14/08/2024 09:28, Jingyi Wang wrote:
> Add the ID for Qualcomm QCS8275/QCS8300 SoC.
> 
> Signed-off-by: Jingyi Wang <quic_jingyw@quicinc.com>
> ---
>  include/dt-bindings/arm/qcom,ids.h | 2 ++
>  1 file changed, 2 insertions(+)

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof

[PATCH 3/3] soc: qcom: socinfo: add QCS8275/QCS8300 SoC ID

[Jingyi Wang]

Add SoC ID for Qualcomm QCS8275/QCS8300.

Signed-off-by: Jingyi Wang <quic_jingyw@quicinc.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 d7359a235e3c..3ba29412b9ba 100644
--- a/drivers/soc/qcom/socinfo.c
+++ b/drivers/soc/qcom/socinfo.c
@@ -441,6 +441,8 @@ static const struct soc_id soc_id[] = {
 	{ qcom_board_id(QCM8550) },
 	{ qcom_board_id(IPQ5300) },
 	{ qcom_board_id(IPQ5321) },
+	{ qcom_board_id(QCS8300) },
+	{ qcom_board_id(QCS8275) },
 };
 
 static const char *socinfo_machine(struct device *dev, unsigned int id)
-- 
2.25.1

Re: (subset) [PATCH 0/3] soc: qcom: socinfo: Add QCS8275/QCS8300 SoC ID

[Bjorn Andersson]

On Wed, 14 Aug 2024 15:28:03 +0800, Jingyi Wang wrote:
> Add support for socinfo for qualcomm QCS8275/QCS8300 SoC. QCS8300
> is an Industrial Safe version SoC while QCS8275 is an Industrial
> Non-Safe version.
> 
> 

Applied, thanks!

[2/3] dt-bindings: arm: qcom,ids: add SoC ID for QCS8275/QCS8300
      commit: 6b34e75c48bb913f3431e66353cad9782e7225c7
[3/3] soc: qcom: socinfo: add QCS8275/QCS8300 SoC ID
      commit: dff75ec5763eb9c7ad64be952cc6930b410beb2d

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