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

[Krzysztof Kozlowski <krzk@kernel.org>]

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