Hi Adrian, On Wed, 27 Feb 2008 09:37:28 +0200, Adrian Bunk wrote: > This patch fixes the following compile error on m32r: > > <-- snip --> > > ... > CC [M] drivers/i2c/busses/i2c-elektor.o > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/i2c/busses/i2c-elektor.c: > In function 'pcf_isa_setbyte': > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/i2c/busses/i2c-elektor.c:80: > error: implicit declaration of function 'iowrite8' > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/i2c/busses/i2c-elektor.c: > In function 'pcf_isa_getbyte': > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/i2c/busses/i2c-elektor.c:90: > error: implicit declaration of function 'ioread8' > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/i2c/busses/i2c-elektor.c: > In function 'pcf_isa_init': > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/i2c/busses/i2c-elektor.c:153: > error: implicit declaration of function 'ioport_map' > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/i2c/busses/i2c-elektor.c:153: > warning: assignment makes pointer from integer without a cast > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/i2c/busses/i2c-elektor.c: > In function 'elektor_probe': > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/i2c/busses/i2c-elektor.c:290: > error: implicit declaration of function 'ioport_unmap' > make[4]: *** [drivers/i2c/busses/i2c-elektor.o] Error 1 > > <-- snip --> > > Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]> > > --- > c8a75f0dd2390ae8c8f91c451ff6b0e113d0ebd0 diff --git > a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig > index 476b0bb..3bed6b9 100644 > --- a/drivers/i2c/busses/Kconfig > +++ b/drivers/i2c/busses/Kconfig > @@ -129,7 +129,7 @@ config I2C_DAVINCI > > config I2C_ELEKTOR > tristate "Elektor ISA card" > - depends on ISA && BROKEN_ON_SMP > + depends on ISA && HAS_IOPORT && BROKEN_ON_SMP > select I2C_ALGOPCF > help > This supports the PCF8584 ISA bus I2C adapter. Say Y if you own >
Looks weird to me. As you can see, I2C_ELEKTOR already depends on ISA and I thought it would be sufficient. Your m32r system has CONFIG_HAS_IOPORT=n but CONFIG_ISA=y? I fail to see how a system could have ISA slots but not have IO ports. -- Jean Delvare _______________________________________________ i2c mailing list [email protected] http://lists.lm-sensors.org/mailman/listinfo/i2c
