[{"id":31449,"web_url":"https://patchwork.libcamera.org/comment/31449/","msgid":"<172745179088.532453.1156288224348230286@ping.linuxembedded.co.uk>","date":"2024-09-27T15:43:10","subject":"Re: [PATCH v8 00/18] Software ISP refactoring","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Quoting Milan Zamazal (2024-09-27 14:46:06)\n> The purpose of this patch series is to bring software ISP code\n> structuring closer to the hardware pipelines.  Most notably, the API\n> around algorithm.h is used now.  This should make software ISP easier to\n> understand, extend, maintain, and code-share with the other pipelines.\n> \n> What is omitted in the patch series:\n> \n> - Any bigger or unrelated functional changes.  The purpose of this\n>   series is code restructuring, which is enough already.\n> - Stats and params buffers are still used in the original way.  Making\n>   their handling closer to the hardware pipelines will be a subject of\n>   followup patches.  This series is a preparation step for that.\n> - Any IPA code sharing with hardware pipelines.  If there is an\n>   opportunity for this, it can be addressed in followup patches.\n\nI've merged this series. I hope that helps you with the items you are\ntackling on top.\n\n--\nKieran","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 0287DC3257\n\tfor <parsemail@patchwork.libcamera.org>;\n\tFri, 27 Sep 2024 15:43:16 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id C45C96350F;\n\tFri, 27 Sep 2024 17:43:15 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id E448F634F2\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 27 Sep 2024 17:43:13 +0200 (CEST)","from pendragon.ideasonboard.com\n\t(cpc89244-aztw30-2-0-cust6594.18-1.cable.virginm.net [86.31.185.195])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id BF0A2163;\n\tFri, 27 Sep 2024 17:41:44 +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=\"oHKORaHd\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1727451704;\n\tbh=UAJPkLtYqqt7f/j77CawO9Rpc1CDB+GKNEtjesNrhaY=;\n\th=In-Reply-To:References:Subject:From:Cc:To:Date:From;\n\tb=oHKORaHd9NLApkOXqpDeM1DSgBwtG8t2l62Sdk2qyRzx28YlkjVSg2WG/ZqPwwmeN\n\tdV1iowf2b3AfsKlFEUnflIop73tUvECi9hg94Y+ntYoNHJwDrShLDl+F4HC4eP1wLV\n\ts47vJtF36Zy2NP5IOIgBnp+f/rZLkNLMdR02thvY=","Content-Type":"text/plain; charset=\"utf-8\"","MIME-Version":"1.0","Content-Transfer-Encoding":"quoted-printable","In-Reply-To":"<20240927134624.801004-1-mzamazal@redhat.com>","References":"<20240927134624.801004-1-mzamazal@redhat.com>","Subject":"Re: [PATCH v8 00/18] Software ISP refactoring","From":"Kieran Bingham <kieran.bingham@ideasonboard.com>","Cc":"Milan Zamazal <mzamazal@redhat.com>,\n\tUmang Jain <umang.jain@ideasonboard.com>,\n\tLaurent Pinchart <laurent.pinchart@ideasonboard.com>,\n\tDaniel Scally <dan.scally@ideasonboard.com>","To":"Milan Zamazal <mzamazal@redhat.com>, libcamera-devel@lists.libcamera.org","Date":"Fri, 27 Sep 2024 16:43:10 +0100","Message-ID":"<172745179088.532453.1156288224348230286@ping.linuxembedded.co.uk>","User-Agent":"alot/0.10","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>"}},{"id":31454,"web_url":"https://patchwork.libcamera.org/comment/31454/","msgid":"<87r095ghpc.fsf@redhat.com>","date":"2024-09-27T16:54:55","subject":"Re: [PATCH v8 00/18] Software ISP refactoring","submitter":{"id":177,"url":"https://patchwork.libcamera.org/api/people/177/","name":"Milan Zamazal","email":"mzamazal@redhat.com"},"content":"Hi Kieran,\n\nKieran Bingham <kieran.bingham@ideasonboard.com> writes:\n\n> Quoting Milan Zamazal (2024-09-27 14:46:06)\n>> The purpose of this patch series is to bring software ISP code\n>> structuring closer to the hardware pipelines.  Most notably, the API\n>\n>> around algorithm.h is used now.  This should make software ISP easier to\n>> understand, extend, maintain, and code-share with the other pipelines.\n>> \n>> What is omitted in the patch series:\n>> \n>> - Any bigger or unrelated functional changes.  The purpose of this\n>>   series is code restructuring, which is enough already.\n>> - Stats and params buffers are still used in the original way.  Making\n>>   their handling closer to the hardware pipelines will be a subject of\n>>   followup patches.  This series is a preparation step for that.\n>> - Any IPA code sharing with hardware pipelines.  If there is an\n>>   opportunity for this, it can be addressed in followup patches.\n>\n> I've merged this series. I hope that helps you with the items you are\n> tackling on top.\n\nDefinitely, thank you!\n\n> --\n> Kieran","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 EEC5BC3257\n\tfor <parsemail@patchwork.libcamera.org>;\n\tFri, 27 Sep 2024 16:55:03 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 0BA666350F;\n\tFri, 27 Sep 2024 18:55:03 +0200 (CEST)","from us-smtp-delivery-124.mimecast.com\n\t(us-smtp-delivery-124.mimecast.com [170.10.129.124])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id C0D5A634F2\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 27 Sep 2024 18:55:00 +0200 (CEST)","from mail-wm1-f71.google.com (mail-wm1-f71.google.com\n\t[209.85.128.71]) by relay.mimecast.com with ESMTP with STARTTLS\n\t(version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id\n\tus-mta-159-v6oSsvYMN_WKsaqgm8kGiQ-1; Fri, 27 Sep 2024 12:54:58 -0400","by mail-wm1-f71.google.com with SMTP id\n\t5b1f17b1804b1-42cb89fbb8bso14518985e9.3\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 27 Sep 2024 09:54:58 -0700 (PDT)","from nuthatch (ip-77-48-47-2.net.vodafone.cz. [77.48.47.2])\n\tby smtp.gmail.com with ESMTPSA id\n\tffacd0b85a97d-37cd57427b9sm2948215f8f.95.2024.09.27.09.54.55\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tFri, 27 Sep 2024 09:54:56 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=redhat.com header.i=@redhat.com\n\theader.b=\"XKs1ElX8\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n\ts=mimecast20190719; t=1727456099;\n\th=from:from:reply-to:subject:subject:date:date:message-id:message-id:\n\tto:to:cc:cc:mime-version:mime-version:content-type:content-type:\n\tin-reply-to:in-reply-to:references:references;\n\tbh=n90GdTeKRGN/omSsF0bn+kG2A3a7DArp+h25IACq2QY=;\n\tb=XKs1ElX8EsjI6dUCAaCKG8buOos+0qjeY0ZD8eMs+R2gPmFTPeOuYzIEQzg7y8xkvb9kju\n\tZB9GmmU+/KhCTYT8qGsZtlinGaiJPeDYANotomCYrK0gwkg2R62MJr9ehtrM++K6d34+1X\n\tvhDRmkJq66hzZPRmARlr+ZfzHl1DWk0=","X-MC-Unique":"v6oSsvYMN_WKsaqgm8kGiQ-1","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20230601; t=1727456097; x=1728060897;\n\th=mime-version:user-agent:message-id:date:references:in-reply-to\n\t:subject:cc:to:from:x-gm-message-state:from:to:cc:subject:date\n\t:message-id:reply-to;\n\tbh=n90GdTeKRGN/omSsF0bn+kG2A3a7DArp+h25IACq2QY=;\n\tb=Pm2ve6gMnjjeHEQ2OonUvW3s96HOrsmJhXtxkuNXzwo1NkWv3SttgkyzEMCdHI9RYP\n\tXLWrA2m2ef1qxPM4MqUFBf14AKbXfHxKJNH7C3OmgsY9hRH7pZnOUpftQL9nXPvOOOzd\n\tTtF1Nc4wrd8U2duLJODs0ubDDR2+GBU6fEZi6zoJJTOm8wxtOKnj9EDJVfHlQUVYlB09\n\tRlL0FpGkuJIUJStez7u3UXvMh+KBxc1++xJgsQ+BYXNcr7B558BKLIyUkOH/ve8psjvO\n\tbH8YcwciJr+/YS3V00ukqES3PNjkSFFsEnPeOcRAyMDKWum8rOdKhsVjT2iQ4OKKqeHp\n\tf/kA==","X-Gm-Message-State":"AOJu0YwMFqmPl/npAZYXfnIoAxMB/ZfBv1EP8GeNupBOoi3R3CYcGbux\n\tnaX4biV/wkOBo/ha9mJYXJJZ8UeVdU+VPYpQrY9+rNP/1DMrow9MwxGioEw9lWhIepAFJZDkbLw\n\t1gesQyqNx8pge5f3B2wrQjAMqc1CJw4Z73HTBtNAzSKJm/mRHjYBFL3FoEYpQvr1CYwUtwMasrv\n\t/ceNo=","X-Received":["by 2002:a05:600c:3c94:b0:426:6326:4cec with SMTP id\n\t5b1f17b1804b1-42f5849081cmr27768545e9.29.1727456097014; \n\tFri, 27 Sep 2024 09:54:57 -0700 (PDT)","by 2002:a05:600c:3c94:b0:426:6326:4cec with SMTP id\n\t5b1f17b1804b1-42f5849081cmr27768395e9.29.1727456096587; \n\tFri, 27 Sep 2024 09:54:56 -0700 (PDT)"],"X-Google-Smtp-Source":"AGHT+IHQn17zpmOwPGoqJWkixHrSyC3VIeefbDnlD3iWVoJU3Fn2nExhD4tXzueOUeaX9/yFBMspPg==","From":"Milan Zamazal <mzamazal@redhat.com>","To":"Kieran Bingham <kieran.bingham@ideasonboard.com>","Cc":"libcamera-devel@lists.libcamera.org,  Umang Jain\n\t<umang.jain@ideasonboard.com>,  Laurent Pinchart\n\t<laurent.pinchart@ideasonboard.com>,  Daniel Scally\n\t<dan.scally@ideasonboard.com>","Subject":"Re: [PATCH v8 00/18] Software ISP refactoring","In-Reply-To":"<172745179088.532453.1156288224348230286@ping.linuxembedded.co.uk>\n\t(Kieran Bingham's message of \"Fri, 27 Sep 2024 16:43:10 +0100\")","References":"<20240927134624.801004-1-mzamazal@redhat.com>\n\t<172745179088.532453.1156288224348230286@ping.linuxembedded.co.uk>","Date":"Fri, 27 Sep 2024 18:54:55 +0200","Message-ID":"<87r095ghpc.fsf@redhat.com>","User-Agent":"Gnus/5.13 (Gnus v5.13)","MIME-Version":"1.0","X-Mimecast-Spam-Score":"0","X-Mimecast-Originator":"redhat.com","Content-Type":"text/plain","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>"}}]