On Sun, Aug 31, 2003 at 09:11:47AM +1000, Eyal Lebedinsky wrote: > Marcelo Tosatti wrote: > > > > Hello, > > > > Here goes -pre2. It contains an USB update, PPC merge, m68k merge, IDE > > gcc -D__KERNEL__ -I/data2/usr/local/src/linux-2.4-pre/include -Wall > -Wstrict-pro > totypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common > -fomit-frame-pointer > -pipe -mpreferred-stack-boundary=2 -march=i686 -malign-functions=4 > -DMODULE -DM > ODVERSIONS -include > /data2/usr/local/src/linux-2.4-pre/include/linux/modversions > .h -nostdinc -iwithprefix include -DKBUILD_BASENAME=airo > -DEXPORT_SYMTAB -c ai > ro.c > airo.c: In function `airo_get_power': > airo.c:5659: parse error before `int' > airo.c:5660: `mode' undeclared (first use in this function) > airo.c:5660: (Each undeclared identifier is reported only once > airo.c:5660: for each function it appears in.) > airo.c: In function `writerids': > airo.c:6673: warning: unused variable `enabled' > make[3]: *** [airo.o] Error 1 > make[3]: Leaving directory > `/data2/usr/local/src/linux-2.4-pre/drivers/net/wirel > ess' > > Cannot tell if it is a bad merge ('int mode =' line should be earlier?) > or bad programming (declaration must come first). I assume you are using gcc 2.95? The patch below fixes it for both 2.6 and 2.4. I've tested the compilation with 2.6.0-test4-mm5 and gcc 2.95. cu Adrian --- linux-2.6.0-test4-mm5-modular-no-smp/drivers/net/wireless/airo.c.old 2003-09-03 17:41:16.000000000 +0200 +++ linux-2.6.0-test4-mm5-modular-no-smp/drivers/net/wireless/airo.c 2003-09-03 17:41:45.000000000 +0200 @@ -5662,9 +5662,10 @@ char *extra) { struct airo_info *local = dev->priv; + int mode; readConfigRid(local, 1); - int mode = local->config.powerSaveMode; + mode = local->config.powerSaveMode; if ((vwrq->disabled = (mode == POWERSAVE_CAM))) return 0; if ((vwrq->flags & IW_POWER_TYPE) == IW_POWER_TIMEOUT) { - : send the line "unsubscribe linux-net" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html