Re: BUG in 2.6.22-rc2-mm1: NIC module b44.c broken (Broadcom 4400)

Uwe Bugla
2007-05-26T19:04:18Z

Am Samstag, 26. Mai 2007 20:41 schrieben Sie:
> On 5/26/07, Michael Buesch <mb@bu3sch.de> wrote:
> > On Saturday 26 May 2007 19:04:04 Uwe Bugla wrote:
> > > Yes, sure! But the help text is very unlucky and humble, and it is not
> > > clear enough in the sense of being distinctive enough, just clear and
> > > comprehensive.
> >
> > Why don't you simply submit a patch to change the helptext then?
>
> Is that ok ?
>
> Discourage people from deselecting B44_PCI
>
> Signed-off-by: Benoit Boissinot <benoit.boissinot@ens-lyon.org>
>
> --- kernel.orig/drivers/net/Kconfig 2007-05-26 20:38:02.000000000 +0200
> +++ kernel/drivers/net/Kconfig 2007-05-26 20:38:15.000000000 +0200
> @@ -1449,7 +1449,7 @@
> help
> Support for b44 PCI devices.
>
> - Say Y
> + Unless you know what you are doing, say Y here.
>
> config FORCEDETH
> tristate "nForce Ethernet support"

No, That is NOT OK! That's nothing but a joke, and if it is a joke : )
it hits well in Mister Buesches direction. In so far it's
well-done : ) : ) : )

To be honest:

I expect at least TWO explained cases:

A. The case in which the b44 module is a PCI module (onboard, additional PCI
card etc.).

B. At least one well described case in which the b44 module is a bus
independent one.

Apart from that, it is Michael's task to explain the what and why, not mine
and not yours! It is him that wants his stuff being pulled into mainline, and
in so far he very well can do the "dirty work" of documentation, explanation.

But the issue is: He is not ready to DO that! Only if it fires back!

Cheers

Uwe