* [PATCH 2/6] Documentation: bindings: document the Alpine MSIX driver [not found] <1454922971-17405-1-git-send-email-antoine.tenart@free-electrons.com> @ 2016-02-08 9:16 ` Antoine Tenart 2016-02-08 14:55 ` Marc Zyngier 0 siblings, 1 reply; 3+ messages in thread From: Antoine Tenart @ 2016-02-08 9:16 UTC (permalink / raw) To: tglx, jason, marc.zyngier, tsahee Cc: thomas.petazzoni, devicetree, Antoine Tenart, linux-kernel, rshitrit, linux-arm-kernel Following the addition of the Alpine MSIX driver, this patch adds the corresponding bindings documentation. Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com> Signed-off-by: Tsahee Zidenberg <tsahee@annapurnalabs.com> --- .../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>; +}; -- 2.7.0 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 2/6] Documentation: bindings: document the Alpine MSIX driver 2016-02-08 9:16 ` [PATCH 2/6] Documentation: bindings: document the Alpine MSIX driver Antoine Tenart @ 2016-02-08 14:55 ` Marc Zyngier 2016-02-08 15:05 ` Antoine Tenart 0 siblings, 1 reply; 3+ messages in thread From: Marc Zyngier @ 2016-02-08 14:55 UTC (permalink / raw) To: Antoine Tenart, tglx, jason, tsahee Cc: rshitrit, thomas.petazzoni, linux-arm-kernel, devicetree, linux-kernel 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@free-electrons.com> > Signed-off-by: Tsahee Zidenberg <tsahee@annapurnalabs.com> > --- > .../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... ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 2/6] Documentation: bindings: document the Alpine MSIX driver 2016-02-08 14:55 ` Marc Zyngier @ 2016-02-08 15:05 ` Antoine Tenart 0 siblings, 0 replies; 3+ messages in thread From: Antoine Tenart @ 2016-02-08 15:05 UTC (permalink / raw) To: Marc Zyngier Cc: Antoine Tenart, tglx, jason, tsahee, rshitrit, thomas.petazzoni, linux-arm-kernel, devicetree, linux-kernel [-- Attachment #1: Type: text/plain, Size: 725 bytes --] On Mon, Feb 08, 2016 at 02:55:57PM +0000, Marc Zyngier wrote: > On 08/02/16 09:16, Antoine Tenart wrote: > > + > > +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. Right. I'll add the interrupt-parent property in the required properties and in the example msix node. Antoine -- Antoine Ténart, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-02-08 15:05 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1454922971-17405-1-git-send-email-antoine.tenart@free-electrons.com>
2016-02-08 9:16 ` [PATCH 2/6] Documentation: bindings: document the Alpine MSIX driver Antoine Tenart
2016-02-08 14:55 ` Marc Zyngier
2016-02-08 15:05 ` Antoine Tenart
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).