[alsa-devel] Creative X-Fi Titanium HD ctxfi bugs 5378 5218 4868
tiwai at suse.de
Sun Jun 12 18:48:27 CEST 2011
At Sat, 11 Jun 2011 18:25:36 +0800,
Harry Butterworth wrote:
> I have redone the patch against today's git repositories.
> Creative have not yet got back to me about the PLL initialization.
> Perhaps we can fix that later.
> I have only put the PCI ID in the pci_ids_compat.h.in file. Is it
> going to be necessary to add the PCI device ID somewhere in the kernel
> tree to avoid breaking the kernel build?
Yes, include/linux/pci_ids.h is the place to change.
But I can fix it by myself, too.
> > A few comment about the patch:
> > - Instead of adding have_dedicated_mic and have_output_switch
> > callbacks in every place, I'd put some bit flags indicating the
> > capabilities, and set at the initialization appropriately.
> I had a go at this in the patch 0002-ALSA...
> > - A few easy coding-style issues; run scripts/checkpatch.pl and fix
> > the errors appropriately.
> checkpatch now reports no errors.
> I've only tested these changes with my Titanium HD card. I don't have
> any of the other X-Fi cards so I couldn't do any regression testing.
> If there's anything else I need to do, please let me know.
I'm going to check patches in a more detail tomorrow.
More information about the Alsa-devel