[{"id":32879,"web_url":"https://patchwork.libcamera.org/comment/32879/","msgid":"<20241219012843.GM5518@pendragon.ideasonboard.com>","date":"2024-12-19T01:28:43","subject":"Re: [PATCH] libcamera: camera_sensor_properties: ov5675: Set correct\n\tdelays","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Kieran,\n\nThank you for the patch.\n\nOn Thu, Dec 19, 2024 at 12:59:58AM +0000, Kieran Bingham wrote:\n> The OV5675 uses different delays for gain and exposure than are configured\n> in the default sensorDelays utilised by the CameraSensorLegacy.\n> \n> Empirical testing using a Lenovo X13s shows that the exposure delay is\n> only a single frame, and the current setting of 2 frame delay produces\n> exceedingly frequent oscillations in the image exposure.\n\nHave you been able to test the other delays ? If not I'd like at least a\ncomment here that indicates this needs to be checked.\n\nAny volunteer to write a measurement tool ?\n\n> Update the OV5675 sensor delays table accordingly.\n> \n> Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>\n> ---\n>  src/libcamera/sensor/camera_sensor_properties.cpp | 7 ++++++-\n>  1 file changed, 6 insertions(+), 1 deletion(-)\n> \n> diff --git a/src/libcamera/sensor/camera_sensor_properties.cpp b/src/libcamera/sensor/camera_sensor_properties.cpp\n> index bd1fc86977ce..813878386a35 100644\n> --- a/src/libcamera/sensor/camera_sensor_properties.cpp\n> +++ b/src/libcamera/sensor/camera_sensor_properties.cpp\n> @@ -360,7 +360,12 @@ const CameraSensorProperties *CameraSensorProperties::get(const std::string &sen\n>  \t\t\t\t{ controls::draft::TestPatternModeOff, 0 },\n>  \t\t\t\t{ controls::draft::TestPatternModeColorBars, 1 },\n>  \t\t\t},\n> -\t\t\t.sensorDelays = { },\n> +\t\t\t.sensorDelays = {\n> +\t\t\t\t.exposureDelay = 1,\n> +\t\t\t\t.gainDelay = 1,\n> +\t\t\t\t.vblankDelay = 2,\n> +\t\t\t\t.hblankDelay = 2\n> +\t\t\t    },\n>  \t\t} },\n>  \t\t{ \"ov5693\", {\n>  \t\t\t.unitCellSize = { 1400, 1400 },","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 2E9CBC32FE\n\tfor <parsemail@patchwork.libcamera.org>;\n\tThu, 19 Dec 2024 01:28:51 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 6DA186845C;\n\tThu, 19 Dec 2024 02:28:50 +0100 (CET)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id BEEE061897\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tThu, 19 Dec 2024 02:28:48 +0100 (CET)","from pendragon.ideasonboard.com (81-175-209-231.bb.dnainternet.fi\n\t[81.175.209.231])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 13EB7415;\n\tThu, 19 Dec 2024 02:28:10 +0100 (CET)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"t3L70SG3\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1734571690;\n\tbh=SRorqCR4pKNfTeU/zf51xtDrxwldisbMwOA7he+TsK4=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=t3L70SG3BZ96ZtaMDgRFYoc6LqF6VjZcssnmRksmfOZAqGwy3LvHXn/fZvbBSNxrT\n\tlxA/AxvXVS3OFrNz4B66XsVndP/+grebd8kNRvpS7Uk4ZERzobLAV+KaC6kT1m/gwA\n\tUQbUiddSf7wFmVXg5y4kToJuJuVgrCuEbSw+idbQ=","Date":"Thu, 19 Dec 2024 03:28:43 +0200","From":"Laurent Pinchart <laurent.pinchart@ideasonboard.com>","To":"Kieran Bingham <kieran.bingham@ideasonboard.com>","Cc":"libcamera devel <libcamera-devel@lists.libcamera.org>,\n\tbryan.odonoghue@linaro.org","Subject":"Re: [PATCH] libcamera: camera_sensor_properties: ov5675: Set correct\n\tdelays","Message-ID":"<20241219012843.GM5518@pendragon.ideasonboard.com>","References":"<20241219005958.16856-1-kieran.bingham@ideasonboard.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","In-Reply-To":"<20241219005958.16856-1-kieran.bingham@ideasonboard.com>","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>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}},{"id":32881,"web_url":"https://patchwork.libcamera.org/comment/32881/","msgid":"<CAEmqJPoDcyFzV2VHBObqv9p+0fRjO0uaZups8fdx4=edEYm8hg@mail.gmail.com>","date":"2024-12-19T09:33:56","subject":"Re: [PATCH] libcamera: camera_sensor_properties: ov5675: Set correct\n\tdelays","submitter":{"id":34,"url":"https://patchwork.libcamera.org/api/people/34/","name":"Naushir Patuck","email":"naush@raspberrypi.com"},"content":"Hi Kieran,\n\nOn Thu, 19 Dec 2024 at 01:00, Kieran Bingham\n<kieran.bingham@ideasonboard.com> wrote:\n>\n> The OV5675 uses different delays for gain and exposure than are configured\n> in the default sensorDelays utilised by the CameraSensorLegacy.\n>\n> Empirical testing using a Lenovo X13s shows that the exposure delay is\n> only a single frame, and the current setting of 2 frame delay produces\n> exceedingly frequent oscillations in the image exposure.\n>\n> Update the OV5675 sensor delays table accordingly.\n>\n> Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>\n> ---\n>  src/libcamera/sensor/camera_sensor_properties.cpp | 7 ++++++-\n>  1 file changed, 6 insertions(+), 1 deletion(-)\n>\n> diff --git a/src/libcamera/sensor/camera_sensor_properties.cpp b/src/libcamera/sensor/camera_sensor_properties.cpp\n> index bd1fc86977ce..813878386a35 100644\n> --- a/src/libcamera/sensor/camera_sensor_properties.cpp\n> +++ b/src/libcamera/sensor/camera_sensor_properties.cpp\n> @@ -360,7 +360,12 @@ const CameraSensorProperties *CameraSensorProperties::get(const std::string &sen\n>                                 { controls::draft::TestPatternModeOff, 0 },\n>                                 { controls::draft::TestPatternModeColorBars, 1 },\n>                         },\n> -                       .sensorDelays = { },\n> +                       .sensorDelays = {\n> +                               .exposureDelay = 1,\n\nFrom my experience, an exposure delay of 1 is very unlikely with a\nrolling shutter sensor because of the integration overlap between\nsuccessive frames.  Of course, I could be wrong since I've not\nencounter this sensor before.  Perhaps it's worth looking at other\ndelays to see if the general interaction between vblank/exposure/gain\nis causing the oscillations?\n\nRegards,\nNaush\n\n> +                               .gainDelay = 1,\n> +                               .vblankDelay = 2,\n> +                               .hblankDelay = 2\n> +                           },\n>                 } },\n>                 { \"ov5693\", {\n>                         .unitCellSize = { 1400, 1400 },\n> --\n> 2.47.1\n>","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 A549BC3272\n\tfor <parsemail@patchwork.libcamera.org>;\n\tThu, 19 Dec 2024 09:34:34 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 9BE9E68462;\n\tThu, 19 Dec 2024 10:34:33 +0100 (CET)","from mail-yb1-xb33.google.com (mail-yb1-xb33.google.com\n\t[IPv6:2607:f8b0:4864:20::b33])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 05C6067FA7\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tThu, 19 Dec 2024 10:34:30 +0100 (CET)","by mail-yb1-xb33.google.com with SMTP id\n\t3f1490d57ef6-e46e574847aso80524276.0\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tThu, 19 Dec 2024 01:34:30 -0800 (PST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (2048-bit key;\n\tunprotected) header.d=raspberrypi.com header.i=@raspberrypi.com\n\theader.b=\"tZW3U7LH\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=raspberrypi.com; s=google; t=1734600869; x=1735205669;\n\tdarn=lists.libcamera.org; \n\th=cc:to:subject:message-id:date:from:in-reply-to:references\n\t:mime-version:from:to:cc:subject:date:message-id:reply-to;\n\tbh=pQboe20DAoYiljpJ3bHw/WP7zoCCXpuWYMuxEbUrPy8=;\n\tb=tZW3U7LHNEeMv+EFkZZejlfiIaXNEi28FfzVV2RxLu78Je6UW22yYb0j/RsBZ3ZAjr\n\twazEC87dno9huFDC2BXQJfeo/YoZZjXQpoCx/CNzYefHQve9SrkWsgf174MjRbdIUsij\n\ti6zDatCfPqi8wrfJfEkz5PnHtfcYYP0IwU5Nr2jbJ/4IHQ4fo4bbw+nT3YJjR2fn8CBp\n\tp84N6vOPkZe1IRea6Zgt38kHdLJqIp2iIPP/zoJGxQNyBP5s4rfXGfyU1oXAFK9wr+EN\n\tBYKXFIUMQV50WQF5fdySSZ2D+6g/PfWODfTW2JslnkjPFzXteERrScOW5V49uRYwxkGY\n\t4HKQ==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20230601; t=1734600869; x=1735205669;\n\th=cc:to:subject:message-id:date:from:in-reply-to:references\n\t:mime-version:x-gm-message-state:from:to:cc:subject:date:message-id\n\t:reply-to;\n\tbh=pQboe20DAoYiljpJ3bHw/WP7zoCCXpuWYMuxEbUrPy8=;\n\tb=AQUpKsY59L+H/Ds6W5qCQ+v9R9eUDtvrhr1EwY31BxuBCzDyfHIe1KEBoWlLUOUHvq\n\tua3IPesR51xNUtq1xCq9ffc4pcTpSueXhqMK0P0C6+FFKkLZjY4XoUwJJ9QgDQmgwHZo\n\tgXdO82c06QuT2d/K4hxASLywWY71Z3fqmDTI/PjbqL/XjW6UenqfkacwXDJDe6x1kaLf\n\twiXpBCps6YDJWwlUi/1aSQC+oIZOnh8ZsX8mXK5kSvSoOfaJn6b+yVja+9vu5rqxv8q3\n\tWWkCS+37h16RX4RRKUzEIf8NwP9cedl3K+jQXxOZlMIJI98tsQib1PdD/eYd0qZin3Fc\n\t2fsA==","X-Gm-Message-State":"AOJu0Yz0Fk4d0Rix22AvOzhAJmqHHWuxh8sArZrvk6Fr9dONwq2SGE12\n\tnHMkB0DF2RQ+NvSMLsvLgy8dbszxlI3FVOjz7+s0t5bdSlfxeySTMf/ja2tPvrapwl4rHBOzUi5\n\t0+7zwGO1l3+cv4L1s5dNUKlSXNBrwozJvGCKCmw==","X-Gm-Gg":"ASbGncs+iC12xuB3g54VsdDCcUeTsRrDHF6/rNs9lPTbZ5DyM94cvS3t/cr9lM9lbNG\n\tYO+MofIEnG+qoaN3mvN62ICv8q4qqRUcAweqVle8JJ4QX/DmzumVGNJ5yU6n5tuX0wH0foQ==","X-Google-Smtp-Source":"AGHT+IEC636YBbSY2/smds+RBrz/ckuYDq5eKTPRA8eCyCsCXDKAjkzrD1KNs6U3MfkRJSfv9cBBCeE66VZ6H/qEuBQ=","X-Received":"by 2002:a05:6902:2382:b0:e4a:ea18:b45c with SMTP id\n\t3f1490d57ef6-e53622ee782mr1947564276.8.1734600869723; Thu, 19 Dec 2024\n\t01:34:29 -0800 (PST)","MIME-Version":"1.0","References":"<20241219005958.16856-1-kieran.bingham@ideasonboard.com>","In-Reply-To":"<20241219005958.16856-1-kieran.bingham@ideasonboard.com>","From":"Naushir Patuck <naush@raspberrypi.com>","Date":"Thu, 19 Dec 2024 09:33:56 +0000","Message-ID":"<CAEmqJPoDcyFzV2VHBObqv9p+0fRjO0uaZups8fdx4=edEYm8hg@mail.gmail.com>","Subject":"Re: [PATCH] libcamera: camera_sensor_properties: ov5675: Set correct\n\tdelays","To":"Kieran Bingham <kieran.bingham@ideasonboard.com>","Cc":"libcamera devel <libcamera-devel@lists.libcamera.org>,\n\tbryan.odonoghue@linaro.org","Content-Type":"text/plain; charset=\"UTF-8\"","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>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}},{"id":32899,"web_url":"https://patchwork.libcamera.org/comment/32899/","msgid":"<8cf10aea-7d2f-43a2-bad0-fcb9d8976453@redhat.com>","date":"2024-12-20T11:49:03","subject":"Re: [PATCH] libcamera: camera_sensor_properties: ov5675: Set correct\n\tdelays","submitter":{"id":102,"url":"https://patchwork.libcamera.org/api/people/102/","name":"Hans de Goede","email":"hdegoede@redhat.com"},"content":"Hi,\n\nOn 19-Dec-24 10:33 AM, Naushir Patuck wrote:\n> Hi Kieran,\n> \n> On Thu, 19 Dec 2024 at 01:00, Kieran Bingham\n> <kieran.bingham@ideasonboard.com> wrote:\n>>\n>> The OV5675 uses different delays for gain and exposure than are configured\n>> in the default sensorDelays utilised by the CameraSensorLegacy.\n>>\n>> Empirical testing using a Lenovo X13s shows that the exposure delay is\n>> only a single frame, and the current setting of 2 frame delay produces\n>> exceedingly frequent oscillations in the image exposure.\n>>\n>> Update the OV5675 sensor delays table accordingly.\n>>\n>> Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>\n>> ---\n>>  src/libcamera/sensor/camera_sensor_properties.cpp | 7 ++++++-\n>>  1 file changed, 6 insertions(+), 1 deletion(-)\n>>\n>> diff --git a/src/libcamera/sensor/camera_sensor_properties.cpp b/src/libcamera/sensor/camera_sensor_properties.cpp\n>> index bd1fc86977ce..813878386a35 100644\n>> --- a/src/libcamera/sensor/camera_sensor_properties.cpp\n>> +++ b/src/libcamera/sensor/camera_sensor_properties.cpp\n>> @@ -360,7 +360,12 @@ const CameraSensorProperties *CameraSensorProperties::get(const std::string &sen\n>>                                 { controls::draft::TestPatternModeOff, 0 },\n>>                                 { controls::draft::TestPatternModeColorBars, 1 },\n>>                         },\n>> -                       .sensorDelays = { },\n>> +                       .sensorDelays = {\n>> +                               .exposureDelay = 1,\n> \n> From my experience, an exposure delay of 1 is very unlikely with a\n> rolling shutter sensor because of the integration overlap between\n> successive frames.  Of course, I could be wrong since I've not\n> encounter this sensor before.  Perhaps it's worth looking at other\n> delays to see if the general interaction between vblank/exposure/gain\n> is causing the oscillations?\n\nI wonder if this is caused by the simple pipeline handler applying\nnew controls coming from the software ISP IPA immediately rather\nthen waiting for the start of the next frame ?\n\nThis issue is fixed by:\n\nhttps://patchwork.libcamera.org/patch/22397/\n\nKieran can you test if that maybe fixes the oscillation\nwithout needing to change the delays ?\n\np.s. regardless of it helping, that patch is ready for merging.\n\nRegards,\n\nHans","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 9715DC3272\n\tfor <parsemail@patchwork.libcamera.org>;\n\tFri, 20 Dec 2024 11:49:11 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 325586808F;\n\tFri, 20 Dec 2024 12:49:10 +0100 (CET)","from us-smtp-delivery-124.mimecast.com\n\t(us-smtp-delivery-124.mimecast.com [170.10.129.124])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id BFEB567F24\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 20 Dec 2024 12:49:08 +0100 (CET)","from mail-ej1-f72.google.com (mail-ej1-f72.google.com\n\t[209.85.218.72]) by relay.mimecast.com with ESMTP with STARTTLS\n\t(version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id\n\tus-mta-507-5EtxJBFxNFS_jp3CdbJbwQ-1; Fri, 20 Dec 2024 06:49:06 -0500","by mail-ej1-f72.google.com with SMTP id\n\ta640c23a62f3a-aa6845cf116so26858766b.1\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 20 Dec 2024 03:49:06 -0800 (PST)","from ?IPV6:2001:1c00:c32:7800:5bfa:a036:83f0:f9ec?\n\t(2001-1c00-0c32-7800-5bfa-a036-83f0-f9ec.cable.dynamic.v6.ziggo.nl.\n\t[2001:1c00:c32:7800:5bfa:a036:83f0:f9ec])\n\tby smtp.gmail.com with ESMTPSA id\n\ta640c23a62f3a-aac0e830af1sm168650166b.14.2024.12.20.03.49.03\n\t(version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128);\n\tFri, 20 Dec 2024 03:49:03 -0800 (PST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=redhat.com header.i=@redhat.com\n\theader.b=\"HUjYmhwk\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n\ts=mimecast20190719; t=1734695347;\n\th=from:from:reply-to:subject:subject:date:date:message-id:message-id:\n\tto:to:cc:cc:mime-version:mime-version:content-type:content-type:\n\tcontent-transfer-encoding:content-transfer-encoding:\n\tin-reply-to:in-reply-to:references:references;\n\tbh=cfXwe0LnUOr8SuAR9IohtaZqd2U+aWzK8T1RgaDvbnk=;\n\tb=HUjYmhwkKRjQdHNE7YzfJRLOsfFWEPbyE3xRnfvtdtTnGtGn2mmmTlmwjr7ZvPFllKAdqZ\n\tvNVODTw1FmXKM7D+7Y/IFivJKnnceqWShz21wZvD1Q7Xg9IHB4eIHSztzQhdqUyUzGr+hQ\n\tpcFj57HHm7XB2uHaUjovKQCQsgQtpao=","X-MC-Unique":"5EtxJBFxNFS_jp3CdbJbwQ-1","X-Mimecast-MFC-AGG-ID":"5EtxJBFxNFS_jp3CdbJbwQ","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20230601; t=1734695345; x=1735300145;\n\th=content-transfer-encoding:in-reply-to:from:content-language\n\t:references:cc:to:subject:user-agent:mime-version:date:message-id\n\t:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to;\n\tbh=cfXwe0LnUOr8SuAR9IohtaZqd2U+aWzK8T1RgaDvbnk=;\n\tb=duGxzY3gvh//3GF61Wx90n1/mNBaFsD5vmc1oXqK7Yi7sXU6kJEfxqnuztzhWaAbY+\n\tz0LN4MxmRvCG25zgBfYfb/FeslzrnJLrPl272hQyD80X/XfQcdtaMWJLYVjscF8KqXB1\n\taZlza+XrJPiKV+hqmiWlkmcPUcUxAkXGzS6fcEIiqpFmgZIK6k7xIhw2XH460eEIafV2\n\tYTAT3bpIXKOHw14EqRk0XamUqr/SnAdVmWomLDG5HXxqOef3Q4ip7bKmRuzzWd4khI/N\n\tICwADJmXfyh7iuEv2uO02IBBEDLKo/B83R+Kr0gvHFKCQJFyAzOBe/NOjwVv66E6NUU6\n\tgGaw==","X-Gm-Message-State":"AOJu0YxJqVab4D87DIzdsHjz4U4KzV+eFl5osuO0aTcjcylsZu+vrEyN\n\tsmQfvPAhXc3DF9KbJ0SvpYkY+fczv8gnQP/HaFimsafdrWG9ib8z6zqa+iW0ZUxukigEECO5tG+\n\thyUWhkHdBhVTy4ZgYLRRws4xtqiZi3DZqjqxLnhWzF7ASv1WY2DraHgWe9cvCYBnFNlcNdypmIl\n\tS3YK2TuA==","X-Gm-Gg":"ASbGnctERBbXqqpPL1ZQkqEqKO6tblqpz0YS+trWpcZHFJdFQAJ80WOjvSXkdtEu9YP\n\tZ5A3/pitoRFR/IgNwZvxR6KJT5SsFCVz1PcdeEAByte8Dd7iN879bS7KMunyFZOC6BeEBl4y17R\n\tFFANfNUxHW71q64Vrp9BrOFRBWLmO09Qy0mIxQmfpCSX8oal9qacLa6JDUPt0ghw608WPEJ9NA9\n\tVsTLtuOeCl8OEL48lsX7na1GRafnu3BOqKleiTueFvbSLee3kKlDrMkLrhxyhzEFic0iPaZXra9\n\tH0zR4uow/jlF7DvVHZF1LOYkhsaN+go5qanEZvN1W3PVvbAA00sckYpLpHHwDB9Ow5f/CKTzrdg\n\t2A1JecWjNBFkbt+hMJHLadEPj+Xl6aTk=","X-Received":["by 2002:a17:907:7208:b0:aac:93b:269a with SMTP id\n\ta640c23a62f3a-aac3469199amr180110466b.20.1734695344800; \n\tFri, 20 Dec 2024 03:49:04 -0800 (PST)","by 2002:a17:907:7208:b0:aac:93b:269a with SMTP id\n\ta640c23a62f3a-aac3469199amr180109466b.20.1734695344411; \n\tFri, 20 Dec 2024 03:49:04 -0800 (PST)"],"X-Google-Smtp-Source":"AGHT+IEk8bug2HbWAz60DOLEZaW6tD0tWVkVEtF4zu/u8TPy39w7A3be0FztLNnAihI881VXytR/5g==","Message-ID":"<8cf10aea-7d2f-43a2-bad0-fcb9d8976453@redhat.com>","Date":"Fri, 20 Dec 2024 12:49:03 +0100","MIME-Version":"1.0","User-Agent":"Mozilla Thunderbird","Subject":"Re: [PATCH] libcamera: camera_sensor_properties: ov5675: Set correct\n\tdelays","To":"Naushir Patuck <naush@raspberrypi.com>,\n\tKieran Bingham <kieran.bingham@ideasonboard.com>","Cc":"libcamera devel <libcamera-devel@lists.libcamera.org>,\n\tbryan.odonoghue@linaro.org,\n\tStanislaw Gruszka <stanislaw.gruszka@linux.intel.com>","References":"<20241219005958.16856-1-kieran.bingham@ideasonboard.com>\n\t<CAEmqJPoDcyFzV2VHBObqv9p+0fRjO0uaZups8fdx4=edEYm8hg@mail.gmail.com>","From":"Hans de Goede <hdegoede@redhat.com>","In-Reply-To":"<CAEmqJPoDcyFzV2VHBObqv9p+0fRjO0uaZups8fdx4=edEYm8hg@mail.gmail.com>","X-Mimecast-Spam-Score":"0","X-Mimecast-MFC-PROC-ID":"V5h91zwNkUahzKPDrvD8EcOldZqYEKIiHadOATaRRyM_1734695345","X-Mimecast-Originator":"redhat.com","Content-Language":"en-US, nl","Content-Type":"text/plain; charset=UTF-8","Content-Transfer-Encoding":"7bit","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>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}}]