From patchwork Tue Jun 2 01:26:01 2026 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Qi Hou X-Patchwork-Id: 26815 Return-Path: X-Original-To: parsemail@patchwork.libcamera.org Delivered-To: parsemail@patchwork.libcamera.org Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) by patchwork.libcamera.org (Postfix) with ESMTPS id 15A22BDCBC for ; Tue, 2 Jun 2026 11:42:40 +0000 (UTC) Received: from monstersaurus.ideasonboard.com (cpc89244-aztw30-2-0-cust6594.18-1.cable.virginm.net [86.31.185.195]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 9665B97F for ; Tue, 2 Jun 2026 13:42:16 +0200 (CEST) Delivered-To: kbingham@ideasonboard.com Received: from perceval.ideasonboard.com by perceval.ideasonboard.com with LMTP id uLVlIh4xHmonFjwA4E0KoQ (envelope-from ) for ; Tue, 02 Jun 2026 03:25:50 +0200 Received: from lancelot.ideasonboard.com (lancelot.ideasonboard.com [92.243.16.209]) by perceval.ideasonboard.com (Postfix) with ESMTPS id 23E4F802; Tue, 2 Jun 2026 03:25:50 +0200 (CEST) Received: from lancelot.ideasonboard.com (localhost [IPv6:::1]) by lancelot.ideasonboard.com (Postfix) with ESMTP id 7B18463035; Tue, 2 Jun 2026 03:26:11 +0200 (CEST) Received: from inva021.nxp.com (inva021.nxp.com [92.121.34.21]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 84D5562010 for ; Tue, 2 Jun 2026 03:26:10 +0200 (CEST) Received: from inva021.nxp.com (localhost [127.0.0.1]) by inva021.eu-rdc02.nxp.com (Postfix) with ESMTP id 26B1F201070; Tue, 2 Jun 2026 03:26:10 +0200 (CEST) Received: from aprdc01srsp001v.ap-rdc01.nxp.com (aprdc01srsp001v.ap-rdc01.nxp.com [165.114.16.16]) by inva021.eu-rdc02.nxp.com (Postfix) with ESMTP id E553320321D; Tue, 2 Jun 2026 03:26:09 +0200 (CEST) Received: from lsvm11u0000124.swis.ap-northeast-2.aws.nxp.com (lsvm11u0000124.swis.ap-northeast-2.aws.nxp.com [10.52.8.111]) by aprdc01srsp001v.ap-rdc01.nxp.com (Postfix) with ESMTP id C6C191800071; Tue, 2 Jun 2026 09:26:08 +0800 (+08) From: Qi Hou To: libcamera-devel@lists.libcamera.org Cc: jared.hu@nxp.com, qi.hou@nxp.com, julien.vuillaumier@nxp.com Subject: [PATCH v2] libcamera: software_isp: debayer_egl: Demote unsupported format log to Info Date: Tue, 2 Jun 2026 10:26:01 +0900 Message-Id: <20260602012601.2799275-1-qi.hou@nxp.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 X-Virus-Scanned: ClamAV using ClamSMTP X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libcamera-devel-bounces@lists.libcamera.org Sender: "libcamera-devel" X-TUID: KV8f7ddUr6s1 Resent-From: Kieran Bingham Resent-To: parsemail@patchwork.libcamera.org From: Hou Qi DebayerEGL::getInputConfig() logs at Error level when it encounters an unsupported input format. This function is called during format enumeration from the simple pipeline handler's tryPipeline(), which iterates over all pixel formats reported by the video capture device, including non-Bayer formats such as UYVY and YUYV. The caller already handles the failure gracefully by checking the return value. Similarly, DebayerEGL::getOutputConfig() logs at Error level for unsupported output formats, but is called from strideAndFrameSize() which also handles failures via the return value. Demote both log messages from Error to Info for consistency with DebayerCpu, which uses Info level in getInputConfig() and getOutputConfig() for the same situation. Signed-off-by: Qi Hou Reviewed-by: Bryan O'Donoghue Reviewed-by: Kieran Bingham --- src/libcamera/software_isp/debayer_egl.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/libcamera/software_isp/debayer_egl.cpp b/src/libcamera/software_isp/debayer_egl.cpp index 7b9e02d9..68779893 100644 --- a/src/libcamera/software_isp/debayer_egl.cpp +++ b/src/libcamera/software_isp/debayer_egl.cpp @@ -78,7 +78,7 @@ int DebayerEGL::getInputConfig(PixelFormat inputFormat, DebayerInputConfig &conf return 0; } - LOG(Debayer, Error) + LOG(Debayer, Info) << "Unsupported input format " << inputFormat; return -EINVAL; @@ -92,7 +92,7 @@ int DebayerEGL::getOutputConfig(PixelFormat outputFormat, DebayerOutputConfig &c return 0; } - LOG(Debayer, Error) + LOG(Debayer, Info) << "Unsupported output format " << outputFormat; return -EINVAL;