From: Mauro Carvalho Chehab <mchehab@kernel.org>
To: Ezequiel Garcia <ezequiel@collabora.com>
Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org,
Tomasz Figa <tfiga@chromium.org>,
Nicolas Dufresne <nicolas@ndufresne.ca>,
kernel@collabora.com, Hans Verkuil <hverkuil@xs4all.nl>,
Sean Young <sean@mess.org>,
Philipp Zabel <p.zabel@pengutronix.de>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Sakari Ailus <sakari.ailus@iki.fi>,
Michael Ira Krufky <mkrufky@gmail.com>,
Helen Koike <helen.koike@collabora.com>
Subject: Re: [PATCH 2/2] media: Remove VIDEO_DEV unneeded dependency
Date: Tue, 24 Mar 2020 10:47:22 +0100 [thread overview]
Message-ID: <20200324104722.1c605792@coco.lan> (raw)
In-Reply-To: <20200323211022.28297-3-ezequiel@collabora.com>
Em Mon, 23 Mar 2020 18:10:22 -0300
Ezequiel Garcia <ezequiel@collabora.com> escreveu:
> Enable VIDEO_DEV (which compiles Video4Linux core)
> when MEDIA_SUPPORT is selected. This is needed, in order
> to be able to enable devices such as stateless codecs.
>
> Signed-off-by: Ezequiel Garcia <ezequiel@collabora.com>
> ---
> drivers/media/Kconfig | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/media/Kconfig b/drivers/media/Kconfig
> index b36a41332867..7de472ad07a2 100644
> --- a/drivers/media/Kconfig
> +++ b/drivers/media/Kconfig
> @@ -93,13 +93,11 @@ source "drivers/media/mc/Kconfig"
>
> #
> # 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 || MEDIA_SDR_SUPPORT
> default y
>
> config VIDEO_V4L2_SUBDEV_API
The rationale of the above is to exclude Digital TV and remote
controller.
Removing the above will make the V4L2 core available every time, even
if all the user wants is remote controller or Digital TV support.
I'm working on a patchset that should hopefully address the issues
you're concerning.
Thanks,
Mauro
next prev parent reply other threads:[~2020-03-24 9:47 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-23 21:10 [PATCH 0/2] media: Clean up stateless codec configuration Ezequiel Garcia
2020-03-23 21:10 ` [PATCH 1/2] media: Allow to enable the media controller API unconditionally Ezequiel Garcia
2020-03-23 21:10 ` [PATCH 2/2] media: Remove VIDEO_DEV unneeded dependency Ezequiel Garcia
2020-03-24 9:47 ` Mauro Carvalho Chehab [this message]
2020-03-24 10:18 ` Ezequiel Garcia
2020-03-24 10:41 ` Mauro Carvalho Chehab
2020-03-27 13:47 ` Ezequiel Garcia
2020-03-27 14:10 ` Mauro Carvalho Chehab
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200324104722.1c605792@coco.lan \
--to=mchehab@kernel.org \
--cc=ezequiel@collabora.com \
--cc=helen.koike@collabora.com \
--cc=hverkuil@xs4all.nl \
--cc=kernel@collabora.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mkrufky@gmail.com \
--cc=nicolas@ndufresne.ca \
--cc=p.zabel@pengutronix.de \
--cc=sakari.ailus@iki.fi \
--cc=sean@mess.org \
--cc=tfiga@chromium.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).