Should DVB depend on I2C?

Got the following compile error :

  LD      .tmp_vmlinux1
drivers/built-in.o: In function `tda8083_writereg':
drivers/media/dvb/frontends/tda8083.c:69: undefined reference to `i2c_transfer'
drivers/built-in.o: In function `tda8083_readregs':
drivers/media/dvb/frontends/tda8083.c:84: undefined reference to `i2c_transfer'
drivers/built-in.o: In function `l64781_writereg':
drivers/media/dvb/frontends/l64781.c:60: undefined reference to `i2c_transfer'
drivers/built-in.o: In function `l64781_readreg':
drivers/media/dvb/frontends/l64781.c:75: undefined reference to `i2c_transfer'
drivers/built-in.o: In function `reset_and_configure':
drivers/media/dvb/frontends/l64781.c:119: undefined reference to `i2c_transfer'
drivers/built-in.o:drivers/media/dvb/frontends/l64781.c:524: more undefined references to `i2c_transfer' follow
make: *** [.tmp_vmlinux1] Error 1

.config is attached

Still waiting on a response at http://marc.theaimsgroup.com/?l=linux-video&m=114787670927881&w=2

Oh wait, there was one, but apparently I wasn't CC'd. Adding the right list now.

DVB people:

Attila Stehr in a Gentoo bug report pointed out that the DVB frontends
do not link without CONFIG_I2C, and there is no dependency listed in the
Kconfig files.

e.g.:
drivers/built-in.o: In function `stv0299_readregs':
stv0299.c:(.text+0xa4d8a): undefined reference to `i2c_transfer'

I checked and I see this is the case for other DVB components too, such
as the dvb-usb drivers.

I'm happy to patch this, but I'm not sure which approach should be best. Should I make CONFIG_DVB or CONFIG_DVB_CORE depend on I2C, or should I modify the config blocks for all of the individual options which use I2C stuff?

Thanks,
Daniel

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

---

• References:
  • compile error: i2c_transfer
    • From: Toralf Förster <[email protected]>

• Prev by Date: Re: [patch 00/61] ANNOUNCE: lock validator -V1
• Next by Date: Re: [RFC 0/5] sched: Add CPU rate caps
• Previous by thread: Re: compile error: i2c_transfer
• Next by thread: /sys/class/net/eth?/carrier and uevents
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]