Cover Letter Detail
Show a cover letter.
GET /api/1.1/covers/11072/?format=api
{ "id": 11072, "url": "https://patchwork.libcamera.org/api/1.1/covers/11072/?format=api", "web_url": "https://patchwork.libcamera.org/cover/11072/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/1.1/projects/1/?format=api", "name": "libcamera", "link_name": "libcamera", "list_id": "libcamera_core", "list_email": "libcamera-devel@lists.libcamera.org", "web_url": "", "scm_url": "", "webscm_url": "" }, "msgid": "<20210131181722.5410-1-laurent.pinchart@ideasonboard.com>", "date": "2021-01-31T18:17:19", "name": "[libcamera-devel,0/3] Restore CameraSensor usage for YUV sensors", "submitter": { "id": 2, "url": "https://patchwork.libcamera.org/api/1.1/people/2/?format=api", "name": "Laurent Pinchart", "email": "laurent.pinchart@ideasonboard.com" }, "mbox": "https://patchwork.libcamera.org/cover/11072/mbox/", "series": [ { "id": 1631, "url": "https://patchwork.libcamera.org/api/1.1/series/1631/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=1631", "date": "2021-01-31T18:17:19", "name": "Restore CameraSensor usage for YUV sensors", "version": 1, "mbox": "https://patchwork.libcamera.org/series/1631/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/covers/11072/comments/", "headers": { "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\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 6DE23BD808\n\tfor <parsemail@patchwork.libcamera.org>;\n\tSun, 31 Jan 2021 18:17:49 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id B6F99683DA;\n\tSun, 31 Jan 2021 19:17:48 +0100 (CET)", "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 96BC260309\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tSun, 31 Jan 2021 19:17:46 +0100 (CET)", "from pendragon.lan (62-78-145-57.bb.dnainternet.fi [62.78.145.57])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id F3F38813\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tSun, 31 Jan 2021 19:17:45 +0100 (CET)" ], "Authentication-Results": "lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"sC6lUZ1c\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1612117066;\n\tbh=FiZkxEO+Qx8apCJurmUkSbWqMAP24Rr9v9zi4jljSLU=;\n\th=From:To:Subject:Date:From;\n\tb=sC6lUZ1ct6cA5VdQScYjJZRio+J3NXrHVlzqC15q+zKsUEiA9tKKH8NfPN4/+86I9\n\tnduGRDPtjK5uHd9yJ58aw8+yyOPkDWMOmc2ij2dZBYZSUgFuslccaGx7Ki9i+xl2eX\n\tk/GxIX7P3IprOn3sgGq55qHH264m3QZMnxwCEVB0=", "From": "Laurent Pinchart <laurent.pinchart@ideasonboard.com>", "To": "libcamera-devel@lists.libcamera.org", "Date": "Sun, 31 Jan 2021 20:17:19 +0200", "Message-Id": "<20210131181722.5410-1-laurent.pinchart@ideasonboard.com>", "X-Mailer": "git-send-email 2.28.0", "MIME-Version": "1.0", "Subject": "[libcamera-devel] [PATCH 0/3] Restore CameraSensor usage for YUV\n\tsensors", "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>,\n\t<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>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>", "Content-Type": "text/plain; charset=\"us-ascii\"", "Content-Transfer-Encoding": "7bit", "Errors-To": "libcamera-devel-bounces@lists.libcamera.org", "Sender": "\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>" }, "content": "Hello,\n\nThe recent changes to the CameraSensor class that made the V4L2\nEXPOSURE, HBLANK and PIXEL_RATE controls mandatory broke usage of YUV\nsensors in the simple pipeline handler, as many YUV sensor drivers don't\nsupport those controls.\n\nDrivers could of course be extended one by one to support the mandatory\ncontrols, but not all of these controls make sense for YUV sensors. YUV\nsensors include an ISP, and typically expose a higher-level API than raw\nsensors. They may not support manual control of exposure time (and while\nmost of the sensors should support this, information to implement manual\nexposure time is not as widely available in sensor documentation than\nfor raw sensors) or horizontal blanking.\n\nSupport for those three controls is needed for raw sensors as they are\nrequired to properly configure the ISP or implement the imaging\nalgorithms. As the simple pipeline handler doesn't support ISPs and rely\non the sensor including an ISP, the information provided by these\ncontrols isn't needed, as shown by the pipeline handler not calling\nCameraSensor::sensorInfo(). The best course of action is thus to make\nthese controls mandatory for raw sensors only, restoring support of YUV\nsensors in the simple pipeline handler.\n\nLaurent Pinchart (3):\n libcamera: camera_sensor: Store Bayer pattern in class member\n libcamera: camera_sensor: Restrict sensor info to raw sensors\n libcamera: camera_sensor: Check control availability from idmap\n\n Documentation/sensor_driver_requirements.rst | 2 +-\n include/libcamera/internal/camera_sensor.h | 2 +\n src/libcamera/camera_sensor.cpp | 98 +++++++++++++-------\n 3 files changed, 65 insertions(+), 37 deletions(-)" }