Skip to contentSkip to conversation

2.4.0-pre compile err (pcxx)

DISCUSSION A. P. Garcia 2 participants 4 messages Jan 1, 2001

lore
Share & tools
lore
#0 lore raw

when i make bzimage with the pc/xx driver configured as a module, it compiles ok. configuring it as built-in gives the following error:

gcc -D__KERNEL__ -I/usr/src/linux-2.4.0-prerelease/include -Wall -Wstrict-prototypes -O2 -fomit-frame-pointer -fno-strict-aliasing -pipe -mpreferred-stack-boundary=2 -march=i586 -c -o pcxx.o pcxx.c
pcxx.c: In function `pcxxdelay':
pcxx.c:1826: `mseconds' undeclared (first use in this function) pcxx.c:1826: (Each undeclared identifier is reported only once pcxx.c:1826: for each function it appears in.)
make[3]: *** [pcxx.o] Error 1
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected]
Please read the FAQ at http://www.tux.org/lkml/

#1 lore raw
Quoted 2 lines

when i make bzimage with the pc/xx driver configured as a module, it compiles ok. configuring it as built-in gives the following error:

Im amazed it built as a module - thats why I missed the error

Quoted 2 lines

pcxx.c:1826: `mseconds' undeclared (first use in this function) pcxx.c:1826: (Each undeclared identifier is reported only once

char/pcxx.c renamed from char/pcxx.c~ +1 -1
@@ -1823,7 +1823,7 @@
  */
 static void pcxxdelay(int msec)
 {
-	mdelay(mseconds);
+	mdelay(msec);
 }
 
 

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected]
Please read the FAQ at http://www.tux.org/lkml/

#2 lore raw
Quoted 4 lines · Alan Cox
Quoted 2 lines

when i make bzimage with the pc/xx driver configured as a module, it compiles ok. configuring it as built-in gives the following error:

Im amazed it built as a module - thats why I missed the error

well, the kernel compiled, but i kind of forgot to make modules

:-)
many thanks
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected]
Please read the FAQ at http://www.tux.org/lkml/

#3 lore raw
In reply to Alan Cox

this is a patch against the pcxx.c in the prerelease distribution [clear thinkos. brain ok now.]

anyhow, before it was awful close ;-)

thanks again

pcxx.c renamed from pcxx.c~ +3 -3
@@ -152,7 +152,7 @@
 DECLARE_TASK_QUEUE(tq_pcxx);
 
 static void pcxxpoll(unsigned long dummy);
-static void pcxxdelay(int);
+static void pcxxdelay(unsigned long);
 static void fepcmd(struct channel *, int, int, int, int, int);
 static void pcxe_put_char(struct tty_struct *, unsigned char);
 static void pcxe_flush_chars(struct tty_struct *);
@@ -1821,9 +1821,9 @@
 /*
  * pcxxdelay - delays a specified number of milliseconds
  */
-static void pcxxdelay(int msec)
+static void pcxxdelay(unsigned long ms)
 {
-	mdelay(mseconds);
+	mdelay(ms);
 }
 
 
Quoted 12 lines · Alan Cox
Quoted 2 lines

when i make bzimage with the pc/xx driver configured as a module, it compiles ok. configuring it as built-in gives the following error:

Im amazed it built as a module - thats why I missed the error

Quoted 2 lines

pcxx.c:1826: `mseconds' undeclared (first use in this function) pcxx.c:1826: (Each undeclared identifier is reported only once

char/pcxx.c line 1828

Quoted 7 lines
char/pcxx.c renamed from char/pcxx.c~ +1 -1
@@ -1823,7 +1823,7 @@
  */
 static void pcxxdelay(int msec)
 {
-	mdelay(mseconds);
+	mdelay(msec);
 }
 

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected]
Please read the FAQ at http://www.tux.org/lkml/