Re: [Announce] Emulex LightPulse Device Driver

From: James Bottomley
Date: Wed Mar 10 2004 - 10:23:27 EST


On Wed, 2004-03-10 at 03:35, Jeff Garzik wrote:
> Embark you shall, and let me join in the chorus of kudos for making the
> leap to open source. :)

Yes, I'll add my welcome to that.

[...]
> That should get you started ;-)

Actually, it would be my interpretation of the FAQ that most of this
work is already intended (although Jeff gave specific instances than the
generalities in the FAQ).

There were many more places than this in the driver that caused me to go
"good grief no". However, it probably makes more sense if you work down
your todo list and come back for a review when you're nearing the end of
it. That way you don't get boat loads of comments about things you were
planning to fix anyway.

James


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