{"id":27265,"url":"https://patchwork.libcamera.org/api/1.1/covers/27265/?format=json","web_url":"https://patchwork.libcamera.org/cover/27265/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/1.1/projects/1/?format=json","name":"libcamera","link_name":"libcamera","list_id":"libcamera_core","list_email":"libcamera-devel@lists.libcamera.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<20260708-libipa-algorithms-v4-0-f0e99f035294@ideasonboard.com>","date":"2026-07-08T09:24:47","name":"[v4,00/35] ipa: libipa: Introduce libipa algorithms","submitter":{"id":143,"url":"https://patchwork.libcamera.org/api/1.1/people/143/?format=json","name":"Jacopo Mondi","email":"jacopo.mondi@ideasonboard.com"},"mbox":"https://patchwork.libcamera.org/cover/27265/mbox/","series":[{"id":6046,"url":"https://patchwork.libcamera.org/api/1.1/series/6046/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=6046","date":"2026-07-08T09:24:47","name":"ipa: libipa: Introduce libipa algorithms","version":4,"mbox":"https://patchwork.libcamera.org/series/6046/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/27265/comments/","headers":{"Return-Path":"","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 7D612C3301\n\tfor ;\n\tWed, 8 Jul 2026 09:25:08 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id E315E66081;\n\tWed, 8 Jul 2026 11:25:06 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 351C065FEF\n\tfor ;\n\tWed, 8 Jul 2026 11:25:05 +0200 (CEST)","from [192.168.1.182] (93-46-82-201.ip106.fastwebnet.it\n\t[93.46.82.201])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 48062E9C;\n\tWed, 8 Jul 2026 11:24:15 +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=\"ugKwXmli\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1783502655;\n\tbh=9bi+DBOGOyc++7DNkJO0uifoYBdtx/lBulP+TSGuilw=;\n\th=From:Subject:Date:To:Cc:From;\n\tb=ugKwXmliELk7zY/7Sx6IZ/44vvaP0aWpcCYC5aOn1BH1Nl89pKdEHczMy0r9x1vSm\n\todNHRubZVXwQFiDXd3WiE6BoObudreGlCLT09MUi0nJTDnSSycPvJa4zE0gbdK47P7\n\tRYWFA0tci1HyyT+/6R6V104iQ+ZscPXa2Q+rIJl0=","From":"Jacopo Mondi ","Subject":"[PATCH v4 00/35] ipa: libipa: Introduce libipa algorithms","Date":"Wed, 08 Jul 2026 11:24:47 +0200","Message-Id":"<20260708-libipa-algorithms-v4-0-f0e99f035294@ideasonboard.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"7bit","X-B4-Tracking":"v=1; b=H4sIAGAXTmoC/33NQQ6CMBCF4auQrq0p09JSV97DuGhhgEmQktYQj\n\teHuVje6QJf/S+abB0sYCRM7FA8WcaFEYcqhdgVrBjf1yKnNzUCAFrqs+EieZsfd2IdI1+GSeGW\n\t11r5xwkHH8t0csaPb2zydcw+UriHe3y+W8rX+05aSC45W2cZKowDwSC26FCYfXGz3TbiwF7rAF\n\twRyC4IMddYoKbsSUP2C5AcyQm9BMkNW16YyXsja+w1oXdcnXY3iWUsBAAA=","X-Change-ID":"20260615-libipa-algorithms-59666bca0a2f","To":"libcamera-devel@lists.libcamera.org, \n\tStefan Klug ","Cc":"Jacopo Mondi , \n\tKieran Bingham , \n\tDaniel Scally , \n\tRobert Mader , \n\tLaurent Pinchart ","X-Mailer":"b4 0.14.3","X-Developer-Signature":"v=1; a=openpgp-sha256; l=6844;\n\ti=jacopo.mondi@ideasonboard.com; h=from:subject:message-id;\n\tbh=9bi+DBOGOyc++7DNkJO0uifoYBdtx/lBulP+TSGuilw=;\n\tb=owEBbQKS/ZANAwAKAXI0Bo8WoVY8AcsmYgBqThdr76c7mGwhwO1BXk2fLm+pkxnJ+rCZdcEmr\n\t6AcCHWReDyJAjMEAAEKAB0WIQS1xD1IgJogio9YOMByNAaPFqFWPAUCak4XawAKCRByNAaPFqFW\n\tPDEJEACt/TEU+9CUTrITiQkMrX8pXK+Lrmq8s9Pz6OO8+XW1pwwMpeq7MMnUUNJNU+LQdaedwuz\n\tGGamU2OZz2YnpBpqupKM0f4yTOh8H4hK5cQwjMYK6LKcmcD7uQ1s3J8cy86XWR8Ng3sCKh0eOlG\n\taNK9OCyLqk88wxsyq54rfDVPVw6d+2M/hbiqBX4s4kM2tAm1N1ZScU23XmttjxVnDGFKZQRtY7M\n\taSQExPHDsphrB8vfWvV1xhRX719AXGlxWdUqx+3nsApgUB40Pi5efA3JYNax9RB+zPZgg290oKq\n\thLi8oWXpWtMWqKSg6hOrPyb4iuUMv/bptnOGdLHJxfoFrD/GmzR/3u1SpYqfe1bkL8Tq8gv88J9\n\t7ZlQ6UvJYVNnq6SbEdZELMdoHQLlsbYfRdvaKFyG6obz18LRHtDCpfOf7km6WB5ANcT+9LbXowp\n\tCh2Aji3l8Jmwjl1uZKKWxvFz2J5LS9sAIRKOXVKE8qgPmSuPM7WQLXNxUYQB2PQSpjbjiyuNDPi\n\tZ/titX4SHzQG/hAT2eR3flTwXnJSw7ze4kwS3PJY1P+sNJn0LP1JyAeb7FwYzkn6vjp5aV98MsW\n\toEB1v6Y0lOtQTmkh4/pLbDqA0tZrE/fYY3L/X4KiXSEVWr3dNuCEbOClZ8iClIwMbcqAVVF4rnD\n\tQGQX+Yd+De1V+8w==","X-Developer-Key":"i=jacopo.mondi@ideasonboard.com; a=openpgp;\n\tfpr=72392EDC88144A65C701EA9BA5826A2587AD026B","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "},"content":"This series introduces the first algorithms in libipa: Awb, Lsc and Ccm.\n\nAs most of the implementations of the common algorithms derive from\nRkISP1 it wasn't possible to separate the algorithm introduction with\nthe RkISP1 IPA modifications to maintain bisectability.\n\nThis series contains:\n\n- Awb (Grey and Bayes)\n - RkISP1\n - Simple\n - Mali-C55\n\n- Ccm\n - RkISP1\n - Simple\n - Mali-C55\n\n- Lsc (grid and polynomial)\n - RkISP1\n - Mali-C55\n\nTested on RkISP1 with imx219 (Grey world awb and grid-based LSC) and on\nMali-C55 with imx708 (Bayes AWB and grid-based LSC).\n\nMore test on Simple and with polynomial LSC would be helpful :)\n\nSigned-off-by: Jacopo Mondi \n---\nChanges in v4:\n- s/awb/AWB in comments everywhere, same for CCM and LSC\n- Awb: changelog per patch\n - Make AwbStats pure virtual and move stats parsing logic to the IPA\n modules\n - Reduce comments/renames to reduce patch size and comments\n- Collect tags\n\nChanges in v3:\n- Address comments on Awb from Stefan:\n - Introduce AwbImplementation::Result\n - Move AwbStats implementation to IPAs\n - Drop AwbStats::rg()/bg() and re-implement AwbGray accordingly\n- Fix Saturation bug in Simple Ccm algorithm\n- Fix Simple Ccm gain limits using Q<2, 8>\n- Update Mali CCM and Gamma to latest uAPI patch version\n- Fix Mali CCM gain masking as reported by Dan\n- Break down libIPA Lsc implementation as requested by Stefan\n\nChanges in v2:\n- Introduce awb::Context to simplify ActiveState and FrameContext\n definition\n- Add mali uAPI header for CCM and Gamma\n- Do not rename Mali functions for hardware configuration in awb and lsc\n- Spelling and documentation\n\n---\nJacopo Mondi (32):\n ipa: libipa: awb: Reimplement AwbAlgorithm\n ipa: mali-c55: awb: Port to use libipa AwbAlgorithm\n ipa: libipa: Add CcmAlgorithm to libipa\n ipa: rkisp1: ccm: Port to use CcmAlgorithm\n include: linux: mali-c55: Update to support CCM and Gamma\n ipa: mali-c55: Implement Ccm algorithm\n ipa: rkisp1: lsc: Re-architecture Lsc support\n ipa: libipa: Introduce lsc_base.h\n ipa: libipa: lsc_polynomial: Rename LscPolynomial\n ipa: libipa: lsc_polynomial: Do not inline functions\n ipa: libipa: lsc: Move Interpolator to lsc_base\n ipa: rkisp1: lsc: Remove rogue empty line\n ipa: libipa: lsc_polynomial: Move LscPolynomial from RkISP1\n ipa: libipa: lsc_table: Move LscTable from RkISP1\n ipa: libipa: lsc_base: Add sampleForCrop documentation\n ipa: rksip1: lsc: Make parseSizes() a class function\n ipa: libipa: lsc: Introduce lsc::ActiveState\n ipa: libipa: lsc: Introduce lsc::FrameContext\n ipa: libipa: lsc: Introduce LscAlgorithm\n ipa: libipa: lsc: Move sizesListToPositions() back to rkisp1\n ipa: rkisp1: lsc: Port to use LscAlgorithm\n ipa: libipa: lsc: Make Components a map<>\n ipa: libipa: lsc: Document Components parsing\n ipa: libipa: lsc_polynomial: Remove sensorSize_\n ipa: libipa: lsc: Document LscDescriptor::sensorSize usage\n ipa: libipa: lsc: Template LscAlgorithm with register format\n ipa: libipa: lsc_base: Describe the function of lsc_base\n ipa: libipa: lsc: Template lsc::Components\n ipa: libipa: lsc: Rename sampleForCrop to resampleLscData\n ipa: libipa: lsc: Specialize Interpolator for uint8_t\n ipa: mali-c55: Add sensorInfo to IPAContext\n ipa: mali-c55: Port to use LscAlgorithm\n\nKieran Bingham (2):\n ipa: simple: awb: Port to use libipa AwbAlgorithm\n ipa: simple: Use libipa CcmAlgorithm\n\nLaurent Pinchart (1):\n ipa: libipa: lsc_polynomial: Don't inline ValueNode accessor\n\n .../internal/software_isp/debayer_params.h | 4 +-\n include/linux/media/arm/mali-c55-config.h | 86 +++-\n src/ipa/libipa/awb.cpp | 418 +++++++++++++++++---\n src/ipa/libipa/awb.h | 100 ++++-\n src/ipa/libipa/awb_bayes.cpp | 52 +--\n src/ipa/libipa/awb_bayes.h | 12 +-\n src/ipa/libipa/awb_grey.cpp | 16 +-\n src/ipa/libipa/awb_grey.h | 6 +-\n src/ipa/libipa/ccm.cpp | 245 ++++++++++++\n src/ipa/libipa/ccm.h | 77 ++++\n src/ipa/libipa/lsc.cpp | 311 +++++++++++++++\n src/ipa/libipa/lsc.h | 137 +++++++\n src/ipa/libipa/lsc_base.cpp | 178 +++++++++\n src/ipa/libipa/lsc_base.h | 90 +++++\n src/ipa/libipa/lsc_polynomial.cpp | 175 ++++++++-\n src/ipa/libipa/lsc_polynomial.h | 152 +++++---\n src/ipa/libipa/lsc_table.cpp | 40 ++\n src/ipa/libipa/lsc_table.h | 118 ++++++\n src/ipa/libipa/meson.build | 8 +\n src/ipa/mali-c55/algorithms/awb.cpp | 223 ++++++-----\n src/ipa/mali-c55/algorithms/awb.h | 28 +-\n src/ipa/mali-c55/algorithms/ccm.cpp | 173 +++++++++\n src/ipa/mali-c55/algorithms/ccm.h | 66 ++++\n src/ipa/mali-c55/algorithms/lsc.cpp | 156 +++++---\n src/ipa/mali-c55/algorithms/lsc.h | 32 +-\n src/ipa/mali-c55/algorithms/meson.build | 1 +\n src/ipa/mali-c55/ipa_context.cpp | 3 +\n src/ipa/mali-c55/ipa_context.h | 20 +-\n src/ipa/mali-c55/mali-c55.cpp | 2 +\n src/ipa/mali-c55/params.h | 2 +\n src/ipa/rkisp1/algorithms/awb.cpp | 191 ++-------\n src/ipa/rkisp1/algorithms/awb.h | 18 +-\n src/ipa/rkisp1/algorithms/ccm.cpp | 97 +----\n src/ipa/rkisp1/algorithms/ccm.h | 17 +-\n src/ipa/rkisp1/algorithms/lsc.cpp | 432 +++------------------\n src/ipa/rkisp1/algorithms/lsc.h | 41 +-\n src/ipa/rkisp1/ipa_context.h | 49 +--\n src/ipa/simple/algorithms/awb.cpp | 131 +++++--\n src/ipa/simple/algorithms/awb.h | 29 ++\n src/ipa/simple/algorithms/blc.cpp | 2 +-\n src/ipa/simple/algorithms/ccm.cpp | 72 ++--\n src/ipa/simple/algorithms/ccm.h | 23 +-\n src/ipa/simple/ipa_context.h | 15 +-\n src/libcamera/software_isp/debayer_cpu.cpp | 12 +-\n 44 files changed, 2933 insertions(+), 1127 deletions(-)\n---\nbase-commit: 709ad59a8d90af3570be1c8d02dee0ec9e011954\nchange-id: 20260615-libipa-algorithms-59666bca0a2f\n\nBest regards,"}