On 06/10/17 12:34, Jassi Brar wrote: > On Wed, Oct 4, 2017 at 5:02 PM, Sudeep Holla <sudeep.holla@xxxxxxx> wrote: > >> Also, I have added shim only for specific controllers that need them. >> E.g. ARM MHU as Jassi disagreed to add doorbell mechanism to that. >> mbox_if provides default implementation that just calls direct mailbox >> APIs. >> > Yeah you could hack away the MHU driver to make your life easy at the > cost of duplicated code and extra DT bindings, but for a moment think > what if your development platform wasn't MHU but, say, Rockchip > mailbox controller? > As mentioned before I understand your concern. But the point is this needs to be replicated with each protocol on that controller. So as Arnd pointed out we can reduce that by generalizing common things like doorbell. -- Regards, Sudeep -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html