[alsa-devel] [PATCH] ASoC: count reaches 10001, not 10000.

Bryan Wu cooloney at kernel.org
Sun Feb 8 18:45:09 CET 2009


On Mon, Feb 9, 2009 at 1:17 AM, Roel Kluin <roel.kluin at gmail.com> wrote:
> With a postfix increment count reaches 10001, not 10000.
>

Thanks a lot, it looks fine for me. Add Cliff to this email, since he
is maintaining this driver.

-Bryan

> diff --git a/sound/soc/blackfin/bf5xx-ad73311.c b/sound/soc/blackfin/bf5xx-ad73311.c
> index 7f2a5e1..edfbdc0 100644
> --- a/sound/soc/blackfin/bf5xx-ad73311.c
> +++ b/sound/soc/blackfin/bf5xx-ad73311.c
> @@ -114,7 +114,7 @@ static int snd_ad73311_configure(void)
>        SSYNC();
>
>        /* When TUVF is set, the data is already send out */
> -       while (!(status & TUVF) && count++ < 10000) {
> +       while (!(status & TUVF) && ++count < 10000) {
>                udelay(1);
>                status = bfin_read_SPORT_STAT();
>                SSYNC();
> @@ -123,7 +123,7 @@ static int snd_ad73311_configure(void)
>        SSYNC();
>        local_irq_enable();
>
> -       if (count == 10000) {
> +       if (count >= 10000) {
>                printk(KERN_ERR "ad73311: failed to configure codec\n");
>                return -1;
>        }
>


More information about the Alsa-devel mailing list