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

From: Jingyi Wang
Date: Wed Aug 14 2024 - 22:27:50 EST




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@xxxxxxxxxxx>
>> ---
>> 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