From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.6 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id F1C0EC43331 for ; Tue, 24 Mar 2020 12:04:55 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id C67BA20714 for ; Tue, 24 Mar 2020 12:04:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1585051495; bh=m2SzPjFv0o6GRub8vX3jUre02/83adkpAzjNbd674pw=; h=Date:From:To:Cc:Subject:In-Reply-To:References:List-ID:From; b=a34oOImX/DJZl3Q+neLmoHMCUXB4TfGbyHisp1imDYsMmLSrEaheHxpqTbccMls3F etLa3k8LfXcKvGO9p5l0Sfj/o541O8fxTtOBwwKt0v6eBTkFN0G3xvve3I0eaXaHJK +X1s3DiSkZpA2L2fSDbMvoOiK75fwforLH83tKSg= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727416AbgCXMEz (ORCPT ); Tue, 24 Mar 2020 08:04:55 -0400 Received: from mail.kernel.org ([198.145.29.99]:53504 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727400AbgCXMEz (ORCPT ); Tue, 24 Mar 2020 08:04:55 -0400 Received: from coco.lan (ip5f5ad4e9.dynamic.kabel-deutschland.de [95.90.212.233]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id B950420714; Tue, 24 Mar 2020 12:04:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1585051494; bh=m2SzPjFv0o6GRub8vX3jUre02/83adkpAzjNbd674pw=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=JsoQKqoQmgU3EnXC0HQ6BRo5deSce9WNQ2jASsFmTj1Oy71wjvBjQiBNrggIwGmPh D72ZKgFak0rRbDaDZ5LOGJ4ul/lorGmCl4GFsPh51pBw+C9SCFjYN0QTajPlzn6Nv/ sUH2y/TwFEcXun8Agxetv27MlG9+F92xvuF+ohIE= Date: Tue, 24 Mar 2020 13:04:50 +0100 From: Mauro Carvalho Chehab To: Linux Doc Mailing List Cc: linux-kernel@vger.kernel.org, Jonathan Corbet Subject: Re: [PATCH] media: Kconfig: make filtering devices optional Message-ID: <20200324130450.4d3341e4@coco.lan> In-Reply-To: <751a1be8d03e1c30d5af17cfd659ffc6a5721a35.1585051247.git.mchehab+huawei@kernel.org> References: <751a1be8d03e1c30d5af17cfd659ffc6a5721a35.1585051247.git.mchehab+huawei@kernel.org> X-Mailer: Claws Mail 3.17.5 (GTK+ 2.24.32; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-doc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-doc@vger.kernel.org Em Tue, 24 Mar 2020 13:01:21 +0100 Mauro Carvalho Chehab escreveu: > The per-device option selection is a feature that some > developers love, while others hate... > > So, let's make both happy by making it optional. In time: title here should be [PATCH v2]. > > Signed-off-by: Mauro Carvalho Chehab > --- > > v2: simplify the Kconfig options by using a visible menu > > drivers/media/Kconfig | 32 +++++++++++++++++++++++++++----- > 1 file changed, 27 insertions(+), 5 deletions(-) > > diff --git a/drivers/media/Kconfig b/drivers/media/Kconfig > index 4c06728a4ab7..e372029ac41f 100644 > --- a/drivers/media/Kconfig > +++ b/drivers/media/Kconfig > @@ -25,14 +25,32 @@ menuconfig MEDIA_SUPPORT > Additional info and docs are available on the web at > > > -menu "Types of devices to be supported" > +if MEDIA_SUPPORT > + > +config MEDIA_SUPPORT_FILTER > + bool "Filter devices by their types" > depends on MEDIA_SUPPORT > + help > + Configuring the media subsystem can be complex, as there are > + hundreds of drivers and other config options. > + > + This menu offers option that will help the Kernel's config > + system to hide drivers that are out of the scope of the > + user needs, and disabling core support for unused APIs. > + > + If not selected, all non-optional media core functionality > + needed to support media drivers will be enabled. Also, all > + media device drivers should be shown. > + > +menu "Media device types" > + visible if MEDIA_SUPPORT_FILTER > > # > # Multimedia support - automatically enable V4L2 and DVB core > # > config MEDIA_CAMERA_SUPPORT > bool "Cameras and video grabbers" > + default y if !MEDIA_SUPPORT_FILTER > help > Enable support for webcams and video grabbers. > > @@ -40,6 +58,7 @@ config MEDIA_CAMERA_SUPPORT > > config MEDIA_ANALOG_TV_SUPPORT > bool "Analog TV" > + default y if !MEDIA_SUPPORT_FILTER > help > Enable analog TV support. > > @@ -52,6 +71,7 @@ config MEDIA_ANALOG_TV_SUPPORT > > config MEDIA_DIGITAL_TV_SUPPORT > bool "Digital TV" > + default y if !MEDIA_SUPPORT_FILTER > help > Enable digital TV support. > > @@ -60,6 +80,7 @@ config MEDIA_DIGITAL_TV_SUPPORT > > config MEDIA_RADIO_SUPPORT > bool "AM/FM radio receivers/transmitters" > + default y if !MEDIA_SUPPORT_FILTER > help > Enable AM/FM radio support. > > @@ -74,6 +95,7 @@ config MEDIA_RADIO_SUPPORT > > config MEDIA_SDR_SUPPORT > bool "Software defined radio" > + default y if !MEDIA_SUPPORT_FILTER > help > Enable software defined radio support. > > @@ -81,6 +103,7 @@ config MEDIA_SDR_SUPPORT > > config MEDIA_CEC_SUPPORT > bool "HDMI CEC support" > + default y if !MEDIA_SUPPORT_FILTER > help > Enable support for HDMI CEC (Consumer Electronics Control), > which is an optional HDMI feature. > @@ -90,6 +113,7 @@ config MEDIA_CEC_SUPPORT > > config MEDIA_EMBEDDED_SUPPORT > bool "Embedded devices (SoC)" > + default y if !MEDIA_SUPPORT_FILTER > help > Enable support for complex cameras, codecs, and other hardware > found on Embedded hardware (SoC). > @@ -98,6 +122,7 @@ config MEDIA_EMBEDDED_SUPPORT > > config MEDIA_TEST_SUPPORT > bool "Test drivers" > + default y if !MEDIA_SUPPORT_FILTER > help > Those drivers should not be used on production Kernels, but > can be useful on debug ones. It enables several dummy drivers > @@ -106,10 +131,7 @@ config MEDIA_TEST_SUPPORT > have regressions. > > Say Y when you have a software defined radio device. > - > -endmenu # media support types > - > -if MEDIA_SUPPORT > +endmenu # media device types > > comment "Media core options" > Thanks, Mauro