Media Kconfig menu reorg and improvements
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v4 Latest33 patches 6 repliesIn review
- v322 patches 6 repliesSuperseded
- v2 posting 2/220 patches 2 repliesSuperseded
Earlier versions · 2
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v2 posting 1/2 | 20 patches | 5 replies | Superseded | |
| v1 | 8 patches | 0 replies | Superseded |
Patches in v4 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/33 | media: dvb-usb: auto-select CYPRESS_FIRMWAREdrivers/ drivers/ | +2 | 0 replies | |
| 2/33 | media: Kconfig: add an option to filter in/out platform driversdrivers/ | +16 | 0 replies | |
| 3/33 | media: Kconfig: not all V4L2 platform drivers are for cameradrivers/ drivers/ | +1 | 0 replies | |
| 4/33 | media: pci: move VIDEO_PCI_SKELETON to a different Kconfigdrivers/ drivers/ | +10 | 0 replies | |
| 5/33 | media: Kconfig: update the MEDIA_SUPPORT help messagedrivers/ | +4 | 0 replies | |
| 6/33 | media: split test drivers from platform directorydrivers/ drivers/ drivers/ +65 more | +51 | 0 replies | |
| 7/33 | media: Kconfig: move comment to siano includedrivers/ drivers/ | +2 | 0 replies | |
| 8/33 | media: Kconfig: move drivers-specific TTPCI_EEPROM Kconfig vardrivers/ drivers/ | +6 | 0 replies | |
| 9/33 | media: Kconfig: mode firewire comment to firewire/Kconfigdrivers/ drivers/ | +4 | 0 replies | |
| 10/33 | media: ddbridge: copy the dvb_dummy_fe driver to ddbridgedrivers/ drivers/ | +322 | 0 replies | |
| 11/33 | media: ddbridge-dummy_fe: do some vars and function renamesdrivers/ drivers/ | +83 | 0 replies | |
| 12/33 | media: ddbridge: use the ddbridge's own dummy fe driverdrivers/ drivers/ drivers/ +2 more | +3 | 0 replies | |
| 13/33 | media: Kconfig: mark other drivers as test driversdrivers/ drivers/ | +5 | 0 replies | |
| 14/33 | media: Kconfig: simplify some dependenciesdrivers/ | +0 | 0 replies | |
| 15/33 | media: Kconfig: better support hybrid TV devicesdrivers/ drivers/ drivers/ +11 more | +46 | 0 replies | |
| 16/33 | media: Kconfig: fix selection for test driversdrivers/ drivers/ drivers/ +1 more | +14 | 0 replies | |
| 17/33 | media: add SPDX headers on Kconfig and Makefile filesdrivers/ drivers/ drivers/ +5 more | +16 | 6 replies | |
| 18/33 | media: dvb-core: Kconfig: default to use dynamic minorsdrivers/ | +1 | 0 replies | |
| 19/33 | media: Kconfig files: use select for V4L2 subdevs and MCdrivers/ drivers/ drivers/ +22 more | +237 | 0 replies | |
| 20/33 | media: Kconfig: reorganize the drivers menu optionsdrivers/ | +7 | 0 replies | |
| 21/33 | media: Kconfig: use a sub-menu to select supported devicesdrivers/ | +14 | 0 replies | |
| 22/33 | media: Kconfig: make filtering devices optionaldrivers/ | +27 | 0 replies | |
| 23/33 | media: Kconfig: warn if drivers are filtereddrivers/ | +4 | 0 replies | |
| 24/33 | media: Kconfig: move CEC-specific options to cec/Kconfigdrivers/ drivers/ | +10 | 0 replies | |
| 25/33 | media: Kconfig: move DVB-specific options to dvb-core/Kconfigdrivers/ drivers/ | +26 | 0 replies | |
| 26/33 | media: Kconfig: move V4L2 subdev API to v4l2-core/Kconfigdrivers/ drivers/ | +9 | 0 replies | |
| 27/33 | media: Kconfig: move media controller core select to main Kconfigdrivers/ drivers/ | +9 | 0 replies | |
| 28/33 | media: Kconfig: place all options under a sub-menudrivers/ | +18 | 0 replies | |
| 29/33 | media: Kconfig: move the position of sub-driver autoselectiondrivers/ | +25 | 0 replies | |
| 30/33 | media: Kconfig: on !EMBEDDED && !EXPERT, enable driver filteringdrivers/ | +1 | 0 replies | |
| 31/33 | media: Kconfig: Better organize the per-API optionsdrivers/ | +18 | 0 replies | |
| 32/33 | media: i2c/Kconfig: reorganize items theredrivers/ | +107 | 0 replies | |
| 33/33 | media: Kconfig: don't use visible for device type selectdrivers/ | +16 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v4 yet.
Cover letter [PATCH v4 00/33] Media Kconfig menu reorg and improvements
The media Kconfig menu is complex. From time to time, it requires some improvements.
This series is organized on two parts:
1. Patches 1 to 19: cleanups and improvements;
- Try to be smarter when selecting features. So, auto-select Cypress firmware support on drivers that use it. Also, auto-select Media Controller and V4L2 subdevice API for devices that won't work without such features.
Please notice that some drivers have can work optionally with those features enabled (or not). For those, users should still explicitly enable such features.
- Platform and test drivers have now an option at the device support selection;
- The test drivers are not platform drivers. They deserve their own directory. When a test driver is selected, the needed API support will be automatically selected.
- Do some Kconfig reorg, in order to keep the main media Kconfig as clean as possible.
- The help messafe for media support was outdated;
- The ddbridge driver was abusing at the dvb dummy frontend driver (meant to be just an example). It will now have its own internal dummy driver, as the original one will soon receive more code and become a virtual driver.
- There's a longstanding issue with hybrid TV devices: for an hybrid device to work, *both* analog and digital TV would need to be selected. As the Kconfig language improved, we can now address it. So, selecting either analog or digital TV support should display all hybrid devices. When an hybrid driver is selected, both V4L2 and DVB core will be available.
2. Patches 20 to 33: visual changes.
- At the beginning, media had just "technical" options to enable/disable each feature. However, that leads into hundreds of different options, being a nightmare for a mortal to set it up.
So, the Kconfig menu options were re-designed in order to bring a more user-oriented view:
The initial items select the type of devices (analog TV, digital TV, cameras, ...). Selecting it would auto-select the core drivers. The next step would be to select either PCI or USB, and the needed driver.
While this makes life simpler, now there are many embedded developers. Most seem to prefer to be able of seeing everything without filters.
The main focus of this part is to make everybody
happy: it is now possible to unselect the filters, making all options visible. Also, now, when the filter is enabled, the media drivers menu will show a message warning that the filter is active.
Hopefully, this should make both embedded developers and normal users happy.
- The media menus now have a more modern visual, grouping options in a way that it is now clearer about what they do.
So, the Media support menu is now:
--- Media support [ ] Filter media drivers [ ] Autoselect ancillary drivers Media device types ---> Media core support ---> Video4Linux options ---> Media controller options ---> Digital TV options ---> HDMI CEC options ---> Media drivers ---> Media ancillary drivers --->
Mauro Carvalho Chehab (33):
media: dvb-usb: auto-select CYPRESS_FIRMWARE
media: Kconfig: add an option to filter in/out platform drivers
media: Kconfig: not all V4L2 platform drivers are for camera
media: pci: move VIDEO_PCI_SKELETON to a different Kconfig
media: Kconfig: update the MEDIA_SUPPORT help message
media: split test drivers from platform directory
media: Kconfig: move comment to siano include
media: Kconfig: move drivers-specific TTPCI_EEPROM Kconfig var
media: Kconfig: mode firewire comment to firewire/Kconfig
media: ddbridge: copy the dvb_dummy_fe driver to ddbridge
media: ddbridge-dummy_fe: do some vars and function renames
media: ddbridge: use the ddbridge's own dummy fe driver
media: Kconfig: mark other drivers as test drivers
media: Kconfig: simplify some dependencies
media: Kconfig: better support hybrid TV devices
media: Kconfig: fix selection for test drivers
media: add SPDX headers on Kconfig and Makefile files
media: dvb-core: Kconfig: default to use dynamic minors
media: Kconfig files: use select for V4L2 subdevs and MC
media: Kconfig: reorganize the drivers menu options
media: Kconfig: use a sub-menu to select supported devices
media: Kconfig: make filtering devices optional
media: Kconfig: warn if drivers are filtered
media: Kconfig: move CEC-specific options to cec/Kconfig
media: Kconfig: move DVB-specific options to dvb-core/Kconfig
media: Kconfig: move V4L2 subdev API to v4l2-core/Kconfig
media: Kconfig: move media controller core select to main Kconfig
media: Kconfig: place all options under a sub-menu
media: Kconfig: move the position of sub-driver autoselection
media: Kconfig: on !EMBEDDED && !EXPERT, enable driver filtering
media: Kconfig: Better organize the per-API options
media: i2c/Kconfig: reorganize items there
media: Kconfig: don't use visible for device type select
drivers/media/Kconfig | 262 ++++++-----
drivers/media/cec/Kconfig | 10 +
drivers/media/common/Kconfig | 2 +-
drivers/media/dvb-core/Kconfig | 27 ++
drivers/media/dvb-frontends/Kconfig | 15 +-
drivers/media/firewire/Kconfig | 5 +-
drivers/media/i2c/Kconfig | 406 +++++++++++-------
drivers/media/i2c/et8ek8/Kconfig | 4 +-
drivers/media/i2c/m5mols/Kconfig | 5 +-
drivers/media/i2c/smiapp/Kconfig | 5 +-
drivers/media/mc/Kconfig | 11 +-
drivers/media/mmc/Kconfig | 1 -
drivers/media/mmc/siano/Kconfig | 2 +
drivers/media/pci/Kconfig | 24 +-
drivers/media/pci/bt8xx/Kconfig | 5 +-
drivers/media/pci/cobalt/Kconfig | 4 +-
drivers/media/pci/cx18/Kconfig | 2 +-
drivers/media/pci/cx23885/Kconfig | 4 +-
drivers/media/pci/cx88/Kconfig | 4 +-
drivers/media/pci/ddbridge/Kconfig | 1 -
drivers/media/pci/ddbridge/Makefile | 2 +-
drivers/media/pci/ddbridge/ddbridge-core.c | 4 +-
.../media/pci/ddbridge/ddbridge-dummy-fe.c | 153 +++++++
.../media/pci/ddbridge/ddbridge-dummy-fe.h | 16 +
drivers/media/pci/intel/ipu3/Kconfig | 4 +-
drivers/media/pci/saa7134/Kconfig | 4 +-
drivers/media/pci/saa7164/Kconfig | 2 +-
drivers/media/pci/sta2x11/Kconfig | 6 +-
drivers/media/platform/Kconfig | 56 +--
drivers/media/platform/Makefile | 5 -
drivers/media/platform/am437x/Kconfig | 4 +-
drivers/media/platform/atmel/Kconfig | 4 +-
drivers/media/platform/cadence/Kconfig | 8 +-
drivers/media/platform/exynos4-is/Kconfig | 5 +-
drivers/media/platform/rcar-vin/Kconfig | 8 +-
drivers/media/platform/sunxi/Kconfig | 2 +
drivers/media/platform/sunxi/Makefile | 2 +
.../media/platform/sunxi/sun4i-csi/Kconfig | 6 +-
.../media/platform/sunxi/sun4i-csi/Makefile | 2 +
.../media/platform/sunxi/sun6i-csi/Kconfig | 4 +-
drivers/media/platform/xilinx/Kconfig | 4 +-
drivers/media/spi/Kconfig | 4 +-
drivers/media/test_drivers/Kconfig | 28 ++
drivers/media/test_drivers/Makefile | 9 +
.../vicodec/Kconfig | 0
.../vicodec/Makefile | 0
.../vicodec/codec-fwht.c | 0
.../vicodec/codec-fwht.h | 0
.../vicodec/codec-v4l2-fwht.c | 0
.../vicodec/codec-v4l2-fwht.h | 0
.../vicodec/vicodec-core.c | 0
.../media/{platform => test_drivers}/vim2m.c | 0
.../{platform => test_drivers}/vimc/Kconfig | 4 +-
.../{platform => test_drivers}/vimc/Makefile | 0
.../vimc/vimc-capture.c | 0
.../vimc/vimc-common.c | 0
.../vimc/vimc-common.h | 0
.../vimc/vimc-core.c | 0
.../vimc/vimc-debayer.c | 0
.../vimc/vimc-scaler.c | 0
.../vimc/vimc-sensor.c | 0
.../vimc/vimc-streamer.c | 0
.../vimc/vimc-streamer.h | 0
.../{platform => test_drivers}/vivid/Kconfig | 0
.../{platform => test_drivers}/vivid/Makefile | 0
.../vivid/vivid-cec.c | 0
.../vivid/vivid-cec.h | 0
.../vivid/vivid-core.c | 0
.../vivid/vivid-core.h | 0
.../vivid/vivid-ctrls.c | 0
.../vivid/vivid-ctrls.h | 0
.../vivid/vivid-kthread-cap.c | 0
.../vivid/vivid-kthread-cap.h | 0
.../vivid/vivid-kthread-out.c | 0
.../vivid/vivid-kthread-out.h | 0
.../vivid/vivid-kthread-touch.c | 0
.../vivid/vivid-kthread-touch.h | 0
.../vivid/vivid-meta-cap.c | 0
.../vivid/vivid-meta-cap.h | 0
.../vivid/vivid-meta-out.c | 0
.../vivid/vivid-meta-out.h | 0
.../vivid/vivid-osd.c | 0
.../vivid/vivid-osd.h | 0
.../vivid/vivid-radio-common.c | 0
.../vivid/vivid-radio-common.h | 0
.../vivid/vivid-radio-rx.c | 0
.../vivid/vivid-radio-rx.h | 0
.../vivid/vivid-radio-tx.c | 0
.../vivid/vivid-radio-tx.h | 0
.../vivid/vivid-rds-gen.c | 0
.../vivid/vivid-rds-gen.h | 0
.../vivid/vivid-sdr-cap.c | 0
.../vivid/vivid-sdr-cap.h | 0
.../vivid/vivid-touch-cap.c | 0
.../vivid/vivid-touch-cap.h | 0
.../vivid/vivid-vbi-cap.c | 0
.../vivid/vivid-vbi-cap.h | 0
.../vivid/vivid-vbi-gen.c | 0
.../vivid/vivid-vbi-gen.h | 0
.../vivid/vivid-vbi-out.c | 0
.../vivid/vivid-vbi-out.h | 0
.../vivid/vivid-vid-cap.c | 0
.../vivid/vivid-vid-cap.h | 0
.../vivid/vivid-vid-common.c | 0
.../vivid/vivid-vid-common.h | 0
.../vivid/vivid-vid-out.c | 0
.../vivid/vivid-vid-out.h | 0
drivers/media/usb/Kconfig | 14 +-
drivers/media/usb/au0828/Kconfig | 6 +-
drivers/media/usb/cx231xx/Kconfig | 4 +-
drivers/media/usb/dvb-usb/Kconfig | 1 +
drivers/media/usb/pvrusb2/Kconfig | 4 +-
drivers/media/usb/tm6000/Kconfig | 4 +-
drivers/media/v4l2-core/Kconfig | 19 +-
drivers/staging/media/hantro/Kconfig | 5 +-
drivers/staging/media/hantro/Makefile | 2 +
drivers/staging/media/imx/Kconfig | 5 +-
drivers/staging/media/ipu3/Kconfig | 3 +-
drivers/staging/media/omap4iss/Kconfig | 4 +-
drivers/staging/media/rkisp1/Kconfig | 4 +-
drivers/staging/media/rkisp1/Makefile | 2 +
drivers/staging/media/sunxi/cedrus/Kconfig | 5 +-
122 files changed, 829 insertions(+), 398 deletions(-)
create mode 100644 drivers/media/pci/ddbridge/ddbridge-dummy-fe.c
create mode 100644 drivers/media/pci/ddbridge/ddbridge-dummy-fe.h
create mode 100644 drivers/media/test_drivers/Kconfig
create mode 100644 drivers/media/test_drivers/Makefile
rename drivers/media/{platform => test_drivers}/vicodec/Kconfig (100%)
rename drivers/media/{platform => test_drivers}/vicodec/Makefile (100%)
rename drivers/media/{platform => test_drivers}/vicodec/codec-fwht.c (100%)
rename drivers/media/{platform => test_drivers}/vicodec/codec-fwht.h (100%)
rename drivers/media/{platform => test_drivers}/vicodec/codec-v4l2-fwht.c (100%)
rename drivers/media/{platform => test_drivers}/vicodec/codec-v4l2-fwht.h (100%)
rename drivers/media/{platform => test_drivers}/vicodec/vicodec-core.c (100%)
rename drivers/media/{platform => test_drivers}/vim2m.c (100%)
rename drivers/media/{platform => test_drivers}/vimc/Kconfig (83%)
rename drivers/media/{platform => test_drivers}/vimc/Makefile (100%)
rename drivers/media/{platform => test_drivers}/vimc/vimc-capture.c (100%)
rename drivers/media/{platform => test_drivers}/vimc/vimc-common.c (100%)
rename drivers/media/{platform => test_drivers}/vimc/vimc-common.h (100%)
rename drivers/media/{platform => test_drivers}/vimc/vimc-core.c (100%)
rename drivers/media/{platform => test_drivers}/vimc/vimc-debayer.c (100%)
rename drivers/media/{platform => test_drivers}/vimc/vimc-scaler.c (100%)
rename drivers/media/{platform => test_drivers}/vimc/vimc-sensor.c (100%)
rename drivers/media/{platform => test_drivers}/vimc/vimc-streamer.c (100%)
rename drivers/media/{platform => test_drivers}/vimc/vimc-streamer.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/Kconfig (100%)
rename drivers/media/{platform => test_drivers}/vivid/Makefile (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-cec.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-cec.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-core.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-core.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-ctrls.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-ctrls.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-kthread-cap.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-kthread-cap.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-kthread-out.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-kthread-out.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-kthread-touch.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-kthread-touch.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-meta-cap.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-meta-cap.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-meta-out.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-meta-out.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-osd.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-osd.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-radio-common.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-radio-common.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-radio-rx.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-radio-rx.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-radio-tx.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-radio-tx.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-rds-gen.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-rds-gen.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-sdr-cap.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-sdr-cap.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-touch-cap.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-touch-cap.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-vbi-cap.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-vbi-cap.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-vbi-gen.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-vbi-gen.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-vbi-out.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-vbi-out.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-vid-cap.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-vid-cap.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-vid-common.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-vid-common.h (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-vid-out.c (100%)
rename drivers/media/{platform => test_drivers}/vivid/vivid-vid-out.h (100%)Signature · 1 line
2.25.1