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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id CC58DC433F5 for ; Fri, 18 Mar 2022 20:59:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:Cc:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=IOIp1338v9T4SH7C1tuHZ+YBxSaqPEHy9JJ8T9TnFv4=; b=E1Pi12zscFNIayeSCyLsclz9rs e0umq2OH9yus8DF34yS3+gvBET7l3BV42pL6VG6GfrOZBbjaFkkPGmfD7xzXGDB7kM7q3TwPA6fFn ubRVlu2xEwvT6lJKl5xuj9wTMxRqbNNHAtNFg8GNSIuiRy7snBIHtn8ZBCoWsq9B3HNk09trUX9F+ cOZcKsPncYMrB1vZ2QTUkk6rTMaXhYmYHtee8J02vKzq1h0IXTg6U7FQ45SUAt22qMWJb/ZKFxwnb bXUhCbAbT62bAXzORD4uHisemLsNur2uEQh9Taer4sh18mT1UHKXtcw/jFiIakfIk3Wzr8pJaQbsr TCR4PiKQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nVJgo-002sEH-VG; Fri, 18 Mar 2022 20:59:14 +0000 Received: from mail-io1-xd31.google.com ([2607:f8b0:4864:20::d31]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nVJgk-002sAt-MN for linux-amlogic@lists.infradead.org; Fri, 18 Mar 2022 20:59:12 +0000 Received: by mail-io1-xd31.google.com with SMTP id l18so10680887ioj.2 for ; Fri, 18 Mar 2022 13:59:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxfoundation.org; s=google; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=oNAPpR+5qx5bHO8wJ1SIgWig3LUKuwuqLo641jvFAak=; b=R3B03hXdzmRL4FMHavxss76+tb9Q0Hy78NbdGoHIgmWRsuIfciGr2JWywHMf/Kpkhf QniqntLtfhiE8E3Wo0oxTFcS1KWerJtG5tNgnrk3j2xUiX43wMMg+/mjp1wKd5A4aRIu ErIGjgvdava6sQlmLmDCO8Qr2P9RfzXvczFjE= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=oNAPpR+5qx5bHO8wJ1SIgWig3LUKuwuqLo641jvFAak=; b=uL6ACwib81dF4Y6aWx3jUwhFDUhRIbquEYKlhZPxVzK4JtY82J/AUb10Aw4nc2iswX Yg5hX979duTf/yER4RSo56mPA2gq+gueTj6U+UWgr7xyQN2rbXQziVyu5OsV57ujC4Da 4c1HW/wmN5i+EK1BRgKv2xkCtgN/LcwzzIlG4yk9ow7sHCtR7YtciMN2tZSRe+t7PuOJ GSjjCki1S0dMN1KgwjL+SQgGoBBqlIcv+AXzNsj2fpqsaTBONxBGXbydgHlxvW8G24qh 99QErizlbr4b9JbyfTa5l6QtSYJm/pcSaQpvhjRffUOomgE7y4PIr2LfxAcjUqouREj9 SFQQ== X-Gm-Message-State: AOAM532BBp6+Nin9a14HFbD3ijRRjdduUwlCC07gbFja0tElc6RpjzNJ u2zRVqv5kCxWounu5q0J1FIA2w== X-Google-Smtp-Source: ABdhPJw+g6qtXnLujFKVRv55AGvVhZcbGPndogV4eznqetPdpLwXYueCN6beIFsP8OoV/l235xaSaw== X-Received: by 2002:a5d:9542:0:b0:648:f92b:7bc6 with SMTP id a2-20020a5d9542000000b00648f92b7bc6mr5223132ios.73.1647637149068; Fri, 18 Mar 2022 13:59:09 -0700 (PDT) Received: from [192.168.1.128] ([71.205.29.0]) by smtp.gmail.com with ESMTPSA id d14-20020a056602328e00b006494aa126c2sm3528385ioz.11.2022.03.18.13.59.05 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 18 Mar 2022 13:59:08 -0700 (PDT) Subject: Re: [PATCH 64/64] media: Kconfig: cleanup VIDEO_DEV dependencies To: Mauro Carvalho Chehab Cc: Fabio Estevam , Felipe Balbi , Florian Fainelli , Geert Uytterhoeven , Giulio Benetti , Greg Kroah-Hartman , Hans Verkuil , Heiko Stuebner , Heungjun Kim , Houlong Wei , Hugues Fruchet , Hyun Kwon , Jacek Anaszewski , Jacob Chen , Jaroslav Kysela , Jean-Christophe Trotin , Jeff LaBundy , Jernej Skrabec , Jerome Brunet , Joe Hung , Joel Stanley , Jonathan Hunter , Kevin Hilman , Kieran Bingham , Krzysztof Kozlowski , Kyungmin Park , Laurent Pinchart , Ludovic Desroches , Marek Szyprowski , Marek Vasut , Martin Blumenstingl , Martina Krasteva , Matthias Brugger , Maxime Coquelin , Maxime Ripard , Michael Grzeschik , Michael Tretter , Michal Simek , Mike Isely , Ming Qian , Minghsiu Tsai , Mirela Rabulea , NXP Linux Team , Neil Armstrong , Nicolas Ferre , Nicolas Saenz Julienne , Olivier Lorin , Paul Kocialkowski , Pavel Machek , Pengutronix Kernel Team , Philipp Zabel , Randy Dunlap , Ray Jui , Rick Chang , Robert Foss , Sakari Ailus , Sascha Hauer , Scott Branden , Sean Young , Shawn Guo , Shawn Tu , Shijie Qin , Sowjanya Komatineni , Stanimir Varbanov , Steve Longerbeam , Sylwester Nawrocki , Takashi Iwai , Thierry Reding , Tianshu Qiu , Tiffany Lin , Todor Tomov , Vincent Knecht , Yong Deng , Yong Zhi , Zhou Peng , alsa-devel@alsa-project.org, bcm-kernel-feedback-list@broadcom.com, linux-amlogic@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-aspeed@lists.ozlabs.org, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-renesas-soc@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-rpi-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-staging@lists.linux.dev, linux-stm32@st-md-mailman.stormreply.com, linux-sunxi@lists.linux.dev, linux-tegra@vger.kernel.org, linux-usb@vger.kernel.org, mjpeg-users@lists.sourceforge.net, openbmc@lists.ozlabs.org, Shuah Khan References: From: Shuah Khan Message-ID: <9e02d88a-3344-8d91-c652-72cb989506de@linuxfoundation.org> Date: Fri, 18 Mar 2022 14:59:05 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.8.1 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220318_135910_809365_A9FAF0CF X-CRM114-Status: GOOD ( 22.89 ) X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-amlogic" Errors-To: linux-amlogic-bounces+linux-amlogic=archiver.kernel.org@lists.infradead.org On 3/14/22 1:55 AM, Mauro Carvalho Chehab wrote: > media Kconfig has two entries associated to V4L API: > VIDEO_DEV and VIDEO_V4L2. > > On Kernel 2.6.x, there were two V4L APIs, each one with its own flag. > VIDEO_DEV were meant to: > 1) enable Video4Linux and make its Kconfig options to appear; > 2) it makes the Kernel build the V4L core. > > while VIDEO_V4L2 where used to distinguish between drivers that > implement the newer API and drivers that implemented the former one. > > With time, such meaning changed, specially after the removal of > all V4L version 1 drivers. > > At the current implementation, VIDEO_DEV only does (1): it enables > the media options related to V4L, that now has: > > menu "Video4Linux options" > visible if VIDEO_DEV > > source "drivers/media/v4l2-core/Kconfig" > endmenu > > but it doesn't affect anymore the V4L core drivers. > > The rationale is that the V4L2 core has a "soft" dependency > at the I2C bus, and now requires to select a number of other > Kconfig options: > > config VIDEO_V4L2 > tristate > depends on (I2C || I2C=n) && VIDEO_DEV > select RATIONAL > select VIDEOBUF2_V4L2 if VIDEOBUF2_CORE > default (I2C || I2C=n) && VIDEO_DEV > > In the past, merging them would be tricky, but it seems that it is now > possible to merge those symbols, in order to simplify V4L dependencies. > > Let's keep VIDEO_DEV, as this one is used on some make *defconfig > configurations. > > Suggested-by: Laurent Pinchart > Reviewed-by: Laurent Pinchart > Signed-off-by: Mauro Carvalho Chehab > --- > > drivers/media/test-drivers/vicodec/Kconfig | 2 +- > drivers/media/test-drivers/vimc/Kconfig | 2 +- > drivers/media/test-drivers/vivid/Kconfig | 2 +- > For vimc change: (trimmed the recipient list to send response) Acked-by: Shuah Khan thanks, -- Shuah _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic