Patch Detail
Show a patch.
GET /api/patches/24305/?format=api
{ "id": 24305, "url": "https://patchwork.libcamera.org/api/patches/24305/?format=api", "web_url": "https://patchwork.libcamera.org/patch/24305/", "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": "<20250911090547.1860863-2-niklas.soderlund+renesas@ragnatech.se>", "date": "2025-09-11T09:05:46", "name": "[v2,1/2] ipa: rkisp1: Add settings for DreamChip RPPX1 ISP", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "e598588c0282b367ddffd9605f9cf2fa3a1faca8", "submitter": { "id": 230, "url": "https://patchwork.libcamera.org/api/people/230/?format=api", "name": "Niklas Söderlund", "email": "niklas.soderlund+renesas@ragnatech.se" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/24305/mbox/", "series": [ { "id": 5430, "url": "https://patchwork.libcamera.org/api/series/5430/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=5430", "date": "2025-09-11T09:05:45", "name": "Add Renesas R-Car Gen4 pipeline with IPA support", "version": 2, "mbox": "https://patchwork.libcamera.org/series/5430/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/24305/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/24305/checks/", "tags": {}, "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 8D057BDB13\n\tfor <parsemail@patchwork.libcamera.org>;\n\tThu, 11 Sep 2025 09:06:43 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 3538069372;\n\tThu, 11 Sep 2025 11:06:42 +0200 (CEST)", "from fhigh-b3-smtp.messagingengine.com\n\t(fhigh-b3-smtp.messagingengine.com [202.12.124.154])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 447276935A\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tThu, 11 Sep 2025 11:06:37 +0200 (CEST)", "from phl-compute-08.internal (phl-compute-08.internal\n\t[10.202.2.48])\n\tby mailfhigh.stl.internal (Postfix) with ESMTP id 0B2657A0010;\n\tThu, 11 Sep 2025 05:06:36 -0400 (EDT)", "from phl-mailfrontend-01 ([10.202.2.162])\n\tby phl-compute-08.internal (MEProxy); Thu, 11 Sep 2025 05:06:36 -0400", "by mail.messagingengine.com (Postfix) with ESMTPA; Thu,\n\t11 Sep 2025 05:06:35 -0400 (EDT)" ], "Authentication-Results": "lancelot.ideasonboard.com; dkim=pass (2048-bit key;\n\tunprotected) header.d=ragnatech.se header.i=@ragnatech.se\n\theader.b=\"ks5WXvw7\"; dkim=pass (2048-bit key;\n\tunprotected) header.d=messagingengine.com\n\theader.i=@messagingengine.com header.b=\"RXNVmzjt\"; \n\tdkim-atps=neutral", "DKIM-Signature": [ "v=1; a=rsa-sha256; c=relaxed/relaxed; d=ragnatech.se; h=\n\tcc:cc:content-transfer-encoding:content-type:content-type:date\n\t:date:from:from:in-reply-to:in-reply-to:message-id:mime-version\n\t:references:reply-to:subject:subject:to:to; s=fm3; t=1757581595;\n\tx=1757667995; bh=DjwXwJbtlgnSg4sO3UddRtGsoB4bpNQ9hqmd0f1RifU=; b=\n\tks5WXvw7IeaFO9ph64Y4gZmvRvD6ruQh8u7qpRLEpYHeJs8Bzq4Teg8psiNQOoa+\n\tqhlccSFApudKX79dyHowWSS97NBsVD0gIoYl1yYoNrdHz8gaDUqRqFhPZ16EyoYe\n\tQ+VOiDIz5ZvwcqFAEu2u7Aj8seO1xhScWG66MFw0CBs2Dq2SqnwL7Uu3kElToH9k\n\tNRrbO61p/QIxekW3CUXXpxoRoQmlhtuGWRIoLQr4fKnzrVfVaMhTMjGTlus8rftn\n\tVooSaaijqVJJ87SuEPYKceqjQOZoOM/qrjCK1Dd3ejs5dElgQ9sJ7KJWcXJvqEDB\n\ta2tn4+9mLco6KWiDvVR5SQ==", "v=1; a=rsa-sha256; c=relaxed/relaxed; d=\n\tmessagingengine.com; h=cc:cc:content-transfer-encoding\n\t:content-type:content-type:date:date:feedback-id:feedback-id\n\t:from:from:in-reply-to:in-reply-to:message-id:mime-version\n\t:references:reply-to:subject:subject:to:to:x-me-proxy\n\t:x-me-sender:x-me-sender:x-sasl-enc; s=fm1; t=1757581595; x=\n\t1757667995; bh=DjwXwJbtlgnSg4sO3UddRtGsoB4bpNQ9hqmd0f1RifU=; b=R\n\tXNVmzjtJFsdJp9IQeRmFFu1vEVxIEGmTmJfPb7i3D7kNKB5jmdXMeAtDWI6pc8mu\n\thiKV1n2svjzta/KxEDEtydNkdhf7SpOtLSNfZYwHi61ybblnNIST01amTdTXvsKt\n\tpUHC8NDsGi5AfR+XORXRGMDUNlJWzYSZIWy3eHw4s8UVWUrB4c6fxmFMo2t2Dapy\n\tImvcv4snIT2Bw3ANBU5msmXD2lINTEUwG7GKFk1A74BTc6Bj8SQUuA4jeZOVgMQb\n\tcLkSSlX7TRabXMm9aQZg1nmsGuaH7jAgp73np8aZ0eq1rmT2vLpsibTf0Om9PtSe\n\tUkPE0IxySlv7lhFTOxQDg==" ], "X-ME-Sender": "<xms:G5HCaOV7l25AVpgXFGbmI6QwNundixrlXMSCvmKqk8-G1Ceg5Mtolg>\n\t<xme:G5HCaFKM1WqXvdql727OQm5ZgcCIIu7To9HEr-iAyRyiXkIYBYzOH-3dhn3Zo0pnI\n\tIMAiDB6QFoi2en0YO4>", "X-ME-Received": "<xmr:G5HCaO94tqnKeEz_K9Xjsi3l7Tl4snt0L5vrbSAclr3z9HdLc7GZZUcnRTq2Q7IfwKDN3V0rCMB3Ny0sly4g9wAlXg>", "X-ME-Proxy-Cause": "gggruggvucftvghtrhhoucdtuddrgeeffedrtdeggddvheejjecutefuodetggdotefrod\n\tftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpuffrtefokffrpgfnqfghnecuuegr\n\tihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenucfjug\n\thrpefhvfevufffkffojghfgggtgfesthekredtredtjeenucfhrhhomheppfhikhhlrghs\n\tucfunpguvghrlhhunhguuceonhhikhhlrghsrdhsohguvghrlhhunhguodhrvghnvghsrg\n\thssehrrghgnhgrthgvtghhrdhsvgeqnecuggftrfgrthhtvghrnhepheeigfeuveeutdef\n\thfehgeekvedtleeuueekveefudehhffhjeffgfegffelfeegnecuvehluhhsthgvrhfuih\n\tiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepnhhikhhlrghsrdhsohguvghrlhhu\n\tnhgusehrrghgnhgrthgvtghhrdhsvgdpnhgspghrtghpthhtohepgedpmhhouggvpehsmh\n\thtphhouhhtpdhrtghpthhtohepjhgrtghophhordhmohhnughisehiuggvrghsohhnsgho\n\trghrugdrtghomhdprhgtphhtthhopehlrghurhgvnhhtrdhpihhntghhrghrthesihguvg\n\tgrshhonhgsohgrrhgurdgtohhmpdhrtghpthhtoheplhhisggtrghmvghrrgdquggvvhgv\n\tlheslhhishhtshdrlhhisggtrghmvghrrgdrohhrghdprhgtphhtthhopehnihhklhgrsh\n\tdrshhouggvrhhluhhnugdorhgvnhgvshgrshesrhgrghhnrghtvggthhdrshgv", "X-ME-Proxy": "<xmx:G5HCaIJt7Z58-T_mecoX-PD9qCNA4xWTf0XQRQoXE7waTQkkDo_v7A>\n\t<xmx:G5HCaDm66Pnl6LvKvEbvA2QwNkPbGIzzWa2NDSfXf3Gi4LePO__B5g>\n\t<xmx:G5HCaMOT3SFGZPGC37MaXYWli8ncSsfK8tfK1xLu3_-e1Z75y615PA>\n\t<xmx:G5HCaJ0CCuvhohItJSsaz0UaWKsSmBFalmIMvdAtAEdi8FaX1ijHwg>\n\t<xmx:G5HCaLmwKztFgIJNWw2_BZWyWe6JK8__DgnlO8FjsBbVI9zJSWrE2GsP>", "Feedback-ID": "i80c9496c:Fastmail", "From": "=?utf-8?q?Niklas_S=C3=B6derlund?=\n\t<niklas.soderlund+renesas@ragnatech.se>", "To": "Jacopo Mondi <jacopo.mondi@ideasonboard.com>,\n\tLaurent Pinchart <laurent.pinchart@ideasonboard.com>,\n\tlibcamera-devel@lists.libcamera.org", "Cc": "=?utf-8?q?Niklas_S=C3=B6derlund?= <niklas.soderlund+renesas@ragnatech.se>", "Subject": "[PATCH v2 1/2] ipa: rkisp1: Add settings for DreamChip RPPX1 ISP", "Date": "Thu, 11 Sep 2025 11:05:46 +0200", "Message-ID": "<20250911090547.1860863-2-niklas.soderlund+renesas@ragnatech.se>", "X-Mailer": "git-send-email 2.51.0", "In-Reply-To": "<20250911090547.1860863-1-niklas.soderlund+renesas@ragnatech.se>", "References": "<20250911090547.1860863-1-niklas.soderlund+renesas@ragnatech.se>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=UTF-8", "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": "The DreamChip RPPX1 ISP appears to be a superset of the Rockchip RkISP1\nISP V10. The RkISP1 interface and hardware operates at 8-bits color\ndepth, while the RPPX1 operates at 24-bits. For this reason the RkISP1\nparameter and buffer formats are deigned around 8-bits.\n\nThe kernel driver for RPPX1 can however compensate for this and scale\ndown it's internal data structures to 8-bits and produce parameter\nbuffers (RK1E) and output statistics (RK1S) that is compatible with the\nRkISP1 IPA.\n\nThe one difference is that the RPPX1 have 32 histogram buckets compared\nto RkISP1 V10 16. Add a dedicated hardware revisions descriptor for the\nRPPX1 device so it can reuse the existing IPA.\n\nSigned-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>\n---\n include/libcamera/ipa/rkisp1.mojom | 8 ++++++++\n src/ipa/rkisp1/rkisp1.cpp | 11 +++++++++++\n 2 files changed, 19 insertions(+)", "diff": "diff --git a/include/libcamera/ipa/rkisp1.mojom b/include/libcamera/ipa/rkisp1.mojom\nindex 043ad27ea199..608ba82a0091 100644\n--- a/include/libcamera/ipa/rkisp1.mojom\n+++ b/include/libcamera/ipa/rkisp1.mojom\n@@ -8,6 +8,14 @@ module ipa.rkisp1;\n \n import \"include/libcamera/ipa/core.mojom\";\n \n+/*\n+ * Hardware revisions in rkisp1-config.h (enum rkisp1_cif_isp_version) start at\n+ * 10 and increment sequentially. Add a namespace starting at 0x80000000 for\n+ * devices not cover by the kernel rkisp1 implementation but still supported\n+ * by the IPA.\n+ */\n+const uint32 HwRevisionExternalRppX1 = 0x80000001;\n+\n struct IPAConfigInfo {\n \tlibcamera.IPACameraSensorInfo sensorInfo;\n \tlibcamera.ControlInfoMap sensorControls;\ndiff --git a/src/ipa/rkisp1/rkisp1.cpp b/src/ipa/rkisp1/rkisp1.cpp\nindex cf66d5553dcd..14600251dfe5 100644\n--- a/src/ipa/rkisp1/rkisp1.cpp\n+++ b/src/ipa/rkisp1/rkisp1.cpp\n@@ -113,6 +113,14 @@ const IPAHwSettings ipaHwSettingsV12{\n \tfalse,\n };\n \n+const IPAHwSettings ipaHwSettingsRPPX1{\n+\tRKISP1_CIF_ISP_AE_MEAN_MAX_V10,\n+\tRKISP1_CIF_ISP_HIST_BIN_N_MAX_V12,\n+\tRKISP1_CIF_ISP_HISTOGRAM_WEIGHT_GRIDS_SIZE_V10,\n+\tRKISP1_CIF_ISP_GAMMA_OUT_MAX_SAMPLES_V10,\n+\tfalse,\n+};\n+\n /* List of controls handled by the RkISP1 IPA */\n const ControlInfoMap::Map rkisp1Controls{\n \t{ &controls::DebugMetadataEnable, ControlInfo(false, true, false) },\n@@ -147,6 +155,9 @@ int IPARkISP1::init(const IPASettings &settings, unsigned int hwRevision,\n \tcase RKISP1_V12:\n \t\tcontext_.hw = &ipaHwSettingsV12;\n \t\tbreak;\n+\tcase HwRevisionExternalRppX1:\n+\t\tcontext_.hw = &ipaHwSettingsRPPX1;\n+\t\tbreak;\n \tdefault:\n \t\tLOG(IPARkISP1, Error)\n \t\t\t<< \"Hardware revision \" << hwRevision\n", "prefixes": [ "v2", "1/2" ] }