[alsa-devel] [PATCH] Revised AT32 ASoC Patch

Geoffrey Wossum geoffrey at pager.net
Wed Jun 4 15:38:33 CEST 2008


On Wednesday 04 June 2008 08:28:07 am Mark Brown wrote:
> On Wed, Jun 04, 2008 at 02:01:55PM +0200, Takashi Iwai wrote:
> > Geoffrey Wossum wrote:
> > > +struct at32_ssc_info {
> > > +	char *name;
> > > +	struct ssc_device *ssc;
> > > +	spinlock_t lock;	/* lock for dir_mask */
> >
> > Just wondering whether this lock is really needed.
> > Is it really racy?
>
> Bidirectional ports could be in two PCMs and have configuration
> happening simultaneously.

To be honest, the AT91 code I was porting to the AT32 had this lock.  I think 
Mark is correct, so it's probably good I wasn't writing this code from 
scratch since I might have missed this detail on the first pass.

---
Geoffrey



More information about the Alsa-devel mailing list