lkml.org 
[lkml]   [2001]   [Jun]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] 2.4.6-pre6 fix drivers/net/Config.in error
Keith Owens wrote:
>
> On Thu, 28 Jun 2001 10:45:55 +0200 (MET DST),
> Andrzej Krzysztofowicz <ankry@pg.gda.pl> wrote:
> >Keith Owens wrote:
> >> Index: 6-pre6.1/drivers/net/Config.in
> >> - dep_bool ' EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI
> >> + if [ "$CONFIG_ISA" = "y" -o "$CONFIG_EISA" = "y" -o "$CONFIG_PCI" = "y" ]; then
>
> >CONFIG_EISA check in this condition is redundant.
>
> True, but the line is a cut and paste from higher up in
> drivers/net/Config.in. Even though it is redundant, it is consistent
> with the rest of the file.

It is not redundant because in theory CONFIG_EISA could exist without
CONFIG_ISA.

> drivers/net/Config.in needs a major cleanup, lots of the if statements
> can go and be replaced by dep_xxx statements, CONFIG_ETHERTAP is marked
> obsolete but is tested against experimental, CONFIG_ZNET is marked
> experimental but is tested against obsolete, etc.

Why not send me an incremental patch for these cleanups, on top of the
cleanup patch that (I hope!) Andrzej will send, in respond to me last
reply.

2.4 will be around for quite a while, and Config.in cleanups should
continue to go in. Sure work should be directed towards 2.5..... but if
somebody sends me a 2.4 patch for drivers/net/Config.in cleanup, I sure
as hell will apply it.

--
Jeff Garzik | Andre the Giant has a posse.
Building 1024 |
MandrakeSoft |
-
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/

\
 
 \ /
  Last update: 2005-03-22 13:17    [W:0.053 / U:0.840 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site