[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200324135359.GA21251@pendragon.ideasonboard.com>
Date: Tue, 24 Mar 2020 15:53:59 +0200
From: Laurent Pinchart <laurent.pinchart@...asonboard.com>
To: Mauro Carvalho Chehab <mchehab+huawei@...nel.org>
Cc: Linux Doc Mailing List <linux-doc@...r.kernel.org>,
linux-kernel@...r.kernel.org, Jonathan Corbet <corbet@....net>,
Ezequiel Garcia <ezequiel@...labora.com>,
Hans Verkuil <hverkuil-cisco@...all.nl>,
Sakari Ailus <sakari.ailus@...ux.intel.com>,
Shuah Khan <skhan@...uxfoundation.org>,
Stefan Richter <stefanr@...6.in-berlin.de>,
linux1394-devel@...ts.sourceforge.net,
Helen Koike <helen.koike@...labora.com>
Subject: Re: [PATCH v2 00/20] Reorganize media Kconfig
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
Powered by blists - more mailing lists