Cover Letter Detail
Show a cover letter.
GET /api/covers/20500/?format=api
{ "id": 20500, "url": "https://patchwork.libcamera.org/api/covers/20500/?format=api", "web_url": "https://patchwork.libcamera.org/cover/20500/", "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": "<20240703104004.184783-1-stefan.klug@ideasonboard.com>", "date": "2024-07-03T10:39:47", "name": "[v2,0/6] ipa: Add black level to camera sensor helpers", "submitter": { "id": 184, "url": "https://patchwork.libcamera.org/api/people/184/?format=api", "name": "Stefan Klug", "email": "stefan.klug@ideasonboard.com" }, "mbox": "https://patchwork.libcamera.org/cover/20500/mbox/", "series": [ { "id": 4435, "url": "https://patchwork.libcamera.org/api/series/4435/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=4435", "date": "2024-07-03T10:39:47", "name": "ipa: Add black level to camera sensor helpers", "version": 2, "mbox": "https://patchwork.libcamera.org/series/4435/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/covers/20500/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 3A866BEFBE\n\tfor <parsemail@patchwork.libcamera.org>;\n\tWed, 3 Jul 2024 10:40:13 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 2343062C96;\n\tWed, 3 Jul 2024 12:40:12 +0200 (CEST)", "from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 7589362C95\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tWed, 3 Jul 2024 12:40:10 +0200 (CEST)", "from ideasonboard.com (unknown\n\t[IPv6:2a00:6020:448c:6c00:9263:c199:9587:576])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 4E5D93E6;\n\tWed, 3 Jul 2024 12:39:42 +0200 (CEST)" ], "Authentication-Results": "lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"h5qpDwC6\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1720003182;\n\tbh=/qwNBNrQCpJ9hNTLdnMx1XYOq2kJH0UC14/6DcCoDsQ=;\n\th=From:To:Cc:Subject:Date:From;\n\tb=h5qpDwC6PQZx1zTs/A6/hiAjESYB+IosXaJ59i+pd1nMVsTlmtjueA80SiiW6jt3O\n\tEu1SygbGm+l0qQTUHiNPdXrYghwd1OcEZYprNx9fXBgIsC9HCBZ3f9sBJlqcFObiof\n\tgOnQv70QW2myYR/a47JT6VJxIRc8beznukH+CPWA=", "From": "Stefan Klug <stefan.klug@ideasonboard.com>", "To": "libcamera-devel@lists.libcamera.org", "Cc": "Stefan Klug <stefan.klug@ideasonboard.com>", "Subject": "[PATCH v2 0/6] ipa: Add black level to camera sensor helpers", "Date": "Wed, 3 Jul 2024 12:39:47 +0200", "Message-ID": "<20240703104004.184783-1-stefan.klug@ideasonboard.com>", "X-Mailer": "git-send-email 2.43.0", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "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": "For the tuning process and the BLC block, the sensor black level needs\nto be known. The black level usually consists of a pedestal value added\nby the sensor and additional physical effects like dark currents. The\nsensors are usually programmed with fixed pedestal value or to\ntarget a defined black level using internal algorithms. We are not aware\nof any sensor driver that changes these values from the defaults\nmentioned in the datasheets. It therefore makes sense to keep that\ninformation inside libcamera. When the need arises to be more flexible\nwe can extend the sensor drivers to report black level information.\n\nThis series adds black level support to the camera sensor helpers and\nthe corresponding metadata to the rkisp1 ipa. Aditionally it removes the\nblack level information from the existing tuning files. When we observe\nthat the hardcoded values are too far off from actual measurements an\nadditional offset can be added to the tuning files.\n\nMy tuning series will be rebased on top of this one.\n\nBest regards,\nStefan\n\nChanges in v2:\n- Store a single int16_t black level inside camera sensor helper. This\n is sufficient for all known use cases.\n- Fix scaling from 16bit down to 12bit for the rkisp1.\n- Add more documentation text and (hopefully) clarify some commit\n messages.\n- Reworked the tuning data handling as proposed by Laurent.\n- Updated all tuning files that contained black level values.\n\n\nStefan Klug (6):\n ipa: libipa: Add black levels to camera sensor helper\n ipa: rkisp1: Move camHelper into IPAContext\n ipa: rkisp1: blc: Query black levels from camera sensor helper\n ipa: rkisp1: blc: Report sensor black levels in metadata\n ipa: rkisp1: data: Update tuning files for imx219 and imx258\n ipa: rkisp1: Move ov4689 and ov5640 black levels into sensor helpers\n\n src/ipa/libipa/camera_sensor_helper.cpp | 29 +++++++++\n src/ipa/libipa/camera_sensor_helper.h | 3 +\n src/ipa/rkisp1/algorithms/blc.cpp | 82 ++++++++++++++++++++++---\n src/ipa/rkisp1/algorithms/blc.h | 5 +-\n src/ipa/rkisp1/data/imx219.yaml | 4 --\n src/ipa/rkisp1/data/imx258.yaml | 1 +\n src/ipa/rkisp1/data/ov4689.yaml | 4 --\n src/ipa/rkisp1/data/ov5640.yaml | 4 --\n src/ipa/rkisp1/data/uncalibrated.yaml | 1 +\n src/ipa/rkisp1/ipa_context.h | 6 ++\n src/ipa/rkisp1/rkisp1.cpp | 26 ++++----\n 11 files changed, 130 insertions(+), 35 deletions(-)" }