* [PATCH 0/2] dt-bindings: PCI: ti,j721e-pci-*: resolve unexpected property warnings
@ 2022-09-22 20:44 ` Matt Ranostay
0 siblings, 0 replies; 14+ messages in thread
From: Matt Ranostay @ 2022-09-22 20:44 UTC (permalink / raw)
To: vigneshr; +Cc: devicetree, linux-arm-kernel, Matt Ranostay
Resolve unexpected property warnings related to interrupts in both J721E PCI EP and host
yaml files.
Matt Ranostay (2):
dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller
definition
dt-bindings: PCI: ti,j721e-pci-*: Add missing interrupt properties
.../bindings/pci/ti,j721e-pci-ep.yaml | 7 +++++++
.../bindings/pci/ti,j721e-pci-host.yaml | 20 +++++++++++++++++++
2 files changed, 27 insertions(+)
--
2.38.0.rc0.52.gdda7228a83
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 0/2] dt-bindings: PCI: ti,j721e-pci-*: resolve unexpected property warnings
@ 2022-09-22 20:44 ` Matt Ranostay
0 siblings, 0 replies; 14+ messages in thread
From: Matt Ranostay @ 2022-09-22 20:44 UTC (permalink / raw)
To: vigneshr; +Cc: devicetree, linux-arm-kernel, Matt Ranostay
Resolve unexpected property warnings related to interrupts in both J721E PCI EP and host
yaml files.
Matt Ranostay (2):
dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller
definition
dt-bindings: PCI: ti,j721e-pci-*: Add missing interrupt properties
.../bindings/pci/ti,j721e-pci-ep.yaml | 7 +++++++
.../bindings/pci/ti,j721e-pci-host.yaml | 20 +++++++++++++++++++
2 files changed, 27 insertions(+)
--
2.38.0.rc0.52.gdda7228a83
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 1/2] dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller definition
2022-09-22 20:44 ` Matt Ranostay
@ 2022-09-22 20:44 ` Matt Ranostay
-1 siblings, 0 replies; 14+ messages in thread
From: Matt Ranostay @ 2022-09-22 20:44 UTC (permalink / raw)
To: vigneshr; +Cc: devicetree, linux-arm-kernel, Matt Ranostay
Add missing 'interrupt-controller' property and related subnodes to resolve
the following warning:
arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dtb: pcie@2910000: Unevaluated properties are not allowed ('interrupt-conroller' was unexpected)
From schema: Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
Signed-off-by: Matt Ranostay <mranostay@ti.com>
---
.../devicetree/bindings/pci/ti,j721e-pci-host.yaml | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
index 2115d5a3f0e1..0f5914a22c14 100644
--- a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
+++ b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
@@ -76,6 +76,19 @@ properties:
msi-map: true
+ interrupt-controller:
+ type: object
+ additionalProperties: false
+
+ properties:
+ interrupt-controller: true
+
+ '#interrupt-cells':
+ const: 1
+
+ interrupts:
+ maxItems: 1
+
required:
- compatible
- reg
--
2.38.0.rc0.52.gdda7228a83
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 1/2] dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller definition
@ 2022-09-22 20:44 ` Matt Ranostay
0 siblings, 0 replies; 14+ messages in thread
From: Matt Ranostay @ 2022-09-22 20:44 UTC (permalink / raw)
To: vigneshr; +Cc: devicetree, linux-arm-kernel, Matt Ranostay
Add missing 'interrupt-controller' property and related subnodes to resolve
the following warning:
arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dtb: pcie@2910000: Unevaluated properties are not allowed ('interrupt-conroller' was unexpected)
From schema: Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
Signed-off-by: Matt Ranostay <mranostay@ti.com>
---
.../devicetree/bindings/pci/ti,j721e-pci-host.yaml | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
index 2115d5a3f0e1..0f5914a22c14 100644
--- a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
+++ b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
@@ -76,6 +76,19 @@ properties:
msi-map: true
+ interrupt-controller:
+ type: object
+ additionalProperties: false
+
+ properties:
+ interrupt-controller: true
+
+ '#interrupt-cells':
+ const: 1
+
+ interrupts:
+ maxItems: 1
+
required:
- compatible
- reg
--
2.38.0.rc0.52.gdda7228a83
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 2/2] dt-bindings: PCI: ti,j721e-pci-*: Add missing interrupt properties
2022-09-22 20:44 ` Matt Ranostay
@ 2022-09-22 20:44 ` Matt Ranostay
-1 siblings, 0 replies; 14+ messages in thread
From: Matt Ranostay @ 2022-09-22 20:44 UTC (permalink / raw)
To: vigneshr; +Cc: devicetree, linux-arm-kernel, Matt Ranostay
Both interrupts, and interrupt names weren't defined in both EP and host
yaml. Also define the only possible interrupt-name as link_state, and
maxItems of interrupts to one.
This patch resolves the following warning:
arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dtb: pcie-ep@2910000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
From schema Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml
Signed-off-by: Matt Ranostay <mranostay@ti.com>
---
Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml | 7 +++++++
.../devicetree/bindings/pci/ti,j721e-pci-host.yaml | 7 +++++++
2 files changed, 14 insertions(+)
diff --git a/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml b/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml
index aed437dac363..10e6eabdff53 100644
--- a/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml
+++ b/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml
@@ -58,6 +58,13 @@ properties:
dma-coherent:
description: Indicates that the PCIe IP block can ensure the coherency
+ interrupts:
+ maxItems: 1
+
+ interrupt-names:
+ items:
+ - const: link_state
+
required:
- compatible
- reg
diff --git a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
index 0f5914a22c14..d9df7cd922f1 100644
--- a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
+++ b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
@@ -76,6 +76,13 @@ properties:
msi-map: true
+ interrupts:
+ maxItems: 1
+
+ interrupt-names:
+ items:
+ - const: link_state
+
interrupt-controller:
type: object
additionalProperties: false
--
2.38.0.rc0.52.gdda7228a83
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 2/2] dt-bindings: PCI: ti,j721e-pci-*: Add missing interrupt properties
@ 2022-09-22 20:44 ` Matt Ranostay
0 siblings, 0 replies; 14+ messages in thread
From: Matt Ranostay @ 2022-09-22 20:44 UTC (permalink / raw)
To: vigneshr; +Cc: devicetree, linux-arm-kernel, Matt Ranostay
Both interrupts, and interrupt names weren't defined in both EP and host
yaml. Also define the only possible interrupt-name as link_state, and
maxItems of interrupts to one.
This patch resolves the following warning:
arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dtb: pcie-ep@2910000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
From schema Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml
Signed-off-by: Matt Ranostay <mranostay@ti.com>
---
Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml | 7 +++++++
.../devicetree/bindings/pci/ti,j721e-pci-host.yaml | 7 +++++++
2 files changed, 14 insertions(+)
diff --git a/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml b/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml
index aed437dac363..10e6eabdff53 100644
--- a/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml
+++ b/Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml
@@ -58,6 +58,13 @@ properties:
dma-coherent:
description: Indicates that the PCIe IP block can ensure the coherency
+ interrupts:
+ maxItems: 1
+
+ interrupt-names:
+ items:
+ - const: link_state
+
required:
- compatible
- reg
diff --git a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
index 0f5914a22c14..d9df7cd922f1 100644
--- a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
+++ b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
@@ -76,6 +76,13 @@ properties:
msi-map: true
+ interrupts:
+ maxItems: 1
+
+ interrupt-names:
+ items:
+ - const: link_state
+
interrupt-controller:
type: object
additionalProperties: false
--
2.38.0.rc0.52.gdda7228a83
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller definition
2022-09-22 20:44 ` Matt Ranostay
@ 2022-09-23 9:31 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-23 9:31 UTC (permalink / raw)
To: Matt Ranostay; +Cc: devicetree, vigneshr, linux-arm-kernel
On Thu, 22 Sep 2022 13:44:33 -0700, Matt Ranostay wrote:
> Add missing 'interrupt-controller' property and related subnodes to resolve
> the following warning:
You did not Cc respective maintainers on your patch. Use
scripts/get_maintainers.pl.
>
> arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dtb: pcie@2910000: Unevaluated properties are not allowed ('interrupt-conroller' was unexpected)
> From schema: Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
>
> Signed-off-by: Matt Ranostay <mranostay@ti.com>
> ---
> .../devicetree/bindings/pci/ti,j721e-pci-host.yaml | 13 +++++++++++++
> 1 file changed, 13 insertions(+)
>
Running 'make dtbs_check' with the schema in this patch gives the
following warnings. Consider if they are expected or the schema is
incorrect. These may not be new warnings.
Note that it is not yet a requirement to have 0 warnings for dtbs_check.
This will change in the future.
Full log is available here: https://patchwork.ozlabs.org/patch/1681334
pcie@2900000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
pcie@2910000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dtb
arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
pcie@2920000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
pcie@2930000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
pcie@f102000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
arch/arm64/boot/dts/ti/k3-am642-evm.dtb
arch/arm64/boot/dts/ti/k3-am642-sk.dtb
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller definition
@ 2022-09-23 9:31 ` Krzysztof Kozlowski
0 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-23 9:31 UTC (permalink / raw)
To: Matt Ranostay; +Cc: devicetree, vigneshr, linux-arm-kernel
On Thu, 22 Sep 2022 13:44:33 -0700, Matt Ranostay wrote:
> Add missing 'interrupt-controller' property and related subnodes to resolve
> the following warning:
You did not Cc respective maintainers on your patch. Use
scripts/get_maintainers.pl.
>
> arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dtb: pcie@2910000: Unevaluated properties are not allowed ('interrupt-conroller' was unexpected)
> From schema: Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
>
> Signed-off-by: Matt Ranostay <mranostay@ti.com>
> ---
> .../devicetree/bindings/pci/ti,j721e-pci-host.yaml | 13 +++++++++++++
> 1 file changed, 13 insertions(+)
>
Running 'make dtbs_check' with the schema in this patch gives the
following warnings. Consider if they are expected or the schema is
incorrect. These may not be new warnings.
Note that it is not yet a requirement to have 0 warnings for dtbs_check.
This will change in the future.
Full log is available here: https://patchwork.ozlabs.org/patch/1681334
pcie@2900000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
pcie@2910000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dtb
arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
pcie@2920000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
pcie@2930000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
pcie@f102000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
arch/arm64/boot/dts/ti/k3-am642-evm.dtb
arch/arm64/boot/dts/ti/k3-am642-sk.dtb
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] dt-bindings: PCI: ti,j721e-pci-*: Add missing interrupt properties
2022-09-22 20:44 ` Matt Ranostay
@ 2022-09-23 11:29 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-23 11:29 UTC (permalink / raw)
To: Matt Ranostay, vigneshr; +Cc: devicetree, linux-arm-kernel
On 22/09/2022 22:44, Matt Ranostay wrote:
> Both interrupts, and interrupt names weren't defined in both EP and host
> yaml. Also define the only possible interrupt-name as link_state, and
> maxItems of interrupts to one.
>
> This patch resolves the following warning:
>
> arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dtb: pcie-ep@2910000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
> From schema Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml
Use scripts/get_maintainers.pl to CC all maintainers and relevant
mailing lists.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/2] dt-bindings: PCI: ti,j721e-pci-*: Add missing interrupt properties
@ 2022-09-23 11:29 ` Krzysztof Kozlowski
0 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-23 11:29 UTC (permalink / raw)
To: Matt Ranostay, vigneshr; +Cc: devicetree, linux-arm-kernel
On 22/09/2022 22:44, Matt Ranostay wrote:
> Both interrupts, and interrupt names weren't defined in both EP and host
> yaml. Also define the only possible interrupt-name as link_state, and
> maxItems of interrupts to one.
>
> This patch resolves the following warning:
>
> arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dtb: pcie-ep@2910000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
> From schema Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml
Use scripts/get_maintainers.pl to CC all maintainers and relevant
mailing lists.
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller definition
2022-09-23 9:31 ` Krzysztof Kozlowski
@ 2022-09-24 20:46 ` Matt Ranostay
-1 siblings, 0 replies; 14+ messages in thread
From: Matt Ranostay @ 2022-09-24 20:46 UTC (permalink / raw)
To: Krzysztof Kozlowski; +Cc: devicetree, vigneshr, linux-arm-kernel
On Fri, Sep 23, 2022 at 11:31:11AM +0200, Krzysztof Kozlowski wrote:
> On Thu, 22 Sep 2022 13:44:33 -0700, Matt Ranostay wrote:
> > Add missing 'interrupt-controller' property and related subnodes to resolve
> > the following warning:
>
> You did not Cc respective maintainers on your patch. Use
> scripts/get_maintainers.pl.
>
Yeah noticed I forgot to Cc Rob Herring :-/
>
> >
> > arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dtb: pcie@2910000: Unevaluated properties are not allowed ('interrupt-conroller' was unexpected)
> > From schema: Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
> >
> > Signed-off-by: Matt Ranostay <mranostay@ti.com>
> > ---
> > .../devicetree/bindings/pci/ti,j721e-pci-host.yaml | 13 +++++++++++++
> > 1 file changed, 13 insertions(+)
> >
>
> Running 'make dtbs_check' with the schema in this patch gives the
> following warnings. Consider if they are expected or the schema is
> incorrect. These may not be new warnings.
>
> Note that it is not yet a requirement to have 0 warnings for dtbs_check.
> This will change in the future.
>
> Full log is available here: https://patchwork.ozlabs.org/patch/1681334
>
>
> pcie@2900000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
> arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
> arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
>
> pcie@2910000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
> arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dtb
> arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
> arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
>
> pcie@2920000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
> arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
> arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
>
> pcie@2930000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
> arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
> arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
>
> pcie@f102000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
> arch/arm64/boot/dts/ti/k3-am642-evm.dtb
> arch/arm64/boot/dts/ti/k3-am642-sk.dtb
This get resolved in the second patchset in this series.
- Matt
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller definition
@ 2022-09-24 20:46 ` Matt Ranostay
0 siblings, 0 replies; 14+ messages in thread
From: Matt Ranostay @ 2022-09-24 20:46 UTC (permalink / raw)
To: Krzysztof Kozlowski; +Cc: devicetree, vigneshr, linux-arm-kernel
On Fri, Sep 23, 2022 at 11:31:11AM +0200, Krzysztof Kozlowski wrote:
> On Thu, 22 Sep 2022 13:44:33 -0700, Matt Ranostay wrote:
> > Add missing 'interrupt-controller' property and related subnodes to resolve
> > the following warning:
>
> You did not Cc respective maintainers on your patch. Use
> scripts/get_maintainers.pl.
>
Yeah noticed I forgot to Cc Rob Herring :-/
>
> >
> > arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dtb: pcie@2910000: Unevaluated properties are not allowed ('interrupt-conroller' was unexpected)
> > From schema: Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
> >
> > Signed-off-by: Matt Ranostay <mranostay@ti.com>
> > ---
> > .../devicetree/bindings/pci/ti,j721e-pci-host.yaml | 13 +++++++++++++
> > 1 file changed, 13 insertions(+)
> >
>
> Running 'make dtbs_check' with the schema in this patch gives the
> following warnings. Consider if they are expected or the schema is
> incorrect. These may not be new warnings.
>
> Note that it is not yet a requirement to have 0 warnings for dtbs_check.
> This will change in the future.
>
> Full log is available here: https://patchwork.ozlabs.org/patch/1681334
>
>
> pcie@2900000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
> arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
> arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
>
> pcie@2910000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
> arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dtb
> arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
> arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
>
> pcie@2920000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
> arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
> arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
>
> pcie@2930000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
> arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb
> arch/arm64/boot/dts/ti/k3-j721e-sk.dtb
>
> pcie@f102000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts' were unexpected)
> arch/arm64/boot/dts/ti/k3-am642-evm.dtb
> arch/arm64/boot/dts/ti/k3-am642-sk.dtb
This get resolved in the second patchset in this series.
- Matt
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller definition
2022-09-24 20:46 ` Matt Ranostay
@ 2022-09-25 7:38 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-25 7:38 UTC (permalink / raw)
To: Matt Ranostay; +Cc: devicetree, vigneshr, linux-arm-kernel
On 24/09/2022 22:46, Matt Ranostay wrote:
> On Fri, Sep 23, 2022 at 11:31:11AM +0200, Krzysztof Kozlowski wrote:
>> On Thu, 22 Sep 2022 13:44:33 -0700, Matt Ranostay wrote:
>>> Add missing 'interrupt-controller' property and related subnodes to resolve
>>> the following warning:
>>
>> You did not Cc respective maintainers on your patch. Use
>> scripts/get_maintainers.pl.
>>
>
> Yeah noticed I forgot to Cc Rob Herring :-/
You did not Cc anyone... No one from PCI maintainers, no one from DT
bindings.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller definition
@ 2022-09-25 7:38 ` Krzysztof Kozlowski
0 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-25 7:38 UTC (permalink / raw)
To: Matt Ranostay; +Cc: devicetree, vigneshr, linux-arm-kernel
On 24/09/2022 22:46, Matt Ranostay wrote:
> On Fri, Sep 23, 2022 at 11:31:11AM +0200, Krzysztof Kozlowski wrote:
>> On Thu, 22 Sep 2022 13:44:33 -0700, Matt Ranostay wrote:
>>> Add missing 'interrupt-controller' property and related subnodes to resolve
>>> the following warning:
>>
>> You did not Cc respective maintainers on your patch. Use
>> scripts/get_maintainers.pl.
>>
>
> Yeah noticed I forgot to Cc Rob Herring :-/
You did not Cc anyone... No one from PCI maintainers, no one from DT
bindings.
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2022-09-25 7:40 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-22 20:44 [PATCH 0/2] dt-bindings: PCI: ti,j721e-pci-*: resolve unexpected property warnings Matt Ranostay
2022-09-22 20:44 ` Matt Ranostay
2022-09-22 20:44 ` [PATCH 1/2] dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller definition Matt Ranostay
2022-09-22 20:44 ` Matt Ranostay
2022-09-23 9:31 ` Krzysztof Kozlowski
2022-09-23 9:31 ` Krzysztof Kozlowski
2022-09-24 20:46 ` Matt Ranostay
2022-09-24 20:46 ` Matt Ranostay
2022-09-25 7:38 ` Krzysztof Kozlowski
2022-09-25 7:38 ` Krzysztof Kozlowski
2022-09-22 20:44 ` [PATCH 2/2] dt-bindings: PCI: ti,j721e-pci-*: Add missing interrupt properties Matt Ranostay
2022-09-22 20:44 ` Matt Ranostay
2022-09-23 11:29 ` Krzysztof Kozlowski
2022-09-23 11:29 ` Krzysztof Kozlowski
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.