Thread (16 messages) 16 messages, 3 authors, 2021-09-23

Re: [PATCH v2 1/3] dt-bindings: PCI: ti,j721e: Add bindings to specify legacy interrupts

From: Rob Herring <robh@kernel.org>
Date: 2021-09-23 15:49:07
Also in: linux-devicetree, linux-omap, linux-pci, lkml

On Wed, Sep 22, 2021 at 11:33 PM Kishon Vijay Abraham I [off-list ref] wrote:
Hi Rob,

On 18/08/21 7:28 pm, Kishon Vijay Abraham I wrote:
quoted
Hi Rob,

On 13/08/21 10:47 pm, Rob Herring wrote:
quoted
On Wed, Aug 04, 2021 at 06:59:10PM +0530, Kishon Vijay Abraham I wrote:
quoted
Add bindings to specify interrupt controller for legacy interrupts.

Signed-off-by: Kishon Vijay Abraham I <redacted>
---
 .../bindings/pci/ti,j721e-pci-host.yaml           | 15 +++++++++++++++
 1 file changed, 15 insertions(+)
diff --git a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
index cc900202df29..f461d7b4c0cc 100644
--- a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
+++ b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
@@ -74,6 +74,11 @@ properties:

   msi-map: true

+patternProperties:
+  "interrupt-controller":
Not a pattern unless you meant for foo-interrupt-controller-bar to be
valid.

Anything is allowed in the node?
It's same as whatever is defined in schemas/interrupt-controller.yaml,
just that it should be a subnode of pcie@. Should I add whatever is
present in schemas/interrupt-controller.yaml here?
Can you suggest how to include this?
You don't. List the properties you are using. You need to define the
#interrupt-cells value for example. I'm sure there's already examples
in the tree doing this same child node.

Rob

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help