On 08/02/16 09:16, Antoine Tenart wrote: > Following the addition of the Alpine MSIX driver, this patch adds the > corresponding bindings documentation. > > Signed-off-by: Antoine Tenart <antoine.tenart@xxxxxxxxxxxxxxxxxx> > Signed-off-by: Tsahee Zidenberg <tsahee@xxxxxxxxxxxxxxxxx> > --- > .../interrupt-controller/al,alpine-msix.txt | 22 ++++++++++++++++++++++ > 1 file changed, 22 insertions(+) > create mode 100644 Documentation/devicetree/bindings/interrupt-controller/al,alpine-msix.txt > > diff --git a/Documentation/devicetree/bindings/interrupt-controller/al,alpine-msix.txt b/Documentation/devicetree/bindings/interrupt-controller/al,alpine-msix.txt > new file mode 100644 > index 000000000000..a77d5a76e8d5 > --- /dev/null > +++ b/Documentation/devicetree/bindings/interrupt-controller/al,alpine-msix.txt > @@ -0,0 +1,22 @@ > +Alpine MSIX controller > + > +Required properties: > + > +- compatible: should be "al,alpine-msix" > +- reg: physical base address and size of the registers > +- interrupt-controller: identifies the node as an interrupt controller > +- msi-controller: identifies the node as an PCI Message Signaled Interrupt > + controller > +- al,msi-base-spi: SPI base of the MSI frame > +- al,msi-num-spis: number of SPIs assigned to the MSI frame > + > +Example: > + > +msix: msix { > + compatible = "al,alpine-msix"; > + reg = <0x0 0xfbe00000 0x0 0x100000>; > + interrupt-controller; > + msi-controller; > + al,msi-base-spi = <160>; > + al,msi-num-spis = <160>; > +}; > This example seems to rely on an implicit interrupt-parent. You probably want to update the example to clarify it. Thanks, M. -- Jazz is not dead. It just smells funny... -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html