{"id":25396,"url":"https://patchwork.libcamera.org/api/covers/25396/?format=json","web_url":"https://patchwork.libcamera.org/cover/25396/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/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":"<20251210003139.43606-1-bryan.odonoghue@linaro.org>","date":"2025-12-10T00:31:17","name":"[v4,00/20] GPUISP precursor series","submitter":{"id":175,"url":"https://patchwork.libcamera.org/api/people/175/?format=json","name":"Bryan O'Donoghue","email":"bryan.odonoghue@linaro.org"},"mbox":"https://patchwork.libcamera.org/cover/25396/mbox/","series":[{"id":5644,"url":"https://patchwork.libcamera.org/api/series/5644/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=5644","date":"2025-12-10T00:31:17","name":"GPUISP precursor series","version":4,"mbox":"https://patchwork.libcamera.org/series/5644/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/25396/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 CCA70BD1F1\n\tfor ;\n\tWed, 10 Dec 2025 00:31:59 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 042A261422;\n\tWed, 10 Dec 2025 01:31:59 +0100 (CET)","from mail-pj1-x1032.google.com (mail-pj1-x1032.google.com\n\t[IPv6:2607:f8b0:4864:20::1032])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 3B1C160C8A\n\tfor ;\n\tWed, 10 Dec 2025 01:31:57 +0100 (CET)","by mail-pj1-x1032.google.com with SMTP id\n\t98e67ed59e1d1-343806688c5so5084500a91.0\n\tfor ;\n\tTue, 09 Dec 2025 16:31:57 -0800 (PST)","from inspiron14p-linux (p96936b06.osaknt01.ap.so-net.ne.jp.\n\t[150.147.107.6]) by smtp.gmail.com with ESMTPSA id\n\t98e67ed59e1d1-34a70427ea6sm566580a91.3.2025.12.09.16.31.53\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tTue, 09 Dec 2025 16:31:54 -0800 (PST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (2048-bit key;\n\tunprotected) header.d=linaro.org header.i=@linaro.org\n\theader.b=\"p7GXkRmB\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=linaro.org; s=google; t=1765326715; x=1765931515;\n\tdarn=lists.libcamera.org; \n\th=content-transfer-encoding:mime-version:message-id:date:subject:cc\n\t:to:from:from:to:cc:subject:date:message-id:reply-to;\n\tbh=ye3BKXPnsOY+539P8QTyrnebjAPJfzkyHmQeDz+jVkI=;\n\tb=p7GXkRmB9djUyTdmQm9lKinKrz7rTT7ATfE/OI75z0GDWokvUJoHLXqxNN4ta3SJfh\n\tVKtBcjhyhw9GTuwHlYEJw4iG5qiRLAwmYWYuVZKtpXwULvEKBrlTn5/4fowUvUWv5BwZ\n\t/ZJF+4CjkcY8Y/j1FTmK4zdEgayXo/kHxKbCdT0v6bgt29gSNAu1TYGIS/6Laqv6LKmz\n\tbZ2vX1u55z8k66Xgs32eCQqyIiTZrq/J4/SPUUY3fAiz5+MImbj5T7xhZ0vc2l2mg2Ea\n\t9K/9/7c61/6NMrQnw5t3QZIxQ9N5Ycyad5QvfQmq2uO/syYIQuPSXwKWov7yftZWWlhx\n\tO6WA==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20230601; t=1765326715; x=1765931515;\n\th=content-transfer-encoding:mime-version:message-id:date:subject:cc\n\t:to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date\n\t:message-id:reply-to;\n\tbh=ye3BKXPnsOY+539P8QTyrnebjAPJfzkyHmQeDz+jVkI=;\n\tb=POh5IV8uuzyJKpWtOa7c2z+i7ymgFXdEqt+FuNK1+u8KPbE/eXfkcYD7a4RI+ZuLEp\n\tEQaiyHBfsl4Sf1aLT9/wz0a5Xf6gPVtkWUxIdrc+JteAuyAUhNapSiHPwzxH7bjVFNj9\n\tuJiPv1j4Tg1KKDSFHwWFIIbLWWYL3RmoQcszgZwcVKT3gD7VDNF+zJZUsPVDVnDlGPJQ\n\tYJBAPfQUU2a9IDOMIBgu3CmJ1xiI+Nw0NtiO3WXqzYNb/fIUvJ5cXWOH2Rzs2HtcG3tg\n\t3rY70KjaXHyZ4ydDwPYjrGwcg/ATXedAkTXVIOo/D81hQ336GsNLJSPhIjyRE3dWhmj1\n\tbLag==","X-Gm-Message-State":"AOJu0YyM8ARTIMdi2MDZv78HQqSDxYcMb3gBSDeNne1IJ9ag859dA2R/\n\tOLIQ/kS8NL5QYC9KxZ2qDESLOU5sCOw5NgEXD+kEnGCmt2NwgZiOQP7KMUzj4Ec9D1F7NXuPNrL\n\tRDclH","X-Gm-Gg":"AY/fxX6Kk73wBWWXdI0t4fxH44knryGadJtB0KgCGSkIcf/PWhttWDOiyo3VZqnmWmy\n\tzxLGRLbjsAxRoXarVgCM8JlCPAMPuhehxx+DlLDzhfLGBQ26SVrKtSry1qm+DGCQncG9/pjiwkl\n\tG8zPrl7LhzGHxqXlPBJwZq4yJ4WnbjNYF8ktuqP5tsy2yeq5FPwb2qMON8+FkDr/BSeawgEr0ym\n\thaP2yfKfnHc4EwSRXskp8/atcjYXTLrxnU7uZ7sToAF91uZoY1c2KflWvX67SftnPj+Evd1nuz7\n\tl7Uav5H+eYX2Pb4OhB6W+lFPQkskW5u4AGP1RM9axuHo384EsibbTzVqiSEfsf1vZW1+tzV8K67\n\tkueI24PVCvE58oGjLfsgfwgFRi6fv46XyiSShoXVCCgAraipIwloR3+tWOEl5J4HWmANTwT5t81\n\t+AIoxTjr2NZ75UTWEvQy6/OZdTPb9eIrbkozvRVqgHI8J4LbB2LgcLQQ6Ll/OrpTp8","X-Google-Smtp-Source":"AGHT+IGKl2P1Q/9kddf5ROtLzBa4Njx+f8uT8F1K1J4hnQDqkfTKh2hGDTnIKgqrl21shw92l1WPhQ==","X-Received":"by 2002:a17:90b:3a90:b0:341:2150:4856 with SMTP id\n\t98e67ed59e1d1-34a72874796mr604732a91.17.1765326715074; \n\tTue, 09 Dec 2025 16:31:55 -0800 (PST)","From":"Bryan O'Donoghue ","To":"libcamera-devel@lists.libcamera.org","Cc":"pavel@ucw.cz,\n\tBryan O'Donoghue ","Subject":"[PATCH v4 00/20] GPUISP precursor series","Date":"Wed, 10 Dec 2025 00:31:17 +0000","Message-ID":"<20251210003139.43606-1-bryan.odonoghue@linaro.org>","X-Mailer":"git-send-email 2.52.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":"","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":"v4:\n- Adds a start() method to the bayer object.\n Previous versions already had a stop() method but, the new start()\n will allow us to do EGL init in start() not configure().\n- Additional comments from Milan missed in previous reviews addressed.\n- Patch 15 changed enough that I dropped Kieran and Milan's previous RB\n please make sure you guys are still happy with this.\n\nv3:\n- Adds Kieran's RBs as given\n- Changes python script - Barnabas\n- Makes isStandardBayerOrder static earlier in series - Barnabas\n- Moves from DebayerCpu to Debayer object type in\n one go for configure(), process() and stop() - Barnabas\n- Rebases on v0.6.0\n\nv2:\n- Moves shaders to src/libcamera/shaders - Laurent\n- Various Doxygen fixes - Kieran/Bryan/Milan\n- Takes Hans' change to process 1/4 frames in processFrame()\n- Implements suggested change in blackLevel calc - Milan\n- Only three patches in this series have no RB\n I think it would be safe to merge this series - bod\n\nv1:\nThis series takes all of the already reviewed-by on-list stuff and\nparcels it into a more digestable series.\n\nRight at the end are some patches which were about making data available in\nthe debayer params strcutre and had some reviewed-by and/or comments from\nKiearn and Milan that I didn't apply review-by to because I wasn't 100%\nsure that was warranted from the list dicussion - CCM was clear enough, AWB\ngot dropped.\n\nAdding in blacklevel, gamma and contrast seem to fit-in with these patches\nand including in this series reduces the overall size of GPUISP as a series\nfrom 44 ! patches in total to 22. \n\nCI is passing both of this branch and for the aggregate branch so, I've\nopted to start sending out the broken apart series, even though I'm\nexpecting an IPU6 fix from Hans it shouldn't affect this early code.\n\nBryan O'Donoghue (15):\n libcamera: shaders: Move GL shader programs to\n src/libcamera/assets/shader\n utils: gen-shader-headers: Add a utility to generate headers from\n shaders\n meson: Automatically generate glsl_shaders.h from specified shader\n programs\n libcamera: software_isp: Move useful items from DebayerCpu to Debayer\n base class\n libcamera: software_isp: Move Bayer params init from DebayerCpu to\n Debayer\n libcamera: software_isp: Move param select code to Debayer base class\n libcamera: software_isp: Move DMA Sync code to Debayer base class\n libcamera: software_isp: Make output DMA sync contingent\n libcamera: software_isp: Move isStandardBayerOrder to base class\n libcamera: software_isp: debayer: Introduce a start() / stop() methods\n to the debayer object\n libcamera: software_isp: debayer: Make the debayer_ object of type\n class Debayer not DebayerCpu\n libcamera: software_isp: lut: Make CCM available in debayer params\n libcamera: software_isp: blacklevel: Make black level available in\n debayer params\n libcamera: software_isp: lut: Make gamma from lut.cpp available in\n debayer params\n libcamera: software_isp: lut: Make contrast available in debayer\n params\n\nHans de Goede (5):\n libcamera: swstats_cpu: Update statsProcessFn() / processLine0()\n documentation\n libcamera: swstats_cpu: Drop patternSize_ documentation\n libcamera: swstats_cpu: Move header to libcamera/internal/software_isp\n libcamera: software_isp: Move benchmark code to its own class\n libcamera: swstats_cpu: Add processFrame() method\n\n .../internal/software_isp/benchmark.h | 39 +++\n .../internal/software_isp/debayer_params.h | 11 +\n .../internal/software_isp/meson.build | 2 +\n .../internal/software_isp/software_isp.h | 5 +-\n .../internal}/software_isp/swstats_cpu.h | 15 +-\n src/apps/qcam/assets/shader/shaders.qrc | 16 +-\n src/apps/qcam/meson.build | 3 +\n src/apps/qcam/viewfinder_gl.cpp | 70 ++---\n src/ipa/simple/algorithms/blc.cpp | 9 +\n src/ipa/simple/algorithms/blc.h | 4 +\n src/ipa/simple/algorithms/lut.cpp | 4 +\n src/libcamera/meson.build | 11 +\n .../shader => libcamera/shaders}/RGB.frag | 0\n .../shaders}/YUV_2_planes.frag | 0\n .../shaders}/YUV_3_planes.frag | 0\n .../shaders}/YUV_packed.frag | 0\n .../shaders}/bayer_1x_packed.frag | 0\n .../shader => libcamera/shaders}/bayer_8.frag | 0\n .../shader => libcamera/shaders}/bayer_8.vert | 0\n .../shaders}/identity.vert | 0\n src/libcamera/shaders/meson.build | 10 +\n src/libcamera/software_isp/benchmark.cpp | 92 +++++++\n src/libcamera/software_isp/debayer.cpp | 251 ++++++++++++++++++\n src/libcamera/software_isp/debayer.h | 44 ++-\n src/libcamera/software_isp/debayer_cpu.cpp | 89 +------\n src/libcamera/software_isp/debayer_cpu.h | 45 +---\n src/libcamera/software_isp/meson.build | 1 +\n src/libcamera/software_isp/software_isp.cpp | 15 +-\n src/libcamera/software_isp/swstats_cpu.cpp | 93 ++++++-\n utils/gen-shader-header.py | 37 +++\n utils/gen-shader-headers.sh | 55 ++++\n utils/meson.build | 2 +\n 32 files changed, 735 insertions(+), 188 deletions(-)\n create mode 100644 include/libcamera/internal/software_isp/benchmark.h\n rename {src/libcamera => include/libcamera/internal}/software_isp/swstats_cpu.h (84%)\n rename src/{apps/qcam/assets/shader => libcamera/shaders}/RGB.frag (100%)\n rename src/{apps/qcam/assets/shader => libcamera/shaders}/YUV_2_planes.frag (100%)\n rename src/{apps/qcam/assets/shader => libcamera/shaders}/YUV_3_planes.frag (100%)\n rename src/{apps/qcam/assets/shader => libcamera/shaders}/YUV_packed.frag (100%)\n rename src/{apps/qcam/assets/shader => libcamera/shaders}/bayer_1x_packed.frag (100%)\n rename src/{apps/qcam/assets/shader => libcamera/shaders}/bayer_8.frag (100%)\n rename src/{apps/qcam/assets/shader => libcamera/shaders}/bayer_8.vert (100%)\n rename src/{apps/qcam/assets/shader => libcamera/shaders}/identity.vert (100%)\n create mode 100644 src/libcamera/shaders/meson.build\n create mode 100644 src/libcamera/software_isp/benchmark.cpp\n create mode 100755 utils/gen-shader-header.py\n create mode 100755 utils/gen-shader-headers.sh"}