Re: [PATCH v4 0/5] Mailbox: Provide support STi based platforms
On Fri, 16 Oct 2015, Jassi Brar wrote: > On 5 October 2015 at 18:32, Lee Jones wrote: > > Hi Jassi, > > > > This should be it. Exciting times! > > > > ST's platforms currently support a maximum of 5 Mailboxes, one for > > each of the supported co-processors situated on the platform. Each > > Mailbox is divided up into 4 instances which consist of 32 channels. > > Messages are passed between the application and co-processors using > > shared memory areas. > > > > Also included in the set is an example Client which should be generic > > enough to use as a testing environment for all Mailbox IPs which use > > shared memory or the API to pass messages. With small changes, it > > should also be able to test more extravagant implementations. > > > > v3 -> v4: > > - More protection in MBOX_BASE() MACRO > > - Remove unused tx-only marker > > > Are you sure about the tx-only marker? :) > > Since it is also a DT property, I think we should have it removed > before we apply the patchset. Lets get it done today. Wow! I can't even explain that. It's gone from my local repo! -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v4 0/5] Mailbox: Provide support STi based platforms
On 5 October 2015 at 18:32, Lee Jones wrote: > Hi Jassi, > > This should be it. Exciting times! > > ST's platforms currently support a maximum of 5 Mailboxes, one for > each of the supported co-processors situated on the platform. Each > Mailbox is divided up into 4 instances which consist of 32 channels. > Messages are passed between the application and co-processors using > shared memory areas. > > Also included in the set is an example Client which should be generic > enough to use as a testing environment for all Mailbox IPs which use > shared memory or the API to pass messages. With small changes, it > should also be able to test more extravagant implementations. > > v3 -> v4: > - More protection in MBOX_BASE() MACRO > - Remove unused tx-only marker > Are you sure about the tx-only marker? :) Since it is also a DT property, I think we should have it removed before we apply the patchset. Lets get it done today. Thanks -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH v4 0/5] Mailbox: Provide support STi based platforms
Hi Jassi, This should be it. Exciting times! ST's platforms currently support a maximum of 5 Mailboxes, one for each of the supported co-processors situated on the platform. Each Mailbox is divided up into 4 instances which consist of 32 channels. Messages are passed between the application and co-processors using shared memory areas. Also included in the set is an example Client which should be generic enough to use as a testing environment for all Mailbox IPs which use shared memory or the API to pass messages. With small changes, it should also be able to test more extravagant implementations. v3 -> v4: - More protection in MBOX_BASE() MACRO - Remove unused tx-only marker v2 => v3: - Controller - Shared (DT <=> Device Driver) defines removed - Return actual error codes instead of NULL in xlate() - Not passing 'direction' configuration via DT anymore - Removed all accompanying configuration checking code - Test F/W - Allow Rx-only and Tx-only controllers to be tested - Cater for controllers requiring pre-Tx 'signal' to be sent - Supply 'wr-' message debugfs file - When written to, will Tx message through Mailbox Framework - When read from, will print out a hexdump of Rx'ed message v1 => v2: - New MACRO() to obtain base address for a given instance - Move locking into the structure it protects - Stop checking for 'ready' state when sending data - Don't clear channel data (that belongs to the API) - #define register offsets instead of providing via pdata - Register driver with module_platform_driver() Lee Jones (5): mailbox: dt: Supply bindings for ST's Mailbox IP mailbox: Add support for ST's Mailbox IP ARM: STi: stih407-family: Add nodes for Mailbox mailbox: Add generic mechanism for testing Mailbox Controllers ARM: STi: DT: STiH407: Enable Mailbox testing facility .../devicetree/bindings/mailbox/sti-mailbox.txt| 52 +++ arch/arm/boot/dts/stih407-family.dtsi | 42 ++ drivers/mailbox/Kconfig| 14 + drivers/mailbox/Makefile | 4 + drivers/mailbox/mailbox-sti.c | 516 + drivers/mailbox/mailbox-test.c | 361 ++ 6 files changed, 989 insertions(+) create mode 100644 Documentation/devicetree/bindings/mailbox/sti-mailbox.txt create mode 100644 drivers/mailbox/mailbox-sti.c create mode 100644 drivers/mailbox/mailbox-test.c -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
