From patchwork Thu Jan 27 11:18:02 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alain Volmat X-Patchwork-Id: 12726595 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 29D7DC433F5 for ; Thu, 27 Jan 2022 11:21:24 +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:MIME-Version:Message-ID:Date:Subject:CC :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=old1IayXxdc/N3lWAtUIbKD2endyH7sXaENG+N2qP9M=; b=YVvGvkqKmD171J bP4oFu018cjugNzxPrfsmyz9b3M+JrUh8ocfc9GuBCWwJ7DIhJn46qcWKSoDAfl5yixUshH982sCX jqWw29ULY4JbF+tjnklyGjVKdmv0E9HOznjC2jgy1NAg8MB1FrBHTrtyFkMHz337A+eLgaLHIBqAN mB/JE8Sb36ocLy2p99UzHiIIL/Md6+KJuh28w1Dltcr3hYeJce6eH5fbfZvn13QhlKMcoyOij/YHG u9a4JCeHSbDFybvLo4H7klabnQ8ek8KDCKil6W4b4CISWpTVxeRyQ5Vljt2sEELoTaVHRvSwOn/9m fKUz0yhl3X99tMNAQAXQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nD2ol-00FO7I-9Y; Thu, 27 Jan 2022 11:19:56 +0000 Received: from mx07-00178001.pphosted.com ([185.132.182.106]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nD2nB-00FNLb-BT for linux-arm-kernel@lists.infradead.org; Thu, 27 Jan 2022 11:18:19 +0000 Received: from pps.filterd (m0288072.ppops.net [127.0.0.1]) by mx07-00178001.pphosted.com (8.16.1.2/8.16.1.2) with ESMTP id 20R9KFG7032351; Thu, 27 Jan 2022 12:18:05 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=foss.st.com; h=from : to : cc : subject : date : message-id : mime-version : content-transfer-encoding : content-type; s=selector1; bh=Mhu35WwCME0IjYf3oxN7vJ4sN+99NzW1K4iP2kRp7m8=; b=o/IrUSr9QZssrAOVhm2r5XCi1N+a+lVLZrcBEMUl46J0tbYfXmCT69OKA3mBCpdEv070 GqLaOItkZwG8DhEF7xVI02PFRAUVNgUqPenLJ2ywSzfb2yV1y7WG0B7GarwV/2EeCX8u 7UkKgWtrymQWoZbU7D94fV7PYGmaCztH2odhROD6/EV5/Qkoy0xdxUx5S3/QwGtLikiK U0HG6UuhnZXdS0uKVFGFIHGQDSLKGCD42X9z4KH8imhTdsfeIFg0JYUIze5MAT2edkgt 9W9gteNiLu2gmrUiLt+SlTZkPPWO0znEqJvfnXW0jxyiFVfYuH5xcr0MzhDUQPcboCX8 sw== Received: from beta.dmz-eu.st.com (beta.dmz-eu.st.com [164.129.1.35]) by mx07-00178001.pphosted.com (PPS) with ESMTPS id 3durp30n2s-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 27 Jan 2022 12:18:05 +0100 Received: from euls16034.sgp.st.com (euls16034.sgp.st.com [10.75.44.20]) by beta.dmz-eu.st.com (STMicroelectronics) with ESMTP id 768D410002A; Thu, 27 Jan 2022 12:18:03 +0100 (CET) Received: from Webmail-eu.st.com (sfhdag2node2.st.com [10.75.127.5]) by euls16034.sgp.st.com (STMicroelectronics) with ESMTP id 6E2ED2128B2; Thu, 27 Jan 2022 12:18:03 +0100 (CET) Received: from localhost (10.75.127.44) by SFHDAG2NODE2.st.com (10.75.127.5) with Microsoft SMTP Server (TLS) id 15.0.1497.26; Thu, 27 Jan 2022 12:18:03 +0100 From: Alain Volmat To: , CC: , , , , , , Subject: [PATCH] media: stm32-dcmi: create video dev once sensor is binded Date: Thu, 27 Jan 2022 12:18:02 +0100 Message-ID: <20220127111802.976275-1-alain.volmat@foss.st.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 X-Originating-IP: [10.75.127.44] X-ClientProxiedBy: SFHDAG2NODE2.st.com (10.75.127.5) To SFHDAG2NODE2.st.com (10.75.127.5) X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.205,Aquarius:18.0.816,Hydra:6.0.425,FMLib:17.11.62.513 definitions=2022-01-27_03,2022-01-27_01,2021-12-02_01 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220127_031817_734932_553DE900 X-CRM114-Status: GOOD ( 28.06 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org In case of an error during the initialization of the sensor, the video device is still available since created at the probe of the dcmi driver. Moreover the device wouldn't be released even when removing the module since the release is performed as part of the notifier unbind callback (not called if no sensor is properly initialized). This patch move the video device creation with the v4l2 notifier complete handler in order to avoid having a video device created when an error happen during the pipe (dcmi - sensor) initialization. This also makes the video device creation symmetric with the release which is already done within the notifier unbind handler. Signed-off-by: Alain Volmat --- v1: this patch is the replacement patch of a previous attempt [1] to move the register within the bound callback. [1] https://lore.kernel.org/linux-media/31ca9ccc-77d4-4368-1024-db70e8e1e7f2@xs4all.nl/ drivers/media/platform/stm32/stm32-dcmi.c | 69 ++++++++++++----------- 1 file changed, 35 insertions(+), 34 deletions(-) diff --git a/drivers/media/platform/stm32/stm32-dcmi.c b/drivers/media/platform/stm32/stm32-dcmi.c index e1b17c05229c..80d0fbeabb4f 100644 --- a/drivers/media/platform/stm32/stm32-dcmi.c +++ b/drivers/media/platform/stm32/stm32-dcmi.c @@ -134,6 +134,7 @@ struct stm32_dcmi { struct video_device *vdev; struct v4l2_async_notifier notifier; struct v4l2_subdev *source; + struct v4l2_subdev *remote; struct v4l2_format fmt; struct v4l2_rect crop; bool do_crop; @@ -579,9 +580,9 @@ static void dcmi_buf_queue(struct vb2_buffer *vb) spin_unlock_irq(&dcmi->irqlock); } -static struct media_entity *dcmi_find_source(struct stm32_dcmi *dcmi) +static struct media_entity *dcmi_find_source(struct v4l2_subdev *subdev) { - struct media_entity *entity = &dcmi->vdev->entity; + struct media_entity *entity = &subdev->entity; struct media_pad *pad; /* Walk searching for entity having no sink */ @@ -1721,6 +1722,7 @@ static int dcmi_framesizes_init(struct stm32_dcmi *dcmi) static int dcmi_graph_notify_complete(struct v4l2_async_notifier *notifier) { struct stm32_dcmi *dcmi = notifier_to_dcmi(notifier); + int src_pad; int ret; /* @@ -1728,7 +1730,7 @@ static int dcmi_graph_notify_complete(struct v4l2_async_notifier *notifier) * we search for the source subdevice * in order to expose it through V4L2 interface */ - dcmi->source = media_entity_to_v4l2_subdev(dcmi_find_source(dcmi)); + dcmi->source = media_entity_to_v4l2_subdev(dcmi_find_source(dcmi->remote)); if (!dcmi->source) { dev_err(dcmi->dev, "Source subdevice not found\n"); return -ENODEV; @@ -1768,6 +1770,34 @@ static int dcmi_graph_notify_complete(struct v4l2_async_notifier *notifier) return ret; } + ret = video_register_device(dcmi->vdev, VFL_TYPE_VIDEO, -1); + if (ret) { + dev_err(dcmi->dev, "Failed to register video device\n"); + return ret; + } + + dev_dbg(dcmi->dev, "Device registered as %s\n", + video_device_node_name(dcmi->vdev)); + + /* + * Link remote sub-device to DCMI, it could be + * a parallel camera sensor or a bridge + */ + src_pad = media_entity_get_fwnode_pad(&dcmi->remote->entity, + dcmi->remote->fwnode, + MEDIA_PAD_FL_SOURCE); + + ret = media_create_pad_link(&dcmi->remote->entity, src_pad, + &dcmi->vdev->entity, 0, + MEDIA_LNK_FL_IMMUTABLE | + MEDIA_LNK_FL_ENABLED); + if (ret) + dev_err(dcmi->dev, "Failed to create media pad link with subdev \"%s\"\n", + dcmi->remote->name); + else + dev_dbg(dcmi->dev, "DCMI is now linked to \"%s\"\n", + dcmi->remote->name); + return 0; } @@ -1788,31 +1818,11 @@ static int dcmi_graph_notify_bound(struct v4l2_async_notifier *notifier, struct v4l2_async_subdev *asd) { struct stm32_dcmi *dcmi = notifier_to_dcmi(notifier); - unsigned int ret; - int src_pad; dev_dbg(dcmi->dev, "Subdev \"%s\" bound\n", subdev->name); + dcmi->remote = subdev; - /* - * Link this sub-device to DCMI, it could be - * a parallel camera sensor or a bridge - */ - src_pad = media_entity_get_fwnode_pad(&subdev->entity, - subdev->fwnode, - MEDIA_PAD_FL_SOURCE); - - ret = media_create_pad_link(&subdev->entity, src_pad, - &dcmi->vdev->entity, 0, - MEDIA_LNK_FL_IMMUTABLE | - MEDIA_LNK_FL_ENABLED); - if (ret) - dev_err(dcmi->dev, "Failed to create media pad link with subdev \"%s\"\n", - subdev->name); - else - dev_dbg(dcmi->dev, "DCMI is now linked to \"%s\"\n", - subdev->name); - - return ret; + return 0; } static const struct v4l2_async_notifier_operations dcmi_graph_notify_ops = { @@ -2008,15 +2018,6 @@ static int dcmi_probe(struct platform_device *pdev) } dcmi->vdev->entity.flags |= MEDIA_ENT_FL_DEFAULT; - ret = video_register_device(dcmi->vdev, VFL_TYPE_VIDEO, -1); - if (ret) { - dev_err(dcmi->dev, "Failed to register video device\n"); - goto err_media_entity_cleanup; - } - - dev_dbg(dcmi->dev, "Device registered as %s\n", - video_device_node_name(dcmi->vdev)); - /* Buffer queue */ q->type = V4L2_BUF_TYPE_VIDEO_CAPTURE; q->io_modes = VB2_MMAP | VB2_READ | VB2_DMABUF;