{"id":25286,"url":"https://patchwork.libcamera.org/api/covers/25286/?format=json","web_url":"https://patchwork.libcamera.org/cover/25286/","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":"<20251202133157.661407-1-bryan.odonoghue@linaro.org>","date":"2025-12-02T13:31:31","name":"[v3,00/22] 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/25286/mbox/","series":[{"id":5630,"url":"https://patchwork.libcamera.org/api/series/5630/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=5630","date":"2025-12-02T13:31:31","name":"GPUISP precursor series","version":3,"mbox":"https://patchwork.libcamera.org/series/5630/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/25286/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 AEFBFBD80A\n\tfor ;\n\tTue, 2 Dec 2025 13:32:04 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 4563C60C77;\n\tTue, 2 Dec 2025 14:32:03 +0100 (CET)","from mail-wr1-x435.google.com (mail-wr1-x435.google.com\n\t[IPv6:2a00:1450:4864:20::435])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 464466096B\n\tfor ;\n\tTue, 2 Dec 2025 14:32:01 +0100 (CET)","by mail-wr1-x435.google.com with SMTP id\n\tffacd0b85a97d-42e2e239ec0so1934475f8f.0\n\tfor ;\n\tTue, 02 Dec 2025 05:32:01 -0800 (PST)","from inspiron14p-linux.ht.home (188-141-3-146.dynamic.upc.ie.\n\t[188.141.3.146]) by smtp.gmail.com with ESMTPSA id\n\tffacd0b85a97d-42e1c5c30c4sm33363322f8f.9.2025.12.02.05.31.59\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tTue, 02 Dec 2025 05:31:59 -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=\"vJockvVv\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=linaro.org; s=google; t=1764682320; x=1765287120;\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=IKAcX0QqWqaGxiVWpPCRUTz+HwO9lyiKPZnrYepitTM=;\n\tb=vJockvVvb43bfa0lWJUosytsMqZs4svTI71dhSQsaDHpR28hNmKZ9gj2L35oPX+WVw\n\tvT0hN+58bbMoQ8wMp2i+gZqkdjY7gi2LhRbjf3QIMBGk0CbszPOLwLJeIzbYN0fmVCzn\n\tJySK7QmpNyedyR3r4/zJBWkd8Qt2hThAN+/7LhMphxWecAKGtAXN1/w0hb4hCRqx6M2Y\n\t0kua7ctd7gxD+mYiKaAzV8Y3S6WWV+49uRqK28INNEL6nSWr0WyFDpk5HicdCVQkZ173\n\tip41VkgJVo57+ssSFlgtH7BpFLm1cpi+9AgRpEQNzX/+8SKiLWXdlENkQNhVrTzXtrQ6\n\taPdQ==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20230601; t=1764682320; x=1765287120;\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=IKAcX0QqWqaGxiVWpPCRUTz+HwO9lyiKPZnrYepitTM=;\n\tb=P50VqV2JL0sHuqgQMQhhxTYT3KRNfWycGJFNebIYPHp1UrM8SiLVg/Wdor2dEo38/2\n\t36Lc+cXkWVXBDbXFXApEgk01lzQwhCKJ6iFvpPHUKiTuqUVdS9SdtFQiE0tEPK9dEgW3\n\tjNkxnarLw++ezxy/8+saexQ4YoA5FcdP+F12T0745D3MupNzI9CiExURjphvn3PfcvDT\n\tzV8afSqweYimTdQzeEF5Y5EQZOx9KurHcYDcNRgKeRgjOst5+qGkTHPmp9e8csVu8zyZ\n\tprhBCnbStUjrNquAd4Fsk8sn8s/Dj+TgFr3DTubcIsR5UWYhi4/SmUh0HFnhRXQSnfpg\n\t2e/g==","X-Gm-Message-State":"AOJu0YzCeVQor1r75AEBtNkyPxAaRVcaOciM4G6yI7JVibWomV+SFeID\n\tbrsHuFT2WUZt4MVEbJT6dLzkjULAiMbuqFBYJbP6JubrJldAhwKCulIxcXAzIoGAkS+NkMuiikx\n\tcETd9","X-Gm-Gg":"ASbGnctunqNHwBF2M0RjgU8+XBEIkWkqhgWum/zAFkYyeswnzQEGnO2g2JT+Y9GVE3m\n\tI8O0oEBJ3iUEcTbjljBhoMHE0t8ORZ1g7PMSN71ukWPEI8eNdmqnLllHf+ZdQszyCv7kXrDW4nI\n\tUo8yBW3/RsWkc1ZK5wgw11emCRS18oneX19EcqoA1Ug6FaFUdmDgMTw8m8BBqN0t8/5PFJ5B6Mr\n\t5Kj+2A/rfsfaixMdmrepChuUOqNQbPkGNqeADgH+WSV/BCyv4ryX0TTGaN5jFJUSWrOIvLy7Mpt\n\tfALgNTWpmPQpEh3w/ilYYjI302PAcWXQxNVkiXa1Q0sIS14kjGu0AhwawtNSYkVWH3b2GLYyOnY\n\tdpEBZe6tUjZlggHq/eULpuisrFxDSKqOTYS72wnBFHxI1gFQHhnGLrnfC8ZZMCtO9cd99/ULBll\n\twDkYs2SPbFN2VT4Xj9CVqigMVX+U4nRkvNr42BheLvCOGTe7nUH7ekNZKAZ2147x78VGr5zB8Zw\n\tcKOyA==","X-Google-Smtp-Source":"AGHT+IGVVU5FN8w62AloHq/LJMensYOTw/0NXVxnn3Hec0vOGx+frHnCLlwAFqta56PCLcvjC5xgOw==","X-Received":"by 2002:a5d:5f44:0:b0:429:d350:802d with SMTP id\n\tffacd0b85a97d-42cc1d34848mr44985733f8f.45.1764682320338; \n\tTue, 02 Dec 2025 05:32:00 -0800 (PST)","From":"Bryan O'Donoghue ","To":"libcamera-devel@lists.libcamera.org","Cc":"pavel@ucw.cz,\n\tBryan O'Donoghue ","Subject":"[PATCH v3 00/22] GPUISP precursor series","Date":"Tue, 2 Dec 2025 13:31:31 +0000","Message-ID":"<20251202133157.661407-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":"v3:\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 (17):\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: Start the ISP thread in configure\n libcamera: software_isp: Move configure to worker thread\n libcamera: software_isp: debayer: Introduce a stop() callback to the\n 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 | 242 ++++++++++++++++++\n src/libcamera/software_isp/debayer.h | 43 +++-\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 | 24 +-\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, 733 insertions(+), 189 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"}