On Wed, Oct 11, 2017 at 6:38 AM, Vinod Koul <vinod.koul@xxxxxxxxx> wrote: > On Fri, Oct 06, 2017 at 05:51:31PM +0200, srinivas.kandagatla@xxxxxxxxxx wrote: >> + ctrl->pending_wr = kcalloc((ctrl->tx.n - 1), >> + sizeof(struct slim_pending), >> + GFP_KERNEL); >> + if (!ctrl->pending_wr) { >> + ret = -ENOMEM; >> + goto wr_alloc_failed; >> + } >> + >> + sema_init(&ctrl->tx_sem, (ctrl->tx.n - 1)); > > i though v5 comment from Arnd was not to use semaphores.. Right, these need to go away. Thanks for spotting it! Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html