Re: ASoC: Merging at91 and avr32 dirs - test report

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

 



On Fri, Oct 10, 2008 at 06:15:17PM +0300, ext-jukka.hynninen@xxxxxxxxxxx wrote:

> However, there seems to be some problem with PM, occasionally ASoC is
> stuck with Standby mode (looking at dapm_widgets) when I start the
> playback. Restarting the player brings the sound back. I'll carry out
> some further investigation and get back to this.

How does this compare with the behaviour of the original at32 machine
support - did that have similar issues?

> The SSC peripheral clock was disabled more often than it was enabled,
> and that catched the bug trap in arch/arm/mach-at91/clock.c. I'm not
> sure if this was the real problem, but it's the way I managed to solve
> it.

Looks reasonable to me.

> diff -uprN a/sound/soc/atmel/atmel_ssc_dai.c
> b/sound/soc/atmel/atmel_ssc_dai.c
> --- a/sound/soc/atmel/atmel_ssc_dai.c   2008-10-10 17:23:46.000000000
> +0300
> +++ b/sound/soc/atmel/atmel_ssc_dai.c   2008-10-10 17:26:06.000000000
> +0300
> @@ -261,11 +261,11 @@ static void atmel_ssc_shutdown(struct sn
>         spin_lock_irq(&ssc_p->lock);
>         ssc_p->dir_mask &= ~dir_mask;
>         if (!ssc_p->dir_mask) {
> -               /* Shutdown the SSC clock. */
> -               pr_debug("atmel_ssc_dau: Stopping clock\n");
> -               clk_disable(ssc_p->ssc->clk);
> -
>                 if (ssc_p->initialized) {
> +                       /* Shutdown the SSC clock. */
> +                       pr_debug("atmel_ssc_dai: Stopping clock\n");
> +                       clk_disable(ssc_p->ssc->clk);
> +
>                         free_irq(ssc_p->ssc->irq, ssc_p);
>                         ssc_p->initialized = 0;
>                 }
_______________________________________________
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