Re: PATCH: C-Media 6206 USB Audio

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

 



At Sun, 7 Jun 2009 22:46:51 -0400,
Dan Allongo wrote:
> 
> I've been posting on the alsa-users list and I think I finally have a
> working patch now that I'd like to submit.
> The C-Media 6206 device is based off of CM106 and just needed a boot quirk
> added to set the registers properly for output.
> At least one other user can confirm that this patch works and that the
> SPDIF-out is also functioning now with this change.
> This is diff'd against today's snapshot so it should be good to go.
> Let me know if there's anything else that I need to do or change with this.

Thanks for the patch!  I'd like certainly to apply it, however...

> +    return snd_usb_cm106_write_int_reg(dev, 0, 0x200c) +
> +        snd_usb_cm106_write_int_reg(dev, 1, 0x3000) +
> +        snd_usb_cm106_write_int_reg(dev, 2, 0xf800) +
> +        snd_usb_cm106_write_int_reg(dev, 3, 0x143f) +
> +        snd_usb_cm106_write_int_reg(dev, 4, 0x0000) +
> +        snd_usb_cm106_write_int_reg(dev, 5, 0x3000);

Summing the return values here look weird.
You want to return an error code, right?  Then use a simple if
sequence, or write a loop there.

Also, run $LINUX/scripts/checkpatch.pl to your patch and fix errors / 
warnings suggested there.  Then please repost the patch with your
sign-off.


thanks,

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