Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <19a54de8-d769-ab65-6332-6a8ff68e1437@quicinc.com>
Date:   Tue, 4 Oct 2022 10:16:47 -0500
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.2.2
Subject: Re: [PATCH 2/5] dt-bindings: arm: qcom: Document QDU1000/QRU1000 SoCs
 and boards
Content-Language: en-US
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Andy Gross <agross@kernel.org>,
        Bjorn Andersson <bjorn.andersson@linaro.org>,
        Stephen Boyd <sboyd@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Ulf Hansson <ulf.hansson@linaro.org>
CC:     Bhupesh Sharma <bhupesh.sharma@linaro.org>,
        <linux-arm-msm@vger.kernel.org>, <linux-mmc@vger.kernel.org>,
        <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>
References: <20221001030641.29354-1-quic_molvera@quicinc.com>
 <20221001030641.29354-3-quic_molvera@quicinc.com>
 <59d306a6-989a-ecf8-0262-7562d110bf0e@linaro.org>
From:   Melody Olvera <quic_molvera@quicinc.com>
In-Reply-To: <59d306a6-989a-ecf8-0262-7562d110bf0e@linaro.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 10/1/2022 4:26 AM, Krzysztof Kozlowski wrote:
> On 01/10/2022 05:06, Melody Olvera wrote:
>> Document the QDU1000 and QRU1000 SoC bindings and the boards that use
>> them.
>>
>> Signed-off-by: Melody Olvera <quic_molvera@quicinc.com>
>> ---
>>  Documentation/devicetree/bindings/arm/qcom.yaml | 16 ++++++++++++++++
> This patch goes with DT.
Will move to the dt PS.
>
>>  1 file changed, 16 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
>> index fb1d00bcc847..1cfd92f4ab5d 100644
>> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
>> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
>> @@ -61,6 +61,8 @@ description: |
>>          sm8250
>>          sm8350
>>          sm8450
>> +        qdu1000
>> +        qru1000
>>  
>>    The 'board' element must be one of the following strings:
>>  
>> @@ -76,6 +78,7 @@ description: |
>>          mtp
>>          qrd
>>          sbc
>> +        x100
>>  
>>    The 'soc_version' and 'board_version' elements take the form of v<Major>.<Minor>
>>    where the minor number may be omitted when it's zero, i.e.  v1.0 is the same
>> @@ -718,6 +721,19 @@ properties:
>>                - qcom,sm8450-qrd
>>            - const: qcom,sm8450
>>  
>> +      - description: Qualcomm Technologies, Inc. Distributed Unit 1000 platform
>> +          - items:
>> +              - enum:
>> +                  - qcom,qdu1000-idp
>> +                  - qcom,qdu1000-x100
>> +              - const: qcom,qdu1000
> Wrong order - you put it in some odd place.
Will put in alphabetical order.
>
>> +
>> +      - description: Qualcomm Technologies, Inc. Radio Unit 1000 platform
>> +          - items:
>> +              - enum:
>> +                  -qcom,qru1000-idp
> Missing space.
Whoops; will fix.
>
>> +              - const: qcom,qru1000
>> +
>>  additionalProperties: true
>>  
>>  ...
> Best regards,
> Krzysztof

Thanks,

Melody