[alsa-devel] [Patch v6 2/7] slimbus: Add messaging APIs to slimbus framework

Arnd Bergmann arnd at arndb.de
Wed Oct 11 09:53:09 CEST 2017


On Wed, Oct 11, 2017 at 6:38 AM, Vinod Koul <vinod.koul at intel.com> wrote:
> On Fri, Oct 06, 2017 at 05:51:31PM +0200, srinivas.kandagatla at linaro.org 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


More information about the Alsa-devel mailing list