[alsa-devel] PATCH: C-Media 6206 USB Audio
tiwai at suse.de
Mon Jun 8 07:41:04 CEST 2009
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
More information about the Alsa-devel