[Alsa-devel] [PATCH] snd-aoa: fix onyx resume

[Takashi Iwai]

At Thu, 14 Dec 2006 15:27:48 +0100,
Johannes Berg wrote:
> 
> When the machine resumes the onyx codec might be in a weird state. Hence,
> simply fully reset it once (and keep the code to take it out of suspend in
> case the suspend of the codec chip survives a reset).
> 
> Signed-off-by: Johannes Berg <johannes at sipsolutions.net>

Applied to ALSA tree now.  Thanks.


Takashi

> 
> ---
> There's another problem with i2sbus suspend, namely that the i2sbus code
> doesn't really handle a suspend well at all, it needs to re-init the
> hardware during resume. I noticed because firmware on the quad powermac
> doesn't set the i2sbus to 44.1kHz but 42kHz or something like that so
> when I was playing a song across suspend/resume it ended up slower...
> 
> I don't want to fix this until that i2sbus control rewrite/fix though
> because it'll most likely collide.
> 
> Takashi, please apply this fix. 
> 
> --- linux-2.6-git.orig/sound/aoa/codecs/snd-aoa-codec-onyx.c	2006-12-13 19:32:38.520763489 +0100
> +++ linux-2.6-git/sound/aoa/codecs/snd-aoa-codec-onyx.c	2006-12-13 19:34:58.597763489 +0100
> @@ -825,7 +825,16 @@ static int onyx_resume(struct codec_info
>  	int err = -ENXIO;
>  
>  	mutex_lock(&onyx->mutex);
> -	/* take codec out of suspend */
> +
> +	/* reset codec */
> +	onyx->codec.gpio->methods->set_hw_reset(onyx->codec.gpio, 0);
> +	msleep(1);
> +	onyx->codec.gpio->methods->set_hw_reset(onyx->codec.gpio, 1);
> +	msleep(1);
> +	onyx->codec.gpio->methods->set_hw_reset(onyx->codec.gpio, 0);
> +	msleep(1);
> +
> +	/* take codec out of suspend (if it still is after reset) */
>  	if (onyx_read_register(onyx, ONYX_REG_CONTROL, &v))
>  		goto out_unlock;
>  	onyx_write_register(onyx, ONYX_REG_CONTROL, v & ~(ONYX_ADPSV | ONYX_DAPSV));
> 
> 
> 
> -------------------------------------------------------------------------
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to share your
> opinions on IT & business topics through brief surveys - and earn cash
> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/alsa-devel
>