2022-08-19 19:17:56

by Apurva Nandan

[permalink] [raw]
Subject: [PATCH 0/3] Fix ti,sci-inta/intr and mmc dtbs_check warnings for TI K3 platforms

This series fixes the following warnings common across all TI K3
platforms:

k3-j721e-common-proc-board.dtb: interrupt-controller@33d00000: Unevaluated properties are not allowed ('#interrupt-cells' was unexpected)
From schema: linux/Documentation/devicetree/bindings/interrupt-controller/ti,sci-inta.yaml

k3-j721s2-common-proc-board.dtb: mmc@4f80000: Unevaluated properties are not allowed ('dma-coherent' was unexpected)
From schema: linux/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml

k3-j721s2-common-proc-board.dtb: interrupt-controller@310e0000: Unevaluated properties are not allowed ('reg' was unexpected)
From schema: linux/Documentation/devicetree/bindings/interrupt-controller/ti,sci-intr.yaml

Apurva Nandan (3):
dt-bindings: irqchip: ti, sci-intr/inta: Fix warning
'#interrupt-cells' was unexpected
dt-bindings: mmc: Fix 'dma-coherent' was unexpected
dt-bindings: interrupt-controller: Fix 'reg' was unexpected

.../devicetree/bindings/interrupt-controller/ti,sci-inta.yaml | 3 +++
.../devicetree/bindings/interrupt-controller/ti,sci-intr.yaml | 3 +++
Documentation/devicetree/bindings/mmc/sdhci-am654.yaml | 3 +++
3 files changed, 9 insertions(+)

--
2.17.1


2022-08-19 19:19:27

by Apurva Nandan

[permalink] [raw]
Subject: [PATCH 3/3] dt-bindings: interrupt-controller: Fix 'reg' was unexpected

Fix the following warning in dtbs_check
interrupt-controller@a00000: Unevaluated properties are not allowed ('reg' was unexpected)

Add the reg property in the schema.

Signed-off-by: Apurva Nandan <[email protected]>
---
.../devicetree/bindings/interrupt-controller/ti,sci-intr.yaml | 3 +++
1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/interrupt-controller/ti,sci-intr.yaml b/Documentation/devicetree/bindings/interrupt-controller/ti,sci-intr.yaml
index e12aee42b126..c99cc7323c71 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/ti,sci-intr.yaml
+++ b/Documentation/devicetree/bindings/interrupt-controller/ti,sci-intr.yaml
@@ -58,6 +58,9 @@ properties:
1 = If intr supports edge triggered interrupts.
4 = If intr supports level triggered interrupts.

+ reg:
+ maxItems: 1
+
interrupt-controller: true

'#interrupt-cells':
--
2.17.1

2022-08-19 19:32:08

by Apurva Nandan

[permalink] [raw]
Subject: [PATCH 2/3] dt-bindings: mmc: Fix 'dma-coherent' was unexpected

dma-coherent is mentioned in almost all TI K3 platform mmc
nodes. Fix warning generated due to its missing match in
yaml schema.

Signed-off-by: Apurva Nandan <[email protected]>
---
Documentation/devicetree/bindings/mmc/sdhci-am654.yaml | 3 +++
1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml b/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml
index 0ab07759b472..ea9121fb188d 100644
--- a/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml
+++ b/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml
@@ -51,6 +51,9 @@ properties:

sdhci-caps-mask: true

+ dma-coherent:
+ type: boolean
+
# PHY output tap delays:
# Used to delay the data valid window and align it to the sampling clock.
# Binding needs to be provided for each supported speed mode otherwise the
--
2.17.1

2022-08-19 19:45:06

by Apurva Nandan

[permalink] [raw]
Subject: [PATCH 1/3] dt-bindings: irqchip: ti,sci-inta: Fix warning '#interrupt-cells' was unexpected

#interrupt-cells = <0> property is required in device tree for
ti,sci-inta nodes, or else we will have following warning when building
device tree files with W=2 warning level.

arch/arm64/boot/dts/ti/k3-j721e-main.dtsi:147.51-156.5: Warning (interrupt_provider): /bus@100000/main-navss/interrupt-controller@33d00000: Missing #interrupt-cells in interrupt provider

And further, #interrupt-cells is required to be in yaml bindings as well
to prevent following schema warnings:

k3-j721e-common-proc-board.dtb: interrupt-controller@33d00000: Unevaluated properties are not allowed ('#interrupt-cells' was unexpected)
From schema: linux/Documentation/devicetree/bindings/interrupt-controller/ti,sci-inta.yaml

Add #interrupt-cells property in ti,sci-inta.yaml

Signed-off-by: Apurva Nandan <[email protected]>
---
.../devicetree/bindings/interrupt-controller/ti,sci-inta.yaml | 3 +++
1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/interrupt-controller/ti,sci-inta.yaml b/Documentation/devicetree/bindings/interrupt-controller/ti,sci-inta.yaml
index 88c46e61732e..1151518859bd 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/ti,sci-inta.yaml
+++ b/Documentation/devicetree/bindings/interrupt-controller/ti,sci-inta.yaml
@@ -59,6 +59,9 @@ properties:

interrupt-controller: true

+ '#interrupt-cells':
+ const: 0
+
msi-controller: true

ti,interrupt-ranges:
--
2.17.1

2022-08-22 19:51:32

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH 1/3] dt-bindings: irqchip: ti,sci-inta: Fix warning '#interrupt-cells' was unexpected

On Sat, 20 Aug 2022 00:37:27 +0530, Apurva Nandan wrote:
> #interrupt-cells = <0> property is required in device tree for
> ti,sci-inta nodes, or else we will have following warning when building
> device tree files with W=2 warning level.
>
> arch/arm64/boot/dts/ti/k3-j721e-main.dtsi:147.51-156.5: Warning (interrupt_provider): /bus@100000/main-navss/interrupt-controller@33d00000: Missing #interrupt-cells in interrupt provider
>
> And further, #interrupt-cells is required to be in yaml bindings as well
> to prevent following schema warnings:
>
> k3-j721e-common-proc-board.dtb: interrupt-controller@33d00000: Unevaluated properties are not allowed ('#interrupt-cells' was unexpected)
> From schema: linux/Documentation/devicetree/bindings/interrupt-controller/ti,sci-inta.yaml
>
> Add #interrupt-cells property in ti,sci-inta.yaml
>
> Signed-off-by: Apurva Nandan <[email protected]>
> ---
> .../devicetree/bindings/interrupt-controller/ti,sci-inta.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>

Acked-by: Rob Herring <[email protected]>

2022-08-22 19:58:11

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH 2/3] dt-bindings: mmc: Fix 'dma-coherent' was unexpected

On Sat, 20 Aug 2022 00:37:28 +0530, Apurva Nandan wrote:
> dma-coherent is mentioned in almost all TI K3 platform mmc
> nodes. Fix warning generated due to its missing match in
> yaml schema.
>
> Signed-off-by: Apurva Nandan <[email protected]>
> ---
> Documentation/devicetree/bindings/mmc/sdhci-am654.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>

Acked-by: Rob Herring <[email protected]>

2022-08-22 20:08:17

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH 3/3] dt-bindings: interrupt-controller: Fix 'reg' was unexpected

On Sat, 20 Aug 2022 00:37:29 +0530, Apurva Nandan wrote:
> Fix the following warning in dtbs_check
> interrupt-controller@a00000: Unevaluated properties are not allowed ('reg' was unexpected)
>
> Add the reg property in the schema.
>
> Signed-off-by: Apurva Nandan <[email protected]>
> ---
> .../devicetree/bindings/interrupt-controller/ti,sci-intr.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>

Acked-by: Rob Herring <[email protected]>

2022-08-26 09:52:40

by Ulf Hansson

[permalink] [raw]
Subject: Re: [PATCH 2/3] dt-bindings: mmc: Fix 'dma-coherent' was unexpected

On Fri, 19 Aug 2022 at 21:07, Apurva Nandan <[email protected]> wrote:
>
> dma-coherent is mentioned in almost all TI K3 platform mmc
> nodes. Fix warning generated due to its missing match in
> yaml schema.
>
> Signed-off-by: Apurva Nandan <[email protected]>

Applied for next, thanks!

Kind regards
Uffe


> ---
> Documentation/devicetree/bindings/mmc/sdhci-am654.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml b/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml
> index 0ab07759b472..ea9121fb188d 100644
> --- a/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml
> +++ b/Documentation/devicetree/bindings/mmc/sdhci-am654.yaml
> @@ -51,6 +51,9 @@ properties:
>
> sdhci-caps-mask: true
>
> + dma-coherent:
> + type: boolean
> +
> # PHY output tap delays:
> # Used to delay the data valid window and align it to the sampling clock.
> # Binding needs to be provided for each supported speed mode otherwise the
> --
> 2.17.1
>

Subject: [irqchip: irq/irqchip-next] dt-bindings: irqchip: ti,sci-inta: Fix warning for missing #interrupt-cells

The following commit has been merged into the irq/irqchip-next branch of irqchip:

Commit-ID: d9fc272bfd76acadf0537901549d07a1b81dbeed
Gitweb: https://git.kernel.org/pub/scm/linux/kernel/git/maz/arm-platforms/d9fc272bfd76acadf0537901549d07a1b81dbeed
Author: Apurva Nandan <[email protected]>
AuthorDate: Sat, 20 Aug 2022 00:37:27 +05:30
Committer: Marc Zyngier <[email protected]>
CommitterDate: Thu, 29 Sep 2022 11:10:10 +01:00

dt-bindings: irqchip: ti,sci-inta: Fix warning for missing #interrupt-cells

ti,sci-inta nodes, or else we will have following warning when building
device tree files with W=2 warning level.

arch/arm64/boot/dts/ti/k3-j721e-main.dtsi:147.51-156.5: Warning (interrupt_provider): /bus@100000/main-navss/interrupt-controller@33d00000: Missing #interrupt-cells in interrupt provider

And further, #interrupt-cells is required to be in yaml bindings as well
to prevent following schema warnings:

k3-j721e-common-proc-board.dtb: interrupt-controller@33d00000: Unevaluated properties are not allowed ('#interrupt-cells' was unexpected)
>From schema: linux/Documentation/devicetree/bindings/interrupt-controller/ti,sci-inta.yaml

Add #interrupt-cells property in ti,sci-inta.yaml

Signed-off-by: Apurva Nandan <[email protected]>
Acked-by: Rob Herring <[email protected]>
Signed-off-by: Marc Zyngier <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
---
Documentation/devicetree/bindings/interrupt-controller/ti,sci-inta.yaml | 3 +++
1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/interrupt-controller/ti,sci-inta.yaml b/Documentation/devicetree/bindings/interrupt-controller/ti,sci-inta.yaml
index 88c46e6..1151518 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/ti,sci-inta.yaml
+++ b/Documentation/devicetree/bindings/interrupt-controller/ti,sci-inta.yaml
@@ -59,6 +59,9 @@ properties:

interrupt-controller: true

+ '#interrupt-cells':
+ const: 0
+
msi-controller: true

ti,interrupt-ranges:

Subject: [irqchip: irq/irqchip-next] dt-bindings: interrupt-controller: ti,sci-intr: Fix missing reg property in the binding

The following commit has been merged into the irq/irqchip-next branch of irqchip:

Commit-ID: daa0b6d0187599a574cb5cb392b259bda3dcf979
Gitweb: https://git.kernel.org/pub/scm/linux/kernel/git/maz/arm-platforms/daa0b6d0187599a574cb5cb392b259bda3dcf979
Author: Apurva Nandan <[email protected]>
AuthorDate: Sat, 20 Aug 2022 00:37:29 +05:30
Committer: Marc Zyngier <[email protected]>
CommitterDate: Thu, 29 Sep 2022 11:13:43 +01:00

dt-bindings: interrupt-controller: ti,sci-intr: Fix missing reg property in the binding

Fix the following warning in dtbs_check
interrupt-controller@a00000: Unevaluated properties are not allowed ('reg' was unexpected)

Add the reg property in the schema.

Signed-off-by: Apurva Nandan <[email protected]>
Acked-by: Rob Herring <[email protected]>
Signed-off-by: Marc Zyngier <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
---
Documentation/devicetree/bindings/interrupt-controller/ti,sci-intr.yaml | 3 +++
1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/interrupt-controller/ti,sci-intr.yaml b/Documentation/devicetree/bindings/interrupt-controller/ti,sci-intr.yaml
index e12aee4..c99cc73 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/ti,sci-intr.yaml
+++ b/Documentation/devicetree/bindings/interrupt-controller/ti,sci-intr.yaml
@@ -58,6 +58,9 @@ properties:
1 = If intr supports edge triggered interrupts.
4 = If intr supports level triggered interrupts.

+ reg:
+ maxItems: 1
+
interrupt-controller: true

'#interrupt-cells':