[alsa-devel] [PATCH] Improved support for different bt87x board configurations

Takashi Iwai tiwai at suse.de
Sat Sep 8 01:50:29 CEST 2007


At Thu, 06 Sep 2007 21:17:12 +0100,
Mauro Carvalho Chehab wrote:
> 
> 
> Em Qua, 2007-09-05 às 15:34 -0700, Trent Piepho escreveu:
> > On Wed, 5 Sep 2007, Mauro Carvalho Chehab wrote:
> > > > Well, at first I misunderstood that we would share the PCI SSIDs
> > > > together with enum items.  Sharing only the enum items doesn't give
> > > > much improvements.  Sharing the card database may do, though.
> > > >
> > > > So, I see Trent's point.  The merge work wouldn't pay for the benifit
> > > > by sharing only the enum id, not the card database.
> > >
> > > I like the idea of sharing the database. This probably will mean that
> > > snd_bt87x would be dependent of bttv driver (or a bttv core driver).
> > > This have the advantage of helping also to fix some conflicts with dvb
> > > bttv driver.
> > 
> > I see how that could be an improvement.  Now the bt87x driver has a
> > whitelist of cards with sound and a blacklist of cards with DVB, while the
> > bt878 DVB driver has the exact opposite lists.  Except the two driver's
> > lists aren't in sync of course.
> > 
> > The cx88 driver keeps the database in the core module.  One drawback is
> > that the database can't be __initdata, as it has to stick around after the
> > core module is loaded so the other drivers that are loaded later can get at
> > it.  Of course, __initdata doesn't do any for most configurations and it's
> > not that big of a drawback in practice.
> 
> Trent,
> 
> Could you work on such patches for 2.6.25? For 2.6.24, my suggestion is
> just to apply your already submitted one.
> 
> Takashi, 
> 
> If you want, you may add my ack to the already-sent Trent's patch:
> 
> Reviewed-by: Mauro Carvalho Chehab <mchehab at infradead.org>

Thanks, but I have already committed, and don't want to rollback now
:)

(BTW, how is this tag supposed to use?  On git or Hg, it's hard to
 change the already committed stuff, especially after openly
 published...)

Takashi


More information about the Alsa-devel mailing list