2022-10-28 15:43:44

by Yassine Oudjana

Subject: [PATCH v4 05/13] dt-bindings: pinctrl: mediatek,pinctrl-mt6795: Fix interrupt count

From: Yassine Oudjana <[email protected]>

The document currently states a maximum of 1 interrupt, but the DT
has 2 specified causing a dtbs_check error. Replace the maximum limit
with a minimum and add per-interrupt descriptions to pass the check.

Fixes: 81557a71564a ("dt-bindings: pinctrl: Add MediaTek MT6795 pinctrl bindings")
Suggested-by: AngeloGioacchino Del Regno <[email protected]>
Signed-off-by: Yassine Oudjana <[email protected]>
Reviewed-by: AngeloGioacchino Del Regno <[email protected]>
---
.../devicetree/bindings/pinctrl/mediatek,pinctrl-mt6795.yaml | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/mediatek,pinctrl-mt6795.yaml b/Documentation/devicetree/bindings/pinctrl/mediatek,pinctrl-mt6795.yaml
index 73ae6e11410b..a3a3f7fb9605 100644
--- a/Documentation/devicetree/bindings/pinctrl/mediatek,pinctrl-mt6795.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/mediatek,pinctrl-mt6795.yaml
@@ -47,7 +47,10 @@ properties:

interrupts:
description: The interrupt outputs to sysirq.
- maxItems: 1
+ minItems: 1
+ items:
+ - description: EINT interrupt
+ - description: EINT event_b interrupt

# PIN CONFIGURATION NODES
patternProperties:
--
2.38.1

2022-10-28 20:16:30

by Krzysztof Kozlowski

Subject: Re: [PATCH v4 05/13] dt-bindings: pinctrl: mediatek,pinctrl-mt6795: Fix interrupt count

On 28/10/2022 11:34, Yassine Oudjana wrote:
> From: Yassine Oudjana <[email protected]>
>
> The document currently states a maximum of 1 interrupt, but the DT
> has 2 specified causing a dtbs_check error. Replace the maximum limit
> with a minimum and add per-interrupt descriptions to pass the check.
>


Acked-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof

2022-11-08 12:37:01

by Linus Walleij

Subject: Re: [PATCH v4 05/13] dt-bindings: pinctrl: mediatek,pinctrl-mt6795: Fix interrupt count

On Fri, Oct 28, 2022 at 5:35 PM Yassine Oudjana
<[email protected]> wrote:

> From: Yassine Oudjana <[email protected]>
>
> The document currently states a maximum of 1 interrupt, but the DT
> has 2 specified causing a dtbs_check error. Replace the maximum limit
> with a minimum and add per-interrupt descriptions to pass the check.
>
> Fixes: 81557a71564a ("dt-bindings: pinctrl: Add MediaTek MT6795 pinctrl bindings")
> Suggested-by: AngeloGioacchino Del Regno <[email protected]>
> Signed-off-by: Yassine Oudjana <[email protected]>
> Reviewed-by: AngeloGioacchino Del Regno <[email protected]>

Patch applied.

Yours,
Linus Walleij