[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5d31facd-8942-d159-a1f1-cd7cae6c09a5@quicinc.com>
Date: Wed, 4 Oct 2023 14:53:45 +0530
From: Devi Priya <quic_devipriy@...cinc.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
<agross@...nel.org>, <andersson@...nel.org>,
<konrad.dybcio@...aro.org>, <lee@...nel.org>, <robh+dt@...nel.org>,
<krzysztof.kozlowski+dt@...aro.org>, <conor+dt@...nel.org>,
<thierry.reding@...il.com>, <ndesaulniers@...gle.com>,
<trix@...hat.com>, <baruch@...s.co.il>,
<linux-arm-msm@...r.kernel.org>, <devicetree@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, <llvm@...ts.linux.dev>
Subject: Re: [PATCH V13 3/4] dt-bindings: mfd: qcom,tcsr: Add simple-mfd
support for IPQ6018
On 10/4/2023 2:50 PM, Krzysztof Kozlowski wrote:
> On 04/10/2023 11:04, Devi Priya wrote:
>> Update the binding to include pwm as the child node to TCSR block and
>> add simple-mfd support for IPQ6018.
>>
>> Signed-off-by: Devi Priya <quic_devipriy@...cinc.com>
>> ---
>
> Thank you for your patch. There is something to discuss/improve.
>
>
>> reg:
>> maxItems: 1
>>
>> + ranges: true
>> +
>> + "#address-cells":
>> + const: 1
>> +
>> + "#size-cells":
>> + const: 1
>> +
>> +patternProperties:
>> + "pwm@[a-f0-9]+$":
>
> type: object
Okay
>
>> + $ref: ../pwm/qcom,ipq6018-pwm.yaml
>
> This should be full patch, so:
> $ref: /schemas/pwm/qcom.......
okay
>
>
>> +
>> required:
>> - compatible
>> - reg
>
> After required (but before additionalProperties), please add:
>
> allOf:if:then: for other compatible (so use keyword "not:") which will
> disallow pwm for them. Disallowing is via:
>
> patternProperties:
> "pwm@[a-f0-9]+$": false
>
> See Documentation/devicetree/bindings/interconnect/qcom,rpm.yaml
> around line 240. We need something similar.
Sure, will add!
>
>
>> @@ -57,8 +74,29 @@ required:
>> additionalProperties: false
>>
>> examples:
>> + # Example 1 - Syscon node found on MSM8960
>> - |
>> syscon@...00000 {
>> compatible = "qcom,tcsr-msm8960", "syscon";
>> reg = <0x1a400000 0x100>;
>> };
>> + # Example 2 - Syscon node found on IPQ6018
>> + - |
>> + #include <dt-bindings/clock/qcom,gcc-ipq6018.h>
>> +
>> + syscon@...7000 {
>> + compatible = "qcom,tcsr-ipq6018", "syscon", "simple-mfd";
>> + reg = <0x01937000 0x21000>;
>> + #address-cells = <1>;
>> + #size-cells = <1>;
>> + ranges = <0 0x1937000 0x21000>;
>
> Please put ranges just after reg.
Sure,okay
Thanks,
Devi Priya
>
>> +
>> + pwm: pwm@...0 {
>> + compatible = "qcom,ipq6018-pwm";
>> + reg = <0xa010 0x20>;
>
> Best regards,
> Krzysztof
>
Powered by blists - more mailing lists