[{"id":25709,"web_url":"https://patchwork.libcamera.org/comment/25709/","msgid":"<CAHW6GYLdhn61oBurwAfuHS2-0VQUXcJPiJMgh_Wte42bQ2dhEw@mail.gmail.com>","date":"2022-11-01T12:12:17","subject":"Re: [libcamera-devel] [PATCH v1 07/10] pipeline: raspberrypi: Add a\n\tparameter to disable startup drop frames","submitter":{"id":42,"url":"https://patchwork.libcamera.org/api/people/42/","name":"David Plowman","email":"david.plowman@raspberrypi.com"},"content":"Hi Naush\n\nThanks for the patch!\n\nOn Fri, 14 Oct 2022 at 14:19, Naushir Patuck via libcamera-devel\n<libcamera-devel@lists.libcamera.org> wrote:\n>\n> Add a new pipeline config parameter \"disable_startup_frame_drops\" to disable\n> any startup drop frames, overriding the IPA request.\n>\n> When this parameter is set, it allows the pipeline handler to run with no\n> internally allocated Unicam buffers (\"min_unicam_buffers\").\n>\n> Add a validation to ensure if \"disable_startup_frame_drops\" is false, at least\n> one internal Unicam buffer is allocated, possibly overriding the\n> \"min_unicam_buffers\" parameter.\n>\n> Signed-off-by: Naushir Patuck <naush@raspberrypi.com>\n> ---\n>  .../pipeline/raspberrypi/data/default.json        |  7 +++++--\n>  .../pipeline/raspberrypi/raspberrypi.cpp          | 15 ++++++++++++++-\n>  2 files changed, 19 insertions(+), 3 deletions(-)\n>\n> diff --git a/src/libcamera/pipeline/raspberrypi/data/default.json b/src/libcamera/pipeline/raspberrypi/data/default.json\n> index a7ea735c87f4..707414bcc5c5 100644\n> --- a/src/libcamera/pipeline/raspberrypi/data/default.json\n> +++ b/src/libcamera/pipeline/raspberrypi/data/default.json\n> @@ -5,7 +5,7 @@\n>          \"pipeline_handler\":\n>          {\n>                  # The minimum number of internal buffers to be allocated for Unicam.\n> -                # This value must be greater than 0, but less than or equal to min_total_unicam_buffers.\n> +                # This value must less than or equal to min_total_unicam_buffers.\n>                  \"min_unicam_buffers\": 2,\n>\n>                  # The minimum total (internal + external) buffer count used for Unicam.\n> @@ -15,6 +15,9 @@\n>                  \"min_total_unicam_buffers\": 4,\n>\n>                  # The number of internal buffers used for ISP Output0.\n> -                \"num_output0_buffers\": 1\n> +                \"num_output0_buffers\": 1,\n> +\n> +                # Override any request from the IPA to drop a number of startup frames.\n> +                \"disable_startup_frame_drops\": false\n>          }\n>  }\n> diff --git a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp\n> index 2aba0430c02e..135948d82f41 100644\n> --- a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp\n> +++ b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp\n> @@ -300,6 +300,7 @@ public:\n>                 unsigned int minUnicamBuffers;\n>                 unsigned int minTotalUnicamBuffers;\n>                 unsigned int numOutput0Buffers;\n> +               bool disableStartupFrameDrops;\n>         };\n>\n>         Config config_;\n> @@ -1044,7 +1045,7 @@ int PipelineHandlerRPi::start(Camera *camera, const ControlList *controls)\n>                 data->setSensorControls(startConfig.controls);\n>\n>         /* Configure the number of dropped frames required on startup. */\n> -       data->dropFrameCount_ = startConfig.dropFrameCount;\n> +       data->dropFrameCount_ = data->config_.disableStartupFrameDrops ? 0 : startConfig.dropFrameCount;\n\nMight this warrant an INFO message, just in case folks are confused by\nthe changing AGC/AWB when things start? Or maybe that would be\nannoying, for example if an application wants to run like this and\nhandles the startup frames for itself?\n\nReviewed-by: David Plowman <david.plowman@raspberrypi.com>\n\nThanks!\nDavid\n\n>\n>         for (auto const stream : data->streams_)\n>                 stream->resetBuffers();\n> @@ -1430,6 +1431,7 @@ int PipelineHandlerRPi::configurePipelineHandler(RPiCameraData *data)\n>                 .minUnicamBuffers = 2,\n>                 .minTotalUnicamBuffers = 4,\n>                 .numOutput0Buffers = 1,\n> +               .disableStartupFrameDrops = false,\n>         };\n>\n>         char const *configFromEnv = utils::secure_getenv(\"LIBCAMERA_RPI_CONFIG_FILE\");\n> @@ -1464,6 +1466,8 @@ int PipelineHandlerRPi::configurePipelineHandler(RPiCameraData *data)\n>                 phConfig[\"min_total_unicam_buffers\"].get<unsigned int>(config.minTotalUnicamBuffers);\n>         config.numOutput0Buffers =\n>                 phConfig[\"num_output0_buffers\"].get<unsigned int>(config.numOutput0Buffers);\n> +       config.disableStartupFrameDrops =\n> +               phConfig[\"disable_startup_frame_drops\"].get<bool>(config.disableStartupFrameDrops);\n>\n>         if (config.minTotalUnicamBuffers < config.minUnicamBuffers || config.minTotalUnicamBuffers < 1) {\n>                 LOG(RPI, Error) << \"Invalid Unicam buffer configuration used!\";\n> @@ -1543,6 +1547,15 @@ int PipelineHandlerRPi::prepareBuffers(Camera *camera)\n>                          */\n>                         numBuffers = std::max<int>(data->config_.minUnicamBuffers,\n>                                                    minBuffers - numRawBuffers);\n> +\n> +                       if (numBuffers == 0 && data->dropFrameCount_) {\n> +                               LOG(RPI, Warning)\n> +                                       << \"Configured with no Unicam buffers,\"\n> +                                          \" but the IPA requested startup frame drops.\"\n> +                                          \" Increasing to one buffer.\";\n> +                               numBuffers = 1;\n> +                       }\n> +\n>                         data->numUnicamBuffers = numBuffers;\n>                 } else if (stream == &data->isp_[Isp::Input]) {\n>                         /*\n> --\n> 2.25.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 D2BD9BD16B\n\tfor <parsemail@patchwork.libcamera.org>;\n\tTue,  1 Nov 2022 12:12:31 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 3EF6F63031;\n\tTue,  1 Nov 2022 13:12:31 +0100 (CET)","from mail-pj1-x1029.google.com (mail-pj1-x1029.google.com\n\t[IPv6:2607:f8b0:4864:20::1029])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id E46E163009\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue,  1 Nov 2022 13:12:29 +0100 (CET)","by mail-pj1-x1029.google.com with SMTP id\n\tu8-20020a17090a5e4800b002106dcdd4a0so18629198pji.1\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 01 Nov 2022 05:12:29 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1667304751;\n\tbh=xPHhoZyGKqvJewgkwUOLuUyTkQPcB9kMPvTLWb9zudQ=;\n\th=References:In-Reply-To:Date:To:Subject:List-Id:List-Unsubscribe:\n\tList-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc:\n\tFrom;\n\tb=To4VkicOnhCmbcWRIZkVXQi6l3MjAnOqv/wyp2XJSk+1OjNTTbUNJ9uJt9PWXQVb+\n\tcYzjtem/o81f+AdzrqaC7ypx4HFhI4a9evQceVrL28Bs8UbxmLOMI7dQ4xC/uK84A6\n\tPnlR5YzTzRGCpRCpx/a10tcB/YLIA1gZ7fVbWcYDXdQzA7gyCsF7MW4YY+w+LHxkp8\n\t4NmpEvRe+6Sn3Y2xCF/tMGJoV2Qb0y7Tg2uxU4cO4L44iu3JuF37hf+f9lutRJbAUN\n\taRISuCFG+y8+98h7ILNsldfn5yfMTAr5/kUuH1eNCq9GiS9mmPkg31hjdMJYfPYJkM\n\tix0TSnDMh9XYA==","v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=raspberrypi.com; s=google;\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=HvmaRYzx7hmuSwi3ShnKapHOXFUJ3bKeonYU9xPd2iA=;\n\tb=tytMPRlcD4Jhb+obFgTsEIZg4zwZg3xGLPISY9znvUWUXl1rrcpks59kW1P1d+3blG\n\t5PTX91aOZ9ziQ4DssESCMRwd4yixEznW5PZSt3roKwso6j9XcdgZP/RM0g4FlnwEVq8J\n\tSqc8l9kpTEXtGWBe5aZZDy4LbFqW6M2JQtdMdCFikCGuf4J7DO7yozboNRtkut6EJdP6\n\t7FViSrd12aLd3ykFcQ6S8MbtG2t8AL/yRFYhfD/WkQHBoyw6q5NVB4uzfLCIPp22gfoP\n\t9YtkKTmQNnqAaK00lzvBtdBfXyD3qqnNGUBZvcQ3Gsbn5XO6Z8DN10pcsCgj5YlgIHps\n\taWIQ=="],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (2048-bit key; \n\tunprotected) header.d=raspberrypi.com\n\theader.i=@raspberrypi.com\n\theader.b=\"tytMPRlc\"; dkim-atps=neutral","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20210112;\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=HvmaRYzx7hmuSwi3ShnKapHOXFUJ3bKeonYU9xPd2iA=;\n\tb=7ldFdbABVvzOUJCtBPhJg9RKXPFsKq2UnK4802shEnJy89F0+j7a0utVSnsfr9/WNI\n\tBpHZdsOobmzPEuEJWndcdCLC6UXkIGgSCDYfJ1oTTz/g6AZPMbATAIo8KK4gXP3hfFrF\n\tcBeh7iYEtrbYV3mDoWiZuT+3pL23CIDcN6h2pJ91T+7kAK4UgDTGrNtzdl/QwIxD1M8h\n\tuh1AEDpDQ7lpyqkDm9z9YD4Hc+kpkckgXIcTHsmSZuWdyxDJKQfp7xyA6q6ALk2ygZhV\n\t2X8LB4w//EK16g42bKudjQ4jfQXhADOpsyudtyHfrXVaHpYVzEIvEZr4+gEvRmo2gK9l\n\tsM4Q==","X-Gm-Message-State":"ACrzQf0mIp0KMMGFektUnJpz7GAfYMx/o5hLpg6OfBLla40kliE4+heI\n\tpSkGiSsrNtAlAF8sziaidjftFQ0qXI7RQfzpUemgOw==","X-Google-Smtp-Source":"AMsMyM7OIF+72smZEGzkYyaJnmmVLibkDhHdE1q9G+klfmTscJrpl4/7L+2CrnQeFgSpkwHyyU2mcoKXrw1176kaIGA=","X-Received":"by 2002:a17:902:720a:b0:181:150c:fcc4 with SMTP id\n\tba10-20020a170902720a00b00181150cfcc4mr19650136plb.109.1667304748388;\n\tTue, 01 Nov 2022 05:12:28 -0700 (PDT)","MIME-Version":"1.0","References":"<20221014131846.27169-1-naush@raspberrypi.com>\n\t<20221014131846.27169-8-naush@raspberrypi.com>","In-Reply-To":"<20221014131846.27169-8-naush@raspberrypi.com>","Date":"Tue, 1 Nov 2022 12:12:17 +0000","Message-ID":"<CAHW6GYLdhn61oBurwAfuHS2-0VQUXcJPiJMgh_Wte42bQ2dhEw@mail.gmail.com>","To":"Naushir Patuck <naush@raspberrypi.com>","Content-Type":"text/plain; charset=\"UTF-8\"","Subject":"Re: [libcamera-devel] [PATCH v1 07/10] pipeline: raspberrypi: Add a\n\tparameter to disable startup drop frames","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>","From":"David Plowman via libcamera-devel <libcamera-devel@lists.libcamera.org>","Reply-To":"David Plowman <david.plowman@raspberrypi.com>","Cc":"libcamera-devel@lists.libcamera.org","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}},{"id":25924,"web_url":"https://patchwork.libcamera.org/comment/25924/","msgid":"<CAEmqJPq13kasK+y8mZU1c-Burc_NSapCEk36W9nczxKTZF8hCg@mail.gmail.com>","date":"2022-11-29T10:46:41","subject":"Re: [libcamera-devel] [PATCH v1 07/10] pipeline: raspberrypi: Add a\n\tparameter to disable startup drop frames","submitter":{"id":34,"url":"https://patchwork.libcamera.org/api/people/34/","name":"Naushir Patuck","email":"naush@raspberrypi.com"},"content":"Hi David,\n\nThank you for your feedback.\n\nOn Tue, 1 Nov 2022 at 12:12, David Plowman <david.plowman@raspberrypi.com>\nwrote:\n\n> Hi Naush\n>\n> Thanks for the patch!\n>\n> On Fri, 14 Oct 2022 at 14:19, Naushir Patuck via libcamera-devel\n> <libcamera-devel@lists.libcamera.org> wrote:\n> >\n> > Add a new pipeline config parameter \"disable_startup_frame_drops\" to\n> disable\n> > any startup drop frames, overriding the IPA request.\n> >\n> > When this parameter is set, it allows the pipeline handler to run with no\n> > internally allocated Unicam buffers (\"min_unicam_buffers\").\n> >\n> > Add a validation to ensure if \"disable_startup_frame_drops\" is false, at\n> least\n> > one internal Unicam buffer is allocated, possibly overriding the\n> > \"min_unicam_buffers\" parameter.\n> >\n> > Signed-off-by: Naushir Patuck <naush@raspberrypi.com>\n> > ---\n> >  .../pipeline/raspberrypi/data/default.json        |  7 +++++--\n> >  .../pipeline/raspberrypi/raspberrypi.cpp          | 15 ++++++++++++++-\n> >  2 files changed, 19 insertions(+), 3 deletions(-)\n> >\n> > diff --git a/src/libcamera/pipeline/raspberrypi/data/default.json\n> b/src/libcamera/pipeline/raspberrypi/data/default.json\n> > index a7ea735c87f4..707414bcc5c5 100644\n> > --- a/src/libcamera/pipeline/raspberrypi/data/default.json\n> > +++ b/src/libcamera/pipeline/raspberrypi/data/default.json\n> > @@ -5,7 +5,7 @@\n> >          \"pipeline_handler\":\n> >          {\n> >                  # The minimum number of internal buffers to be\n> allocated for Unicam.\n> > -                # This value must be greater than 0, but less than or\n> equal to min_total_unicam_buffers.\n> > +                # This value must less than or equal to\n> min_total_unicam_buffers.\n> >                  \"min_unicam_buffers\": 2,\n> >\n> >                  # The minimum total (internal + external) buffer count\n> used for Unicam.\n> > @@ -15,6 +15,9 @@\n> >                  \"min_total_unicam_buffers\": 4,\n> >\n> >                  # The number of internal buffers used for ISP Output0.\n> > -                \"num_output0_buffers\": 1\n> > +                \"num_output0_buffers\": 1,\n> > +\n> > +                # Override any request from the IPA to drop a number of\n> startup frames.\n> > +                \"disable_startup_frame_drops\": false\n> >          }\n> >  }\n> > diff --git a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp\n> b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp\n> > index 2aba0430c02e..135948d82f41 100644\n> > --- a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp\n> > +++ b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp\n> > @@ -300,6 +300,7 @@ public:\n> >                 unsigned int minUnicamBuffers;\n> >                 unsigned int minTotalUnicamBuffers;\n> >                 unsigned int numOutput0Buffers;\n> > +               bool disableStartupFrameDrops;\n> >         };\n> >\n> >         Config config_;\n> > @@ -1044,7 +1045,7 @@ int PipelineHandlerRPi::start(Camera *camera,\n> const ControlList *controls)\n> >                 data->setSensorControls(startConfig.controls);\n> >\n> >         /* Configure the number of dropped frames required on startup. */\n> > -       data->dropFrameCount_ = startConfig.dropFrameCount;\n> > +       data->dropFrameCount_ = data->config_.disableStartupFrameDrops ?\n> 0 : startConfig.dropFrameCount;\n>\n> Might this warrant an INFO message, just in case folks are confused by\n> the changing AGC/AWB when things start? Or maybe that would be\n> annoying, for example if an application wants to run like this and\n> handles the startup frames for itself?\n>\n\nThere are already DEBUG level messages for dropping convergence frames,\ndo you think that is enough?\n\nRegards,\nNaush\n\n\n\n>\n> Reviewed-by: David Plowman <david.plowman@raspberrypi.com>\n>\n> Thanks!\n> David\n>\n> >\n> >         for (auto const stream : data->streams_)\n> >                 stream->resetBuffers();\n> > @@ -1430,6 +1431,7 @@ int\n> PipelineHandlerRPi::configurePipelineHandler(RPiCameraData *data)\n> >                 .minUnicamBuffers = 2,\n> >                 .minTotalUnicamBuffers = 4,\n> >                 .numOutput0Buffers = 1,\n> > +               .disableStartupFrameDrops = false,\n> >         };\n> >\n> >         char const *configFromEnv =\n> utils::secure_getenv(\"LIBCAMERA_RPI_CONFIG_FILE\");\n> > @@ -1464,6 +1466,8 @@ int\n> PipelineHandlerRPi::configurePipelineHandler(RPiCameraData *data)\n> >                 phConfig[\"min_total_unicam_buffers\"].get<unsigned\n> int>(config.minTotalUnicamBuffers);\n> >         config.numOutput0Buffers =\n> >                 phConfig[\"num_output0_buffers\"].get<unsigned\n> int>(config.numOutput0Buffers);\n> > +       config.disableStartupFrameDrops =\n> > +\n>  phConfig[\"disable_startup_frame_drops\"].get<bool>(config.disableStartupFrameDrops);\n> >\n> >         if (config.minTotalUnicamBuffers < config.minUnicamBuffers ||\n> config.minTotalUnicamBuffers < 1) {\n> >                 LOG(RPI, Error) << \"Invalid Unicam buffer configuration\n> used!\";\n> > @@ -1543,6 +1547,15 @@ int PipelineHandlerRPi::prepareBuffers(Camera\n> *camera)\n> >                          */\n> >                         numBuffers =\n> std::max<int>(data->config_.minUnicamBuffers,\n> >                                                    minBuffers -\n> numRawBuffers);\n> > +\n> > +                       if (numBuffers == 0 && data->dropFrameCount_) {\n> > +                               LOG(RPI, Warning)\n> > +                                       << \"Configured with no Unicam\n> buffers,\"\n> > +                                          \" but the IPA requested\n> startup frame drops.\"\n> > +                                          \" Increasing to one buffer.\";\n> > +                               numBuffers = 1;\n> > +                       }\n> > +\n> >                         data->numUnicamBuffers = numBuffers;\n> >                 } else if (stream == &data->isp_[Isp::Input]) {\n> >                         /*\n> > --\n> > 2.25.1\n> >\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 33983BE08B\n\tfor <parsemail@patchwork.libcamera.org>;\n\tTue, 29 Nov 2022 10:46:59 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 982E46333A;\n\tTue, 29 Nov 2022 11:46:58 +0100 (CET)","from mail-io1-xd2d.google.com (mail-io1-xd2d.google.com\n\t[IPv6:2607:f8b0:4864:20::d2d])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id C12B963314\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 29 Nov 2022 11:46:56 +0100 (CET)","by mail-io1-xd2d.google.com with SMTP id q21so9712045iod.4\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 29 Nov 2022 02:46:56 -0800 (PST)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1669718818;\n\tbh=ckb3Y4V/0ZCgQARDmd53MrK9lMG4jHiWtmH94KTSaw0=;\n\th=References:In-Reply-To:Date:To:Subject:List-Id:List-Unsubscribe:\n\tList-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc:\n\tFrom;\n\tb=bpkcLmJ/HcZ+VlZFpUxF9MPF8+HZ/wzi8wo+gWM1IjiLH0n/rGogG+EKUzmghzIy5\n\tWS2q/G9WvnsDPfGd5fzaGE5VybBPVCOL9K+ng1r11oCpPjLiXGUgL1A0yg3sfAoSoP\n\t5ce1LGFRA5pV8T20UMLFSYLkqWgNJDWbWK26t4XMYEpzfg0NKWqB6bFVOJZligJpRx\n\tBJ5AwdxpTxdk01YUCH2YdNp2eMyXgOs9QLaNn1GdGuK3OhTeBavPCBGHAJ0GZ7CA8k\n\t1S0HGfp4QzztqB4Vh+V0WGL9F71XngdU0DyAt5BafM3D+z+gNz2lnnao+2icA3h3Nn\n\tqOjffRYHELUqg==","v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=raspberrypi.com; s=google;\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=p7cxwHRsLofJ2hEcz5h4Z6HyJd1H09wV6k8qZWTKG0I=;\n\tb=riJfjEPoSZmNiAke06PjpC/7Zq09KBqb9MUR6s7t2rWS8GIfFOOgxqSO8vsaZD1dhT\n\tTY/eSS00EVyqx44eyLqvYJr3bmTGC32R7seqJICc8jn4xnchWPSP5OE9QUkgDnYuqTKz\n\tPY5XZCvSXynPSKPmGp8/89h/i00OlSgH+O0kBOxhQvdkCSltO/O6eMULfwi++qPhsylx\n\tu8+iKLFTBDD26I4lFl7PoEBROf1q0zEGxBQD1fadLlkugWVBTZneZW4jPusSvG4g73qI\n\tm/HjUZBupfCpvPfotLLVvDyJ119LmWZhFle0SQ73BoDc5vMke0vq56RLOG+deHfZJFm7\n\tnw8A=="],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (2048-bit key; \n\tunprotected) header.d=raspberrypi.com\n\theader.i=@raspberrypi.com\n\theader.b=\"riJfjEPo\"; dkim-atps=neutral","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20210112;\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=p7cxwHRsLofJ2hEcz5h4Z6HyJd1H09wV6k8qZWTKG0I=;\n\tb=DDOL0F3BKDJ50rNztHzriI43xy/3kS40ZXkZnT37zUE/lBls0OOd8OyXxHXiHTdBp6\n\tVwzD/Hh1E3UE/x/1ecxgJeBIXGJU6cJ2CmpQ2fqqcsOlQ9UF+beiPy/QFoZXsW6xFmqS\n\tF0gaKO84wUl7RVsOFD6NkSImhDaSD1X1wIk8eKeSbmLP6QiCsah76u4bj/pPSAiP5/mM\n\t/TEoD0FXxQK8mh/IX6hO4CpyiYALQC040HN41VS+EdV6WqBJP4LQTPcMT9CgrIH+0cKa\n\tti3j+3lCdXfMQekp6TiRD4YsDk4pXDdrkZmg4GDPFX1ZWTxomyZgG02JOVuJ/KjGyfPq\n\t/TvA==","X-Gm-Message-State":"ANoB5pn2C0bUgVrkM1zbgxPopgva2EZEfl05IaYEWPoSs3sZMia8JxqF\n\tp5DrSik4Qi5U78ESQecCvGXUqBSniyCHSVWA0WVoxZspyL0=","X-Google-Smtp-Source":"AA0mqf5UPxOn3hOh24iCh8j5m8UnVcO7i0LUwJoiSGT7Xu3n6FpCMx0wrljsyR1ytguUHLfvp7wHqIekV10QzXMF40w=","X-Received":"by 2002:a02:6a43:0:b0:375:4725:4b4f with SMTP id\n\tm3-20020a026a43000000b0037547254b4fmr26007326jaf.52.1669718815523;\n\tTue, 29 Nov 2022 02:46:55 -0800 (PST)","MIME-Version":"1.0","References":"<20221014131846.27169-1-naush@raspberrypi.com>\n\t<20221014131846.27169-8-naush@raspberrypi.com>\n\t<CAHW6GYLdhn61oBurwAfuHS2-0VQUXcJPiJMgh_Wte42bQ2dhEw@mail.gmail.com>","In-Reply-To":"<CAHW6GYLdhn61oBurwAfuHS2-0VQUXcJPiJMgh_Wte42bQ2dhEw@mail.gmail.com>","Date":"Tue, 29 Nov 2022 10:46:41 +0000","Message-ID":"<CAEmqJPq13kasK+y8mZU1c-Burc_NSapCEk36W9nczxKTZF8hCg@mail.gmail.com>","To":"David Plowman <david.plowman@raspberrypi.com>","Content-Type":"multipart/alternative; boundary=\"000000000000dfdea305ee99b448\"","Subject":"Re: [libcamera-devel] [PATCH v1 07/10] pipeline: raspberrypi: Add a\n\tparameter to disable startup drop frames","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>","From":"Naushir Patuck via libcamera-devel\n\t<libcamera-devel@lists.libcamera.org>","Reply-To":"Naushir Patuck <naush@raspberrypi.com>","Cc":"libcamera-devel@lists.libcamera.org","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}}]