Re: [PATCH 002b/003] snd-powermac: style awacs.s and awacs.h

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

 



2008/4/16, Takashi Iwai <tiwai@xxxxxxx>:
>
> Thanks, that works.  But, I get checkpatch errors:
>
>  ERROR: do not use assignment in if condition
>  #116: FILE: ppc/awacs.c:704:
>
> +               if ((err = snd_ctl_add(chip->card,
>
>
> ERROR: do not use assignment in if condition
>  #171: FILE: ppc/awacs.c:982:
>
> +               if ((err = build_mixers(chip,
>
>
> ERROR: do not use assignment in if condition
>  #191: FILE: ppc/awacs.c:999:
>
> +               if ((err = build_mixers(chip,
>
>
>
> Please fix?
>

I can do that but it will make the code much clumsier. Actually, I
think this is checkpatch misunderstanding something. There is a '< 0'
at the end of those conditions. Is it really this kind of assingment
they want to forbid? This is the style there has been all the time.

Risto
_______________________________________________
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