Re: Creative X-Fi Titanium HD ctxfi bugs 5378 5218 4868

[Takashi Iwai <tiwai@suse.de>]

At Sun, 12 Jun 2011 18:48:27 +0200,
Takashi Iwai wrote:
> 
> 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.

Now both patches were applied to sound git tree.
Just minor fixes were applied in addition (coding-style issues in the
second patch and added the definition in include/linux/pci_ids.h in
the first patch).


thanks,

Takashi
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel