On 17/04/2023 09:55, Krzysztof Kozlowski wrote: > On 17/04/2023 09:27, Bartosz Golaszewski wrote: >> On Sun, Apr 16, 2023 at 5:04 PM Krzysztof Kozlowski >> <krzysztof.kozlowski@xxxxxxxxxx> wrote: >>> >>> On 14/04/2023 11:33, Bartosz Golaszewski wrote: >>>> On Thu, Apr 6, 2023 at 4:10 PM Bartosz Golaszewski <brgl@xxxxxxxx> wrote: >>>>> >>>>> On Mon, Mar 27, 2023 at 2:53 PM Bartosz Golaszewski <brgl@xxxxxxxx> wrote: >>>>>> >>>>>> From: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx> >>>>>> >>>>>> Add a compatible for the Power Domain Controller on SA8775p platforms. >>>>>> Increase the number of PDC pin mappings. >>>>>> >>>>>> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx> >>>>>> Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx> >>>>>> Cc: Marc Zyngier <maz@xxxxxxxxxx> >>>>>> --- >>>>>> .../devicetree/bindings/interrupt-controller/qcom,pdc.yaml | 3 ++- >>>>>> 1 file changed, 2 insertions(+), 1 deletion(-) >>>>>> >>>>>> diff --git a/Documentation/devicetree/bindings/interrupt-controller/qcom,pdc.yaml b/Documentation/devicetree/bindings/interrupt-controller/qcom,pdc.yaml >>>>>> index 94791e261c42..641ff32e4a6c 100644 >>>>>> --- a/Documentation/devicetree/bindings/interrupt-controller/qcom,pdc.yaml >>>>>> +++ b/Documentation/devicetree/bindings/interrupt-controller/qcom,pdc.yaml >>>>>> @@ -26,6 +26,7 @@ properties: >>>>>> compatible: >>>>>> items: >>>>>> - enum: >>>>>> + - qcom,sa8775p-pdc >>>>>> - qcom,sc7180-pdc >>>>>> - qcom,sc7280-pdc >>>>>> - qcom,sc8280xp-pdc >>>>>> @@ -53,7 +54,7 @@ properties: >>>>>> qcom,pdc-ranges: >>>>>> $ref: /schemas/types.yaml#/definitions/uint32-matrix >>>>>> minItems: 1 >>>>>> - maxItems: 32 # no hard limit >>>>>> + maxItems: 38 # no hard limit >>> >>> I don't think the limit is correct. I still see warnings with this >>> patch. We already have 57 elements, so limit should be I guess 128 or >>> something. >>> >> >> You mean for other platforms? This limit applies to sa8775p as the > > I see errors on sa8775p. OK, so as you pointed out it's about other platform - sa8540p-ride. Anyway you update now this line to bump the limit, so bump it to something high instead of changing it in two patches. Best regards, Krzysztof
