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/
This archive was generated by hypermail 2b29 : Tue Apr 30 2002 - 22:00:10 EST