[alsa-devel] [PATCH] ASoC: Intel: remove unneeded variable

Mark Brown broonie at kernel.org
Sun Apr 12 13:40:21 CEST 2015


On Sun, Apr 12, 2015 at 07:55:30AM +0200, Julia Lawall wrote:
> Removes unneeded variable used to store return value.

> +++ b/sound/soc/intel/baytrail/sst-baytrail-ipc.c
> @@ -511,7 +511,6 @@ static int sst_byt_process_notification(
>  	struct sst_byt_stream *stream;
>  	u64 header;
>  	u8 msg_id, stream_id;
> -	int handled = 1;
>  
>  	header = sst_dsp_shim_read64_unlocked(sst, SST_IPCD);
>  	msg_id = sst_byt_header_msg_id(header);
> @@ -531,7 +530,7 @@ static int sst_byt_process_notification(
>  		break;
>  	}
>  
> -	return handled;
> +	return 1;

This seems like the script has identified an error - it looks like we
should be managing handled more actively in the function.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20150412/fb4c794f/attachment.sig>


More information about the Alsa-devel mailing list