lkml.org 
[lkml]   [2001]   [Oct]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Subjectlittle bug in drivers/parport/ieee1284_ops.c
Date
Hi,

Think I found a bug in linux-2.4.12 in the module ieee1284_ops
It fails to compile the modules :

gcc -D__KERNEL__ -I/usr/src/linux-2.4.12/include -Wall -Wstrict-prototypes
-Wno-trigraphs -O2 -fomit-frame-pointer -fno-strict-aliasing -fno-common
-pipe -mpreferred-stack-boundary=2 -march=i586 -DMODULE -c -o
ieee1284_ops.o ieee1284_ops.c
ieee1284_ops.c: In function `ecp_forward_to_reverse':
ieee1284_ops.c:365: `IEEE1284_PH_DIR_UNKNOWN' undeclared (first use in this
function)
ieee1284_ops.c:365: (Each undeclared identifier is reported only once
ieee1284_ops.c:365: for each function it appears in.)
ieee1284_ops.c: In function `ecp_reverse_to_forward':
ieee1284_ops.c:397: `IEEE1284_PH_DIR_UNKNOWN' undeclared (first use in this
function)
make[2]: *** [ieee1284_ops.o] Error 1


I checked include/linux/parport.h and decided that it probably had to be
EEE1284_PH_ECP_DIR_UNKNOWN.

After this modification everything compiles fine.

--
groeten,
Kees Lemmens.

-----------------------------------------------------------------------
Department of Applied Mathematical Analysis, Faculty of Information
Technology and Systems, Delft University of Technology,

Address : PO Box 5031, NL-2600 GA, Delft, The Netherlands.
Phone work/home: (+31) 015-2787224/010-4740254, Fax : 015-2787209
Email : C.W.J.Lemmens@its.tudelft.nl.
WWW : http://ta.twi.tudelft.nl/DV/Staff/C.W.J.Lemmens.html
-----------------------------------------------------------------------
-
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:08    [W:0.058 / U:0.540 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site