Messages in this thread | | | Date | Thu, 28 Jun 2001 16:40:51 -0400 | From | Jeff Garzik <> | Subject | Re: [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/
| |