Re: [PATCH v5 1/4] dt-bindings: arm: qcom: Document QCS8550 SoC and the AIM300 AIoT board

[Tengfei Fan]

On 3/4/2024 3:42 PM, Krzysztof Kozlowski wrote:
> On 01/03/2024 14:41, Tengfei Fan wrote:
> > Document QCS8550 SoC and the AIM300 AIoT board bindings.
> > QCS8550 is derived from SM8550. The difference between SM8550 and
> > QCS8550 is QCS8550 doesn't have modem RF system. QCS8550 is mainly used
> > in IoT scenarios.
> > AIM300 Series is a highly optimized family of modules designed to
> > support AIoT applications. It integrates QCS8550 SoC, UFS and PMIC chip
> > etc.
> > AIM stands for Artificial Intelligence Module. AIoT stands for AI IoT.
> >
> > Signed-off-by: Tengfei Fan <quic_tengfan@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 66beaac60e1d..0ca4333fa8cf 100644
> > --- a/Documentation/devicetree/bindings/arm/qcom.yaml
> > +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
> > @@ -42,6 +42,7 @@ description: |
> >           msm8996
> >           msm8998
> >           qcs404
> > +        qcs8550
> >           qcm2290
> >           qcm6490
> >           qdu1000
> > @@ -868,6 +869,13 @@ properties:
> >             - const: qcom,qcs404-evb
> >             - const: qcom,qcs404
> >   
> > +      - items:
> > +          - enum:
> > +              - qcom,qcs8550-aim300-aiot
> > +          - const: qcom,qcs8550-aim300
> > +          - const: qcom,qcs8550
> > +          - const: qcom,sm8550
>
> This should be after sm8550 boards, not after qcs404.

Yes, I will adjust the order in the next patch series.

> Best regards,
> Krzysztof

--
Thx and BRs,
Tengfei Fan