Re: [PATCH] 2.4.6-pre6 fix drivers/net/Config.in error

2001-06-28 Thread Keith Owens

On Thu, 28 Jun 2001 10:45:55 +0200 (MET DST), 
Andrzej Krzysztofowicz <[EMAIL PROTECTED]> 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.

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.  I started to clean
up the file then decided to leave it, the entire thing is being
replaced in 2.5 anyway.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



Re: [PATCH] 2.4.6-pre6 fix drivers/net/Config.in error

2001-06-28 Thread Andrzej Krzysztofowicz

Keith Owens wrote:
> On Thu, 28 Jun 2001 00:07:13 -0400, 
> Jeff Garzik <[EMAIL PROTECTED]> wrote:
> >Steven Cole wrote:
> >> -   dep_bool '  EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI
> >> +   dep_bool '  EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI 
>$CONFIG_PCI
> >
> >See the "EISA" and "VLB" parts in there?  EISA != PCI
> 
> Against 2.4.6-pre6.
> 
> Index: 6-pre6.1/drivers/net/Config.in
> --- 6-pre6.1/drivers/net/Config.in Thu, 28 Jun 2001 10:34:32 +1000 kaos 
>(linux-2.4/l/c/9_Config.in 1.1.2.2.1.3.1.8 644)
> +++ 6-pre6.1(w)/drivers/net/Config.in Thu, 28 Jun 2001 16:07:03 +1000 kaos 
>(linux-2.4/l/c/9_Config.in 1.1.2.2.1.3.1.8 644)
> @@ -142,7 +142,11 @@ if [ "$CONFIG_NET_ETHERNET" = "y" ]; the
>tristate '  NE/2 (ne2000 MCA version) support' CONFIG_NE2_MCA
>tristate '  IBM LAN Adapter/A support' CONFIG_IBMLANA
> fi
> -   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.
Intentionally ?

> + bool '  EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI

Andrzej
-- 
===
  Andrzej M. Krzysztofowicz   [EMAIL PROTECTED]
  phone (48)(58) 347 14 61
Faculty of Applied Phys. & Math.,   Technical University of Gdansk
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



Re: [PATCH] 2.4.6-pre6 fix drivers/net/Config.in error

2001-06-28 Thread Andrzej Krzysztofowicz

Keith Owens wrote:
 On Thu, 28 Jun 2001 00:07:13 -0400, 
 Jeff Garzik [EMAIL PROTECTED] wrote:
 Steven Cole wrote:
  -   dep_bool '  EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI
  +   dep_bool '  EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI 
$CONFIG_PCI
 
 See the EISA and VLB parts in there?  EISA != PCI
 
 Against 2.4.6-pre6.
 
 Index: 6-pre6.1/drivers/net/Config.in
 --- 6-pre6.1/drivers/net/Config.in Thu, 28 Jun 2001 10:34:32 +1000 kaos 
(linux-2.4/l/c/9_Config.in 1.1.2.2.1.3.1.8 644)
 +++ 6-pre6.1(w)/drivers/net/Config.in Thu, 28 Jun 2001 16:07:03 +1000 kaos 
(linux-2.4/l/c/9_Config.in 1.1.2.2.1.3.1.8 644)
 @@ -142,7 +142,11 @@ if [ $CONFIG_NET_ETHERNET = y ]; the
tristate '  NE/2 (ne2000 MCA version) support' CONFIG_NE2_MCA
tristate '  IBM LAN Adapter/A support' CONFIG_IBMLANA
 fi
 -   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.
Intentionally ?

 + bool '  EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI

Andrzej
-- 
===
  Andrzej M. Krzysztofowicz   [EMAIL PROTECTED]
  phone (48)(58) 347 14 61
Faculty of Applied Phys.  Math.,   Technical University of Gdansk
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



Re: [PATCH] 2.4.6-pre6 fix drivers/net/Config.in error

2001-06-28 Thread Keith Owens

On Thu, 28 Jun 2001 10:45:55 +0200 (MET DST), 
Andrzej Krzysztofowicz [EMAIL PROTECTED] 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.

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.  I started to clean
up the file then decided to leave it, the entire thing is being
replaced in 2.5 anyway.

-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



Re: [PATCH] 2.4.6-pre6 fix drivers/net/Config.in error

2001-06-27 Thread Jeff Garzik

Steven Cole wrote:
> -   dep_bool '  EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI
> +   dep_bool '  EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI $CONFIG_PCI

See the "EISA" and "VLB" parts in there?  EISA != PCI

-- 
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 [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



[PATCH] 2.4.6-pre6 fix drivers/net/Config.in error

2001-06-27 Thread Steven Cole

I got this error for 2.4.6-pre6 for make xconfig

drivers/net/Config.in: 145: can't handle dep_bool/dep_mbool/dep_tristate condition
make[1]: *** [kconfig.tk] Error 1
make[1]: Leaving directory `/usr/src/linux-2.4.6-pre6/scripts'
make: *** [xconfig] Error 2

This may be the proper fix.
Steven

--- linux/drivers/net/Config.in.originalWed Jun 27 20:39:50 2001
+++ linux/drivers/net/Config.in Wed Jun 27 20:41:28 2001
@@ -142,7 +142,7 @@
   tristate '  NE/2 (ne2000 MCA version) support' CONFIG_NE2_MCA
   tristate '  IBM LAN Adapter/A support' CONFIG_IBMLANA
fi
-   dep_bool '  EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI
+   dep_bool '  EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI $CONFIG_PCI
if [ "$CONFIG_NET_PCI" = "y" ]; then
   dep_tristate 'AMD PCnet32 PCI support' CONFIG_PCNET32 $CONFIG_PCI
   dep_tristate 'Adaptec Starfire support (EXPERIMENTAL)' 
CONFIG_ADAPTEC_STARFIRE $CONFIG_PCI $CONFIG_EXPERIMENTAL
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



[PATCH] 2.4.6-pre6 fix drivers/net/Config.in error

2001-06-27 Thread Steven Cole

I got this error for 2.4.6-pre6 for make xconfig

drivers/net/Config.in: 145: can't handle dep_bool/dep_mbool/dep_tristate condition
make[1]: *** [kconfig.tk] Error 1
make[1]: Leaving directory `/usr/src/linux-2.4.6-pre6/scripts'
make: *** [xconfig] Error 2

This may be the proper fix.
Steven

--- linux/drivers/net/Config.in.originalWed Jun 27 20:39:50 2001
+++ linux/drivers/net/Config.in Wed Jun 27 20:41:28 2001
@@ -142,7 +142,7 @@
   tristate '  NE/2 (ne2000 MCA version) support' CONFIG_NE2_MCA
   tristate '  IBM LAN Adapter/A support' CONFIG_IBMLANA
fi
-   dep_bool '  EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI
+   dep_bool '  EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI $CONFIG_PCI
if [ $CONFIG_NET_PCI = y ]; then
   dep_tristate 'AMD PCnet32 PCI support' CONFIG_PCNET32 $CONFIG_PCI
   dep_tristate 'Adaptec Starfire support (EXPERIMENTAL)' 
CONFIG_ADAPTEC_STARFIRE $CONFIG_PCI $CONFIG_EXPERIMENTAL
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



Re: [PATCH] 2.4.6-pre6 fix drivers/net/Config.in error

2001-06-27 Thread Jeff Garzik

Steven Cole wrote:
 -   dep_bool '  EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI
 +   dep_bool '  EISA, VLB, PCI and on board controllers' CONFIG_NET_PCI $CONFIG_PCI

See the EISA and VLB parts in there?  EISA != PCI

-- 
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 [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/