[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <9858ecfc-6a51-71a9-70e7-dd1e3eb9ba65@linaro.org>
Date: Wed, 8 Mar 2023 08:45:44 +0100
From: Konrad Dybcio <konrad.dybcio@...aro.org>
To: Shawn Guo <shawn.guo@...aro.org>
Cc: linux-arm-msm@...r.kernel.org, andersson@...nel.org,
agross@...nel.org, marijn.suijten@...ainline.org,
Thomas Gleixner <tglx@...utronix.de>,
Marc Zyngier <maz@...nel.org>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
linux-kernel@...r.kernel.org, devicetree@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: interrupt-controller: Allow
#power-domain-cells
On 8.03.2023 08:24, Shawn Guo wrote:
> On Wed, Mar 8, 2023 at 9:17 AM Konrad Dybcio <konrad.dybcio@...aro.org> wrote:
>>
>> MPM provides a single genpd. Allow #power-domain-cells = <0>.
>
> Indeed! It was missing from the initial submission. Thanks for fixing it up!
>
>>
>> Fixes: 54fc9851c0e0 ("dt-bindings: interrupt-controller: Add Qualcomm MPM support")
>> Signed-off-by: Konrad Dybcio <konrad.dybcio@...aro.org>
>> ---
>> .../devicetree/bindings/interrupt-controller/qcom,mpm.yaml | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/interrupt-controller/qcom,mpm.yaml b/Documentation/devicetree/bindings/interrupt-controller/qcom,mpm.yaml
>> index 509d20c091af..4c470dc143e5 100644
>> --- a/Documentation/devicetree/bindings/interrupt-controller/qcom,mpm.yaml
>> +++ b/Documentation/devicetree/bindings/interrupt-controller/qcom,mpm.yaml
>> @@ -62,6 +62,9 @@ properties:
>> - description: MPM pin number
>> - description: GIC SPI number for the MPM pin
>>
>> + '#power-domain-cells':
>
> Should we add it to the "required" list and examples?
I was debating it.. it's definitely not *required*, as the
irqchip part works even without it and then on SoCs with MPM
but without PSCI the setup may be tricky..
About the example, both yes/no are okay for me
Konrad
>
> Shawn
>
>> + const: 0
>> +
>> required:
>> - compatible
>> - reg
>> --
>> 2.39.2
>>
Powered by blists - more mailing lists