Cover Letter Detail
Show a cover letter.
GET /api/1.1/covers/15056/?format=api
{ "id": 15056, "url": "https://patchwork.libcamera.org/api/1.1/covers/15056/?format=api", "web_url": "https://patchwork.libcamera.org/cover/15056/", "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": "<20211206233948.1351206-1-kieran.bingham@ideasonboard.com>", "date": "2021-12-06T23:39:40", "name": "[libcamera-devel,RFC,0/8] Request metadata: SensorSequence", "submitter": { "id": 4, "url": "https://patchwork.libcamera.org/api/1.1/people/4/?format=api", "name": "Kieran Bingham", "email": "kieran.bingham@ideasonboard.com" }, "mbox": "https://patchwork.libcamera.org/cover/15056/mbox/", "series": [ { "id": 2822, "url": "https://patchwork.libcamera.org/api/1.1/series/2822/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=2822", "date": "2021-12-06T23:39:40", "name": "Request metadata: SensorSequence", "version": 1, "mbox": "https://patchwork.libcamera.org/series/2822/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/covers/15056/comments/", "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 85538BF415\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon, 6 Dec 2021 23:39:57 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 8B02F60898;\n\tTue, 7 Dec 2021 00:39:56 +0100 (CET)", "from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 5FA3C60117\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 7 Dec 2021 00:39:54 +0100 (CET)", "from Monstersaurus.local\n\t(cpc89244-aztw30-2-0-cust3082.18-1.cable.virginm.net [86.31.172.11])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id D0FAE556;\n\tTue, 7 Dec 2021 00:39:53 +0100 (CET)" ], "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=\"ZxzsNUPD\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1638833994;\n\tbh=ujAZAywscOsx91urn6hF+Mm7lSA6aqxdlP3eSwKsVIs=;\n\th=From:To:Cc:Subject:Date:From;\n\tb=ZxzsNUPDdNkFMFftShxUPoctxuY/BAIAjdfpADqs0m2AoMUbOO2Bz6usWs1/uFSgX\n\tp51D3I1Y1HUpEShVj6oJDsOqBpKfaM0qi36mggtOEjEwdcmNxJOYM6qzY9Lbxnn2JR\n\t44EU2OnwkzW05Z0XeTY57t3zmt9dy4z3L1xMzGGM=", "From": "Kieran Bingham <kieran.bingham@ideasonboard.com>", "To": "libcamera devel <libcamera-devel@lists.libcamera.org>", "Date": "Mon, 6 Dec 2021 23:39:40 +0000", "Message-Id": "<20211206233948.1351206-1-kieran.bingham@ideasonboard.com>", "X-Mailer": "git-send-email 2.30.2", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[libcamera-devel] [RFC PATCH 0/8] Request metadata: SensorSequence", "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>", "Errors-To": "libcamera-devel-bounces@lists.libcamera.org", "Sender": "\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>" }, "content": "When completing a request, the individual stream buffers contain a\nsequence number. This number is generated by the device that ultimately\nfills that stream, but it might not be the sensor. Processing through an\nISP could cause the sequence numbers and timestamp data associated with\nthe completed buffer to be values representative of the ISP processing\nrather than the (intended) capture device.\n\nProvide a new metadata control, still to be fully sketched out which\ngives us a defined value to report the Camera sequence number. This\nallows pipeline handlers to correctly set the value according to the\ndevice that represents the capture from the sensor.\n\nThis plumbing then allows applications to detect frame drops, which were\notherwise going unnoticed, and as such some basic additions have been\nmade to cam, qcam, and gstreamer to support this new data.\n\nStill possible:\n - Adding a validation to lc-compliance to make sure pipelines set the\n SensorSequence on every frame.\n\n - Probably expecting some better gstreamer event integration perhaps? \n\n - qcam should report more statistics on the processing overall\n\n - libcamera Tracepoints could be added as an event to track when\n frames are detected as dropped by the core framework.\n\nAnything else?\n\n\nKieran Bingham (8):\n libcamera: controls: Add SensorSequence metadata control\n libcamera: pipeline: Set the Sensor sequence number for all pipelines\n cam: Use SensorTimestamp rather than buffer metadata\n cam: Use Sensor sequence numbers and detect frame drop\n qcam: main_window: Fix include ordering\n qcam: Use Sensor sequence numbers and detect frame drop\n gstreamer: gstlibcamerasrc: Fix include ordering\n gstreamer: Use Sensor sequence numbers and detect frame drop\n\n src/cam/camera_session.cpp | 24 ++++++++--\n src/cam/camera_session.h | 1 +\n src/gstreamer/gstlibcamerasrc.cpp | 46 +++++++++++++++----\n src/libcamera/control_ids.yaml | 8 ++++\n src/libcamera/pipeline/ipu3/ipu3.cpp | 4 +-\n .../pipeline/raspberrypi/raspberrypi.cpp | 3 ++\n src/libcamera/pipeline/rkisp1/rkisp1.cpp | 4 +-\n src/libcamera/pipeline/simple/simple.cpp | 12 +++--\n src/libcamera/pipeline/uvcvideo/uvcvideo.cpp | 2 +\n src/qcam/main_window.cpp | 28 +++++++++--\n src/qcam/main_window.h | 1 +\n 11 files changed, 111 insertions(+), 22 deletions(-)" }