Re: [PATCH v3 6/9] ASoC: qcom: Add QAIF PCM operations

Harendra Gautam <[email protected]>
Newsgroups gmane.linux.sound,gmane.linux.drivers.devicetree,gmane.linux.ports.arm.msm,gmane.linux.kernel
Message-ID <CAC-tS8Cxy9b2CwBTno5fAcjeXBd4jYj7KV2NqCjOccu9i5HxmA@mail.gmail.com>
On Mon, Aug 24, 2026 at 2:55 PM Markus Elfring <[email protected]> wrote:
>
> …
> > +++ b/sound/soc/qcom/qaif-platform.c
> > @@ -0,0 +1,544 @@
> …
> > +static int qaif_platform_pcmops_close(struct snd_soc_component *component,
> > +                                   struct snd_pcm_substream *substream)
> > +{
> …
> > +     mutex_lock(&drvdata->stream_lock);
> > +     if (v->free_stream_dma_idx)
> > +             v->free_stream_dma_idx(drvdata, data->stream_dma_idx, dai_id);
> > +     mutex_unlock(&drvdata->stream_lock);
> …
>
> Under which circumstances would you become interested to apply a statement
> like “guard(mutex)(&drvdata->stream_lock);”?
> https://elixir.bootlin.com/linux/v7.2/source/include/linux/mutex.h#L253
>
> Regards,
> Markus
Thanks for the pointer. I believe guard() can be used across all
stream_lock. I Will address it in the next version.
-Harendra
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.