Re: [linux-dvb-maintainer] Re: [PATCH] b2c2: make front-ends selectable and include noob option

[Johannes Stezenbach]

On Tue, Dec 06, 2005, Prakash Punnoor wrote:
> Am Dienstag Dezember 6 2005 21:20 schrieb Michael Krufky:
> > On 12/6/05, Prakash Punnoor <prakash@xxxxxxxxxx> wrote:
> > > this patch probably needs some touch-up but mainly I am sking the dvb
> > > guys why
> > > don't they do something like this. Current situation:
> >
> > NACK.
> >
> > You are going to run into some problems with this patch... For instance,
> > What if the user chooses to compile the b2c2-flexcop driver in-kernel, but
> > only compiles the frontend drivers as modules...  Then, the frontend
> > support will be built into the flexcop driver, but the module will not yet
> > be available at the time when the kernel is looking for it...
> 
> Well, I said it needed touch up. ;-) After all I didn't seriously believe it 
> gets merged in current state (and yes, I didn't think about the module issue, 
> but you're right , of course). But it simply didn't seem like dvb guys are 
> caring about the problem. I once (probably half a year ago already) mailed to 
> linux-dvb and got zero response. That told me everything.

I make it a point to ignore postings which ignore
the recent mailing list history ;-)

This had been discussed on linux-dvb and the consensus was that
no one wants to invest time to maintain an #ifdef mess
just so that people can save a few KB in their kernel.

Also, most users don't know and don't care what demodulator
their card has, the dependency on all of them, plus the
implied auto probing saves them some headaches and us a lot of
newbie questions.

> Personally I won't invest more time in perfecting the patch. I just wanted to 
> get some attention to this problem and will use the patch privately for my 
> own happiness...

The b2c2-flexcop-pci driver could certainly use some fixing. Your
patch just hides the driver problems by deselecting functionality
that _you_ don't need.


Johannes
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/