* [RFC LINUX PATCH] Dcoumentation: dt: mailbox: Add Xilinx IPI Mailbox [not found] <1506034724-14639-1-git-send-email-jliang@xilinx.com> @ 2017-09-22 6:05 ` Jiaying Liang 2017-09-22 11:10 ` Sudeep Holla 0 siblings, 1 reply; 3+ messages in thread From: Jiaying Liang @ 2017-09-22 6:05 UTC (permalink / raw) To: linux-arm-kernel > -----Original Message----- > From: Wendy Liang [mailto:wendy.liang at xilinx.com] > Sent: Thursday, September 21, 2017 3:59 PM > To: linux-kernel at vger.kernel.org > Cc: jassisinghbrar at gmail.com; Cyril Chemparathy <cyrilc@xilinx.com>; > Michal Simek <michals@xilinx.com>; Jiaying Liang <jliang@xilinx.com> > Subject: [RFC LINUX PATCH] Dcoumentation: dt: mailbox: Add Xilinx IPI > Mailbox > > Xilinx ZynqMP IPI(Inter Processor Interrupt) is a hardware block in ZynqMP > SoC used for the communication between various processor systems. > > Signed-off-by: Wendy Liang <jliang@xilinx.com> > --- > .../bindings/mailbox/xlnx,zynqmp-ipi-mailbox.txt | 88 > ++++++++++++++++++++++ > 1 file changed, 88 insertions(+) > create mode 100644 > Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.txt > > diff --git a/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi- > mailbox.txt b/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi- > mailbox.txt > new file mode 100644 > index 0000000..5d915d1 > --- /dev/null > +++ b/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox. > +++ txt > @@ -0,0 +1,88 @@ > +Xilinx IPI Mailbox Driver > +======================================== > + > +The Xilinx IPI(Inter Processor Interrupt) mailbox driver is a mailbox > +controller that manages the messaging between two IPI agents. Each IPI > +mailbox has request and response buffers between the two IPI agents. > + > ++-------------------------------------+ > +| | Xilinx ZynqMP IPI Mailbox > +| Controller| > +| | > +| +-------------+ > +| | SMC | > +| | | > ++--------+--------------+------+------+ > + | | > + | +-----------------+ > + | | ATF (ARM trusted firmware) > + | | > ++-------------------------------------+ > + | | Hardware > + | | > + +--------------------------------------+ > + | | > + +----------------------+ +-----------+ | > + | | Buffers between | | IPI Agent | | > + | | two IPI agents | | Registers | | > + | +--------------------+ +-----------+ | > + | | > + | Xilinx ZynqMP IPI | > + +--------------------------------------+ > + > + > +Message Manager Device Node: > +=========================== > +Required properties: > +-------------------- > +- compatible: Shall be: "xlnx,zynqmp-ipi-mailbox" > +- ipi-smc-fid-base Base offset of SMC function IDs for IPI mailbox SMC. > + It contains the IPI IDs of the two IPI agents. > +- reg: IPI request and response buffers address range. It > + can be the IPI buffers from the hardware or it can > + be carved out shared memory. > +- reg-names: Reg resource name of the IPI request and response > + buffers. > +- #mbox-cells: Shall be 1. Contains the logical channel IDs of the > + channels on the IPI mailbox. > +- interrupt-parent: Phandle for the interrupt controller. > +- interrupts: Interrupt mapping. > + > +Required properties: > +-------------------- > +- method: The method of accessing the IPI agent registers. > + Permitted values are: "smc" and "hvc". Default is > + "smc". > +Example: > +------------ > + /* APU IPI mailbox driver */ > + ipis { > + #address-cells = <1>; > + #size-cells = <0>; > + ipi_mailbox_apu_rpu0: ipi_mailbox at 0 { > + compatible = "xlnx,zynqmp-ipi-mailbox"; > + reg = <0 0xff990400 40>; > + reg-names = "apu-rpu0"; > + ipi-smc-fid-base = <0x1010>; > + method = "smc"; > + #mbox-cells = <1>; > + interrupt-parent = <&gic>; > + interrupts = <0 35 4>; > + }; > + ipi_mailbox_apu_rpu1: ipi_mailbox at 1 { > + compatible = "xlnx,zynqmp-ipi-mailbox"; > + reg = <0 0xff990440 40>; > + reg-names = "apu-rpu1"; > + ipi-smc-fid-base = <0x1020>; > + method = "smc"; > + #mbox-cells = <1>; > + interrupt-parent = <&gic>; > + interrupts = <0 35 4>; > + }; > + }; > + device0: device0 { > + ... > + mbox-names = "rpu0", "rpu1", > + mboxes = <&ipi_mailbox_apu_rpu0 0>, > + < &ipi_mailbox_apu_rpu1 0>; > + }; > -- > 2.7.4 cc: device tree and linux arm kernel mailing lists. ^ permalink raw reply [flat|nested] 3+ messages in thread
* [RFC LINUX PATCH] Dcoumentation: dt: mailbox: Add Xilinx IPI Mailbox 2017-09-22 6:05 ` [RFC LINUX PATCH] Dcoumentation: dt: mailbox: Add Xilinx IPI Mailbox Jiaying Liang @ 2017-09-22 11:10 ` Sudeep Holla 2017-09-22 17:37 ` Jiaying Liang 0 siblings, 1 reply; 3+ messages in thread From: Sudeep Holla @ 2017-09-22 11:10 UTC (permalink / raw) To: linux-arm-kernel On Fri, Sep 22, 2017 at 06:05:18AM +0000, Jiaying Liang wrote: > > Xilinx ZynqMP IPI(Inter Processor Interrupt) is a hardware block in ZynqMP > SoC used for the communication between various processor systems. > > Signed-off-by: Wendy Liang <jliang@xilinx.com> > --- > .../bindings/mailbox/xlnx,zynqmp-ipi-mailbox.txt | 88 ++++++++++++++++++++++ > 1 file changed, 88 insertions(+) > create mode 100644 > Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.txt > > diff --git a/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi- > mailbox.txt b/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi- > mailbox.txt > new file mode 100644 > index 0000000..5d915d1 > --- /dev/null > +++ b/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox. > +++ txt > @@ -0,0 +1,88 @@ > +Xilinx IPI Mailbox Driver > +======================================== > + > +The Xilinx IPI(Inter Processor Interrupt) mailbox driver is a mailbox > +controller that manages the messaging between two IPI agents. Each IPI > +mailbox has request and response buffers between the two IPI agents. > + > ++-------------------------------------+ > +| | Xilinx ZynqMP IPI Mailbox > +| Controller| > +| | > +| +-------------+ > +| | SMC | > +| | | > ++--------+--------------+------+------+ > + | | > + | +-----------------+ > + | | ATF (ARM trusted firmware) I suppose it should work with any EL3 firmware, ATF reference can be removed IMO. > + | | > ++-------------------------------------+ > + | | Hardware > + | | > + +--------------------------------------+ > + | | > + +----------------------+ +-----------+ | > + | | Buffers between | | IPI Agent | | > + | | two IPI agents | | Registers | | > + | +--------------------+ +-----------+ | > + | | > + | Xilinx ZynqMP IPI | > + +--------------------------------------+ > + > + > +Message Manager Device Node: > +=========================== > +Required properties: > +-------------------- > +- compatible: Shall be: "xlnx,zynqmp-ipi-mailbox" > +- ipi-smc-fid-base Base offset of SMC function IDs for IPI mailbox SMC. > + It contains the IPI IDs of the two IPI agents. Why is "SMC" associated with this hardware block ? Is it secure device ? Can Linux access it ? If so, why do you need SMC ? > +- reg: IPI request and response buffers address range. It > + can be the IPI buffers from the hardware or it can > + be carved out shared memory. It sounds like buffer used for communication and not part of this IP. Shouldn't this be part of mailbox client binding rather than controller binding. > +- reg-names: Reg resource name of the IPI request and response > + buffers. > +- #mbox-cells: Shall be 1. Contains the logical channel IDs of the > + channels on the IPI mailbox. > +- interrupt-parent: Phandle for the interrupt controller. > +- interrupts: Interrupt mapping. > + > +Required properties: > +-------------------- > +- method: The method of accessing the IPI agent registers. > + Permitted values are: "smc" and "hvc". Default is > + "smc". > +Example: > +------------ > + /* APU IPI mailbox driver */ > + ipis { > + #address-cells = <1>; > + #size-cells = <0>; > + ipi_mailbox_apu_rpu0: ipi_mailbox at 0 { > + compatible = "xlnx,zynqmp-ipi-mailbox"; > + reg = <0 0xff990400 40>; > + reg-names = "apu-rpu0"; > + ipi-smc-fid-base = <0x1010>; > + method = "smc"; > + #mbox-cells = <1>; > + interrupt-parent = <&gic>; > + interrupts = <0 35 4>; > + }; > + ipi_mailbox_apu_rpu1: ipi_mailbox at 1 { > + compatible = "xlnx,zynqmp-ipi-mailbox"; > + reg = <0 0xff990440 40>; > + reg-names = "apu-rpu1"; > + ipi-smc-fid-base = <0x1020>; > + method = "smc"; > + #mbox-cells = <1>; > + interrupt-parent = <&gic>; > + interrupts = <0 35 4>; > + }; Why do you need the above 2 ? They don't look like 2 controller blocks. You just need to represent single mailbox controller. -- Regards, Sudeep ^ permalink raw reply [flat|nested] 3+ messages in thread
* [RFC LINUX PATCH] Dcoumentation: dt: mailbox: Add Xilinx IPI Mailbox 2017-09-22 11:10 ` Sudeep Holla @ 2017-09-22 17:37 ` Jiaying Liang 0 siblings, 0 replies; 3+ messages in thread From: Jiaying Liang @ 2017-09-22 17:37 UTC (permalink / raw) To: linux-arm-kernel HI Sudeep, > -----Original Message----- > From: Sudeep Holla [mailto:sudeep.holla at arm.com] > Sent: Friday, September 22, 2017 4:10 AM > To: Jiaying Liang <jliang@xilinx.com> > Cc: linux-kernel at vger.kernel.org; linux-arm-kernel at lists.infradead.org; > devicetree at vger.kernel.org; jassisinghbrar at gmail.com; Cyril Chemparathy > <cyrilc@xilinx.com>; Michal Simek <michals@xilinx.com>; > robh+dt at kernel.org; mark.rutland at arm.com; Soren Brinkmann > <sorenb@xilinx.com>; Sudeep Holla <sudeep.holla@arm.com> > Subject: Re: [RFC LINUX PATCH] Dcoumentation: dt: mailbox: Add Xilinx IPI > Mailbox > > On Fri, Sep 22, 2017 at 06:05:18AM +0000, Jiaying Liang wrote: > > > > Xilinx ZynqMP IPI(Inter Processor Interrupt) is a hardware block in > > ZynqMP SoC used for the communication between various processor > systems. > > > > Signed-off-by: Wendy Liang <jliang@xilinx.com> > > --- > > .../bindings/mailbox/xlnx,zynqmp-ipi-mailbox.txt | 88 > ++++++++++++++++++++++ > > 1 file changed, 88 insertions(+) > > create mode 100644 > > Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.txt > > > > diff --git > > a/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi- > > mailbox.txt > > b/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi- > > mailbox.txt > > new file mode 100644 > > index 0000000..5d915d1 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi- > mailbox. > > +++ txt > > @@ -0,0 +1,88 @@ > > +Xilinx IPI Mailbox Driver > > +======================================== > > + > > +The Xilinx IPI(Inter Processor Interrupt) mailbox driver is a mailbox > > +controller that manages the messaging between two IPI agents. Each > > +IPI mailbox has request and response buffers between the two IPI agents. > > + > > ++-------------------------------------+ > > +| | Xilinx ZynqMP IPI Mailbox > > +| Controller| > > +| | > > +| +-------------+ > > +| | SMC | > > +| | | > > ++--------+--------------+------+------+ > > + | | > > + | +-----------------+ > > + | | ATF (ARM trusted firmware) > > I suppose it should work with any EL3 firmware, ATF reference can be > removed IMO. > > > + | | > > ++-------------------------------------+ > > + | | Hardware > > + | | > > + +--------------------------------------+ > > + | | > > + +----------------------+ +-----------+ | > > + | | Buffers between | | IPI Agent | | > > + | | two IPI agents | | Registers | | > > + | +--------------------+ +-----------+ | > > + | | > > + | Xilinx ZynqMP IPI | > > + +--------------------------------------+ > > + > > + > > +Message Manager Device Node: > > +=========================== > > +Required properties: > > +-------------------- > > +- compatible: Shall be: "xlnx,zynqmp-ipi-mailbox" > > +- ipi-smc-fid-base Base offset of SMC function IDs for IPI mailbox SMC. > > + It contains the IPI IDs of the two IPI agents. > > Why is "SMC" associated with this hardware block ? Is it secure device ? > Can Linux access it ? If so, why do you need SMC ? [Wendy] one IPI agent uses its own IPI agent registers to notify (write to register to raise interrupt) Other IPI agents. The IPI agent registers are shared between secure and non-secure. And thus, I think about to access the IPI agent registers in ATF. And thus Use SMC for registers access. > > > +- reg: IPI request and response buffers address > range. It > > + can be the IPI buffers from the hardware or it can > > + be carved out shared memory. > > It sounds like buffer used for communication and not part of this IP. > Shouldn't this be part of mailbox client binding rather than controller > binding. [Wendy] the IPI hardware has IPI buffers (32bytes request buffer and 32bytes response buffer per) But we also want user to be able to use other shared memory. The reason I am considering to make it part of the mailbox is, we can have logical Channels on top of the physical channels. Each mailbox controller controls the physical connection(IPI agent registers and buffers), each Mailbox client requests for a logical channel. > > > +- reg-names: Reg resource name of the IPI request and response > > + buffers. > > +- #mbox-cells: Shall be 1. Contains the logical channel IDs of > the > > + channels on the IPI mailbox. > > +- interrupt-parent: Phandle for the interrupt controller. > > +- interrupts: Interrupt mapping. > > + > > +Required properties: > > +-------------------- > > +- method: The method of accessing the IPI agent registers. > > + Permitted values are: "smc" and "hvc". Default is > > + "smc". > > +Example: > > +------------ > > + /* APU IPI mailbox driver */ > > + ipis { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + ipi_mailbox_apu_rpu0: ipi_mailbox at 0 { > > + compatible = "xlnx,zynqmp-ipi-mailbox"; > > + reg = <0 0xff990400 40>; > > + reg-names = "apu-rpu0"; > > + ipi-smc-fid-base = <0x1010>; > > + method = "smc"; > > + #mbox-cells = <1>; > > + interrupt-parent = <&gic>; > > + interrupts = <0 35 4>; > > + }; > > + ipi_mailbox_apu_rpu1: ipi_mailbox at 1 { > > + compatible = "xlnx,zynqmp-ipi-mailbox"; > > + reg = <0 0xff990440 40>; > > + reg-names = "apu-rpu1"; > > + ipi-smc-fid-base = <0x1020>; > > + method = "smc"; > > + #mbox-cells = <1>; > > + interrupt-parent = <&gic>; > > + interrupts = <0 35 4>; > > + }; > > Why do you need the above 2 ? They don't look like 2 controller blocks. > You just need to represent single mailbox controller. [Wendy] I just wanted to have single mailbox controller to control one Physical connection and allows logical channels on top of it. Is there any framework to support logical channels on top of mailbox? Or we need to write the mailbox client driver to support logical channels on top of one mailbox client? Thanks, Wendy > > -- > Regards, > Sudeep ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-09-22 17:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1506034724-14639-1-git-send-email-jliang@xilinx.com>
2017-09-22 6:05 ` [RFC LINUX PATCH] Dcoumentation: dt: mailbox: Add Xilinx IPI Mailbox Jiaying Liang
2017-09-22 11:10 ` Sudeep Holla
2017-09-22 17:37 ` Jiaying Liang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox