Re: [announce] new rt2800 drivers for Ralink wireless & project tree

From: Ivo van Doorn
Date: Fri Nov 06 2009 - 12:59:10 EST


Hi,

> > > Look at the diffstat of Bart's driver:
> > >
> > > 15 files changed, 4036 insertions(+), 7158 deletions(-)
> > >
> > > He reduced your 5.2 KLOC non-working driver into a 1.8 KLOC _working_
> > > driver.
> >
> > Bullshit, read the mails again.
>
> This was uncalled for, right?

The way I said it, yes.

But I was frustrated by Ingo with his comment that the driver was magically fixed,
while even Bartlomiej indicated that was not the case. With these comments,
and those were he were his ignorance about the Ralink drivers in the kernel
was apparent, he managed to get me completely upset because I really
got the impression I was talking to a wall or something and overreacted.

> > > And _still_ your complaint about Bart's series is that he updated the
> > > MAINTAINERS entry and added an entry for rt2800? Heck _sure_ he should
> > > update it, he is the one doing the hard work of trying to bring it to
> > > users, trying to clean up a messy driver space, trying to turn crap into
> > > gold.
> >
> > So if I want to focus on something different in the kernel, I just send 1 patch,
> > and a second to claim the maintainership of it even though there is an active
> > maintainer available?
>
> What about listing yourself as a maintainer for a start?

The rt2x00 project is listed as maintainer for everything in the
drivers/net/wireless/rt2x00 folder.

> > > The thing is, if you dont have the time or interest to listen to and act
> > > upon review feedback, be constructive about it and fix (obvious)
> > > structural problems in your rt2800 code, you should just step aside and
> > > let Bart maintain what he is apparently more capable of maintaining than
> > > you are.
> > >
> > > What you are doing here is a thinly veiled land-grab: you did a minimal
> > > token driver for rt2800 that doesnt work, kept it in your private tree
> > > for _1.5 years_, and the moment someone _else_ came along and did
> > > something better and more functional in drivers/staging/, you discovered
> > > your sudden interest for it and moved the crappy driver upstream at
> > > lightning's speed (it is already in net-next AFAICS, despite negative
> > > test and review feedback) - ignoring and throwing away all the work that
> > > Bart has done.
> >
> > Get your facts straight, the bullshit level in your mail is staggering.
> >
> > You have no fucking clue who wrote the rt2800 driver which is in
> > drivers/staging/,
>
> Perhaps you should not be a maintainer if you can't behave yourself?

Ok let me explain clearer then:

drivers/net/staging/rt28{6,7}0 was developed by Ralink and I acked
the merged for those drivers after the asurance that it was only merged
to please the users so developers could focus on the rt2x00 version of
the driver.

Anyway, I'll do what John requested and go back into my hole since I
have absolutely no desire to continue this flamewar.

Ivo
--
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/