Patch Detail
Show a patch.
GET /api/1.1/patches/8724/?format=api
{ "id": 8724, "url": "https://patchwork.libcamera.org/api/1.1/patches/8724/?format=api", "web_url": "https://patchwork.libcamera.org/patch/8724/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/1.1/projects/1/?format=api", "name": "libcamera", "link_name": "libcamera", "list_id": "libcamera_core", "list_email": "libcamera-devel@lists.libcamera.org", "web_url": "", "scm_url": "", "webscm_url": "" }, "msgid": "<20200709132835.112593-10-paul.elder@ideasonboard.com>", "date": "2020-07-09T13:28:21", "name": "[libcamera-devel,v5,09/23] libcamera: pipeline_handler: Fatal if registering camera without media devices", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": false, "hash": "f0f9c49df10eacc2d28f79ec3a2ebd5f1e9a13f0", "submitter": { "id": 17, "url": "https://patchwork.libcamera.org/api/1.1/people/17/?format=api", "name": "Paul Elder", "email": "paul.elder@ideasonboard.com" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/8724/mbox/", "series": [ { "id": 1093, "url": "https://patchwork.libcamera.org/api/1.1/series/1093/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=1093", "date": "2020-07-09T13:28:12", "name": "Clean up formats in v4l2-compat and pipeline handlers", "version": 5, "mbox": "https://patchwork.libcamera.org/series/1093/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/8724/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/8724/checks/", "tags": {}, "headers": { "Return-Path": "<libcamera-devel-bounces@lists.libcamera.org>", "X-Original-To": "parsemail@patchwork.libcamera.org", "Delivered-To": "parsemail@patchwork.libcamera.org", "Received": [ "from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 058DDBD792\n\tfor <parsemail@patchwork.libcamera.org>;\n\tThu, 9 Jul 2020 13:29:13 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id C409B61221;\n\tThu, 9 Jul 2020 15:29:12 +0200 (CEST)", "from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id DF383611BA\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tThu, 9 Jul 2020 15:29:11 +0200 (CEST)", "from pyrite.rasen.tech (unknown\n\t[IPv6:2400:4051:61:600:2c71:1b79:d06d:5032])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 238C8525;\n\tThu, 9 Jul 2020 15:29:09 +0200 (CEST)" ], "Authentication-Results": "lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"KpTlZn17\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1594301351;\n\tbh=SZ6JKm2SfkqebE7nZSCnGt56qj/+Uqx9h2hx8UxIjds=;\n\th=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n\tb=KpTlZn17SKPylTy6xtuXExzTp+rvuHhemjm0hdsrZ4lSRq9st2+IaC7O67ZoCr+5j\n\tLIPtwN7MveDVq6OsgG3lb5WelLCXNBIy8HICxtPEcEaj7WxXgYkNti0ffOGOh0a1O/\n\t6hgcdNM96UCX0yTI+7Bms1ov8yFZRjweXCYFwNXc=", "From": "Paul Elder <paul.elder@ideasonboard.com>", "To": "libcamera-devel@lists.libcamera.org", "Date": "Thu, 9 Jul 2020 22:28:21 +0900", "Message-Id": "<20200709132835.112593-10-paul.elder@ideasonboard.com>", "X-Mailer": "git-send-email 2.27.0", "In-Reply-To": "<20200709132835.112593-1-paul.elder@ideasonboard.com>", "References": "<20200709132835.112593-1-paul.elder@ideasonboard.com>", "MIME-Version": "1.0", "Subject": "[libcamera-devel] [PATCH v5 09/23] libcamera: pipeline_handler:\n\tFatal if registering camera without media devices", "X-BeenThere": "libcamera-devel@lists.libcamera.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "<libcamera-devel.lists.libcamera.org>", "List-Unsubscribe": "<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>", "List-Archive": "<https://lists.libcamera.org/pipermail/libcamera-devel/>", "List-Post": "<mailto:libcamera-devel@lists.libcamera.org>", "List-Help": "<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>", "List-Subscribe": "<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>", "Content-Type": "text/plain; charset=\"us-ascii\"", "Content-Transfer-Encoding": "7bit", "Errors-To": "libcamera-devel-bounces@lists.libcamera.org", "Sender": "\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>" }, "content": "Pipeline handlers must acquire media devices via\nPipelineHander::acquireMediaDevice so that the media devices can be\nregistered with the pipeline handler, so that they can be automatically\nadded to the devnum map for the v4l2 compatibility layer to use. Die\nfatally if any camera trying to be registered has not acquired any media\ndevices via acquireMediaDevice.\n\nSigned-off-by: Paul Elder <paul.elder@ideasonboard.com>\nReviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>\n\n---\nNo change in v5\n\nCosmetic change in v4\n\nNo change in v3\n---\n src/libcamera/pipeline_handler.cpp | 4 ++++\n 1 file changed, 4 insertions(+)", "diff": "diff --git a/src/libcamera/pipeline_handler.cpp b/src/libcamera/pipeline_handler.cpp\nindex de5ca09..ccd45ed 100644\n--- a/src/libcamera/pipeline_handler.cpp\n+++ b/src/libcamera/pipeline_handler.cpp\n@@ -496,6 +496,10 @@ void PipelineHandler::registerCamera(std::shared_ptr<Camera> camera,\n \tcameraData_[camera.get()] = std::move(data);\n \tcameras_.push_back(camera);\n \n+\tif (mediaDevices_.empty())\n+\t\tLOG(Pipeline, Fatal)\n+\t\t\t<< \"Registering camera with no media devices!\";\n+\n \t/*\n \t * Walk the entity list and map the devnums of all capture video nodes\n \t * to the camera.\n", "prefixes": [ "libcamera-devel", "v5", "09/23" ] }