diff mbox series

[16/19] dt-bindings: arm: qcom: document QCS8275/QCS8300 SoC and reference board

Message ID 20240904-qcs8300_initial_dtsi-v1-16-d0ea9afdc007@quicinc.com (mailing list archive)
State Not Applicable
Headers show
Series Add initial support for QCS8300 | expand

Commit Message

Jingyi Wang Sept. 4, 2024, 8:33 a.m. UTC
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(+)

Comments

Krzysztof Kozlowski Sept. 4, 2024, 9:38 a.m. UTC | #1
On 04/09/2024 10:33, 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 c0529486810f..ccf9a166368f 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
> @@ -895,6 +897,12 @@ properties:
>            - const: qcom,qcs404-evb
>            - const: qcom,qcs404
>  
> +      - items:
> +          - enum:
> +              - qcom,qcs8300-ride
> +          - const: qcom,qcs8275

So the qcs8300 ride comes with non-safe SoC?

Best regards,
Krzysztof
Jingyi Wang Sept. 5, 2024, 4:42 a.m. UTC | #2
On 9/4/2024 5:38 PM, Krzysztof Kozlowski wrote:
> On 04/09/2024 10:33, 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 c0529486810f..ccf9a166368f 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
>> @@ -895,6 +897,12 @@ properties:
>>            - const: qcom,qcs404-evb
>>            - const: qcom,qcs404
>>  
>> +      - items:
>> +          - enum:
>> +              - qcom,qcs8300-ride
>> +          - const: qcom,qcs8275
> 
> So the qcs8300 ride comes with non-safe SoC?
> 
> Best regards,
> Krzysztof
> 
Both QCS8275 and QCS8300 SoC can reference qcs8300 ride board. Could you
describe your suggestion in more detail?

Thanks,
Jingyi
Krzysztof Kozlowski Sept. 5, 2024, 6:26 a.m. UTC | #3
On 05/09/2024 06:42, Jingyi Wang wrote:
> 
> 
> On 9/4/2024 5:38 PM, Krzysztof Kozlowski wrote:
>> On 04/09/2024 10:33, 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 c0529486810f..ccf9a166368f 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
>>> @@ -895,6 +897,12 @@ properties:
>>>            - const: qcom,qcs404-evb
>>>            - const: qcom,qcs404
>>>  
>>> +      - items:
>>> +          - enum:
>>> +              - qcom,qcs8300-ride
>>> +          - const: qcom,qcs8275
>>
>> So the qcs8300 ride comes with non-safe SoC?
>>
>> Best regards,
>> Krzysztof
>>
> Both QCS8275 and QCS8300 SoC can reference qcs8300 ride board. Could you
> describe your suggestion in more detail?

I did not suggest anything. I am confused that you claim that every
qcs8300 is using the non-safe flavor of the SoC. I am fine with this but
I want to understand it and be sure you will not change it next month
when you learn what this means.

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index c0529486810f..ccf9a166368f 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
@@ -895,6 +897,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