[alsa-devel] BT87x ALSA driver patches

[Takashi Iwai]

At Wed, 25 Jul 2007 15:43:50 -0700 (PDT),
Trent Piepho wrote:
> 
> On Mon, 23 Jul 2007, Mauro Carvalho Chehab wrote:
> > Em Sáb, 2007-07-21 às 17:36 -0700, Trent Piepho escreveu:
> > > 01/01: bttv: Fix Viewcast Osprey 440 support
> > > http://linuxtv.org/hg/~tap/v4l-dvb?cmd=changeset;node=8e72fe5675b3
> > >
> > > I have two other changes that only affect the ALSA driver part.  Should I
> > > send those to ALSA or to you?
> >
> > Please send to ALSA guys, c/c me.
> 
> Ok, I'll do that.  What about the two lines to bt87x.c that this patch added?
> I seemed logical to me that they should be part of the same patch at git as
> the changes to bttv-cards.c, but I suppose it's not 100% necessary to do so,
> the kernel will still compile and not crash.

In such a case, I think one patch makes more sense.

Mauro, please apply it from your tree all together?

> Do you plan to send this to git as one patch, or to strip out the ALSA change?
> I'm just wondering if I should send these two lines to the ALSA people as a
> separate patch, of if they will get them into their hg via a kernel-sync
> patch.  My bt87x patches depend on this one, so somehow the ALSA hg needs
> to get this change before I can send them the next two.
> 
> The problem really, is that having two divers for the same chip split into
> v4l-dvb and ALSA like this a pain.  IMHO, it would make a lot more sense
> to put the bttv alsa driver into v4l-dvb, like the saa7134 alsa and oss
> and cx88 alsa drivers are.

OTOH, fixing cx88 and saa7134 codes are also painful for us :)


thanks,

Takashi