Patch Detail
Show a patch.
GET /api/patches/11039/?format=api
{ "id": 11039, "url": "https://patchwork.libcamera.org/api/patches/11039/?format=api", "web_url": "https://patchwork.libcamera.org/patch/11039/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/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": "<20210128091050.881815-3-naush@raspberrypi.com>", "date": "2021-01-28T09:10:47", "name": "[libcamera-devel,v3,2/5] libcamera: camera_sensor: Add frame length limits to CameraSensorInfo", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "d9a8657e6b4d600f3ab2c6ee5ae55e0221ee7683", "submitter": { "id": 34, "url": "https://patchwork.libcamera.org/api/people/34/?format=api", "name": "Naushir Patuck", "email": "naush@raspberrypi.com" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/11039/mbox/", "series": [ { "id": 1619, "url": "https://patchwork.libcamera.org/api/series/1619/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=1619", "date": "2021-01-28T09:10:45", "name": "Raspberry Pi: FrameDurations control refinements", "version": 3, "mbox": "https://patchwork.libcamera.org/series/1619/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/11039/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/11039/checks/", "tags": {}, "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 42212BD808\n\tfor <parsemail@patchwork.libcamera.org>;\n\tThu, 28 Jan 2021 09:10:59 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 1167868383;\n\tThu, 28 Jan 2021 10:10:59 +0100 (CET)", "from mail-wr1-x42e.google.com (mail-wr1-x42e.google.com\n\t[IPv6:2a00:1450:4864:20::42e])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 6FB406837A\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tThu, 28 Jan 2021 10:10:56 +0100 (CET)", "by mail-wr1-x42e.google.com with SMTP id v15so4612796wrx.4\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tThu, 28 Jan 2021 01:10:56 -0800 (PST)", "from naushir-VirtualBox.patuck.local ([88.97.76.4])\n\tby smtp.gmail.com with ESMTPSA id\n\tf17sm6517056wrv.0.2021.01.28.01.10.54\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tThu, 28 Jan 2021 01:10:55 -0800 (PST)" ], "Authentication-Results": "lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=raspberrypi.com header.i=@raspberrypi.com\n\theader.b=\"rNuqhqHv\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=raspberrypi.com; s=google;\n\th=from:to:cc:subject:date:message-id:in-reply-to:references\n\t:mime-version:content-transfer-encoding;\n\tbh=Vjiy0X5vCxiIh0kqLnE5/yh2+p1/5u96Km9Vbm5Nhag=;\n\tb=rNuqhqHvbXxbnOQzcjoGf2m0c2lBIWouTC9R6KUmb8A/uuAPTU4dzDDPFN4Y8A97xK\n\t8Iu5gGHrpbxmoHDOPCp/LRjDKe0YCOrHTJqgmA2JaytisdAjCY12laajaCwS0oFXxoK2\n\tOE7yVwyCVouaQOPBCfEcAFIgdqFdnM5mNAEjIDgdyLyVV5VarsE8Vs+orvH6bb7LlI2s\n\t1gSoR8wQ2KKEO4+v4L0SmWVjcwxHTpTwoWFfzlpJuX4ViipkcUtF7w3WueqvQw+qE+EV\n\tXIxhB8G5az+Rl2I6R0qqkNdsESjgH1PLEN+lCYVsZ2YbYSnka58ILoQNcjWi06W9YZ8/\n\t3MQA==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\n\t:references:mime-version:content-transfer-encoding;\n\tbh=Vjiy0X5vCxiIh0kqLnE5/yh2+p1/5u96Km9Vbm5Nhag=;\n\tb=J3RHv/u3hLgWrbUEkTVmHw6htTED6A//dGiFjAva5T+qJaPviTHSwRJ8hVEeo4+rzy\n\t9ZFKSptcTYVe2wO7dmE9o6DSj6I+4QhdMBfbU5LJ5YiUUfbT37YGH2yO2dNYgkh/pCuB\n\tbstStYzkYHBurlk/oL5gI2JlJxqAjue5H12iOySB4WafKO7xBoOokqE+gkVRy8au4ioQ\n\t1YWbGD4S3tf2nxC72VCjfL/QH/htJlYSniV2I9EIgjsQmbV4nTVdHwVrZUntImEG0F5i\n\tavpnjIujJmj34o3EGs11yu3WlFYujQAkeNwR1UPyJ1UcKmPP9rc8ugADJRSiO60LeeAK\n\tnORA==", "X-Gm-Message-State": "AOAM532NExsq9ylpumnWhvHiHps5KoH3GqLMURdrh9APVPB4Xw+LrQ2Z\n\tPtlP8qmJlDtN1B4lVwOlHlglr201Yex2Qmdf", "X-Google-Smtp-Source": "ABdhPJxM2OmdeaqnGJFd8KfMV4KA5jahZlTVgMLoPxScmDkH09vQGCWAnxQetyCgg0uRQ5O3sIdIXw==", "X-Received": "by 2002:adf:fc8a:: with SMTP id\n\tg10mr15702631wrr.189.1611825055865; \n\tThu, 28 Jan 2021 01:10:55 -0800 (PST)", "From": "Naushir Patuck <naush@raspberrypi.com>", "To": "libcamera-devel@lists.libcamera.org", "Date": "Thu, 28 Jan 2021 09:10:47 +0000", "Message-Id": "<20210128091050.881815-3-naush@raspberrypi.com>", "X-Mailer": "git-send-email 2.25.1", "In-Reply-To": "<20210128091050.881815-1-naush@raspberrypi.com>", "References": "<20210128091050.881815-1-naush@raspberrypi.com>", "MIME-Version": "1.0", "Subject": "[libcamera-devel] [PATCH v3 2/5] libcamera: camera_sensor: Add\n\tframe length limits to CameraSensorInfo", "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": "Sensor frame length is made up of active and inactive (blanking) lines.\nThe minimum and maximum frame length values may be used by pipeline\nhandlers to limit frame durations based on the sensor mode capabilities.\n\nStore the minimum and maximum allowable frame length values (in lines)\nin the CameraSensorInfo structure. These values are computed in\nCameraSensor::sensorInfo() by querying the sensor subdevice\nV4L2_CID_VBLANK control limits. This in turn means that V4L2_CID_VBLANK\nis now a mandatory subdevice control.\n\nSigned-off-by: Naushir Patuck <naush@raspberrypi.com>\n---\n include/libcamera/internal/camera_sensor.h | 3 ++\n src/libcamera/camera_sensor.cpp | 43 ++++++++++++++++++++--\n test/ipa/ipa_wrappers_test.cpp | 2 +\n 3 files changed, 44 insertions(+), 4 deletions(-)", "diff": "diff --git a/include/libcamera/internal/camera_sensor.h b/include/libcamera/internal/camera_sensor.h\nindex fed36bf26e47..5d8c9b1a3121 100644\n--- a/include/libcamera/internal/camera_sensor.h\n+++ b/include/libcamera/internal/camera_sensor.h\n@@ -33,6 +33,9 @@ struct CameraSensorInfo {\n \n \tuint64_t pixelRate;\n \tuint32_t lineLength;\n+\n+\tuint32_t minFrameLength;\n+\tuint32_t maxFrameLength;\n };\n \n class CameraSensor : protected Loggable\ndiff --git a/src/libcamera/camera_sensor.cpp b/src/libcamera/camera_sensor.cpp\nindex ab315bdc468c..f60d0cc9c6fa 100644\n--- a/src/libcamera/camera_sensor.cpp\n+++ b/src/libcamera/camera_sensor.cpp\n@@ -113,6 +113,36 @@ LOG_DEFINE_CATEGORY(CameraSensor)\n * The total line length in pixel clock periods, including blanking.\n */\n \n+/**\n+ * \\var CameraSensorInfo::minFrameLength\n+ * \\brief The minimum allowable frame length in units of lines\n+ *\n+ * The sensor frame length comprises of active output lines and blanking lines\n+ * in a frame. The minimum frame length value dictates the minimum allowable\n+ * frame duration of the sensor mode.\n+ *\n+ * To obtain the minimum frame duration:\n+ *\n+ * \\verbatim\n+\tframeDuration(s) = minFrameLength(lines) * lineLength(pixels) / pixelRate(pixels per second)\n+ \\endverbatim\n+ */\n+\n+/**\n+ * \\var CameraSensorInfo::maxFrameLength\n+ * \\brief The maximum allowable frame length in units of lines\n+ *\n+ * The sensor frame length comprises of active output lines and blanking lines\n+ * in a frame. The maximum frame length value dictates the maximum allowable\n+ * frame duration of the sensor mode.\n+ *\n+ * To obtain the maximum frame duration:\n+ *\n+ * \\verbatim\n+\tframeDuration(s) = maxFrameLength(lines) * lineLength(pixels) / pixelRate(pixels per second)\n+ \\endverbatim\n+ */\n+\n /**\n * \\class CameraSensor\n * \\brief A camera sensor based on V4L2 subdevices\n@@ -699,12 +729,13 @@ int CameraSensor::sensorInfo(CameraSensorInfo *info) const\n \tinfo->outputSize = format.size;\n \n \t/*\n-\t * Retrieve the pixel rate and the line length through V4L2 controls.\n-\t * Support for the V4L2_CID_PIXEL_RATE and V4L2_CID_HBLANK controls is\n-\t * mandatory.\n+\t * Retrieve the pixel rate, line length and minimum/maximum frame\n+\t * duration through V4L2 controls. Support for the V4L2_CID_PIXEL_RATE,\n+\t * V4L2_CID_HBLANK and V4L2_CID_VBLANK controls is mandatory.\n \t */\n \tControlList ctrls = subdev_->getControls({ V4L2_CID_PIXEL_RATE,\n-\t\t\t\t\t\t V4L2_CID_HBLANK });\n+\t\t\t\t\t\t V4L2_CID_HBLANK,\n+\t\t\t\t\t\t V4L2_CID_VBLANK });\n \tif (ctrls.empty()) {\n \t\tLOG(CameraSensor, Error)\n \t\t\t<< \"Failed to retrieve camera info controls\";\n@@ -715,6 +746,10 @@ int CameraSensor::sensorInfo(CameraSensorInfo *info) const\n \tinfo->lineLength = info->outputSize.width + hblank;\n \tinfo->pixelRate = ctrls.get(V4L2_CID_PIXEL_RATE).get<int64_t>();\n \n+\tconst ControlInfo vblank = ctrls.infoMap()->at(V4L2_CID_VBLANK);\n+\tinfo->minFrameLength = info->outputSize.height + vblank.min().get<int32_t>();\n+\tinfo->maxFrameLength = info->outputSize.height + vblank.max().get<int32_t>();\n+\n \treturn 0;\n }\n \ndiff --git a/test/ipa/ipa_wrappers_test.cpp b/test/ipa/ipa_wrappers_test.cpp\nindex 47533d105d03..eb6d783e8489 100644\n--- a/test/ipa/ipa_wrappers_test.cpp\n+++ b/test/ipa/ipa_wrappers_test.cpp\n@@ -313,6 +313,8 @@ protected:\n \t\t\t.outputSize = { 2560, 1940 },\n \t\t\t.pixelRate = 96000000,\n \t\t\t.lineLength = 2918,\n+\t\t\t.minFrameLength = 1940,\n+\t\t\t.maxFrameLength = 2880\n \t\t};\n \t\tstd::map<unsigned int, IPAStream> config{\n \t\t\t{ 1, { V4L2_PIX_FMT_YUYV, { 1024, 768 } } },\n", "prefixes": [ "libcamera-devel", "v3", "2/5" ] }