Cover Letter Detail
Show a cover letter.
GET /api/1.1/covers/3739/?format=api
{ "id": 3739, "url": "https://patchwork.libcamera.org/api/1.1/covers/3739/?format=api", "web_url": "https://patchwork.libcamera.org/cover/3739/", "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": "<20200510115810.21938-1-laurent.pinchart@ideasonboard.com>", "date": "2020-05-10T11:58:04", "name": "[libcamera-devel,v5,0/6] Simple pipeline handler", "submitter": { "id": 2, "url": "https://patchwork.libcamera.org/api/1.1/people/2/?format=api", "name": "Laurent Pinchart", "email": "laurent.pinchart@ideasonboard.com" }, "mbox": "https://patchwork.libcamera.org/cover/3739/mbox/", "series": [ { "id": 890, "url": "https://patchwork.libcamera.org/api/1.1/series/890/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=890", "date": "2020-05-10T11:58:04", "name": "Simple pipeline handler", "version": 5, "mbox": "https://patchwork.libcamera.org/series/890/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/covers/3739/comments/", "headers": { "Return-Path": "<laurent.pinchart@ideasonboard.com>", "Received": [ "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 3B4BA60BDC\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tSun, 10 May 2020 13:58:20 +0200 (CEST)", "from pendragon.bb.dnainternet.fi (81-175-216-236.bb.dnainternet.fi\n\t[81.175.216.236])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 90630304;\n\tSun, 10 May 2020 13:58:19 +0200 (CEST)" ], "Authentication-Results": "lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"hBbVA64g\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1589111899;\n\tbh=6rf32P+7R/gOW7ew/+XGwjkUFX/KgWoCSupR/pCOa4M=;\n\th=From:To:Cc:Subject:Date:From;\n\tb=hBbVA64gRrydD9Ybr1dtWbFryjdCu7MA+0yeKQ4ygfgvyffQHJjLTKji2+bDYT510\n\t2kqA0aEn/v/P6T5GSazcdTVJFgLCOpdAajD0YQhqhWHygWkmsUnyrNHDmOjGd0+RqT\n\tFJ69mY7MDhQZPUCFPdiQEokvAAwpeg9XdSGnqq5o=", "From": "Laurent Pinchart <laurent.pinchart@ideasonboard.com>", "To": "libcamera-devel@lists.libcamera.org", "Cc": "Martijn Braam <martijn@brixit.nl>", "Date": "Sun, 10 May 2020 14:58:04 +0300", "Message-Id": "<20200510115810.21938-1-laurent.pinchart@ideasonboard.com>", "X-Mailer": "git-send-email 2.26.2", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit", "Subject": "[libcamera-devel] [PATCH v5 0/6] Simple pipeline handler", "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>", "X-List-Received-Date": "Sun, 10 May 2020 11:58:20 -0000" }, "content": "Hello,\n\nThis patch series builds on Martijn's work to implement a pipeline\nhandler for simple pipelines. Additional information about the pipeline\nhandler itself can be found in patch 3/6.\n\nCompared to v4, this version incorporates the review comments and has\nbeen rebased on top of master, without major changes.\n\nPatch 1/6 adds a V4L2 API extension for the VIDIOC_ENUM_FMT ioctl,\nrequired for pipeline auto-configuration. The kernel changes have been\nmerged in the linux-media master branch and is scheduled for kernel\nv5.8. The patch description has been updated accordingly. Patch 2/6\nextends the V4L2VideoDevice class to implement the extension.\n\nPatch 3/6 adds the new simple pipeline handler, patch 4/6 the simple\nconverter, and patch 5/6 integrates converter support into the simple\npipeline handler. The converter API hasn't changed and should still be\nimproved, it's currently a proof of concept. Other options also exist,\nsuch as making the simple pipeline handler a base class from which more\nspecialized pipeline handlers could derive from, and implement support\nfor converters in device-specific code.\n\nPatch 6/6 extends the simple pipeline handler to support devices that\ncontain multiple video nodes. It could be squashed with 3/6, but keeping\nit separate should ease review.\n\nThe code has been tested on an i.MX7 platform with a greyscale sensor\nand the i.MX PXP as format converter to produce RGB. I've kept sun6i-csi\nfrom Martijn's work in the list of supported drivers, but supporting\nthat platform will require extending the corresponding driver to support\nthe VIDIOC_ENUM_FMT extension. Once done, if all goes well, the platform\nshould work out of the box.\n\nLaurent Pinchart (5):\n include: linux: Extend VIDIOC_ENUM_FMT to support MC-centric devices\n libcamera: v4l2_videodevice: Support filtering formats by media bus\n code\n libcamera: pipeline: simple: Add simple format converter\n libcamera: pipeline: simple: Integrate converter support\n libcamera: pipeline: simple: Support multiple capture video nodes\n\nMartijn Braam (1):\n libcamera: pipeline: Add a simple pipeline handler\n\n include/linux/videodev2.h | 5 +-\n meson_options.txt | 2 +-\n src/libcamera/include/v4l2_videodevice.h | 4 +-\n src/libcamera/pipeline/simple/converter.cpp | 217 +++++\n src/libcamera/pipeline/simple/converter.h | 60 ++\n src/libcamera/pipeline/simple/meson.build | 4 +\n src/libcamera/pipeline/simple/simple.cpp | 894 ++++++++++++++++++++\n src/libcamera/v4l2_videodevice.cpp | 16 +-\n 8 files changed, 1195 insertions(+), 7 deletions(-)\n create mode 100644 src/libcamera/pipeline/simple/converter.cpp\n create mode 100644 src/libcamera/pipeline/simple/converter.h\n create mode 100644 src/libcamera/pipeline/simple/meson.build\n create mode 100644 src/libcamera/pipeline/simple/simple.cpp" }