Re: [PATCH] Soc: wm8940: Fix check on error code form snd_soc_codec_set_cache_io

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Oct 02, 2009 at 04:09:49PM +0100, Jonathan Cameron wrote:
> 
> Soc: wm8940: Fix check on error code form snd_soc_codec_set_cache_io

Applied, thanks.

Please don't duplicate the first line of the commit message into the
message body like this, it means the commit message needs to be manually
mangled after applying.

> 
> Fix for typo in commit 8d50e447d19fec64adebeef55f2b60d695435412
> ASoC: Factor out I/O for Wolfson 8 bit data 16 bit register CODECs
> 
> Signed-off-by: Jonathan Cameron <jic23@xxxxxxxxx>
> 
> ---
> 
>  sound/soc/codecs/wm8940.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/sound/soc/codecs/wm8940.c b/sound/soc/codecs/wm8940.c
> index da97aae..1ef2454 100644
> --- a/sound/soc/codecs/wm8940.c
> +++ b/sound/soc/codecs/wm8940.c
> @@ -790,7 +790,7 @@ static int wm8940_register(struct wm8940_priv *wm8940,
>  	codec->reg_cache = &wm8940->reg_cache;
>  
>  	ret = snd_soc_codec_set_cache_io(codec, 8, 16, control);
> -	if (ret == 0) {
> +	if (ret < 0) {
>  		dev_err(codec->dev, "Failed to set cache I/O: %d\n", ret);
>  		return ret;
>  	}
> -- 
> 1.6.3.3
> 
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@xxxxxxxxxxxxxxxx
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> 

-- 
"You grabbed my hand and we fell into it, like a daydream - or a fever."
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux