[alsa-devel] [patch 1/2] ASoC: sst-haswell-ipc: restore IRQs on error

Liam Girdwood liam.r.girdwood at intel.com
Tue Feb 25 14:12:11 CET 2014


On Tue, 2014-02-25 at 11:32 +0300, Dan Carpenter wrote:
> This should be spin_unlock_irqrestore() instead of spin_unlock()
> 
> Fixes: 22981243589c ('ASoC: Intel: Add Haswell/Broadwell IPC')
> Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com>
> 
> diff --git a/sound/soc/intel/sst-haswell-ipc.c b/sound/soc/intel/sst-haswell-ipc.c
> index 668d486520ae..552aebf2c8f5 100644
> --- a/sound/soc/intel/sst-haswell-ipc.c
> +++ b/sound/soc/intel/sst-haswell-ipc.c
> @@ -527,7 +527,7 @@ static int ipc_tx_message(struct sst_hsw *hsw, u32 header, void *tx_data,
>  
>  	msg = msg_get_empty(hsw);
>  	if (msg == NULL) {
> -		spin_unlock(&hsw->dsp->spinlock);
> +		spin_unlock_irqrestore(&hsw->dsp->spinlock, flags);
>  		return -EBUSY;
>  	}
>  

Both 

Acked-by: Liam Girdwood <liam.r.girdwood at linux.intel.com>


---------------------------------------------------------------------
Intel Corporation (UK) Limited
Registered No. 1134945 (England)
Registered Office: Pipers Way, Swindon SN3 1RJ
VAT No: 860 2173 47

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.


More information about the Alsa-devel mailing list