mirror of
https://github.com/torvalds/linux.git
synced 2024-11-01 17:51:43 +00:00
bb69ee27b9
There are a large number of reports that the media build is
not compiling when some drivers are compiled as builtin, while
the needed frontends are compiled as module.
On the last one of such reports:
From: kbuild test robot <fengguang.wu@intel.com>
Subject: saa7134-dvb.c:undefined reference to `zl10039_attach'
The .config file has:
CONFIG_VIDEO_SAA7134=y
CONFIG_VIDEO_SAA7134_DVB=y
# CONFIG_MEDIA_ATTACH is not set
CONFIG_DVB_ZL10039=m
And it produces all those errors:
drivers/built-in.o: In function `set_type':
tuner-core.c:(.text+0x2f263e): undefined reference to `tea5767_attach'
tuner-core.c:(.text+0x2f273e): undefined reference to `tda9887_attach'
drivers/built-in.o: In function `tuner_probe':
tuner-core.c:(.text+0x2f2d20): undefined reference to `tea5767_autodetection'
drivers/built-in.o: In function `av7110_attach':
av7110.c:(.text+0x330bda): undefined reference to `ves1x93_attach'
av7110.c:(.text+0x330bf7): undefined reference to `stv0299_attach'
av7110.c:(.text+0x330c63): undefined reference to `tda8083_attach'
av7110.c:(.text+0x330d09): undefined reference to `ves1x93_attach'
av7110.c:(.text+0x330d33): undefined reference to `tda8083_attach'
av7110.c:(.text+0x330d5d): undefined reference to `stv0297_attach'
av7110.c:(.text+0x330dbe): undefined reference to `stv0299_attach'
drivers/built-in.o: In function `tuner_attach_dtt7520x':
ngene-cards.c:(.text+0x3381cb): undefined reference to `dvb_pll_attach'
drivers/built-in.o: In function `demod_attach_lg330x':
ngene-cards.c:(.text+0x33828a): undefined reference to `lgdt330x_attach'
drivers/built-in.o: In function `demod_attach_stv0900':
ngene-cards.c:(.text+0x3383d5): undefined reference to `stv090x_attach'
drivers/built-in.o: In function `cineS2_probe':
ngene-cards.c:(.text+0x338b7f): undefined reference to `drxk_attach'
drivers/built-in.o: In function `configure_tda827x_fe':
saa7134-dvb.c:(.text+0x346ae7): undefined reference to `tda10046_attach'
drivers/built-in.o: In function `dvb_init':
saa7134-dvb.c:(.text+0x347283): undefined reference to `mt352_attach'
saa7134-dvb.c:(.text+0x3472cd): undefined reference to `mt352_attach'
saa7134-dvb.c:(.text+0x34731c): undefined reference to `tda10046_attach'
saa7134-dvb.c:(.text+0x34733c): undefined reference to `tda10046_attach'
saa7134-dvb.c:(.text+0x34735c): undefined reference to `tda10046_attach'
saa7134-dvb.c:(.text+0x347378): undefined reference to `tda10046_attach'
saa7134-dvb.c:(.text+0x3473db): undefined reference to `tda10046_attach'
drivers/built-in.o:saa7134-dvb.c:(.text+0x347502): more undefined references to `tda10046_attach' follow
drivers/built-in.o: In function `dvb_init':
saa7134-dvb.c:(.text+0x347812): undefined reference to `mt352_attach'
saa7134-dvb.c:(.text+0x347951): undefined reference to `mt312_attach'
saa7134-dvb.c:(.text+0x3479a9): undefined reference to `mt312_attach'
>> saa7134-dvb.c:(.text+0x3479c1): undefined reference to `zl10039_attach'
This is happening because a builtin module can't use directly a symbol
found on a module. By enabling CONFIG_MEDIA_ATTACH, the configuration
becomes valid, as dvb_attach() macro loads the module if needed, making
the symbol available to the builtin module.
While this bug started to appear after the patches that use IS_DEFINED
macro (like changeset 7b34be71db), this
bug is a way ancient than that.
The thing is that, before the IS_DEFINED() patches, the logic used to be:
&& defined(MODULE))
struct dvb_frontend *zl10039_attach(struct dvb_frontend *fe,
u8 i2c_addr,
struct i2c_adapter *i2c);
static inline struct dvb_frontend *zl10039_attach(struct dvb_frontend *fe,
u8 i2c_addr,
struct i2c_adapter *i2c)
{
printk(KERN_WARNING "%s: driver disabled by Kconfig\n", __func__);
return NULL;
}
The above code, with the .config file used, was evoluting to FALSE
(instead of TRUE as it should be, as CONFIG_DVB_ZL10039 is 'm'),
and were adding the static inline code at saa7134-dvb, instead
of the external call. So, while it weren't producing any compilation
error, the code weren't working either.
So, as the overhead for using CONFIG_MEDIA_ATTACH is minimal, just
enable it, if MODULES is defined.
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
203 lines
5.5 KiB
Plaintext
203 lines
5.5 KiB
Plaintext
#
|
|
# Multimedia device configuration
|
|
#
|
|
|
|
menuconfig MEDIA_SUPPORT
|
|
tristate "Multimedia support"
|
|
depends on HAS_IOMEM
|
|
help
|
|
If you want to use Webcams, Video grabber devices and/or TV devices
|
|
enable this option and other options below.
|
|
Additional info and docs are available on the web at
|
|
<http://linuxtv.org>
|
|
|
|
if MEDIA_SUPPORT
|
|
|
|
comment "Multimedia core support"
|
|
|
|
#
|
|
# Multimedia support - automatically enable V4L2 and DVB core
|
|
#
|
|
config MEDIA_CAMERA_SUPPORT
|
|
bool "Cameras/video grabbers support"
|
|
---help---
|
|
Enable support for webcams and video grabbers.
|
|
|
|
Say Y when you have a webcam or a video capture grabber board.
|
|
|
|
config MEDIA_ANALOG_TV_SUPPORT
|
|
bool "Analog TV support"
|
|
---help---
|
|
Enable analog TV support.
|
|
|
|
Say Y when you have a TV board with analog support or with a
|
|
hybrid analog/digital TV chipset.
|
|
|
|
Note: There are several DVB cards that are based on chips that
|
|
support both analog and digital TV. Disabling this option
|
|
will disable support for them.
|
|
|
|
config MEDIA_DIGITAL_TV_SUPPORT
|
|
bool "Digital TV support"
|
|
---help---
|
|
Enable digital TV support.
|
|
|
|
Say Y when you have a board with digital support or a board with
|
|
hybrid digital TV and analog TV.
|
|
|
|
config MEDIA_RADIO_SUPPORT
|
|
bool "AM/FM radio receivers/transmitters support"
|
|
---help---
|
|
Enable AM/FM radio support.
|
|
|
|
Additional info and docs are available on the web at
|
|
<http://linuxtv.org>
|
|
|
|
Say Y when you have a board with radio support.
|
|
|
|
Note: There are several TV cards that are based on chips that
|
|
support radio reception. Disabling this option will
|
|
disable support for them.
|
|
|
|
config MEDIA_RC_SUPPORT
|
|
bool "Remote Controller support"
|
|
depends on INPUT
|
|
---help---
|
|
Enable support for Remote Controllers on Linux. This is
|
|
needed in order to support several video capture adapters,
|
|
standalone IR receivers/transmitters, and RF receivers.
|
|
|
|
Enable this option if you have a video capture board even
|
|
if you don't need IR, as otherwise, you may not be able to
|
|
compile the driver for your adapter.
|
|
|
|
Say Y when you have a TV or an IR device.
|
|
|
|
#
|
|
# Media controller
|
|
# Selectable only for webcam/grabbers, as other drivers don't use it
|
|
#
|
|
|
|
config MEDIA_CONTROLLER
|
|
bool "Media Controller API"
|
|
depends on MEDIA_CAMERA_SUPPORT
|
|
---help---
|
|
Enable the media controller API used to query media devices internal
|
|
topology and configure it dynamically.
|
|
|
|
This API is mostly used by camera interfaces in embedded platforms.
|
|
|
|
#
|
|
# Video4Linux support
|
|
# Only enables if one of the V4L2 types (ATV, webcam, radio) is selected
|
|
#
|
|
|
|
config VIDEO_DEV
|
|
tristate
|
|
depends on MEDIA_SUPPORT
|
|
depends on MEDIA_CAMERA_SUPPORT || MEDIA_ANALOG_TV_SUPPORT || MEDIA_RADIO_SUPPORT
|
|
default y
|
|
|
|
config VIDEO_V4L2_SUBDEV_API
|
|
bool "V4L2 sub-device userspace API"
|
|
depends on VIDEO_DEV && MEDIA_CONTROLLER
|
|
---help---
|
|
Enables the V4L2 sub-device pad-level userspace API used to configure
|
|
video format, size and frame rate between hardware blocks.
|
|
|
|
This API is mostly used by camera interfaces in embedded platforms.
|
|
|
|
source "drivers/media/v4l2-core/Kconfig"
|
|
|
|
#
|
|
# DVB Core
|
|
# Only enables if one of DTV is selected
|
|
#
|
|
|
|
config DVB_CORE
|
|
tristate
|
|
depends on MEDIA_SUPPORT
|
|
depends on MEDIA_DIGITAL_TV_SUPPORT
|
|
default y
|
|
select CRC32
|
|
|
|
config DVB_NET
|
|
bool "DVB Network Support"
|
|
default (NET && INET)
|
|
depends on NET && INET && DVB_CORE
|
|
help
|
|
This option enables DVB Network Support which is a part of the DVB
|
|
standard. It is used, for example, by automatic firmware updates used
|
|
on Set-Top-Boxes. It can also be used to access the Internet via the
|
|
DVB card, if the network provider supports it.
|
|
|
|
You may want to disable the network support on embedded devices. If
|
|
unsure say Y.
|
|
|
|
# This Kconfig option is used by both PCI and USB drivers
|
|
config TTPCI_EEPROM
|
|
tristate
|
|
depends on I2C
|
|
default n
|
|
|
|
source "drivers/media/dvb-core/Kconfig"
|
|
|
|
comment "Media drivers"
|
|
source "drivers/media/rc/Kconfig"
|
|
|
|
#
|
|
# V4L platform/mem2mem drivers
|
|
#
|
|
|
|
source "drivers/media/usb/Kconfig"
|
|
source "drivers/media/pci/Kconfig"
|
|
source "drivers/media/platform/Kconfig"
|
|
source "drivers/media/mmc/Kconfig"
|
|
source "drivers/media/parport/Kconfig"
|
|
source "drivers/media/radio/Kconfig"
|
|
|
|
comment "Supported FireWire (IEEE 1394) Adapters"
|
|
depends on DVB_CORE && FIREWIRE
|
|
source "drivers/media/firewire/Kconfig"
|
|
|
|
# Common driver options
|
|
source "drivers/media/common/Kconfig"
|
|
|
|
comment "Media ancillary drivers (tuners, sensors, i2c, frontends)"
|
|
|
|
#
|
|
# Ancillary drivers (tuners, i2c, frontends)
|
|
#
|
|
|
|
config MEDIA_SUBDRV_AUTOSELECT
|
|
bool "Autoselect ancillary drivers (tuners, sensors, i2c, frontends)"
|
|
depends on MEDIA_ANALOG_TV_SUPPORT || MEDIA_DIGITAL_TV_SUPPORT || MEDIA_CAMERA_SUPPORT
|
|
default y
|
|
help
|
|
By default, a media driver auto-selects all possible ancillary
|
|
devices such as tuners, sensors, video encoders/decoders and
|
|
frontends, that are used by any of the supported devices.
|
|
|
|
This is generally the right thing to do, except when there
|
|
are strict constraints with regards to the kernel size,
|
|
like on embedded systems.
|
|
|
|
Use this option with care, as deselecting ancillary drivers which
|
|
are, in fact, necessary will result in the lack of the needed
|
|
functionality for your device (it may not tune or may not have
|
|
the needed demodulators).
|
|
|
|
If unsure say Y.
|
|
|
|
config MEDIA_ATTACH
|
|
bool
|
|
depends on MEDIA_ANALOG_TV_SUPPORT || MEDIA_DIGITAL_TV_SUPPORT || MEDIA_RADIO_SUPPORT
|
|
depends on MODULES
|
|
default MODULES
|
|
|
|
source "drivers/media/i2c/Kconfig"
|
|
source "drivers/media/tuners/Kconfig"
|
|
source "drivers/media/dvb-frontends/Kconfig"
|
|
|
|
endif # MEDIA_SUPPORT
|