Hello Arnd, On 9 April 2013 16:25, Arnd Bergmann <arnd@xxxxxxxx> wrote: > On Thursday 04 April 2013, Anna, Suman wrote: >> OMAP and ST-Ericsson platforms are both using mailbox to communicate with some coprocessors. This series creates a consolidated framework, living under drivers/mailbox. >> The changes mainly contain: >> - create a mailbox framework independent from OMAP h/w >> - creates dbx500 mailbox driver for ST-Ericsson platforms >> - move the omap mailbox out of plat-omap/mach-omapX adapting to the new framework. >> - minor bug fixes in mailbox code > > Pulled into a new next/mailbox branch, to keep it separate from the > existing subsystems. > I am going to be a heavy user of the Mailbox API. And I have reviewed this API quite in detail. I pointed out many aspects that might have worked for TI's usage but are not going to be work on many platforms (including one of mine). Suman and Loic also acknowledged most (if not all) from 'generic' POV. Here is the thread ... http://www.spinics.net/lists/arm-kernel/msg239433.html Not to mean only talk and no do, I prepared another proposal that addressed all the concerns shared so far in the RFC http://www.spinics.net/lists/kernel/msg1523873.html (its not ready for primetime yet) And also converted the PL320 driver to the new API, unlike the pulled patchset which leaves that out in the cold. http://www.spinics.net/lists/kernel/msg1523874.html Now, we could either call it (effectively the TI's framework with quirks for STE) as the "Common API" and then dismantle and convert it patch by patch (authors and I seem to agree many things need to be changed and implemented). OR we do it reasonably right the first time even if that means yet another release. IMHO we should go for slow but steady thing. Hi Suman, Hi Loic, Please feel free to object to anything you think I might have misrepresented. I am OK if you are not sure about my implementation either - we could co-work on a new one. Regards, -Jassi -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
