Hi Marcelo,
IMHO it gives a bad picture of the quality of Linux if a stable kernel
contains options that doesn't compile. CONFIG_IEEE1394_PCILYNX_PORTS
doesn't compile (the error message is at the end of the mail) and Andreas
Bombe stated in a private mail to me four months ago that it shouldn't
have been a public option.
My patch doesn't do any harm because currently the kernel doesn't compile
when this option is enabled and if someone fixes pcilynx.c it's pretty
trivial to revert this patch.
--- drivers/ieee1394/Config.in.old Fri May 3 11:11:06 2002
+++ drivers/ieee1394/Config.in Fri May 3 11:12:18 2002
@@ -12,7 +12,7 @@
dep_tristate ' Texas Instruments PCILynx support' CONFIG_IEEE1394_PCILYNX $CONFIG_IEEE1394
if [ "$CONFIG_IEEE1394_PCILYNX" != "n" ]; then
bool ' Use PCILynx local RAM' CONFIG_IEEE1394_PCILYNX_LOCALRAM
- bool ' Support for non-IEEE1394 local ports' CONFIG_IEEE1394_PCILYNX_PORTS
+# bool ' Support for non-IEEE1394 local ports' CONFIG_IEEE1394_PCILYNX_PORTS
fi
dep_tristate ' OHCI-1394 support' CONFIG_IEEE1394_OHCI1394 $CONFIG_IEEE1394
TIA
Adrian
<-- snip -->
gcc -D__KERNEL__ -I/home/bunk/linux/kernel-2.4/linux-full/include -Wall
-Wstrict-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common -pipe
-mpreferred-stack-boundary=2 -march=k6 -nostdinc -I
/usr/lib/gcc-lib/i386-linux/2.95.4/include -DKBUILD_BASENAME=pcilynx -c -o pcilynx.o pcilynx.c
pcilynx.c: In function `mem_open':
pcilynx.c:647: `num_of_cards' undeclared (first use in this function)
pcilynx.c:647: (Each undeclared identifier is reported only once
pcilynx.c:647: for each function it appears in.)
pcilynx.c:647: `cards' undeclared (first use in this function)
pcilynx.c: In function `aux_poll':
pcilynx.c:706: `cards' undeclared (first use in this function)
make[3]: *** [pcilynx.o] Error 1
make[3]: Leaving directory
`/home/bunk/linux/kernel-2.4/linux-full/drivers/ieee1394'
<-- snip -->
On Tue, Jun 04, 2002 at 11:19:19AM +0200, Adrian Bunk wrote:
> Hi Marcelo,
>
> IMHO it gives a bad picture of the quality of Linux if a stable kernel
> contains options that doesn't compile. CONFIG_IEEE1394_PCILYNX_PORTS
> doesn't compile (the error message is at the end of the mail) and Andreas
> Bombe stated in a private mail to me four months ago that it shouldn't
> have been a public option.
>
> My patch doesn't do any harm because currently the kernel doesn't compile
> when this option is enabled and if someone fixes pcilynx.c it's pretty
> trivial to revert this patch.
We've already done this locally in our repo. Go ahead and apply this
patch if you want.
Ben
--
Debian - http://www.debian.org/
Linux 1394 - http://linux1394.sourceforge.net/
Subversion - http://subversion.tigris.org/
Deqo - http://www.deqo.com/