Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

media: Clarify how menus are hidden by SUBDRV_AUTOSELECT

Some users have been having a hard time finding the hidden
menus. A typically case are camera sensor drivers
(e.g IMX219, OV5645, etc), which are common on embedded
platforms and not really "ancillary" devices.

The problem with MEDIA_SUBDRV_AUTOSELECT seems to be related
to the fact that it uses the "visible" syntax to hide
the menus.

This is not obvious and it normally takes some time to
figure out.

To fix the problem, add a comment on each of hidden menus,
which should clarify what option is causing menus to be hidden.

Signed-off-by: Ezequiel Garcia <ezequiel@collabora.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>

authored by

Ezequiel Garcia and committed by
Mauro Carvalho Chehab
4fa4ef39 31d5d15d

+22 -4
+5
drivers/media/Kconfig
··· 207 207 208 208 If unsure say Y. 209 209 210 + config MEDIA_HIDE_ANCILLARY_SUBDRV 211 + bool 212 + depends on MEDIA_SUBDRV_AUTOSELECT && !COMPILE_TEST && !EXPERT 213 + default y 214 + 210 215 config MEDIA_ATTACH 211 216 bool 212 217 depends on MEDIA_ANALOG_TV_SUPPORT || MEDIA_DIGITAL_TV_SUPPORT || MEDIA_RADIO_SUPPORT
+4 -1
drivers/media/dvb-frontends/Kconfig
··· 1 + comment "DVB Frontend drivers hidden by 'Autoselect ancillary drivers'" 2 + depends on MEDIA_HIDE_ANCILLARY_SUBDRV 3 + 1 4 menu "Customise DVB Frontends" 2 - visible if !MEDIA_SUBDRV_AUTOSELECT || COMPILE_TEST || EXPERT 5 + visible if !MEDIA_HIDE_ANCILLARY_SUBDRV 3 6 4 7 comment "Multistandard (satellite) frontends" 5 8 depends on DVB_CORE
+4 -1
drivers/media/i2c/Kconfig
··· 22 22 # Encoder / Decoder module configuration 23 23 # 24 24 25 + comment "I2C drivers hidden by 'Autoselect ancillary drivers'" 26 + depends on MEDIA_HIDE_ANCILLARY_SUBDRV 27 + 25 28 menu "I2C Encoders, decoders, sensors and other helper chips" 26 - visible if !MEDIA_SUBDRV_AUTOSELECT || COMPILE_TEST || EXPERT 29 + visible if !MEDIA_HIDE_ANCILLARY_SUBDRV 27 30 28 31 comment "Audio decoders, processors and mixers" 29 32
+4 -1
drivers/media/spi/Kconfig
··· 1 1 # SPDX-License-Identifier: GPL-2.0-only 2 2 if VIDEO_V4L2 3 3 4 + comment "SPI drivers hidden by 'Autoselect ancillary drivers'" 5 + depends on MEDIA_HIDE_ANCILLARY_SUBDRV 6 + 4 7 menu "SPI helper chips" 5 - visible if !MEDIA_SUBDRV_AUTOSELECT || COMPILE_TEST || EXPERT 8 + visible if !MEDIA_HIDE_ANCILLARY_SUBDRV 6 9 7 10 config VIDEO_GS1662 8 11 tristate "Gennum Serializers video"
+5 -1
drivers/media/tuners/Kconfig
··· 15 15 select MEDIA_TUNER_TDA9887 if MEDIA_SUBDRV_AUTOSELECT 16 16 select MEDIA_TUNER_MC44S803 if MEDIA_SUBDRV_AUTOSELECT 17 17 18 + comment "Tuner drivers hidden by 'Autoselect ancillary drivers'" 19 + depends on MEDIA_HIDE_ANCILLARY_SUBDRV 20 + depends on MEDIA_ANALOG_TV_SUPPORT || MEDIA_DIGITAL_TV_SUPPORT || MEDIA_RADIO_SUPPORT || MEDIA_SDR_SUPPORT 21 + 18 22 menu "Customize TV tuners" 19 - visible if !MEDIA_SUBDRV_AUTOSELECT || COMPILE_TEST || EXPERT 23 + visible if !MEDIA_HIDE_ANCILLARY_SUBDRV 20 24 depends on MEDIA_ANALOG_TV_SUPPORT || MEDIA_DIGITAL_TV_SUPPORT || MEDIA_RADIO_SUPPORT || MEDIA_SDR_SUPPORT 21 25 22 26 config MEDIA_TUNER_SIMPLE