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