On Wed, Jul 11, 2018 at 4:07 PM, A.s. Dong <[email protected]> wrote: > > > -----Original Message----- > > From: Sascha Hauer [mailto:[email protected]] > > Sent: Wednesday, July 11, 2018 3:55 PM > > To: A.s. Dong <[email protected]> > > Cc: [email protected]; [email protected]; Jassi Brar > > <[email protected]>; [email protected]; Oleksij Rempel > > <[email protected]>; dl-linux-imx <[email protected]>; > > [email protected]; Fabio Estevam <[email protected]>; > > [email protected] > > Subject: Re: [PATCH V4 3/5] mailbox: imx: add imx mu support > > > > On Wed, Jul 11, 2018 at 07:29:38AM +0000, A.s. Dong wrote: > > > Hi Sascha, > > > > > > > -----Original Message----- > > > > From: Sascha Hauer [mailto:[email protected]] > > > > Sent: Tuesday, July 10, 2018 10:20 PM > > > > To: A.s. Dong <[email protected]> > > > > Cc: [email protected]; [email protected]; Jassi > > > > Brar <[email protected]>; [email protected]; > > > > Oleksij Rempel <[email protected]>; dl-linux-imx > > > > <[email protected]>; [email protected]; Fabio Estevam > > > > <[email protected]>; [email protected] > > > > Subject: Re: [PATCH V4 3/5] mailbox: imx: add imx mu support > > > > > > > > Hi, > > > > > > > > On Sun, Jul 08, 2018 at 10:56:55PM +0800, Dong Aisheng wrote: > > > > > This is used for i.MX multi core communication. > > > > > e.g. A core to SCU firmware(M core) on MX8. > > > > > > > > > > Tx is using polling mode while Rx is interrupt driven and schedule > > > > > a hrtimer to receive remain words if have more than > > > > > 4 words. > > > > > > > > You told us that using interrupts is not possible due to miserable > > > > performance, we then provided you a way with which you could poll. > > > > Why are you using interrupts now? > > > > > > > > > > Because mailbox framework does not support sync rx now, I think we do > > > not need to wait for that feature done first as it's independent and > > > separate features of framework. > > > > You can wait forever for this feature, nobody will add it for you. It's up > > to you > > to add support for that feature. Who else should add this feature if not > > you? > > And when will you add that feature if not now when you actually need it? > > It is common practice that you adjust the frameworks to your needs rather > > than working around them. > > > > I'm willing to add it. Just because you said Jassi already had the idea on > how to > Implement it and does not add much complexity. So I just want to see his > patches. > But if he did not work on it, I can also help on it. > I am not much aware of the history of this conversation... but it seems you need to make use of mbox_chan_ops.peek_data().
If not that, please let me know the requirement. Cheers!
