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 02EDEC433EF for ; Sun, 19 Jun 2022 00:36:27 +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-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=iw8uFKw8LbF+aq6m0lIZeYT76VAh9e//J3rKTSvSgnU=; b=Xc865W36/4YaDC D3/Hoe4vAcoWZRRyUjJ9ar85TMVxChFk+UncyJFKWta0EJtMSCv4ViaBwg1Z2dmj6lLGxuXgfhtaR vE6CGqmS0sMHeutzAZR2EQa5D9kISXUQU3qhsyWTfpE2pF21KDVa0X7LjKBeZevCoPeIckuHtBOs5 He4y3zoR1tJaisUP8bizKT0NqecPcEehZAkOJ2DR+XxHjTre2ee2KbTEJ6KWx9KpfPfv8HSrQ4pS3 QB+34HymPVQm49r8jFUvQutA0rudtBY9bKfdRjO6/Vk3mNBCD14JkWXfVIgcpEis4t/rXUSX5sPIN 42tkd2ksCmlIkQF/upcw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1o2itJ-00Cjtu-UW; Sun, 19 Jun 2022 00:34:14 +0000 Received: from perceval.ideasonboard.com ([213.167.242.64]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1o2itH-00CjtO-6f for linux-arm-kernel@lists.infradead.org; Sun, 19 Jun 2022 00:34:12 +0000 Received: from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi [62.78.145.57]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 1C75D993; Sun, 19 Jun 2022 02:34:08 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1655598848; bh=CJwMmUGuykwFuJQyzvqCXHaGSd5nFKNyUK+sRtc9VGk=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=sRZy8PpTEX+A3G3UbOHcNnLYEmY1FD54xuSBX+oXu8FgfxBx13mNxsgty6MEqLXpd 3Uq5aJ81qlj2YPQnxozIEhLK6uva/mlS66ceLnzVy48VdFKjCzFhhqXW7ZQqKUffZx ZONUjfSrFOnUEmuIcDlipqbARxPaENREzCPUQnsw= Date: Sun, 19 Jun 2022 03:33:54 +0300 From: Laurent Pinchart To: Marek Vasut Cc: linux-media@vger.kernel.org, Alain Volmat , Alexandre Torgue , Amelie DELAUNAY , Hugues FRUCHET , Philippe CORNU , linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH] media: stm32: dcmi: Register V4L2 subdev nodes Message-ID: References: <20220618222335.478205-1-marex@denx.de> <4dc7b39c-35cd-663a-98f8-6034693df3c8@denx.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <4dc7b39c-35cd-663a-98f8-6034693df3c8@denx.de> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220618_173411_432141_544326D7 X-CRM114-Status: GOOD ( 15.27 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Sun, Jun 19, 2022 at 02:28:55AM +0200, Marek Vasut wrote: > On 6/19/22 01:20, Laurent Pinchart wrote: > > Hi Marek, > > > > Thank you for the patch. > > > > On Sun, Jun 19, 2022 at 12:23:35AM +0200, Marek Vasut wrote: > >> Unless the V4L2 device calls v4l2_device_register_subdev_nodes(), > >> the /dev/v4l-subdev* sub-device nodes are not registered and thus > >> not created. Add the missing call. > > > > This driver configures the subdevs internally. Exposing the subdev nodes > > to userspace would conflict with driver operation. > > > > If you need the subdev nodes only to query subdev information, you can > > use v4l2_device_register_ro_subdev_nodes(). Ideally I'd prefer this > > driver to expose subdev nodes to userspace and stop propagating formats > > internally, but that may break applications :-( > > I need those subdevs to configure e.g. test pattern on the sensor side. Doesn't the dcmi driver expose the sensor controls on the video node ? -- Regards, Laurent Pinchart _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel