Cover Letter Detail
Show a cover letter.
GET /api/covers/334/?format=api
{ "id": 334, "url": "https://patchwork.libcamera.org/api/covers/334/?format=api", "web_url": "https://patchwork.libcamera.org/cover/334/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/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": "<20190122234505.32634-1-niklas.soderlund@ragnatech.se>", "date": "2019-01-22T23:44:57", "name": "[libcamera-devel,0/8] libcamera: add basic support for Streams and format configuration", "submitter": { "id": 5, "url": "https://patchwork.libcamera.org/api/people/5/?format=api", "name": "Niklas Söderlund", "email": "niklas.soderlund@ragnatech.se" }, "mbox": "https://patchwork.libcamera.org/cover/334/mbox/", "series": [ { "id": 118, "url": "https://patchwork.libcamera.org/api/series/118/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=118", "date": "2019-01-22T23:44:57", "name": "libcamera: add basic support for Streams and format configuration", "version": 1, "mbox": "https://patchwork.libcamera.org/series/118/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/covers/334/comments/", "headers": { "Return-Path": "<niklas.soderlund@ragnatech.se>", "Received": [ "from bin-mail-out-05.binero.net (bin-mail-out-05.binero.net\n\t[195.74.38.228])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 44DDA60C7D\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tWed, 23 Jan 2019 00:46:37 +0100 (CET)", "from bismarck.berto.se (unknown [89.233.230.99])\n\tby bin-vsp-out-03.atm.binero.net (Halon) with ESMTPA\n\tid dca1ec54-1e9f-11e9-911a-0050569116f7;\n\tWed, 23 Jan 2019 00:46:15 +0100 (CET)" ], "X-Halon-ID": "dca1ec54-1e9f-11e9-911a-0050569116f7", "Authorized-sender": "niklas@soderlund.pp.se", "From": "=?utf-8?q?Niklas_S=C3=B6derlund?= <niklas.soderlund@ragnatech.se>", "To": "libcamera-devel@lists.libcamera.org", "Date": "Wed, 23 Jan 2019 00:44:57 +0100", "Message-Id": "<20190122234505.32634-1-niklas.soderlund@ragnatech.se>", "X-Mailer": "git-send-email 2.20.1", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit", "Subject": "[libcamera-devel] [PATCH 0/8] libcamera: add basic support for\n\tStreams and format configuration", "X-BeenThere": "libcamera-devel@lists.libcamera.org", "X-Mailman-Version": "2.1.23", "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>", "X-List-Received-Date": "Tue, 22 Jan 2019 23:46:37 -0000" }, "content": "Hi,\n\nThis series aims to add support of Streams to the Camera object. A \nStream is a single video stream origination from a video source, \nmultiple video streams might form the same Camera is possible as long as \nthey all originate from the same video source.\n\nThis series depends on the association between Camera and \nPipleineHandlers series [1]. For obvious reasons patch 8/8 is not meant \nfor upstream consumption but a proof of concept for how a application \ncould interact with the API.\n\nAs this is the first draft of this series I expect a few things to \nchange. While working and discussing this series potential topics for \nchange are.\n\n- Provide a convenience Camera::configure() implementation which takes a \n single StreamConfiguration object. Rational is that there might a lot \n of applications which will be designed to only work with one Stream at \n a time so to make life a little easier for them might be a good ting.\n\n- Have the Stream objects registered with the camera device at the same \n time it's created by the PipelineHandler::match() and not propagate \n the Camera::streams() call to the PipelineHandler. This is an \n appealing thought.\n\n- Have the base class PipelineHandler deal with disconnecting all \n cameras belonging to the specific implementations. This if possible to \n do in a neat way would reduce a lot of boiler plate code.\n\n- Over all if possible move more validation to the Camera and base class \n PipelineHandler where possible to reduce boiler plate code from the \n specific implementations. Suggestions on where this would be neat or \n really meaningful are appreciated.\n\n\n1. [PATCH 0/3] libcamera: add association between Camera and \n PipelineHandlers\n\nLaurent Pinchart (1):\n libcamera: camera: Add acquire() and release()\n\nNiklas Söderlund (7):\n libcamera: stream: add basic Stream class\n libcamera: stream: add basic StreamConfiguration class\n libcamera: pipelines: add log category for each pipeline\n libcamera: pipelines: add method to retrieve streams\n libcamera: pipelines: add method to configure streams\n libcamera: camera: integrate streams and configuration\n [POC] cam: add hack option to staticly configure a pipeline\n\n include/libcamera/camera.h | 12 +++\n include/libcamera/libcamera.h | 1 +\n include/libcamera/meson.build | 1 +\n include/libcamera/stream.h | 45 ++++++++\n src/cam/main.cpp | 38 +++++++\n src/libcamera/camera.cpp | 83 ++++++++++++++-\n src/libcamera/include/pipeline_handler.h | 6 ++\n src/libcamera/meson.build | 1 +\n src/libcamera/pipeline/ipu3/ipu3.cpp | 51 +++++++++\n src/libcamera/pipeline/uvcvideo.cpp | 40 +++++++\n src/libcamera/pipeline/vimc.cpp | 40 +++++++\n src/libcamera/pipeline_handler.cpp | 30 ++++++\n src/libcamera/stream.cpp | 130 +++++++++++++++++++++++\n 13 files changed, 477 insertions(+), 1 deletion(-)\n create mode 100644 include/libcamera/stream.h\n create mode 100644 src/libcamera/stream.cpp" }