On Friday, August 19, 2016 7:32:23 PM CEST Thierry Reding wrote: > +static int tegra_hsp_doorbell_send_data(struct tegra_hsp_channel *channel, > + void *data) > +{ > + tegra_hsp_channel_writel(channel, 1, HSP_DB_TRIGGER); > + > + return 0; > +} > It seems very odd that you don't write the data at all but just trigger the communication here. My interpretation is that you bypass the mailbox infrastructure for the actual message, and instead reimplement it on top of the mailbox infrastructure as part of the bpmp driver. Why not expose the bpmp itself as the mailbox here? Arnd -- 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