On Thu, Jun 12, 2014 at 10:31:19PM +0530, Jassi Brar wrote: A couple of tiny nits, I'll send followup patches for these.
> +bool mbox_client_peek_data(struct mbox_chan *chan)
> +{
> + if (chan->mbox->ops->peek_data)
> + return chan->mbox->ops->peek_data(chan);
> +
> + return false;
> +}
> +EXPORT_SYMBOL_GPL(mbox_client_peek_data);
This isn't declared in the header to allow users to use it.
> +struct mbox_chan *mbox_request_channel(const struct mbox_client *cl)
> + spin_lock_irqsave(&chan->lock, flags);
> + chan->msg_free = 0;
> + chan->msg_count = 0;
> + chan->active_req = NULL;
> + chan->cl = cl;
chan->cl is non-const but cl is const so this assignment is invalid.
signature.asc
Description: Digital signature
