Thread (21 messages) 21 messages, 5 authors, 2018-08-02

[PATCH v8 4/4] mailbox: Add support for i.MX7D messaging unit

From: o.rempel@pengutronix.de (Oleksij Rempel)
Date: 2018-08-01 08:00:43
Also in: linux-devicetree

On Wed, Aug 01, 2018 at 07:52:43AM +0000, A.s. Dong wrote:
quoted
-----Original Message-----
From: Oleksij Rempel [mailto:o.rempel at pengutronix.de]
Sent: Tuesday, July 31, 2018 10:12 PM
To: Shawn Guo <shawnguo@kernel.org>; Fabio Estevam
[off-list ref]; Rob Herring [off-list ref]; Mark
Rutland [off-list ref]; A.s. Dong [off-list ref];
Vladimir Zapolskiy [off-list ref]; Jassi Brar
[off-list ref]
Cc: Oleksij Rempel <o.rempel@pengutronix.de>; kernel at pengutronix.de;
linux-arm-kernel at lists.infradead.org; devicetree at vger.kernel.org; dl-linux-
imx [off-list ref]
Subject: [PATCH v8 4/4] mailbox: Add support for i.MX7D messaging unit

The Mailbox controller is able to send messages (up to 4 32 bit words)
between the endpoints.
I'm not sure but is this still valid after change to multi channels mode?
We actually lost the HW capability to send messages up to 4 words
in the multi channels mode.
And such commit message probably will confuse people.
Worth a improve?

Otherwise this patch mostly is okay to me except a few bits already
pointed out by Jassi.
Ok, I schedule this task for Thursday or Friday.

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180801/b9a2ebf3/attachment.sig>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help