All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH RESEND v2 0/2] dt-bindings: PCI: ti,j721e-pci-*: resolve unexpected property warnings
@ 2022-10-25  8:19 ` Matt Ranostay
  0 siblings, 0 replies; 8+ messages in thread
From: Matt Ranostay @ 2022-10-25  8:19 UTC (permalink / raw)
  To: bhelgaas, robh+dt, krzysztof.kozlowski+dt
  Cc: linux-pci, linux-arm-kernel, Matt Ranostay

Resolve unexpected property warnings related to interrupts in both J721E PCI EP and host
yaml files.

Changes from v1:
* Fix typo in commit message
* Add missing Cc to maintainers

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.GIT


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH RESEND v2 0/2] dt-bindings: PCI: ti,j721e-pci-*: resolve unexpected property warnings
@ 2022-10-25  8:19 ` Matt Ranostay
  0 siblings, 0 replies; 8+ messages in thread
From: Matt Ranostay @ 2022-10-25  8:19 UTC (permalink / raw)
  To: bhelgaas, robh+dt, krzysztof.kozlowski+dt
  Cc: linux-pci, linux-arm-kernel, Matt Ranostay

Resolve unexpected property warnings related to interrupts in both J721E PCI EP and host
yaml files.

Changes from v1:
* Fix typo in commit message
* Add missing Cc to maintainers

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.GIT


_______________________________________________
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] 8+ messages in thread

* [PATCH RESEND v2 1/2] dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller definition
  2022-10-25  8:19 ` Matt Ranostay
@ 2022-10-25  8:19   ` Matt Ranostay
  -1 siblings, 0 replies; 8+ messages in thread
From: Matt Ranostay @ 2022-10-25  8:19 UTC (permalink / raw)
  To: bhelgaas, robh+dt, krzysztof.kozlowski+dt
  Cc: linux-pci, linux-arm-kernel, Matt Ranostay, Rob Herring

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-controller' was unexpected)
        From schema: Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml

Acked-by: Rob Herring <robh@kernel.org>
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.GIT


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH RESEND v2 1/2] dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller definition
@ 2022-10-25  8:19   ` Matt Ranostay
  0 siblings, 0 replies; 8+ messages in thread
From: Matt Ranostay @ 2022-10-25  8:19 UTC (permalink / raw)
  To: bhelgaas, robh+dt, krzysztof.kozlowski+dt
  Cc: linux-pci, linux-arm-kernel, Matt Ranostay, Rob Herring

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-controller' was unexpected)
        From schema: Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml

Acked-by: Rob Herring <robh@kernel.org>
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.GIT


_______________________________________________
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] 8+ messages in thread

* [PATCH RESEND v2 2/2] dt-bindings: PCI: ti,j721e-pci-*: Add missing interrupt properties
  2022-10-25  8:19 ` Matt Ranostay
@ 2022-10-25  8:19   ` Matt Ranostay
  -1 siblings, 0 replies; 8+ messages in thread
From: Matt Ranostay @ 2022-10-25  8:19 UTC (permalink / raw)
  To: bhelgaas, robh+dt, krzysztof.kozlowski+dt
  Cc: linux-pci, linux-arm-kernel, Matt Ranostay, Rob Herring

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

Acked-by: Rob Herring <robh@kernel.org>
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.GIT


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH RESEND v2 2/2] dt-bindings: PCI: ti,j721e-pci-*: Add missing interrupt properties
@ 2022-10-25  8:19   ` Matt Ranostay
  0 siblings, 0 replies; 8+ messages in thread
From: Matt Ranostay @ 2022-10-25  8:19 UTC (permalink / raw)
  To: bhelgaas, robh+dt, krzysztof.kozlowski+dt
  Cc: linux-pci, linux-arm-kernel, Matt Ranostay, Rob Herring

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

Acked-by: Rob Herring <robh@kernel.org>
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.GIT


_______________________________________________
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] 8+ messages in thread

* Re: [PATCH RESEND v2 0/2] dt-bindings: PCI: ti,j721e-pci-*: resolve unexpected property warnings
  2022-10-25  8:19 ` Matt Ranostay
@ 2022-10-27  8:10   ` Lorenzo Pieralisi
  -1 siblings, 0 replies; 8+ messages in thread
From: Lorenzo Pieralisi @ 2022-10-27  8:10 UTC (permalink / raw)
  To: bhelgaas, krzysztof.kozlowski+dt, Matt Ranostay, robh+dt
  Cc: Lorenzo Pieralisi, linux-pci, linux-arm-kernel

On Tue, 25 Oct 2022 01:19:07 -0700, Matt Ranostay wrote:
> Resolve unexpected property warnings related to interrupts in both J721E PCI EP and host
> yaml files.
> 
> Changes from v1:
> * Fix typo in commit message
> * Add missing Cc to maintainers
> 
> [...]

Applied to pci/dt, thanks!

[1/2] dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller definition
      https://git.kernel.org/lpieralisi/pci/c/ba4ff1cb6cac
[2/2] dt-bindings: PCI: ti,j721e-pci-*: Add missing interrupt properties
      https://git.kernel.org/lpieralisi/pci/c/598418e60356

Thanks,
Lorenzo

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH RESEND v2 0/2] dt-bindings: PCI: ti,j721e-pci-*: resolve unexpected property warnings
@ 2022-10-27  8:10   ` Lorenzo Pieralisi
  0 siblings, 0 replies; 8+ messages in thread
From: Lorenzo Pieralisi @ 2022-10-27  8:10 UTC (permalink / raw)
  To: bhelgaas, krzysztof.kozlowski+dt, Matt Ranostay, robh+dt
  Cc: Lorenzo Pieralisi, linux-pci, linux-arm-kernel

On Tue, 25 Oct 2022 01:19:07 -0700, Matt Ranostay wrote:
> Resolve unexpected property warnings related to interrupts in both J721E PCI EP and host
> yaml files.
> 
> Changes from v1:
> * Fix typo in commit message
> * Add missing Cc to maintainers
> 
> [...]

Applied to pci/dt, thanks!

[1/2] dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller definition
      https://git.kernel.org/lpieralisi/pci/c/ba4ff1cb6cac
[2/2] dt-bindings: PCI: ti,j721e-pci-*: Add missing interrupt properties
      https://git.kernel.org/lpieralisi/pci/c/598418e60356

Thanks,
Lorenzo

_______________________________________________
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] 8+ messages in thread

end of thread, other threads:[~2022-10-27  8:12 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-25  8:19 [PATCH RESEND v2 0/2] dt-bindings: PCI: ti,j721e-pci-*: resolve unexpected property warnings Matt Ranostay
2022-10-25  8:19 ` Matt Ranostay
2022-10-25  8:19 ` [PATCH RESEND v2 1/2] dt-bindings: PCI: ti,j721e-pci-host: add interrupt controller definition Matt Ranostay
2022-10-25  8:19   ` Matt Ranostay
2022-10-25  8:19 ` [PATCH RESEND v2 2/2] dt-bindings: PCI: ti,j721e-pci-*: Add missing interrupt properties Matt Ranostay
2022-10-25  8:19   ` Matt Ranostay
2022-10-27  8:10 ` [PATCH RESEND v2 0/2] dt-bindings: PCI: ti,j721e-pci-*: resolve unexpected property warnings Lorenzo Pieralisi
2022-10-27  8:10   ` Lorenzo Pieralisi

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.