From patchwork Fri Jan 23 12:59:03 2026 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Julien Vuillaumier X-Patchwork-Id: 25955 Return-Path: X-Original-To: parsemail@patchwork.libcamera.org Delivered-To: parsemail@patchwork.libcamera.org Received: from lancelot.ideasonboard.com (lancelot.ideasonboard.com [92.243.16.209]) by patchwork.libcamera.org (Postfix) with ESMTPS id 9A3BCBDCBF for ; Fri, 23 Jan 2026 12:59:45 +0000 (UTC) Received: from lancelot.ideasonboard.com (localhost [IPv6:::1]) by lancelot.ideasonboard.com (Postfix) with ESMTP id 4D03F61FCB; Fri, 23 Jan 2026 13:59:45 +0100 (CET) Authentication-Results: lancelot.ideasonboard.com; dkim=pass (2048-bit key; unprotected) header.d=nxp.com header.i=@nxp.com header.b="lqcnFaew"; dkim-atps=neutral Received: from DU2PR03CU002.outbound.protection.outlook.com (mail-northeuropeazlp170110003.outbound.protection.outlook.com [IPv6:2a01:111:f403:c200::3]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 94BAC61FBB for ; Fri, 23 Jan 2026 13:59:43 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=WZytnqg6FS0KxrDMWfPczy+N8tpZC2XEOFR23vHaFIkYGwPTsdB40Ob8qSdU5/y8Gbmy/Bq56PZL0TjAIWzFxBxlx48NickSdtyfENrdh0KJvtWmDi/bYHWCWrudthaUBrRtDAMThanYWFzlQJaZCuLjmxZIPiWlfPl+ZXlQl9Pmh+8aNuZB3ggh+p4FS2HJ5pnS3dLWzo4uc22zS6ML6+MIvCq0DTnH5uDS5CnOinq2pLen7tCaTVcvjNljldFCFzJ93VyUOSUg1yousoZ8lg1txgIr3TrVNWZ6Gl6SVH7ttr/s5AP0MosYIhq5siuObpJQ2G2PndiHEbRu4kt+uA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=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; bh=XzMDiKtbhoV3YKgamfso7Tx+JNJTdnnkTdMpjeOGLf8=; b=GnkCJKXVzSNXbH6saseNqT0zruQISywtOBfOlMg91rTjamBQppkPyv66QKmh97JCkDQVkIrJxaz7aYyEq/rxnCdqAZiJirH0N8+zI6bCooBj3bsYtxmLCuHmfj/JnwN9qmMsUx55usqzeQV3LD5XJtT+w6dGs76dPHdVKHjS5jhfZod3FvfNAmVEkNXLhALotnrm1jStbe5H3sD0iUqZAM6SLQ4iNoQbYnBIRk4wq7RkTiKmy0QGgwXdWIPt88srs+T4q7RkV1oDjs/eHyeSXkb4tz9LaWljjV7v48wY4RSKz/9U01UKissV7YN6a/t1KITLXc1Ba/1Qp/lA42u9JQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=XzMDiKtbhoV3YKgamfso7Tx+JNJTdnnkTdMpjeOGLf8=; b=lqcnFaewQNjOHllyp0klGFY7Fn/Wl6BzqvjMW0ryCfmozjQzgxu5SEOKEHmrwYEE+AjOMgEAFwG4Sp+duZhshs8oFkQxODpuP3kbWeIEwoQPijzfDEgpuvz1JO5d77qqtF93R5NRnnUlmVUv6huoF/wfgNGY/AfpsZ7/21CJjkSXGhxcZmPWWdUzLx0ZiBl2Hk9p3xWa+A9/1ey+BRFyXIyhwMriAlPAs1eYi3WZmA59lysZMwEgm1DsSEztQIdEsWQZNNb3MatM04Zy4vJR5Cjvhw1MWFbNDNcVUYAEmpWIrvV7tIxd+c7ihvSMHrv2rjHnIJhJdIN7hPp3yTt5Mw== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM9PR04MB8147.eurprd04.prod.outlook.com (2603:10a6:20b:3e0::22) by GVXPR04MB11690.eurprd04.prod.outlook.com (2603:10a6:150:2e5::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9542.11; Fri, 23 Jan 2026 12:59:42 +0000 Received: from AM9PR04MB8147.eurprd04.prod.outlook.com ([fe80::b387:72c6:e33c:8656]) by AM9PR04MB8147.eurprd04.prod.outlook.com ([fe80::b387:72c6:e33c:8656%7]) with mapi id 15.20.9542.009; Fri, 23 Jan 2026 12:59:42 +0000 From: Julien Vuillaumier To: libcamera-devel@lists.libcamera.org Cc: Julien Vuillaumier Subject: [PATCH v2 1/1] libcamera: camera_sensor: Fix imageStream() for CameraSensorLegacy class Date: Fri, 23 Jan 2026 13:59:03 +0100 Message-Id: <20260123125903.2469016-2-julien.vuillaumier@nxp.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20260123125903.2469016-1-julien.vuillaumier@nxp.com> References: <20260123125903.2469016-1-julien.vuillaumier@nxp.com> X-ClientProxiedBy: AS4P189CA0069.EURP189.PROD.OUTLOOK.COM (2603:10a6:20b:659::7) To AM9PR04MB8147.eurprd04.prod.outlook.com (2603:10a6:20b:3e0::22) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM9PR04MB8147:EE_|GVXPR04MB11690:EE_ X-MS-Office365-Filtering-Correlation-Id: 33c9dd89-0ced-402e-950a-08de5a7f4645 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; ARA:13230040|19092799006|1800799024|366016|52116014|376014|38350700014; X-Microsoft-Antispam-Message-Info: 86mRlxpP8iO0aUbp3raCsZe+Km9ApfeKoxmx8SoD00lNNKbBckcKlaMzzzlsuhUd2wwWXrdaqWGixg7cIArkVj1d0GDofPoRZXeA34j8GivdcgBin/spdot93lb3Afb5AS+8Y2jXUPmp5hfe8gzDFmbS8PfJpstfMo2eDtksfH6/A4oVoew7ADIFg+cYeYqRPDEUSpEpanXX+1NnzMTG/p2x3jHy5Fp32bnrL5+a3vgw+l4yO9x/bnqXeyOVw7qyngnu5HUNnZEyW9j/BqAz+6RGVLYP/w7WZwyLC1fWP/DKzJl/KZFNSnRNBrha9LlSlVlq/gJRpW1BzUaZgW3O029Y8nLOaMQw0SBIf/KNzfaDuEMIVy0OhlwQw4vcPfLTWn3XrguPF2QCDC4xBxNHTc7LmsJSRL4iisHj1Vmv2Fi8qn6mfC/aIcZIFgsYDcj3COhvhWIK4/5AiVK5+Ti3UErGGcQjP2CNb0q+D8JDujB53Voaw9NgbdXbQkolM1iGCe7Uz/ifW5n8uUZyh0b1APMIGK+rXVeWZSGvY98dI7qflgMc2vlRkuuTkKLjUfyF1ktrR9CQKaZ9chmA6GuTLhFK1pQGq0MCDRn2uVdlC5xyokzKaA0lK8OeYk+tKbIgMj/wJHkrZNpfAw9GG1bhtdySHWrmDhHfdZLSlnEE6lzT2xjARuh++VL+ajwX7zW0cNtA5yOm2j78wGW+dreF4CuBgOEy7q+VjbUgv8DH76HWCIi4JUt9RgYJaLgX8CTQhxQQA0ySnhL+WgL50QI4wOXYTvOeQlv07F7viA3NCe+Z381h/1gzg5CHyHf+XiozDyFk3J+SNKrUpIlcEPVTi8D6F0iIbHO9XMmwoCMJYXnZhCCLEK4gkOpkahRQ7GI1sxoz6MhVBsLCB+0CDVAEYpt+UcoHWcftDXE1Ces/3hmNNSjwWmdwMM6mnVK39DZb1ST/v+WSn/HaGpUkMhZby+/ThowuhzorMQ4LL19B5EZ1th/bzhIuwEgx3oePqTFgJXmF78Nd+4/UfO8d5cKv4gIQQD0HVw4N46AM1nUAb+uzIkLa0xehdmLTD3pepKpxihova4i93fjUQA9n1osOfgTnPktDv2+Fqv2d4UBvQs4luVjLZcT+My0W+zgEbx8k75gUoZqDp3Fwu9zyHVLdLmpGXDvyOC//h62IFtQpyTS1H1ySNGt46gb5W5ojOfJ1uw7jtCZf4higtIRYcheEWfG+YtkK1kMzc2wg0OGgVreHo+rTdyXOAGUE3jwf9YqBIHg8WeO7Vv8BVcdqrG1Zx34tZZdNoEllbhZ6fi/CshbowFvrnBwQZjpmtE6kiwYaxG5wnNBpyZ/mN+Ky9OeBFRxGuZnq4Y1DokpCur6pJZlWdfAccDSnKaeVmKda/3i1v4QaaRbGsNw307Y9MyRT689cDNhIG9PLi0E2BwfPdsOVSwq6hPzbhVYYUtr3FEQ34R10YRk8mIDtiuEpc94N6GYKLfNnKiaZaxAkz2O8Q0HkQabTfb5837s8DvSL7Q34Orjk01usdVqpTBjSLSZFJBR5GE/iVMClKSJxtfC7RiPVA/neM19FwGoDcpf6Kr17k+LqMhZyy+77MGWz0z6Vr4It4IBmwYlUvQ9IwNpr18k= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM9PR04MB8147.eurprd04.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230040)(19092799006)(1800799024)(366016)(52116014)(376014)(38350700014); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: J5BG3Fs+ddfqZDDj2R8hK2j+uMCzBIn8aeTAYbWjjsYREwJ2/WcQGB2Pn3GLkiy/twfnEHW5+LscobK/ZlkNIDKm1QCtWcf5Ru2wcH47O7A68roxwTSt2L2A4hF/qUEu81Px3qKE28dyG/YTMnNVDpmCHbsuXrvPASuzycSWgthPxjwf78Ea0gF787PwCI/9GpbAmHekbYTcqa/rEeKgySnLuN0LoHZouSs5c4/73glmK76c2ZWzWFQlxM3xWxdTuV2PcCRLKbW+g/B9U3H/BFzjMdjA4RpF2+diueLlhBfM4grVUWNyIxGG1VCF3qcrB4+oo7VlWQCT2bLPApSnz786ZLFQmVrEb8p9zt+Fm4RmgoSVA7nZNudq59Xt8ukP1zIGww3WsIT33Bdr+Fab/p3JgyoJOJ3N3HZau/4+besBfU4miBhMzYMp6qPXXZ1DVUoZp9eHlcORQO83hEd8YKGBJCXrrBO1KEbh/7ruRJH9yJ/7noEbf9DNYtB3mu3VodoTfGnWPMzshza8J3dqrBmuTvOWgwq31ZoSnC6Xy7f972ftudxR5vU47C3Emq78KylOZHpjjSKChidn5btXKxsLLwYHE+EpqLZRd3cO9jMGH1R5g2l2jketv4zwnH/WvSQBoO21FJURW1MIGNz7UOBRat0JmSrdCKP+O8X0JdCedoSgOWl2oRtb+EPzjL5JtCxdDci6NtWDeE2s1pNlIa7tNkkRFXxzgoJ0iIf5jPZFlAl/qaPSLgfBKCLdKoicy/A1tlvqO92rmeku85mT7wqZV5NXfW+LLu2kNr5QA8MJS0o+lvAD1VHnz30wB+EmTXPV6h4LbIru1AAmfesIP5MiGXUoKf55GRFfBZC3SyY0IWq+BHsK70GOJcEOKPwPnqxVjZOUdn+tNrX2KNIdXKPEBpILkz6sjNaqOQ5XwFuNCLyddSiS5YLSPylZ5QPB4jJsnqKNXiGVU/o0KWi/cWS6j/ki0fy99dP59qet5XRW2eogmwQX1A4fAbGGSRmxsnJd8tW3rp3Vn7ohMbMO4Li1dsGkMm8FKxBI9bhx7BEws1N0FzTMSCuEPg3134CrHwpgCr8G3LUNFr65FPwC9rQFzQ38VIV3rNZ1gnQvUrfKKwqwbFoKUCog1Fxc653INnLfGcCyn126UPLf5qv2ChaN1bOLA5+VtOrdx7HOv7CWWrSRP6nu70vAV6ia/TP/PJvE6xnUcce76cS4Pdzq5WsS3zoiIjFdgVHE+MWsKfQJrNLloWfArCp/ZaBPSLJjEX10KHsx4OEAiTpHRwi7OV+1xHyxeMJ28cDopnpNyV6OfOfyh68/FtO1naOWCiu0B1+pfilGmW+IAYqx9ebsw/GBVIHrHNaue1d/Jwebt3DCC4GgPcVraKBkN7b7DkPr8D3d+Wns8fNyHmAFqFTxN2Ot9qeESn5pmULfGEs3d4Sdlz7cVvf+DD7qkUNzaMFnI1HvzWp5J964IVIzcYA0IVwrlYCfPPE2bWPGgmFcMUiS8NqYBR1CoWacCPZeozi4rKale5VnZpcIuQ5/Jdk6gbNr5ns2+hOW5YSCxjVSYT20uBYTPoahI6otzZZbaw/LviNM6PMSvwZC0aLvsjvnSdtn3qm36adYKAQ53kGUbSiVMIMOvbWw2AcI9OOtTwGouCK4sOBK8BHgN5nlUWwAkQ77TdlQfloIa+onKJeCebodQj76kuRrWb90nY0SpoYsFYT82hYpzvuA87b9R1ZfEZyQrB68DCaPtyOuG7Qt/mg= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 33c9dd89-0ced-402e-950a-08de5a7f4645 X-MS-Exchange-CrossTenant-AuthSource: AM9PR04MB8147.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Jan 2026 12:59:42.1015 (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: 4vXXmW0AQZTZlF1mqyFnQtM6jHTDCJPn56lWnI+fPeAS8IIVuZ8IEmqvDvni+YfVnSfC5IoHlaTCUfa4sosJRxcR/RJGXEEJHx6ULHeZcRs= X-MS-Exchange-Transport-CrossTenantHeadersStamped: GVXPR04MB11690 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" The CameraSensor::imageStream() function has a default implementation in the base class that returns {0, 0} as {pad, stream}, assuming that the image pad is located on the pad index 0. This assumption is correct most of the time, but not in some other cases, for instance when an external ISP entity acts as the sensor. Such entity would typically have sink pad(s) connected to the actual sensor and source pad(s) connected to the downstream graph. Associated pad indexes would likely be different from zero. CameraSensorLegacy subclass correctly handles this case in its functions, using the pad_ variable discovered at init() time to access the source pad index, instead of using a hardcoded zero value. Exception is imageStream() that is not overriden in the CameraSensorLegacy definition so keeps the default implementation of the parent class, hardcoding the returned source pad index to zero. This change declares CameraSensor::imageStream() as a pure virtual to let the subclasses provide an implementation. Implementation for CameraSensorLegacy is added, based on pad_ variable usage. Signed-off-by: Julien Vuillaumier --- include/libcamera/internal/camera_sensor.h | 2 +- src/libcamera/sensor/camera_sensor.cpp | 5 +---- src/libcamera/sensor/camera_sensor_legacy.cpp | 6 ++++++ 3 files changed, 8 insertions(+), 5 deletions(-) diff --git a/include/libcamera/internal/camera_sensor.h b/include/libcamera/internal/camera_sensor.h index e6b72d22a..58e7df4a4 100644 --- a/include/libcamera/internal/camera_sensor.h +++ b/include/libcamera/internal/camera_sensor.h @@ -64,7 +64,7 @@ public: Transform transform = Transform::Identity, V4L2SubdeviceFormat *sensorFormat = nullptr) = 0; - virtual V4L2Subdevice::Stream imageStream() const; + virtual V4L2Subdevice::Stream imageStream() const = 0; virtual std::optional embeddedDataStream() const; virtual V4L2SubdeviceFormat embeddedDataFormat() const; virtual int setEmbeddedDataEnabled(bool enable); diff --git a/src/libcamera/sensor/camera_sensor.cpp b/src/libcamera/sensor/camera_sensor.cpp index 05390d1e1..58affd8f5 100644 --- a/src/libcamera/sensor/camera_sensor.cpp +++ b/src/libcamera/sensor/camera_sensor.cpp @@ -200,6 +200,7 @@ CameraSensor::~CameraSensor() = default; */ /** + * \fn CameraSensor::imageStream() * \brief Retrieve the image source stream * * Sensors that produce multiple streams do not guarantee that the image stream @@ -209,10 +210,6 @@ CameraSensor::~CameraSensor() = default; * * \return The image source stream */ -V4L2Subdevice::Stream CameraSensor::imageStream() const -{ - return { 0, 0 }; -} /** * \brief Retrieve the embedded data source stream diff --git a/src/libcamera/sensor/camera_sensor_legacy.cpp b/src/libcamera/sensor/camera_sensor_legacy.cpp index 39c34200b..d2583a615 100644 --- a/src/libcamera/sensor/camera_sensor_legacy.cpp +++ b/src/libcamera/sensor/camera_sensor_legacy.cpp @@ -84,6 +84,7 @@ public: Transform transform = Transform::Identity, V4L2SubdeviceFormat *sensorFormat = nullptr) override; + V4L2Subdevice::Stream imageStream() const override; const ControlList &properties() const override { return properties_; } int sensorInfo(IPACameraSensorInfo *info) const override; Transform computeTransform(Orientation *orientation) const override; @@ -856,6 +857,11 @@ int CameraSensorLegacy::applyConfiguration(const SensorConfiguration &config, return 0; } +V4L2Subdevice::Stream CameraSensorLegacy::imageStream() const +{ + return { pad_, 0 }; +} + int CameraSensorLegacy::sensorInfo(IPACameraSensorInfo *info) const { if (!bayerFormat_)