| Message ID | 20261102-kbingham-orientation-v2-0-21b1cdd0d167@ideasonboard.com |
|---|---|
| Headers | show
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 [92.243.16.209]) by patchwork.libcamera.org (Postfix) with ESMTPS id 052F2C324C for <parsemail@patchwork.libcamera.org>; Mon, 2 Nov 2026 12:11:44 +0000 (UTC) Received: from lancelot.ideasonboard.com (localhost [IPv6:::1]) by lancelot.ideasonboard.com (Postfix) with ESMTP id 1646A691F3; Mon, 2 Nov 2026 13:11:43 +0100 (CET) Authentication-Results: lancelot.ideasonboard.com; dkim=pass (1024-bit key; unprotected) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="ZeP5T+lm"; dkim-atps=neutral Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 77202691DC for <libcamera-devel@lists.libcamera.org>; Mon, 2 Nov 2026 13:11:41 +0100 (CET) Received: from ping.linuxembedded.co.uk (cpc89244-aztw30-2-0-cust6594.18-1.cable.virginm.net [86.31.185.195]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id CE92A4DC; Mon, 2 Nov 2026 13:09:24 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1793621365; bh=M0G48bU13701J31hfw8h3xPCugpQ/J0UfzQX+H1rRSE=; h=From:Subject:Date:To:Cc:From; b=ZeP5T+lm21MGVEP3IlMqwc+99L+PQU+WobkIjesa+6x4H93PfoNU2x/PoDAOwsXby K8YIr8sDEQEBHtGNICnmIG8eeJXG1AzzT3cxEKXqETGPg/uASakvug4zRx0cTAXjJU f8znUgWV9ajOyAT1dQaHK+8R1D6y6B9oCkT7nBmY= From: Kieran Bingham <kieran.bingham@ideasonboard.com> Subject: [PATCH v2 0/3] libcamera: rkisp1: Camera Sensor Mode Validation Date: Mon, 02 Nov 2026 12:11:32 +0000 Message-Id: <20261102-kbingham-orientation-v2-0-21b1cdd0d167@ideasonboard.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-B4-Tracking: v=1; b=H4sIAPV96GoC/x3MQQqAIBBA0avErBN0qKiuEi0sxxoiC5UIors3t PyL9x9IFJkS9MUDkS5OfAQJLAuYVxsWUuykATU2xmhU28RhWe2uDnEh2yxAVWic89p3dduB0DO S5/vfDuP7fid9zaNmAAAA X-Change-ID: 20261102-kbingham-orientation-421ddf0f9589 To: libcamera-devel@lists.libcamera.org Cc: Kieran Bingham <kieran.bingham@ideasonboard.com>, Jacopo Mondi <jacopo.mondi@ideasonboard.com>, =?utf-8?q?Barnab=C3=A1s_P=C5=91cze?= <barnabas.pocze@ideasonboard.com> X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=ed25519-sha256; t=1793621500; l=1522; i=kieran.bingham@ideasonboard.com; s=20260207; h=from:subject:message-id; bh=M0G48bU13701J31hfw8h3xPCugpQ/J0UfzQX+H1rRSE=; b=+gS4mAUOYM5EHShSVWSasI0Cv7CLWHeztBis6DY00tnQ5zdSZ51cHkmt8909kNY1aYAShYoWx XJlXZnItaZMD4p9UeOZwMcZzoRf7/SOWVQjEw3i2BNKOZ6i1STseVz+ X-Developer-Key: i=kieran.bingham@ideasonboard.com; a=ed25519; pk=FVXKN7YuwHc6UtbRUeTMAmranfsQomA+vnilfglWdaY= 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>, <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>, <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> |
| Series |
|
| Related |
show
|
While testing the RKISP1 with dewarp features enabled, it is possible to have a camera with a default 90* rotation. During testing it becomes very easy to transpose the width and height - which can lead to asking for an invalid camera configuration. This demonstrated itself as a configuration that was successfully validated, but that could never successfully configure. Expand the CameraSensor classes to aid reporting this instance to the user, and ensure that if an invalid sensor configuration is selected that the Configuration is rejected as Invalid. Further extend the 'cam' tool to support setting the rotation(Orientations) accordingly which then allows the desired configuration to be tested correctly. Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com> --- Kieran Bingham (3): libcamera: pipeline: rkisp1: Detect invalid sensor configurations libcamera: camera_sensor: Extend reporting in getFormat() apps: cam: Support full orientation options src/apps/cam/camera_session.cpp | 9 +++++++++ src/apps/cam/main.cpp | 5 ++++- src/libcamera/pipeline/rkisp1/rkisp1.cpp | 12 ++++++++++++ src/libcamera/sensor/camera_sensor_legacy.cpp | 4 +++- src/libcamera/sensor/camera_sensor_raw.cpp | 4 +++- 5 files changed, 31 insertions(+), 3 deletions(-) --- base-commit: df8e348d64b4932a0676256ed24f0da3ee9438bc change-id: 20261102-kbingham-orientation-421ddf0f9589 Best regards,