2.7.0 testings

Mark D. Studebaker mds at paradyne.com
Thu Nov 28 18:26:10 CET 2002


I like your suggestion for fixing LINUX :=
If you fix that I'll fix pport.

The build symlink comes from the top level kernel Makefile,
from 'make modules_install'.

Jean Delvare wrote:
> 
> > I'll test with a 2.2 series kernel on another machine early tomorrow.
> 
> We have a problem. 2.2 kernels may not have a
> /lib/modules/$VERSION/build symlink. I don't know what's responsible for
> creating it. Curiously, when I tried on my laptop with a 2.2.22 kernel,
> the symlink was created. But on my old desktop system, kernels 2.2.14,
> 2.2.16 and 2.2.20 don't have one.
> 
> So we can't keep the things as they are. I suggest to use the following:
> 
> LINUX := $(shell if [ -L /lib/modules/$(KERNELVERSION)/build ] ; then
> echo "/lib/modules/$(KERNELVERSION)/build" ; else echo "/usr/src/linux"
> ; fi)
> 
> Someone's got a better idea?
> 
> Another problem:
> 
> /usr/src/i2c> make
> gcc -I/usr/src/linux/include -O2 -DLM_SENSORS -D__KERNEL__ -DMODULE
> -fomit-frame-pointer -DEXPORT_SYMTAB -DMODVERSIONS -include
> /usr/src/linux/include/linux/modversions.h -c kernel/i2c-pport.c -o
> kernel/i2c-pport.o
> kernel/i2c-pport.c: In function `bit_pport_init':
> kernel/i2c-pport.c:115: wrong type argument to unary exclamation mark
> make: *** [kernel/i2c-pport.o] Error 1
> /usr/src/i2c>
> 
> This is a 2.2.20 kernel. The code is:
> 
>     112
>     113 static int bit_pport_init(void)
>     114 {
>     115     if (!request_region((base+2),1, "i2c (PPORT adapter)")) {
>     116         return -ENODEV;
>     117     } else {
>     118
> 
> >From /usr/src/linux-2.2.20:
> kernel/resource.c:void request_region(unsigned long from, unsigned long
> num, const char *name)
> So actually you are not supposed to check a returned value. Who can fix
> this?
> 
> I also get this warning for many files:
> /usr/src/linux-2.2.20/include/linux/module.h:13: warning: `_set_ver'
> redefined
> /usr/src/linux-2.2.20/include/linux/modsetver.h:9: warning: this is the
> location of the previous definition
> But I guess this is a kernel-side problem, right? Gonna try on a 2.2.22
> to see if it has been fixed.
> 
> --
> Jean Delvare
> http://www.ensicaen.ismra.fr/~delvare/



More information about the lm-sensors mailing list