[{"id":36101,"web_url":"https://patchwork.libcamera.org/comment/36101/","msgid":"<5bcf1259-aa22-4740-bcf6-8cd21b34bbad@ideasonboard.com>","date":"2025-10-03T09:52:42","subject":"Re: [PATCH 2/2] ipa: ipa_module: Remove pipelineName","submitter":{"id":216,"url":"https://patchwork.libcamera.org/api/people/216/","name":"Barnabás Pőcze","email":"barnabas.pocze@ideasonboard.com"},"content":"2025. 10. 03. 11:27 keltezéssel, Jacopo Mondi írta:\n> All the IPA modules declare a pipelineName that is identical\n> to their name. As we now support creating IPAs by name\n> (either explicitly provided by the pipeline handlers or by using the\n> pipeline name), the duplicated information in IPAModuleInfo is\n> redundant.\n> \n> Remove it.\n> \n> Signed-off-by: Jacopo Mondi \n> ---\n\nReviewed-by: Barnabás Pőcze \n\n\n> include/libcamera/ipa/ipa_module_info.h | 1 -\n> src/ipa/ipu3/ipu3.cpp | 1 -\n> src/ipa/mali-c55/mali-c55.cpp | 1 -\n> src/ipa/rkisp1/rkisp1.cpp | 1 -\n> src/ipa/rpi/pisp/pisp.cpp | 1 -\n> src/ipa/rpi/vc4/vc4.cpp | 1 -\n> src/ipa/simple/soft_simple.cpp | 1 -\n> src/ipa/vimc/vimc.cpp | 1 -\n> src/libcamera/ipa_module.cpp | 15 ++++++---------\n> test/ipa/ipa_module_test.cpp | 3 ---\n> 10 files changed, 6 insertions(+), 20 deletions(-)\n> \n> diff --git a/include/libcamera/ipa/ipa_module_info.h b/include/libcamera/ipa/ipa_module_info.h\n> index 3507a6d7678a3fd171c9239967d4667381ac3ea6..436ec82d7750d1ab7901499f4c4d88f0a173b563 100644\n> --- a/include/libcamera/ipa/ipa_module_info.h\n> +++ b/include/libcamera/ipa/ipa_module_info.h\n> @@ -16,7 +16,6 @@ namespace libcamera {\n> struct IPAModuleInfo {\n> \tint moduleAPIVersion;\n> \tuint32_t pipelineVersion;\n> -\tchar pipelineName[256];\n> \tchar name[256];\n> } __attribute__((packed));\n> \n> diff --git a/src/ipa/ipu3/ipu3.cpp b/src/ipa/ipu3/ipu3.cpp\n> index 1cae08bf255f6f53a0ebee02ba3aadda5a650ae3..a714024129b688b0579662ac7d6d9c3ffc15b62a 100644\n> --- a/src/ipa/ipu3/ipu3.cpp\n> +++ b/src/ipa/ipu3/ipu3.cpp\n> @@ -672,7 +672,6 @@ const struct IPAModuleInfo ipaModuleInfo = {\n> \tIPA_MODULE_API_VERSION,\n> \t1,\n> \t\"ipu3\",\n> -\t\"ipu3\",\n> };\n> \n> /**\n> diff --git a/src/ipa/mali-c55/mali-c55.cpp b/src/ipa/mali-c55/mali-c55.cpp\n> index 7d45e7310aecdae0e47655e6d2e8830e776d74cd..e52e89be7c7e34310ca866bcbea2134833e967db 100644\n> --- a/src/ipa/mali-c55/mali-c55.cpp\n> +++ b/src/ipa/mali-c55/mali-c55.cpp\n> @@ -387,7 +387,6 @@ const struct IPAModuleInfo ipaModuleInfo = {\n> \tIPA_MODULE_API_VERSION,\n> \t1,\n> \t\"mali-c55\",\n> -\t\"mali-c55\",\n> };\n> \n> IPAInterface *ipaCreate()\n> diff --git a/src/ipa/rkisp1/rkisp1.cpp b/src/ipa/rkisp1/rkisp1.cpp\n> index fa22bfc349043002345d275b11a60ac983e329d7..a8e46094bdb9c431eb9143cccfa88e8ae7a3c04f 100644\n> --- a/src/ipa/rkisp1/rkisp1.cpp\n> +++ b/src/ipa/rkisp1/rkisp1.cpp\n> @@ -480,7 +480,6 @@ const struct IPAModuleInfo ipaModuleInfo = {\n> \tIPA_MODULE_API_VERSION,\n> \t1,\n> \t\"rkisp1\",\n> -\t\"rkisp1\",\n> };\n> \n> IPAInterface *ipaCreate()\n> diff --git a/src/ipa/rpi/pisp/pisp.cpp b/src/ipa/rpi/pisp/pisp.cpp\n> index 829b912585223a0296a9709aca433d5240caea1b..9424cb4d129dda541a3cc5f9bacac56dc44f15da 100644\n> --- a/src/ipa/rpi/pisp/pisp.cpp\n> +++ b/src/ipa/rpi/pisp/pisp.cpp\n> @@ -1089,7 +1089,6 @@ const IPAModuleInfo ipaModuleInfo = {\n> \tIPA_MODULE_API_VERSION,\n> \t1,\n> \t\"rpi/pisp\",\n> -\t\"rpi/pisp\",\n> };\n> \n> IPAInterface *ipaCreate()\n> diff --git a/src/ipa/rpi/vc4/vc4.cpp b/src/ipa/rpi/vc4/vc4.cpp\n> index b2fec934480487c0c0566d510afd7a72129e441c..dc3455642ec71865d73662e6cd2d42bba61cadb8 100644\n> --- a/src/ipa/rpi/vc4/vc4.cpp\n> +++ b/src/ipa/rpi/vc4/vc4.cpp\n> @@ -606,7 +606,6 @@ const struct IPAModuleInfo ipaModuleInfo = {\n> \tIPA_MODULE_API_VERSION,\n> \t1,\n> \t\"rpi/vc4\",\n> -\t\"rpi/vc4\",\n> };\n> \n> IPAInterface *ipaCreate()\n> diff --git a/src/ipa/simple/soft_simple.cpp b/src/ipa/simple/soft_simple.cpp\n> index b147aca2e3436cd79586063c4ee23f6698f266bf..9122f1c019ce1bb1c6a4d96e3fda664023391464 100644\n> --- a/src/ipa/simple/soft_simple.cpp\n> +++ b/src/ipa/simple/soft_simple.cpp\n> @@ -350,7 +350,6 @@ const struct IPAModuleInfo ipaModuleInfo = {\n> \tIPA_MODULE_API_VERSION,\n> \t0,\n> \t\"simple\",\n> -\t\"simple\",\n> };\n> \n> IPAInterface *ipaCreate()\n> diff --git a/src/ipa/vimc/vimc.cpp b/src/ipa/vimc/vimc.cpp\n> index a1351a0f45ce4ce6afed3151001eb74b10090c16..822e00aff2033e42254823b07ec396a50f71f182 100644\n> --- a/src/ipa/vimc/vimc.cpp\n> +++ b/src/ipa/vimc/vimc.cpp\n> @@ -202,7 +202,6 @@ const struct IPAModuleInfo ipaModuleInfo = {\n> \tIPA_MODULE_API_VERSION,\n> \t0,\n> \t\"vimc\",\n> -\t\"vimc\",\n> };\n> \n> IPAInterface *ipaCreate()\n> diff --git a/src/libcamera/ipa_module.cpp b/src/libcamera/ipa_module.cpp\n> index 0bd6f14626fe2038072f48b70ca4341b0eb8cef5..53cea6b187e86f20628d3b51c47097836801b094 100644\n> --- a/src/libcamera/ipa_module.cpp\n> +++ b/src/libcamera/ipa_module.cpp\n> @@ -215,18 +215,15 @@ Span elfLoadSymbol(Span elf, const char *symbol)\n> * \\var IPAModuleInfo::pipelineVersion\n> * \\brief The pipeline handler version that the IPA module is for\n> *\n> - * \\var IPAModuleInfo::pipelineName\n> - * \\brief The name of the pipeline handler that the IPA module is for\n> - *\n> - * This name is used to match a pipeline handler with the module.\n> - *\n> * \\var IPAModuleInfo::name\n> * \\brief The name of the IPA module\n> *\n> - * The name may be used to build file system paths to IPA-specific resources.\n> - * It shall only contain printable characters, and may not contain '*', '?' or\n> - * '\\'. For IPA modules included in libcamera, it shall match the directory of\n> - * the IPA module in the source tree.\n> + * This name is used to match a the IPA module.\n> + *\n> + * The name may also be used to build file system paths to IPA-specific\n> + * resources. It shall only contain printable characters, and may not contain\n> + * '*', '?' or '\\'. For IPA modules included in libcamera, it shall match the\n> + * directory of the IPA module in the source tree.\n> *\n> * \\todo Allow user to choose to isolate open source IPAs\n> */\n> diff --git a/test/ipa/ipa_module_test.cpp b/test/ipa/ipa_module_test.cpp\n> index 1c97da3242a7d558c52dce6ed3bc9b5f76742944..af71c3285328263da71643da97e7b8c440d9a7ad 100644\n> --- a/test/ipa/ipa_module_test.cpp\n> +++ b/test/ipa/ipa_module_test.cpp\n> @@ -37,12 +37,10 @@ protected:\n> \t\t\tcerr << \"IPA module information mismatch: expected:\" << endl\n> \t\t\t << \"moduleAPIVersion = \" << testInfo.moduleAPIVersion << endl\n> \t\t\t << \"pipelineVersion = \" << testInfo.pipelineVersion << endl\n> -\t\t\t << \"pipelineName = \" << testInfo.pipelineName << endl\n> \t\t\t << \"name = \" << testInfo.name\n> \t\t\t << \"got: \" << endl\n> \t\t\t << \"moduleAPIVersion = \" << info.moduleAPIVersion << endl\n> \t\t\t << \"pipelineVersion = \" << info.pipelineVersion << endl\n> -\t\t\t << \"pipelineName = \" << info.pipelineName << endl\n> \t\t\t << \"name = \" << info.name << endl;\n> \t\t}\n> \n> @@ -58,7 +56,6 @@ protected:\n> \t\t\tIPA_MODULE_API_VERSION,\n> \t\t\t0,\n> \t\t\t\"vimc\",\n> -\t\t\t\"vimc\",\n> \t\t};\n> \n> \t\tcount += runTest(\"src/ipa/vimc/ipa_vimc.so\", testInfo);\n>","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 CF66FC324C\n\tfor ;\n\tFri, 3 Oct 2025 09:52:47 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 6EB396B5F9;\n\tFri, 3 Oct 2025 11:52:47 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 9708D6B5AA\n\tfor ;\n\tFri, 3 Oct 2025 11:52:45 +0200 (CEST)","from [192.168.33.17] (185.182.214.142.nat.pool.zt.hu\n\t[185.182.214.142])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 3F1E419DC;\n\tFri, 3 Oct 2025 11:51: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=\"X+Dxf3PX\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1759485075;\n\tbh=SSIPVWuTXlDgOM91gff8RdehymG1cy6yFrJMhlCoFsM=;\n\th=Date:Subject:To:References:From:In-Reply-To:From;\n\tb=X+Dxf3PXVs8JUaN9YIuZGZe7IdUtUwZ1locJrVOFqLIOCXyp/+wAGfX0O9QVbeThe\n\tdYTCFwJnwLVJ9hGpDbwJlxOvMiA2+9xSknCFmletdriC/sCXaJjXTyYa6xv8+PRB/J\n\tcq15g+JxEBfTJhPyjIPOJZ4NgtvSdpRNFZmoc6Hg=","Message-ID":"<5bcf1259-aa22-4740-bcf6-8cd21b34bbad@ideasonboard.com>","Date":"Fri, 3 Oct 2025 11:52:42 +0200","MIME-Version":"1.0","User-Agent":"Mozilla Thunderbird","Subject":"Re: [PATCH 2/2] ipa: ipa_module: Remove pipelineName","To":"Jacopo Mondi ,\n\tlibcamera-devel@lists.libcamera.org, =?utf-8?q?Niklas_S=C3=B6derlund?=\n\t, Hans de Goede ","References":"<20251003-ipa-match-by-name-v1-0-07b796729412@ideasonboard.com>\n\t<20251003-ipa-match-by-name-v1-2-07b796729412@ideasonboard.com>","From":"=?utf-8?q?Barnab=C3=A1s_P=C5=91cze?= ","Content-Language":"en-US, hu-HU","In-Reply-To":"<20251003-ipa-match-by-name-v1-2-07b796729412@ideasonboard.com>","Content-Type":"text/plain; charset=UTF-8; format=flowed","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\" "}},{"id":36103,"web_url":"https://patchwork.libcamera.org/comment/36103/","msgid":"<175948782998.935713.7629570970676555548@ping.linuxembedded.co.uk>","date":"2025-10-03T10:37:09","subject":"Re: [PATCH 2/2] ipa: ipa_module: Remove pipelineName","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Quoting Jacopo Mondi (2025-10-03 10:27:30)\n> All the IPA modules declare a pipelineName that is identical\n> to their name. As we now support creating IPAs by name\n> (either explicitly provided by the pipeline handlers or by using the\n> pipeline name), the duplicated information in IPAModuleInfo is\n> redundant.\n> \n> Remove it.\n> \n> Signed-off-by: Jacopo Mondi \n\nReviewed-by: Kieran Bingham \n\n> ---\n> include/libcamera/ipa/ipa_module_info.h | 1 -\n> src/ipa/ipu3/ipu3.cpp | 1 -\n> src/ipa/mali-c55/mali-c55.cpp | 1 -\n> src/ipa/rkisp1/rkisp1.cpp | 1 -\n> src/ipa/rpi/pisp/pisp.cpp | 1 -\n> src/ipa/rpi/vc4/vc4.cpp | 1 -\n> src/ipa/simple/soft_simple.cpp | 1 -\n> src/ipa/vimc/vimc.cpp | 1 -\n> src/libcamera/ipa_module.cpp | 15 ++++++---------\n> test/ipa/ipa_module_test.cpp | 3 ---\n> 10 files changed, 6 insertions(+), 20 deletions(-)\n> \n> diff --git a/include/libcamera/ipa/ipa_module_info.h b/include/libcamera/ipa/ipa_module_info.h\n> index 3507a6d7678a3fd171c9239967d4667381ac3ea6..436ec82d7750d1ab7901499f4c4d88f0a173b563 100644\n> --- a/include/libcamera/ipa/ipa_module_info.h\n> +++ b/include/libcamera/ipa/ipa_module_info.h\n> @@ -16,7 +16,6 @@ namespace libcamera {\n> struct IPAModuleInfo {\n> int moduleAPIVersion;\n> uint32_t pipelineVersion;\n> - char pipelineName[256];\n> char name[256];\n> } __attribute__((packed));\n> \n> diff --git a/src/ipa/ipu3/ipu3.cpp b/src/ipa/ipu3/ipu3.cpp\n> index 1cae08bf255f6f53a0ebee02ba3aadda5a650ae3..a714024129b688b0579662ac7d6d9c3ffc15b62a 100644\n> --- a/src/ipa/ipu3/ipu3.cpp\n> +++ b/src/ipa/ipu3/ipu3.cpp\n> @@ -672,7 +672,6 @@ const struct IPAModuleInfo ipaModuleInfo = {\n> IPA_MODULE_API_VERSION,\n> 1,\n> \"ipu3\",\n> - \"ipu3\",\n> };\n> \n> /**\n> diff --git a/src/ipa/mali-c55/mali-c55.cpp b/src/ipa/mali-c55/mali-c55.cpp\n> index 7d45e7310aecdae0e47655e6d2e8830e776d74cd..e52e89be7c7e34310ca866bcbea2134833e967db 100644\n> --- a/src/ipa/mali-c55/mali-c55.cpp\n> +++ b/src/ipa/mali-c55/mali-c55.cpp\n> @@ -387,7 +387,6 @@ const struct IPAModuleInfo ipaModuleInfo = {\n> IPA_MODULE_API_VERSION,\n> 1,\n> \"mali-c55\",\n> - \"mali-c55\",\n> };\n> \n> IPAInterface *ipaCreate()\n> diff --git a/src/ipa/rkisp1/rkisp1.cpp b/src/ipa/rkisp1/rkisp1.cpp\n> index fa22bfc349043002345d275b11a60ac983e329d7..a8e46094bdb9c431eb9143cccfa88e8ae7a3c04f 100644\n> --- a/src/ipa/rkisp1/rkisp1.cpp\n> +++ b/src/ipa/rkisp1/rkisp1.cpp\n> @@ -480,7 +480,6 @@ const struct IPAModuleInfo ipaModuleInfo = {\n> IPA_MODULE_API_VERSION,\n> 1,\n> \"rkisp1\",\n> - \"rkisp1\",\n> };\n> \n> IPAInterface *ipaCreate()\n> diff --git a/src/ipa/rpi/pisp/pisp.cpp b/src/ipa/rpi/pisp/pisp.cpp\n> index 829b912585223a0296a9709aca433d5240caea1b..9424cb4d129dda541a3cc5f9bacac56dc44f15da 100644\n> --- a/src/ipa/rpi/pisp/pisp.cpp\n> +++ b/src/ipa/rpi/pisp/pisp.cpp\n> @@ -1089,7 +1089,6 @@ const IPAModuleInfo ipaModuleInfo = {\n> IPA_MODULE_API_VERSION,\n> 1,\n> \"rpi/pisp\",\n> - \"rpi/pisp\",\n> };\n> \n> IPAInterface *ipaCreate()\n> diff --git a/src/ipa/rpi/vc4/vc4.cpp b/src/ipa/rpi/vc4/vc4.cpp\n> index b2fec934480487c0c0566d510afd7a72129e441c..dc3455642ec71865d73662e6cd2d42bba61cadb8 100644\n> --- a/src/ipa/rpi/vc4/vc4.cpp\n> +++ b/src/ipa/rpi/vc4/vc4.cpp\n> @@ -606,7 +606,6 @@ const struct IPAModuleInfo ipaModuleInfo = {\n> IPA_MODULE_API_VERSION,\n> 1,\n> \"rpi/vc4\",\n> - \"rpi/vc4\",\n> };\n> \n> IPAInterface *ipaCreate()\n> diff --git a/src/ipa/simple/soft_simple.cpp b/src/ipa/simple/soft_simple.cpp\n> index b147aca2e3436cd79586063c4ee23f6698f266bf..9122f1c019ce1bb1c6a4d96e3fda664023391464 100644\n> --- a/src/ipa/simple/soft_simple.cpp\n> +++ b/src/ipa/simple/soft_simple.cpp\n> @@ -350,7 +350,6 @@ const struct IPAModuleInfo ipaModuleInfo = {\n> IPA_MODULE_API_VERSION,\n> 0,\n> \"simple\",\n> - \"simple\",\n> };\n> \n> IPAInterface *ipaCreate()\n> diff --git a/src/ipa/vimc/vimc.cpp b/src/ipa/vimc/vimc.cpp\n> index a1351a0f45ce4ce6afed3151001eb74b10090c16..822e00aff2033e42254823b07ec396a50f71f182 100644\n> --- a/src/ipa/vimc/vimc.cpp\n> +++ b/src/ipa/vimc/vimc.cpp\n> @@ -202,7 +202,6 @@ const struct IPAModuleInfo ipaModuleInfo = {\n> IPA_MODULE_API_VERSION,\n> 0,\n> \"vimc\",\n> - \"vimc\",\n> };\n> \n> IPAInterface *ipaCreate()\n> diff --git a/src/libcamera/ipa_module.cpp b/src/libcamera/ipa_module.cpp\n> index 0bd6f14626fe2038072f48b70ca4341b0eb8cef5..53cea6b187e86f20628d3b51c47097836801b094 100644\n> --- a/src/libcamera/ipa_module.cpp\n> +++ b/src/libcamera/ipa_module.cpp\n> @@ -215,18 +215,15 @@ Span elfLoadSymbol(Span elf, const char *symbol)\n> * \\var IPAModuleInfo::pipelineVersion\n> * \\brief The pipeline handler version that the IPA module is for\n> *\n> - * \\var IPAModuleInfo::pipelineName\n> - * \\brief The name of the pipeline handler that the IPA module is for\n> - *\n> - * This name is used to match a pipeline handler with the module.\n> - *\n> * \\var IPAModuleInfo::name\n> * \\brief The name of the IPA module\n> *\n> - * The name may be used to build file system paths to IPA-specific resources.\n> - * It shall only contain printable characters, and may not contain '*', '?' or\n> - * '\\'. For IPA modules included in libcamera, it shall match the directory of\n> - * the IPA module in the source tree.\n> + * This name is used to match a the IPA module.\n> + *\n> + * The name may also be used to build file system paths to IPA-specific\n> + * resources. It shall only contain printable characters, and may not contain\n> + * '*', '?' or '\\'. For IPA modules included in libcamera, it shall match the\n> + * directory of the IPA module in the source tree.\n> *\n> * \\todo Allow user to choose to isolate open source IPAs\n> */\n> diff --git a/test/ipa/ipa_module_test.cpp b/test/ipa/ipa_module_test.cpp\n> index 1c97da3242a7d558c52dce6ed3bc9b5f76742944..af71c3285328263da71643da97e7b8c440d9a7ad 100644\n> --- a/test/ipa/ipa_module_test.cpp\n> +++ b/test/ipa/ipa_module_test.cpp\n> @@ -37,12 +37,10 @@ protected:\n> cerr << \"IPA module information mismatch: expected:\" << endl\n> << \"moduleAPIVersion = \" << testInfo.moduleAPIVersion << endl\n> << \"pipelineVersion = \" << testInfo.pipelineVersion << endl\n> - << \"pipelineName = \" << testInfo.pipelineName << endl\n> << \"name = \" << testInfo.name\n> << \"got: \" << endl\n> << \"moduleAPIVersion = \" << info.moduleAPIVersion << endl\n> << \"pipelineVersion = \" << info.pipelineVersion << endl\n> - << \"pipelineName = \" << info.pipelineName << endl\n> << \"name = \" << info.name << endl;\n> }\n> \n> @@ -58,7 +56,6 @@ protected:\n> IPA_MODULE_API_VERSION,\n> 0,\n> \"vimc\",\n> - \"vimc\",\n> };\n> \n> count += runTest(\"src/ipa/vimc/ipa_vimc.so\", testInfo);\n> \n> -- \n> 2.51.0\n>","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 2815DC324C\n\tfor ;\n\tFri, 3 Oct 2025 10:37:17 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 2C5826B5AA;\n\tFri, 3 Oct 2025 12:37:16 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id C816069318\n\tfor ;\n\tFri, 3 Oct 2025 12:37: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 DE243192C;\n\tFri, 3 Oct 2025 12:35:42 +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=\"kKez2ETJ\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1759487743;\n\tbh=qTZCAWmG+kxjw2IshUFk95MXJoiJVfXhdbyFkPP8O0M=;\n\th=In-Reply-To:References:Subject:From:Cc:To:Date:From;\n\tb=kKez2ETJ2owmVlhvtUdoAz6f4yD9IX7+F9i3+Dslk+vOci7OEJ9DGiIZJ1TsvwHma\n\t3PH67IwvLSutM4aMAKT2ui2/xR+te3JJG3hV1RMl/QdiW1bTiICoABTQWJNy/qwyQO\n\t9ZUJPujJrkXoN/y7CmlNxrPAsW9HbWjMdIn5my78=","Content-Type":"text/plain; charset=\"utf-8\"","MIME-Version":"1.0","Content-Transfer-Encoding":"quoted-printable","In-Reply-To":"<20251003-ipa-match-by-name-v1-2-07b796729412@ideasonboard.com>","References":"<20251003-ipa-match-by-name-v1-0-07b796729412@ideasonboard.com>\n\t<20251003-ipa-match-by-name-v1-2-07b796729412@ideasonboard.com>","Subject":"Re: [PATCH 2/2] ipa: ipa_module: Remove pipelineName","From":"Kieran Bingham ","Cc":"Jacopo Mondi ","To":"Hans de Goede , Jacopo Mondi\n\t, Niklas =?utf-8?q?S=C3=B6derlund?=\n\t,\n\tlibcamera-devel@lists.libcamera.org","Date":"Fri, 03 Oct 2025 11:37:09 +0100","Message-ID":"<175948782998.935713.7629570970676555548@ping.linuxembedded.co.uk>","User-Agent":"alot/0.9.1","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\" "}}]