Re: [PATCH v2 00/20] Reorganize media Kconfig

From: Laurent Pinchart
Date: Tue Mar 24 2020 - 09:54:06 EST

Next message: Enric Balletbo i Serra: "Re: [PATCH] platform/chrome: wilco_ec: Provide correct output format to 'h1_gpio' file"
Previous message: Geert Uytterhoeven: "[PATCH v6 0/8] gpio: Add GPIO Aggregator"
In reply to: Mauro Carvalho Chehab: "[PATCH v2 04/20] media: reorganize the drivers menu options"
Next in thread: Mauro Carvalho Chehab: "Re: [PATCH v2 00/20] Reorganize media Kconfig"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Mauro,

Thank you for the patches.

On Tue, Mar 24, 2020 at 02:42:53PM +0100, Mauro Carvalho Chehab wrote:
> This patch series do lots of reorg at the media Kconfig options.
> It also move test drivers from platform dir to a new one.
>
> After this change, the main config is organized on menus, allowing to
> select:
>
> - type of devices selection - the filtering options
> - Media core options - with API and other core stuff
> - Media core extra options
> - Media drivers
> - Media ancillary drivers
>
> The "type of devices" menu has the filtering options for:
>
> - Cameras and video grabbers
> - Analog TV
> - Digital TV
> - AM/FM radio receivers/transmitters
> - SDR
> - CEC
> - Embeded devices (SoC)
> - Test drivers
>
> This way, one interested only on embedded devices can unselect
> everything but "Embedded devices (SoC)" option.
>
> Distros for PC/Laptops can enable everything but
> "Embedded devices (SoC)" and "Test drivers".

How about a device such as the Intel IPU3 ? It's a SoC, and is present
in laptops. Unlike the physical interface which is a fairly well defined
way to categorize devices, creating artificial classes will always leave
some devices without a home. We could have a capture card that supports
both analog and digital TV. A digital TV capture card with an HDMI input
can have a CEC device. Lots of combinations are possible.

> Users can select just what they want, without bothering with
> hundreds of options that he won't have any clue about their
> meanings.
>
> Mauro Carvalho Chehab (20):
> media: dvb-usb: auto-select CYPRESS_FIRMWARE
> media: Kconfig: not all V4L2 platform drivers are for camera
> media: pci: move VIDEO_PCI_SKELETON to a different Kconfig
> media: reorganize the drivers menu options
> media: Kconfig: update the MEDIA_SUPPORT help message
> media: Kconfig: use a sub-menu to select supported devices
> media: Kconfig: add an option to filter in/out the embedded drivers
> media: split test drivers from platform directory
> media: Kconfig: make filtering devices optional
> media: Kconfig: move CEC-specific options to cec/Kconfig
> media: Kconfig: move comment to siano include
> media: Kconfig: move DVB-specific options to dvb-core/Kconfig
> media: Kconfig: move V4L2 subdev API to v4l2-core/Kconfig
> media: Kconfig: move drivers-specific TTPCI_EEPROM Kconfig var
> media: Kconfig: move media controller core select to main Kconfig
> media: Kconfig: better organize menu items
> media: Kconfig: mode firewire comment to firewire/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
>
> drivers/media/Kconfig | 216 ++++++++++--------
> drivers/media/cec/Kconfig | 10 +
> drivers/media/common/Kconfig | 2 +-
> drivers/media/dvb-core/Kconfig | 26 +++
> drivers/media/firewire/Kconfig | 5 +-
> drivers/media/mc/Kconfig | 9 -
> drivers/media/mmc/Kconfig | 1 -
> drivers/media/mmc/siano/Kconfig | 2 +
> drivers/media/pci/Kconfig | 10 +
> drivers/media/platform/Kconfig | 24 --
> drivers/media/platform/Makefile | 5 -
> 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 | 0
> .../{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 | 6 +
> drivers/media/usb/dvb-usb/Kconfig | 1 +
> drivers/media/v4l2-core/Kconfig | 19 +-
> 79 files changed, 222 insertions(+), 151 deletions(-)
> 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 (100%)
> 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%)

--
Regards,

Laurent Pinchart

Next message: Enric Balletbo i Serra: "Re: [PATCH] platform/chrome: wilco_ec: Provide correct output format to 'h1_gpio' file"
Previous message: Geert Uytterhoeven: "[PATCH v6 0/8] gpio: Add GPIO Aggregator"
In reply to: Mauro Carvalho Chehab: "[PATCH v2 04/20] media: reorganize the drivers menu options"
Next in thread: Mauro Carvalho Chehab: "Re: [PATCH v2 00/20] Reorganize media Kconfig"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]