Cover Letter Detail
Show a cover letter.
GET /api/1.1/covers/22022/?format=api
{ "id": 22022, "url": "https://patchwork.libcamera.org/api/1.1/covers/22022/?format=api", "web_url": "https://patchwork.libcamera.org/cover/22022/", "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": "<20241120085753.1993436-1-stefan.klug@ideasonboard.com>", "date": "2024-11-20T08:57:39", "name": "[0/7] rkisp1: Fix aspect ratio and ScalerCrop", "submitter": { "id": 184, "url": "https://patchwork.libcamera.org/api/1.1/people/184/?format=api", "name": "Stefan Klug", "email": "stefan.klug@ideasonboard.com" }, "mbox": "https://patchwork.libcamera.org/cover/22022/mbox/", "series": [ { "id": 4807, "url": "https://patchwork.libcamera.org/api/1.1/series/4807/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=4807", "date": "2024-11-20T08:57:39", "name": "rkisp1: Fix aspect ratio and ScalerCrop", "version": 1, "mbox": "https://patchwork.libcamera.org/series/4807/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/covers/22022/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 61ADFC32F5\n\tfor <parsemail@patchwork.libcamera.org>;\n\tWed, 20 Nov 2024 08:58:02 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id DAD3E65F32;\n\tWed, 20 Nov 2024 09:58:00 +0100 (CET)", "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 7FA36658B6\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tWed, 20 Nov 2024 09:57:58 +0100 (CET)", "from ideasonboard.com (unknown\n\t[IPv6:2a00:6020:448c:6c00:bbd:82cc:f3f3:e12e])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 5073E55A;\n\tWed, 20 Nov 2024 09:57:40 +0100 (CET)" ], "Authentication-Results": "lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"byMGHFuI\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1732093060;\n\tbh=icixU+vvjaooFuLU+5d3UJa0TMHWYpLPEXimfTrDyV4=;\n\th=From:To:Cc:Subject:Date:From;\n\tb=byMGHFuIZ07LF7p2Mk1kxcxj6HpjIJezIO77+BPgwqmMCwpKQdvaEYwPsqNAgIT6M\n\tdeWqfqfawDWYgjetCn+5nzMuXT3uoV3GBj0vD5Hlz7nonNF3XIqhGyL6PuaH9h4O5v\n\tXlQJ2UxwelV0GzoTdWZVdKpfxEpbrZPM2LV3fCjs=", "From": "Stefan Klug <stefan.klug@ideasonboard.com>", "To": "libcamera-devel@lists.libcamera.org", "Cc": "Stefan Klug <stefan.klug@ideasonboard.com>", "Subject": "[PATCH 0/7] rkisp1: Fix aspect ratio and ScalerCrop", "Date": "Wed, 20 Nov 2024 09:57:39 +0100", "Message-ID": "<20241120085753.1993436-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": "Hi all,\n\nThis series contains fixes for a few issues on the imx8mp that I'd like\nto get in before the next release.\n\nPatch 1/7 is a drive by fix\n\nPatch 2/7 Fixes the aspect ratio on the imx8mp. There exists code to\nkeep the aspect ratio in the rkisp1 pipeline that adjust the input crop\non the resizer stage. On the imx8mp however the resizer hardware is\nmissing that functionality and cropping needs to be done on the ISP\noutput. Patch 2 implements that.\n\nPatches 3-7 fix the ScalerCrop control. The existing implementation\nimplements ScalerCrop in dewarper coordinates (After scaling down).\nThis is incorrect as ScalerCrop is specified to be in sensor\ncoordinates. To fix that, patches 3-5 do preparatory changes and patch 6\nimplements the correct ScalerCrop control. Patch 7 adds\nScalerMaximumCrop for completeness sake.\n\nNote that this solution while being correct on the numbers is still not\nideal, because the aspect ratio preserving crop on the ISP output is\nstill applied unconditionally. That will get fixed in an upcoming\nseries, where the full sensor size get's passed to the dewarper and all\ncropping/scaling gets handled there.\n\nBest regards,\nStefan\n\nStefan Klug (7):\n libcamera: rkisp1: Fix scope of dewarper stop() exit action\n libcamera: rkisp1: Keep aspect ratio on imx8mp\n libcamera: geometry: Add Rectangle::mappedBetween()\n pipeline: rkisp1: Split inputCrop and outputCrop\n pipeline: rkisp1: Reorder sensorInfo collection code\n pipeline: rkisp1: Fix ScalerCrop to be in sensor coordinates\n pipeline: rkisp1: Add ScalerMaximumCrop property\n\n include/libcamera/geometry.h | 2 +\n src/libcamera/geometry.cpp | 33 +++++++\n src/libcamera/pipeline/rkisp1/rkisp1.cpp | 97 ++++++++++++++-----\n src/libcamera/pipeline/rkisp1/rkisp1_path.cpp | 6 ++\n 4 files changed, 114 insertions(+), 24 deletions(-)" }