[{"id":38979,"web_url":"https://patchwork.libcamera.org/comment/38979/","msgid":"","date":"2026-06-02T08:28:53","subject":"Re: [PATCH v2] libcamera: software_isp: debayer_egl: Demote\n\tunsupported format log to Info","submitter":{"id":175,"url":"https://patchwork.libcamera.org/api/people/175/","name":"Bryan O'Donoghue","email":"bryan.odonoghue@linaro.org"},"content":"On 02/06/2026 02:26, Qi Hou wrote:\n> From: Hou Qi \n> \n> DebayerEGL::getInputConfig() logs at Error level when it encounters an\n> unsupported input format. This function is called during format\n> enumeration from the simple pipeline handler's tryPipeline(), which\n> iterates over all pixel formats reported by the video capture device,\n> including non-Bayer formats such as UYVY and YUYV. The caller already\n> handles the failure gracefully by checking the return value. Similarly,\n> DebayerEGL::getOutputConfig() logs at Error level for unsupported\n> output formats, but is called from strideAndFrameSize() which also\n> handles failures via the return value.\n> \n> Demote both log messages from Error to Info for consistency with\n> DebayerCpu, which uses Info level in getInputConfig() and\n> getOutputConfig() for the same situation.\n> \n> Signed-off-by: Qi Hou \n> ---\n> src/libcamera/software_isp/debayer_egl.cpp | 4 ++--\n> 1 file changed, 2 insertions(+), 2 deletions(-)\n> \n> diff --git a/src/libcamera/software_isp/debayer_egl.cpp b/src/libcamera/software_isp/debayer_egl.cpp\n> index 7b9e02d9..68779893 100644\n> --- a/src/libcamera/software_isp/debayer_egl.cpp\n> +++ b/src/libcamera/software_isp/debayer_egl.cpp\n> @@ -78,7 +78,7 @@ int DebayerEGL::getInputConfig(PixelFormat inputFormat, DebayerInputConfig &conf\n> \t\treturn 0;\n> \t}\n> \n> -\tLOG(Debayer, Error)\n> +\tLOG(Debayer, Info)\n> \t\t<< \"Unsupported input format \" << inputFormat;\n> \n> \treturn -EINVAL;\n> @@ -92,7 +92,7 @@ int DebayerEGL::getOutputConfig(PixelFormat outputFormat, DebayerOutputConfig &c\n> \t\treturn 0;\n> \t}\n> \n> -\tLOG(Debayer, Error)\n> +\tLOG(Debayer, Info)\n> \t\t<< \"Unsupported output format \" << outputFormat;\n> \n> \treturn -EINVAL;\n> --\n> 2.34.1\n> \n\nReviewed-by: Bryan O'Donoghue \n\n---\nbod","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":["parsemail@patchwork.libcamera.org","kbingham@ideasonboard.com"],"Received":["from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id D053DC328C\n\tfor ;\n\tTue, 2 Jun 2026 11:47:05 +0000 (UTC)","from monstersaurus.ideasonboard.com\n\t(cpc89244-aztw30-2-0-cust6594.18-1.cable.virginm.net\n\t[86.31.185.195])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 6D6AF55\n\tfor ;\n\tTue, 2 Jun 2026 13:46:42 +0200 (CEST)","from perceval.ideasonboard.com\n\tby perceval.ideasonboard.com with LMTP id 6BoiEjSUHmoEOT0A4E0KoQ\n\t(envelope-from )\n\tfor ; Tue, 02 Jun 2026 10:28:36 +0200","from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\tby perceval.ideasonboard.com (Postfix) with ESMTPS\n\tid BA03E802;\tTue, 2 Jun 2026 10:28:35 +0200 (CEST)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id A5FA263035;\n\tTue, 2 Jun 2026 10:28:57 +0200 (CEST)","from mail-wm1-x32b.google.com (mail-wm1-x32b.google.com\n\t[IPv6:2a00:1450:4864:20::32b])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 08E4763035\n\tfor ;\n\tTue, 2 Jun 2026 10:28:56 +0200 (CEST)","by mail-wm1-x32b.google.com with SMTP id\n\t5b1f17b1804b1-4905529b933so85691105e9.0\n\tfor ;\n\tTue, 02 Jun 2026 01:28:55 -0700 (PDT)","from [192.168.0.101] ([109.76.64.105])\n\tby smtp.gmail.com with ESMTPSA id\n\t5b1f17b1804b1-490b0e7639csm51303965e9.10.2026.06.02.01.28.54\n\t(version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128);\n\tTue, 02 Jun 2026 01:28:54 -0700 (PDT)"],"Authentication-Results":["perceval.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=linaro.org header.i=@linaro.org\n\theader.a=rsa-sha256 header.s=google header.b=n8kULTuT; \n\tdkim-atps=neutral","lancelot.ideasonboard.com; dkim=pass (2048-bit key; \n\tunprotected) header.d=linaro.org\n\theader.i=@linaro.org header.b=\"n8kULTuT\"; \n\tdkim-atps=neutral"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google;\n\tt=1780388935; x=1780993735; darn=lists.libcamera.org; \n\th=content-transfer-encoding:in-reply-to:content-language:from\n\t:references:cc:to:subject:user-agent:mime-version:date:message-id\n\t:from:to:cc:subject:date:message-id:reply-to;\n\tbh=SfJ7vispKC5EH0JLHhp+uPr9N/XXNNDRI9Bkyn0+ApM=;\n\tb=n8kULTuTZPe4X8m0ogGC5hDHp8j13gp0jmPUBCHizR9ROjPKco/aaH7IN7FVzVzvcb\n\t3Ni+DCR/CweDwjUJk+1mhEVRdaGQBnrcjqSvCf7ABGVrPklbIeyY+EVBOam7MfITxpUC\n\tpUWky3H+oIocKsT8pk81VbOb6Ed2bRLGPrqC/ClBKZEJDkacq7ZevC2zbQJYsm4plXNx\n\tv+NefJy/ujaYx9Q44JR0wzjIFU2RNEu0f88YUMu1OT/VKBQPUDazDcUfvd46wJVCqYyR\n\tUeROpT4Hmj80xe9jtvQ1aFn6yc57mfREJlsut5RHKDEHVGagVv0kfJ/LVyHnUrjI+rWY\n\tiXWA==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20251104; t=1780388935; x=1780993735;\n\th=content-transfer-encoding:in-reply-to:content-language:from\n\t:references:cc:to:subject:user-agent:mime-version:date:message-id\n\t:x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id\n\t:reply-to;\n\tbh=SfJ7vispKC5EH0JLHhp+uPr9N/XXNNDRI9Bkyn0+ApM=;\n\tb=hRV8oWAQm9pTmKQVr2Z0Ni82UBSbADXBtbVm0VXEv6C+rinRW7pHwvu71mGalid5hD\n\tFpjjSVdg+uBOaEHmd3gfkw2kzoiCaFvyU0OTKz0Wx/do/OBy+SmLGC27HkdnP4wRRNkU\n\tDgpyAsp3AbVjQSnwYVOrWdmdmjYj0g/Uz4wDJUZMyZuKDWo3udlP/ROy9AR3ayU4BWTk\n\tfDmvi1qs3o7ZBihaiAL63jYfAWIsYAKdxlSp4S2d9ecfQq65SnRdSCVrYUpEnZHnfm7T\n\toqU07hrVNMOAeQbewAzOGFS3j//kEkOtgeMfEk+epAVptdmDdON60Nk3Jjn6qVniFfyh\n\t6IHw==","X-Forwarded-Encrypted":"i=1; =?utf-8?q?AFNElJ8o7jzShXgyIY0NO1sWXqSSA78N4eiJ?=\n\t=?utf-8?q?FhKIaSRT7m/bQuUt2XDZfS9BgCyKi6s3omW/eah7H5XtJv9wOQ9KP/Y?=\n\t=?utf-8?b?PUBsaXN0cy5saWJjYW1lcmEub3Jn?=","X-Gm-Message-State":"AOJu0Yxy9L8daDzrjgTOQCfx6oFqsgBT90eAy3Gy+nfY5Gtv/fAGLtvw\n\tDwswULm9frXxGUSLP4q1A0a42Ev1RqqpBVosdGD/IcOC1esi/ohpUyAQAdQW5eItErJHWFdRC3x\n\tTTCjqUfY=","X-Gm-Gg":"Acq92OFjhjogHMNgwTwPxj62Z7Niyx7CQLwXszPoA6GIWBTUDfnzQCN9sOi5hSjbsVh\n\tFjDw80znBGEaXd/jLBWtXuPpW2tjUg4PXsi8pZW9xMmsouwvBO+cE+lj6mCfyARUu1KsOcV8Z/6\n\tzHuD2F4qs3E4QzFAtg5wwtc8nvdNYMfml9W3CItquG+kwDJUJHvHFLih7OfYx5mMNWuOAiJNInD\n\tUM/moMM0NWf23JD/yWhiLQSMqK+Is+/qrpNAh23gOOp1qFmT4PntdvOG5b92vQNf1A7JVIMTAGs\n\tYPqCBMwObhOfA1mdrI02IVB97oaEkKQ15+6cKQC1mmoYhIGWcLVww6u++zqVBs63H/Xu0lHrtft\n\tgv8JKQG+nAqDvZwAVmHibsETlr3t7rCOeW1fL6qIBwk0zszpzxzXST7L7LDliBl6oM9yBNG55nm\n\tK0fuagXg8VZGYydAuBNfVWmRdGye01OmBWN8U1BYtkTb8ojA==","X-Received":"by 2002:a05:600c:3496:b0:490:b294:c652 with SMTP id\n\t5b1f17b1804b1-490b294c6c7mr23251775e9.20.1780388935220; \n\tTue, 02 Jun 2026 01:28:55 -0700 (PDT)","Message-ID":"","Date":"Tue, 2 Jun 2026 09:28:53 +0100","MIME-Version":"1.0","User-Agent":"Mozilla Thunderbird","Subject":"Re: [PATCH v2] libcamera: software_isp: debayer_egl: Demote\n\tunsupported format log to Info","To":"Qi Hou , libcamera-devel@lists.libcamera.org","Cc":"jared.hu@nxp.com, julien.vuillaumier@nxp.com","References":"=?utf-8?q?=3CsYRPmksJNVmP6m7xn3KOjUbJ0L7CCDu1ddf9uuoHPQm-9H7b?=\n\t=?utf-8?q?JzfD2D8mITk60z3zFxUpBudQmUAWBII3TPLplQ=3D=3D=40protonmai?=\n\t=?utf-8?q?l=2Einternalid=3E?=\n\t<20260602012601.2799275-1-qi.hou@nxp.com>","From":"Bryan O'Donoghue ","Content-Language":"en-US","In-Reply-To":"<20260602012601.2799275-1-qi.hou@nxp.com>","Content-Type":"text/plain; charset=UTF-8; format=flowed","Content-Transfer-Encoding":"7bit","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\" ","X-TUID":"Vg91JnjZjQRb","Resent-From":"Kieran Bingham ","Resent-To":"parsemail@patchwork.libcamera.org"}},{"id":38980,"web_url":"https://patchwork.libcamera.org/comment/38980/","msgid":"<178039317209.815980.16798958776686332830@ping.linuxembedded.co.uk>","date":"2026-06-02T09:39:32","subject":"Re: [PATCH v2] libcamera: software_isp: debayer_egl: Demote\n\tunsupported format log to Info","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Quoting Qi Hou (2026-06-02 02:26:01)\n> From: Hou Qi \n> \n> DebayerEGL::getInputConfig() logs at Error level when it encounters an\n> unsupported input format. This function is called during format\n> enumeration from the simple pipeline handler's tryPipeline(), which\n> iterates over all pixel formats reported by the video capture device,\n> including non-Bayer formats such as UYVY and YUYV. The caller already\n> handles the failure gracefully by checking the return value. Similarly,\n> DebayerEGL::getOutputConfig() logs at Error level for unsupported\n> output formats, but is called from strideAndFrameSize() which also\n> handles failures via the return value.\n> \n> Demote both log messages from Error to Info for consistency with\n> DebayerCpu, which uses Info level in getInputConfig() and\n> getOutputConfig() for the same situation.\n> \n\n\nReviewed-by: Kieran Bingham \n\n> Signed-off-by: Qi Hou \n> ---\n> src/libcamera/software_isp/debayer_egl.cpp | 4 ++--\n> 1 file changed, 2 insertions(+), 2 deletions(-)\n> \n> diff --git a/src/libcamera/software_isp/debayer_egl.cpp b/src/libcamera/software_isp/debayer_egl.cpp\n> index 7b9e02d9..68779893 100644\n> --- a/src/libcamera/software_isp/debayer_egl.cpp\n> +++ b/src/libcamera/software_isp/debayer_egl.cpp\n> @@ -78,7 +78,7 @@ int DebayerEGL::getInputConfig(PixelFormat inputFormat, DebayerInputConfig &conf\n> return 0;\n> }\n> \n> - LOG(Debayer, Error)\n> + LOG(Debayer, Info)\n> << \"Unsupported input format \" << inputFormat;\n> \n> return -EINVAL;\n> @@ -92,7 +92,7 @@ int DebayerEGL::getOutputConfig(PixelFormat outputFormat, DebayerOutputConfig &c\n> return 0;\n> }\n> \n> - LOG(Debayer, Error)\n> + LOG(Debayer, Info)\n> << \"Unsupported output format \" << outputFormat;\n> \n> return -EINVAL;\n> -- \n> 2.34.1\n>","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id D0627C328C\n\tfor ;\n\tTue, 2 Jun 2026 11:47:07 +0000 (UTC)","from monstersaurus.ideasonboard.com\n\t(cpc89244-aztw30-2-0-cust6594.18-1.cable.virginm.net\n\t[86.31.185.195])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id B00FA55\n\tfor ;\n\tTue, 2 Jun 2026 13:46:44 +0200 (CEST)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1780400804;\n\tbh=u8RnsLKQAmnhiJX6J5wK2dWe7RxMCh7LVhFgf3R/aZY=;\n\th=In-Reply-To:References:Subject:From:Cc:To:Date:Resent-From:\n\tResent-To:From;\n\tb=kY/YalwEJishU+9hdw7gmy6zGVincw0MAsAUkd5BG4m7oZhHi+7whX9IeujevDkiL\n\t4VnXeTXLYa7BCDcdrSL5K9q6QqZOSslbcKU2uPKUWM0a9gqZtPn3sQ5Z5uGXBhcuoe\n\tJyQ+w/gHV/xahBhqlv2K3RsP7pPgS1tpShLw8ysI=","Content-Type":"text/plain; charset=\"utf-8\"","MIME-Version":"1.0","Content-Transfer-Encoding":"quoted-printable","In-Reply-To":"<20260602012601.2799275-1-qi.hou@nxp.com>","References":"<20260602012601.2799275-1-qi.hou@nxp.com>","Subject":"Re: [PATCH v2] libcamera: software_isp: debayer_egl: Demote\n\tunsupported format log to Info","From":"Kieran Bingham ","Cc":"jared.hu@nxp.com, qi.hou@nxp.com, julien.vuillaumier@nxp.com","To":"Qi Hou , libcamera-devel@lists.libcamera.org","Date":"Tue, 02 Jun 2026 10:39:32 +0100","Message-ID":"<178039317209.815980.16798958776686332830@ping.linuxembedded.co.uk>","User-Agent":"alot/0.9.1","Resent-From":"Kieran Bingham ","Resent-To":"parsemail@patchwork.libcamera.org"}},{"id":38982,"web_url":"https://patchwork.libcamera.org/comment/38982/","msgid":"<178041027424.815980.4111742283835686485@ping.linuxembedded.co.uk>","date":"2026-06-02T14:24:34","subject":"Re: [PATCH v2] libcamera: software_isp: debayer_egl: Demote\n\tunsupported format log to Info","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Hello,\n\nQuoting Qi Hou (2026-06-02 02:26:01)\n> From: Hou Qi \n> Signed-off-by: Qi Hou \n\nThe 'From' tag and the 'Signed off by' tags do not match, and the git\nserver has hooks to reject this.\n\nI can rewrite either one, which is your preferred or correct ordering?\n\n Hou Qi\nor\n Qi Hou\n?\n\n--\nKieran\n\n\n> \n> DebayerEGL::getInputConfig() logs at Error level when it encounters an\n> unsupported input format. This function is called during format\n> enumeration from the simple pipeline handler's tryPipeline(), which\n> iterates over all pixel formats reported by the video capture device,\n> including non-Bayer formats such as UYVY and YUYV. The caller already\n> handles the failure gracefully by checking the return value. Similarly,\n> DebayerEGL::getOutputConfig() logs at Error level for unsupported\n> output formats, but is called from strideAndFrameSize() which also\n> handles failures via the return value.\n> \n> Demote both log messages from Error to Info for consistency with\n> DebayerCpu, which uses Info level in getInputConfig() and\n> getOutputConfig() for the same situation.\n> \n> Signed-off-by: Qi Hou \n> ---\n> src/libcamera/software_isp/debayer_egl.cpp | 4 ++--\n> 1 file changed, 2 insertions(+), 2 deletions(-)\n> \n> diff --git a/src/libcamera/software_isp/debayer_egl.cpp b/src/libcamera/software_isp/debayer_egl.cpp\n> index 7b9e02d9..68779893 100644\n> --- a/src/libcamera/software_isp/debayer_egl.cpp\n> +++ b/src/libcamera/software_isp/debayer_egl.cpp\n> @@ -78,7 +78,7 @@ int DebayerEGL::getInputConfig(PixelFormat inputFormat, DebayerInputConfig &conf\n> return 0;\n> }\n> \n> - LOG(Debayer, Error)\n> + LOG(Debayer, Info)\n> << \"Unsupported input format \" << inputFormat;\n> \n> return -EINVAL;\n> @@ -92,7 +92,7 @@ int DebayerEGL::getOutputConfig(PixelFormat outputFormat, DebayerOutputConfig &c\n> return 0;\n> }\n> \n> - LOG(Debayer, Error)\n> + LOG(Debayer, Info)\n> << \"Unsupported output format \" << outputFormat;\n> \n> return -EINVAL;\n> -- \n> 2.34.1\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 1F2E4BDCBC\n\tfor ;\n\tTue, 2 Jun 2026 14:24:40 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 30DEB63045;\n\tTue, 2 Jun 2026 16:24:39 +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 867CF61754\n\tfor ;\n\tTue, 2 Jun 2026 16:24:37 +0200 (CEST)","from monstersaurus.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 3338397F;\n\tTue, 2 Jun 2026 16:24:14 +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=\"k5WrW4g4\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1780410254;\n\tbh=MCTPBqY4m5an/XQr+5W0KZHQb8ntcr1+g22lH5ddY8I=;\n\th=In-Reply-To:References:Subject:From:Cc:To:Date:From;\n\tb=k5WrW4g4LTuS0fbXnLqAbLlNkQftHIfN5vNwoZJPnx2rsLC2x9nbVa8jm7fIfOmFu\n\tUglr9XFqvu3COcIMYTycwZp92tKBK/6WxYUSWniGvrGe2ebLyDxdWGLsgREadwyEK4\n\tJTKcz0h/SfWymr+ANj9/XZ/U0m/Sy1vn8w9uTscU=","Content-Type":"text/plain; charset=\"utf-8\"","MIME-Version":"1.0","Content-Transfer-Encoding":"quoted-printable","In-Reply-To":"<20260602012601.2799275-1-qi.hou@nxp.com>","References":"<20260602012601.2799275-1-qi.hou@nxp.com>","Subject":"Re: [PATCH v2] libcamera: software_isp: debayer_egl: Demote\n\tunsupported format log to Info","From":"Kieran Bingham ","Cc":"jared.hu@nxp.com, qi.hou@nxp.com, julien.vuillaumier@nxp.com","To":"Qi Hou , libcamera-devel@lists.libcamera.org","Date":"Tue, 02 Jun 2026 15:24:34 +0100","Message-ID":"<178041027424.815980.4111742283835686485@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\" "}},{"id":38984,"web_url":"https://patchwork.libcamera.org/comment/38984/","msgid":"","date":"2026-06-03T01:04:16","subject":"RE: [EXT] Re: [PATCH v2] libcamera: software_isp: debayer_egl:\n\tDemote unsupported format log to Info","submitter":{"id":195,"url":"https://patchwork.libcamera.org/api/people/195/","name":"Qi Hou","email":"qi.hou@nxp.com"},"content":"\"Qi Hou\". Thanks\n\nRegards,\nQi Hou\n\n-----Original Message-----\nFrom: Kieran Bingham \nSent: Tuesday, June 2, 2026 10:25 PM\nTo: Qi Hou ; libcamera-devel@lists.libcamera.org\nCc: Jared Hu ; Qi Hou ; Julien Vuillaumier \nSubject: [EXT] Re: [PATCH v2] libcamera: software_isp: debayer_egl: Demote unsupported format log to Info\n\nCaution: This is an external email. Please take care when clicking links or opening attachments. When in doubt, report the message using the 'Report this email' button\n\n\nHello,\n\nQuoting Qi Hou (2026-06-02 02:26:01)\n> From: Hou Qi \n> Signed-off-by: Qi Hou \n\nThe 'From' tag and the 'Signed off by' tags do not match, and the git server has hooks to reject this.\n\nI can rewrite either one, which is your preferred or correct ordering?\n\n Hou Qi\nor\n Qi Hou\n?\n\n--\nKieran\n\n\n>\n> DebayerEGL::getInputConfig() logs at Error level when it encounters an \n> unsupported input format. This function is called during format \n> enumeration from the simple pipeline handler's tryPipeline(), which \n> iterates over all pixel formats reported by the video capture device, \n> including non-Bayer formats such as UYVY and YUYV. The caller already \n> handles the failure gracefully by checking the return value. \n> Similarly,\n> DebayerEGL::getOutputConfig() logs at Error level for unsupported \n> output formats, but is called from strideAndFrameSize() which also \n> handles failures via the return value.\n>\n> Demote both log messages from Error to Info for consistency with \n> DebayerCpu, which uses Info level in getInputConfig() and\n> getOutputConfig() for the same situation.\n>\n> Signed-off-by: Qi Hou \n> ---\n> src/libcamera/software_isp/debayer_egl.cpp | 4 ++--\n> 1 file changed, 2 insertions(+), 2 deletions(-)\n>\n> diff --git a/src/libcamera/software_isp/debayer_egl.cpp \n> b/src/libcamera/software_isp/debayer_egl.cpp\n> index 7b9e02d9..68779893 100644\n> --- a/src/libcamera/software_isp/debayer_egl.cpp\n> +++ b/src/libcamera/software_isp/debayer_egl.cpp\n> @@ -78,7 +78,7 @@ int DebayerEGL::getInputConfig(PixelFormat inputFormat, DebayerInputConfig &conf\n> return 0;\n> }\n>\n> - LOG(Debayer, Error)\n> + LOG(Debayer, Info)\n> << \"Unsupported input format \" << inputFormat;\n>\n> return -EINVAL;\n> @@ -92,7 +92,7 @@ int DebayerEGL::getOutputConfig(PixelFormat outputFormat, DebayerOutputConfig &c\n> return 0;\n> }\n>\n> - LOG(Debayer, Error)\n> + LOG(Debayer, Info)\n> << \"Unsupported output format \" << outputFormat;\n>\n> return -EINVAL;\n> --\n> 2.34.1\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 9741BC324C\n\tfor ;\n\tWed, 3 Jun 2026 01:04:22 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id B92206308C;\n\tWed, 3 Jun 2026 03:04:21 +0200 (CEST)","from MRWPR03CU001.outbound.protection.outlook.com\n\t(mail-francesouthazlp170110003.outbound.protection.outlook.com\n\t[IPv6:2a01:111:f403:c207::3])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 121256139A\n\tfor ;\n\tWed, 3 Jun 2026 03:04:20 +0200 (CEST)","from PAXPR04MB8285.eurprd04.prod.outlook.com\n\t(2603:10a6:102:1ca::15)\n\tby DU0PR04MB9395.eurprd04.prod.outlook.com (2603:10a6:10:35a::20)\n\twith Microsoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.21.92.7;\n\tWed, 3 Jun 2026 01:04:17 +0000","from PAXPR04MB8285.eurprd04.prod.outlook.com\n\t([fe80::81d0:4221:c6e5:9b82]) by\n\tPAXPR04MB8285.eurprd04.prod.outlook.com\n\t([fe80::81d0:4221:c6e5:9b82%4]) with mapi id 15.21.0092.006;\n\tWed, 3 Jun 2026 01:04:16 +0000"],"Authentication-Results":["lancelot.ideasonboard.com; dkim=pass (2048-bit key;\n\tunprotected) header.d=nxp.com header.i=@nxp.com header.b=\"c5bhN+EI\";\n\tdkim-atps=neutral","dkim=none (message not signed)\n\theader.d=none;dmarc=none action=none header.from=nxp.com;"],"ARC-Seal":"i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none;\n\tb=NwKULpDA/UHrqRTzh43fA7jTkC92/UQk72RDzJscOBLo2iYlHdxGjedtLFNEXKgZvTdtKy1a8t4y8Ql86b3mpAbWOpWCeuweUjS1kiYQ+2zqSr2yG1PV3y9hr55UEhLcicOMentMeWSCT7PgXL9Azgr+pbYh3jw5EyuKYe02jKrBlpPzIg1tXf77y9VZMRk9jbAsILG1ukr7jG5+tscvbwghu8AbXPnGW4XDQRhoOSA4UKRZbVFZ/Mty15KJqMGBqwC5ATG/8XyFwpF/VbBoVC/8a9T94347b+lFLuXGDycceuXp+mvUNAGVHaWCCNaEoG+1FrEO668XY7HurWhVYg==","ARC-Message-Signature":"i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;\n\ts=arcselector10001;\n\th=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1;\n\tbh=o3EVMD8MeiZ9lxSKKRoJrqjbARXi6vwVnb92oxwXWQ4=;\n\tb=I5llFBh8HOTXv4rMCDnuXPNO8pAnjMze3+35tEHBtJ/tsj4WCrrnwGCWTvNhYVGekUT2ectYzxyFyT58437ogWFV9nqe9u3wGF8HvylhdRuHkih9ptCN7iZYDtXTIwSN5mgBZBTqECrmX60ZOqIs/b2KFI/QqBK0QiGmsbJIgiurFcMkHWkiUNTXMdRvvEB4HONrPvIDE0JUmO9/U3zbCcSHcmbqnQOvHyOcayNJEwgmf/AWn+OjHisXDx08yn+Royi42vZxA2g1G+ZYejrQzm7rt0v5zSwGwWz2juOaDZTISha9qVE7CWv8k5CaVCktK9NLw/a3oeU8ZtFHcsMqQw==","ARC-Authentication-Results":"i=1; mx.microsoft.com 1; spf=pass\n\tsmtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com;\n\tdkim=pass header.d=nxp.com; arc=none","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1;\n\th=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n\tbh=o3EVMD8MeiZ9lxSKKRoJrqjbARXi6vwVnb92oxwXWQ4=;\n\tb=c5bhN+EIuXS+Bc4Iw7fCujGIMjIaDgxGpyx41rwul72W5dmvlEKU5hV3OTQqAEbmnJ/dzeWBAyPzjfYAZRniS07F5eT4BLN9+h4ZWMJ1frdnEwxulyQnxMJkYqcyJ8/z9LpfxNhxL6bD6lmMFr65onOKRaTR39ch8WIzD7v11CFV+hsLQgULXnaKsoH4mSs+PMlMcvh06PEGhfeIAr1Vd5MjJhf27nrmRTb51GuBTHFf8FEohh/vUkV4rf4OsRjJOyHMiSDborHci3941iiVffeNjvAaBdIYXJR2niRo8cD65MuAHajrgstr8UwwaJTA0dT4pfqnB8PdGPnkiaFPCw==","From":"Qi Hou ","To":"Kieran Bingham ,\n\t\"libcamera-devel@lists.libcamera.org\"\n\t","CC":"Jared Hu , Julien Vuillaumier\n\t","Subject":"RE: [EXT] Re: [PATCH v2] libcamera: software_isp: debayer_egl:\n\tDemote unsupported format log to Info","Thread-Topic":"[EXT] Re: [PATCH v2] libcamera: software_isp: debayer_egl:\n\tDemote unsupported format log to Info","Thread-Index":"AQHc8i7MySrGNfvHfE65SJf7LBcoILYrUrMAgACyhlA=","Date":"Wed, 3 Jun 2026 01:04:16 +0000","Message-ID":"","References":"<20260602012601.2799275-1-qi.hou@nxp.com>\n\t<178041027424.815980.4111742283835686485@ping.linuxembedded.co.uk>","In-Reply-To":"<178041027424.815980.4111742283835686485@ping.linuxembedded.co.uk>","Accept-Language":"zh-CN, en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","authentication-results":["lancelot.ideasonboard.com; dkim=pass (2048-bit key;\n\tunprotected) header.d=nxp.com header.i=@nxp.com header.b=\"c5bhN+EI\";\n\tdkim-atps=neutral","dkim=none (message not signed)\n\theader.d=none;dmarc=none action=none header.from=nxp.com;"],"x-ms-publictraffictype":"Email","x-ms-traffictypediagnostic":"PAXPR04MB8285:EE_|DU0PR04MB9395:EE_","x-ms-office365-filtering-correlation-id":"947faf15-ff27-40f9-2bb0-08dec10c0916","x-ms-exchange-senderadcheck":"1","x-ms-exchange-antispam-relay":"0","x-microsoft-antispam":"BCL:0;\n\tARA:13230040|376014|1800799024|42112799006|19092799006|366016|38070700021|56012099006|11063799006|4143699003|3023799007|22082099003|18002099003;","x-microsoft-antispam-message-info":"ud8IPQIkJ4Dd3POq3HuOt4TaOzjiFiKOujxF0z/ok6mPTtPX7r6gTjdfKc/6AXjvzfEu2ATlrpWU7UHAymWCmYj0ug+M4o8mhIWo1Hi2fKTtb2yPYblvIyE7zUBJ125tgrZLNBMtvaEoTnQ/CDLsI/qqArarTyivRdIGCvVrh5CjgQSIdrOe6IRX3lZU/RtQccSRdCd+G1FQQF01knKvCpd+WZpzp62NRQ8ztCOHE09F3H8sMeUUQlUERq3iO8z95q6H+rE+6pWReeDDCKqNi48nx84sWSiX1uDQlurW4FWKliKJ5FLK2I4VtFjALfkje31YWTNVCEWC9IVjFxNMMZ+VYKccF2HR7zQA7x/R03XeLmFFUPl2NqbEncLtF4mw1skYbsJiDqfmax06Yr1jE7pNG2YcQq7JGOiMycaXJM7JC/p9Camb1DBye4rTnqghpQeEbqRH+RHuSWZojdP5x4UysZolH6GZib5R83qg4cKpFO3BHKipd3fRgxnRu6KZO6RiOQxxitzhy2Bmixw5fhJMpAoMM9qI13R/xxCLwHSEKrxZ1NFPp4QvqddQZLBmjFsFjizI2sUTFdzaZCJqfZuFK6XW+o3gj0Zin4mOTLoav7NOipZJ5DdnvNGgfCgNAOHLBbORZU7It/pwxhR1wNingLkINkBZobHw4e0poLgbzrrl8htsDLORLntj6fYIHLXGmM0i5+vT9SwNTIXMpcpIRh5HsMQfHOdgmlz6XWTRBHRPyEw+6p+O++wYDhhz","x-forefront-antispam-report":"CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:;\n\tIPV:NLI; SFV:NSPM; H:PAXPR04MB8285.eurprd04.prod.outlook.com; PTR:;\n\tCAT:NONE; \n\tSFS:(13230040)(376014)(1800799024)(42112799006)(19092799006)(366016)(38070700021)(56012099006)(11063799006)(4143699003)(3023799007)(22082099003)(18002099003);\n\tDIR:OUT; SFP:1101; ","x-ms-exchange-antispam-messagedata-chunkcount":"1","x-ms-exchange-antispam-messagedata-0":"=?utf-8?q?iVJdFAlQU+fubNegUgi/f0M2g?=\n\t=?utf-8?q?aFMJD6WBn2IKNv4vQr6xbRLlbtISNa29Q7J22BWuFPQJkty2r52OeJ0G?=\n\t=?utf-8?q?UfTQX5co/YMy/P9hLvuQG3DkJ9SdJ8LyAb6YC+YsVz+dSaZkyy07BaQJ?=\n\t=?utf-8?q?HMC8QhGiiU9BHOCAUqoOKwUi1Q/9qXUJnL3ptcVxOjsLHrlJFS71n+sl?=\n\t=?utf-8?q?WGdxlfiZOcDiPEBKJJP6Sui3zzE+eYqniSPIP9uBteU5NvkSGVcQBK/W?=\n\t=?utf-8?q?lDoTyG7QvNEE1b+Jwz+PvJ6oD+M6JyUXBKH6haudtZHjdn4G9gMV0rYX?=\n\t=?utf-8?q?I2GQV5u2tT9faw4ktYqZJg2CRsI1hNV2PAzMo01a6Lb0Idubq8XMwAaf?=\n\t=?utf-8?q?rP/Q4EglcdWQbhFwnTtkL2raG4QzPkU/zeWQn1AXyTy6CZEnI7knDCgW?=\n\t=?utf-8?q?SpVvS4wuN3sBI/KV/VPXy1WQvc/wtyP5Q0V/JCgfyaT1MaTIhxcF4ScL?=\n\t=?utf-8?q?99tqPAo24tOzuhHD/ftQsKTrzMCdfxOcUw+bTHIMmdqvXdKsocBBNWVh?=\n\t=?utf-8?q?AaMUe0QN41e8DQ5OVck+hH0hNCooXLVegYpTKJld4Gth+MwjiAf7rdbu?=\n\t=?utf-8?q?u9+QvnoOXvo0ra9vyHZAYMRLNc7tddBe5Cf1nULLurRy2ZAqyS0NRWDb?=\n\t=?utf-8?q?5IA9We7khb3dW4D2aJml8s290Cw0S9CeHYDlz6L1GLDnGhdCRT2CyBjG?=\n\t=?utf-8?q?rpY0e+Fwcxw7kAqlAtvftl92VYkkhioRioXWiXesG64MZRHdyQeYo5yA?=\n\t=?utf-8?q?4hQ/c07wSXXSj4Konr9eYyoCQHBnm515ZNUNsRyujaTMa+cpqeTSkMHo?=\n\t=?utf-8?q?ubtYOxqfgn0HAFqgHhdx1PUmIc1yw7AZ9qVQB6NVnNvBknXOLnROCoY8?=\n\t=?utf-8?q?2V+MVEikG7AnKNXylzMyCBJ8OKqNjffAkqjAYafzkAPMT5oHrUURdbp0?=\n\t=?utf-8?q?+0B0b2B2fPOA+wL33/h4n5x7MDGxQepMpw5Kb0C6H5xSgfr4N0FrRMmU?=\n\t=?utf-8?q?ZD+1m5oZFkvn5TS3anAuKpbOX89AaD0BCD7d2eGcWU6NkAuAodZkYERP?=\n\t=?utf-8?q?Le6fSF88NixjrI4Qw5rsfrmETF1tkf3ungYc/t4SxRvMYG/p0yFXPOBd?=\n\t=?utf-8?q?Uvuohd9n05aq3ioy4gcfeOJEy5rmwUewU0tDxiD6d8LMSFwzxIPprggl?=\n\t=?utf-8?q?u7U1XU4PYkBin9B/Ktp2IaEnv9T5zirQL2YRLkPnhNuI2MEpP2+yZpn2?=\n\t=?utf-8?q?plPpIfe1NcFGKXy4H296HZl49VuZcj7eQQF5rJo6g/uw82mkyxLlyDd+?=\n\t=?utf-8?q?9AAHyVZl4E0v9GSIV4zlMhMvKs3XpVyzKV8B1FDR408g86v9RY/2+gHy?=\n\t=?utf-8?q?uTLoq3L6xu6m9y8MOvRFi+TfJ5eGXQ7c63K33wr6Vy6gdD4mdLja/9PO?=\n\t=?utf-8?q?GF4AhYNkQGstx11v1B2nL2AtBP2c9xkcjHFPLJgHVxVjsvoClfC9Sqw2?=\n\t=?utf-8?q?w+ybh8a1PE4Y4ryIse07CqB2IzPYo4ae5QuEeb3pLwH1RpOS2QxMgkec?=\n\t=?utf-8?q?R4q4pKLAqXf2pZQ6DF9zau9oFM16vy59LMdy4OpSBAcRgJP82Ok5+4nW?=\n\t=?utf-8?q?gPGvKWbjnlmJ1hGKMR+Y22B3eGyU/d3E9LwQswQQ2k5I21fqUs5mGSvd?=\n\t=?utf-8?q?rBuR7MmMdL1aVbPXy8bbU3+VFlFz7sOGGO/oeUR+2ithgrwI0cDf+2jZ?=\n\t=?utf-8?q?ZS4pbcIZ237llx+F5LCGlckccgQRPrfQJspoqLjrtLEs22pRC4/wIMTD?=\n\t=?utf-8?q?4uf/YEOtxbyECMXgSOdCv7o?=","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","MIME-Version":"1.0","X-OriginatorOrg":"nxp.com","X-MS-Exchange-CrossTenant-AuthAs":"Internal","X-MS-Exchange-CrossTenant-AuthSource":"PAXPR04MB8285.eurprd04.prod.outlook.com","X-MS-Exchange-CrossTenant-Network-Message-Id":"947faf15-ff27-40f9-2bb0-08dec10c0916","X-MS-Exchange-CrossTenant-originalarrivaltime":"03 Jun 2026 01:04:16.9173\n\t(UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"686ea1d3-bc2b-4c6f-a92c-d99c5c301635","X-MS-Exchange-CrossTenant-mailboxtype":"HOSTED","X-MS-Exchange-CrossTenant-userprincipalname":"9c/S3fXiZvHdiv6I3+XDqZlPninQxhq6Uba7n9pOaAa40fSgQ1SNi6zErHe/68Ig","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"DU0PR04MB9395","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\" "}}]