Messages in this thread | | | Date | Wed, 24 Apr 2002 12:43:18 -0700 | From | Ben Greear <> | Subject | Re: AW: Re: AW: Re: VLAN and Network Drivers 2.4.x |
| |
David S. Miller wrote:
> From: Ben Greear <greearb@candelatech.com> > Date: Wed, 24 Apr 2002 10:58:06 -0700 > > > But the changes are wrong, just because they work for some people > > doesn't make the change mergeable into the main tree. > > Wrong is a strong word for a change that makes it work for some people without > obvious negative side effects. > > Ummm, sed 's/obvious/known/' We don't know what the patch > even does.
We may not know EVERYTHING the patch does, but we do know that it enables VLANs to work, and does not degrade other functionality in any _observable_ way (to this point in time, at least.)
If someone wants to use VLANS, and wants to use EEPRO nics, then this patch is obviously better than the unpatched driver. Thus my suggestion that we make it easier for users to enable this patch/hack/whatever.
Allowing a #define switch, or even clearly commented driver code that a FAQ can point to will help the VLAN user, and will not AT ALL affect non-vlan aware users.
If/when the e100 guys show us a better way, no one will argue that you shouldn't modify the eepro100 to be more kosher.
Thanks, Ben
-- Ben Greear <greearb@candelatech.com> <Ben_Greear AT excite.com> President of Candela Technologies Inc http://www.candelatech.com ScryMUD: http://scry.wanfear.com http://scry.wanfear.com/~greear
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |