* RE: [RFC LINUX PATCH] Dcoumentation: dt: mailbox: Add Xilinx IPI Mailbox [not found] ` <1506034724-14639-1-git-send-email-jliang-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org> @ 2017-09-22 6:05 ` Jiaying Liang [not found] ` <BY1PR0201MB0885E6563EBAFE83DF8822A4B0670-QYJsKn8jqXIWAWX5OQc4qhrHTHEw16jenBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org> 0 siblings, 1 reply; 3+ messages in thread From: Jiaying Liang @ 2017-09-22 6:05 UTC (permalink / raw) To: Jiaying Liang, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, jassisinghbrar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, Cyril Chemparathy, Michal Simek, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, Soren Brinkmann > -----Original Message----- > From: Wendy Liang [mailto:wendy.liang-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org] > Sent: Thursday, September 21, 2017 3:59 PM > To: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > Cc: jassisinghbrar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org; Cyril Chemparathy <cyrilc-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>; > Michal Simek <michals-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>; Jiaying Liang <jliang-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org> > 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-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org> > --- > .../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@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@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. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 3+ messages in thread
[parent not found: <BY1PR0201MB0885E6563EBAFE83DF8822A4B0670-QYJsKn8jqXIWAWX5OQc4qhrHTHEw16jenBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>]
* Re: [RFC LINUX PATCH] Dcoumentation: dt: mailbox: Add Xilinx IPI Mailbox [not found] ` <BY1PR0201MB0885E6563EBAFE83DF8822A4B0670-QYJsKn8jqXIWAWX5OQc4qhrHTHEw16jenBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org> @ 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: Jiaying Liang Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, jassisinghbrar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, Cyril Chemparathy, Michal Simek, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, Soren Brinkmann, Sudeep Holla 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-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org> > --- > .../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@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@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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [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: Sudeep Holla Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, jassisinghbrar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, Cyril Chemparathy, Michal Simek, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, Soren Brinkmann HI Sudeep, > -----Original Message----- > From: Sudeep Holla [mailto:sudeep.holla-5wv7dgnIgG8@public.gmane.org] > Sent: Friday, September 22, 2017 4:10 AM > To: Jiaying Liang <jliang-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org> > Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org; > devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; jassisinghbrar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org; Cyril Chemparathy > <cyrilc-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>; Michal Simek <michals-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>; > robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org; mark.rutland-5wv7dgnIgG8@public.gmane.org; Soren Brinkmann > <sorenb-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>; Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org> > 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-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org> > > --- > > .../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@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@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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ 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>
[not found] ` <1506034724-14639-1-git-send-email-jliang-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
2017-09-22 6:05 ` [RFC LINUX PATCH] Dcoumentation: dt: mailbox: Add Xilinx IPI Mailbox Jiaying Liang
[not found] ` <BY1PR0201MB0885E6563EBAFE83DF8822A4B0670-QYJsKn8jqXIWAWX5OQc4qhrHTHEw16jenBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>
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; as well as URLs for NNTP newsgroup(s).