[alsa-devel] Creative X-Fi Titanium HD ctxfi bugs 5378 5218 4868

[Takashi Iwai]

At Sat, 11 Jun 2011 18:25:36 +0800,
Harry Butterworth wrote:
> 
> Hi,
> 
>     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.

Thanks!


Takashi