[{"id":17658,"web_url":"https://patchwork.libcamera.org/comment/17658/","msgid":"<ef611e03-e6eb-013c-2926-075612dd163e@ideasonboard.com>","date":"2021-06-21T13:44:56","subject":"Re: [libcamera-devel] [PATCH v1] libcamera: Add OV5647 sensor\n\tproperties","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Hi Vedant,\n\nOn 21/06/2021 14:34, Vedant Paranjape wrote:\n> Brief specifications available at\n> https://cdn.sparkfun.com/datasheets/Dev/RaspberryPi/ov5647_full.pdf\n> \n\nI presume this comes from the features on page 5 stating\n\n\"\"\"\n1.4 μm x 1.4 μm pixel with OmniBSI technology for\nhigh performance (high sensitivity, low crosstalk, low\nnoise\n\"\"\"\n\nIt might be nice to reference that specifically rather than the whole\ndocument, so readers can quickly and easily find the relevant source.\n\n\n> Signed-off-by: Vedant Paranjape <vedantparanjape160201@gmail.com>\n> ---\n>  src/libcamera/camera_sensor_properties.cpp | 5 +++++\n>  1 file changed, 5 insertions(+)\n> \n> diff --git a/src/libcamera/camera_sensor_properties.cpp b/src/libcamera/camera_sensor_properties.cpp\n> index f660743a..43030e8b 100644\n> --- a/src/libcamera/camera_sensor_properties.cpp\n> +++ b/src/libcamera/camera_sensor_properties.cpp\n> @@ -81,6 +81,11 @@ const CameraSensorProperties *CameraSensorProperties::get(const std::string &sen\n>  \t\t\t\t{ 1, controls::draft::TestPatternModeColorBars },\n>  \t\t\t},\n>  \t\t} },\n> +\t\t{ \"ov5647\", {\n> +\t\t\t.unitCellSize = { 1400, 1400 },\n> +\t\t\t/* \\todo fill test pattern modes for ov5647. */\n\nIt seems this todo is quite involved, as it will require updating the\nkernel driver to actually have test patterns :-S\n\nI can't see any available at:\n\nhttps://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/media/i2c/ov5647.c\n\nSo the todo is likely sufficient, but I'd be more explicit saying there\nare no current supported test modes in the kernel.\n\nOther than those comments,\n\nReviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>\n\n\n> +\t\t\t.testPatternModes = {},\n> +\t\t} },\n>  \t\t{ \"ov5693\", {\n>  \t\t\t.unitCellSize = { 1400, 1400 },\n>  \t\t\t.testPatternModes = {\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 20FB7BE58C\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon, 21 Jun 2021 13:45:01 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 9C94568935;\n\tMon, 21 Jun 2021 15:45:00 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 9FF6160295\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon, 21 Jun 2021 15:44:59 +0200 (CEST)","from [192.168.0.20]\n\t(cpc89244-aztw30-2-0-cust3082.18-1.cable.virginm.net [86.31.172.11])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 23C9E4C2C;\n\tMon, 21 Jun 2021 15:44:59 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"WBOSl9PA\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1624283099;\n\tbh=R/4wCNIUzdkeGDigXWvDxfjVz/eX0JibZ+n8/NntSKc=;\n\th=Subject:To:References:From:Date:In-Reply-To:From;\n\tb=WBOSl9PApZuCumZrA8vTgyGt/aeOsmUsgaqyp8mPgwsNtUCwGWTXHPcXQLq1W1/In\n\tNOh4LBhf5L93FSy+2emoHbyvYDUrXzpeJHGWwz/E8OOzcVasjJnGauTczejJmM8HpJ\n\teDA6mCKuvLZH3ImS4xS3vpfpHoFbvIVbKBgd/tAQ=","To":"Vedant Paranjape <vedantparanjape160201@gmail.com>,\n\tlibcamera-devel@lists.libcamera.org","References":"<20210621133449.80586-1-vedantparanjape160201@gmail.com>","From":"Kieran Bingham <kieran.bingham@ideasonboard.com>","Message-ID":"<ef611e03-e6eb-013c-2926-075612dd163e@ideasonboard.com>","Date":"Mon, 21 Jun 2021 14:44:56 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101\n\tThunderbird/78.8.1","MIME-Version":"1.0","In-Reply-To":"<20210621133449.80586-1-vedantparanjape160201@gmail.com>","Content-Type":"text/plain; charset=utf-8","Content-Language":"en-GB","Content-Transfer-Encoding":"8bit","Subject":"Re: [libcamera-devel] [PATCH v1] libcamera: Add OV5647 sensor\n\tproperties","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":17659,"web_url":"https://patchwork.libcamera.org/comment/17659/","msgid":"<YNCXwu7OXfgbWTiU@pendragon.ideasonboard.com>","date":"2021-06-21T13:44:34","subject":"Re: [libcamera-devel] [PATCH v1] libcamera: Add OV5647 sensor\n\tproperties","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Vedant,\n\nThank you for the patch.\n\nOn Mon, Jun 21, 2021 at 07:04:49PM +0530, Vedant Paranjape wrote:\n> Brief specifications available at\n> https://cdn.sparkfun.com/datasheets/Dev/RaspberryPi/ov5647_full.pdf\n> \n> Signed-off-by: Vedant Paranjape <vedantparanjape160201@gmail.com>\n> ---\n>  src/libcamera/camera_sensor_properties.cpp | 5 +++++\n>  1 file changed, 5 insertions(+)\n> \n> diff --git a/src/libcamera/camera_sensor_properties.cpp b/src/libcamera/camera_sensor_properties.cpp\n> index f660743a..43030e8b 100644\n> --- a/src/libcamera/camera_sensor_properties.cpp\n> +++ b/src/libcamera/camera_sensor_properties.cpp\n> @@ -81,6 +81,11 @@ const CameraSensorProperties *CameraSensorProperties::get(const std::string &sen\n>  \t\t\t\t{ 1, controls::draft::TestPatternModeColorBars },\n>  \t\t\t},\n>  \t\t} },\n> +\t\t{ \"ov5647\", {\n> +\t\t\t.unitCellSize = { 1400, 1400 },\n> +\t\t\t/* \\todo fill test pattern modes for ov5647. */\n> +\t\t\t.testPatternModes = {},\n\nHow about filling them ? :-)\n\n> +\t\t} },\n>  \t\t{ \"ov5693\", {\n>  \t\t\t.unitCellSize = { 1400, 1400 },\n>  \t\t\t.testPatternModes = {","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 4E560BE58C\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon, 21 Jun 2021 13:45:03 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id F27DF6893B;\n\tMon, 21 Jun 2021 15:45:02 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 0145468932\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon, 21 Jun 2021 15:45:01 +0200 (CEST)","from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi\n\t[62.78.145.57])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 89EC64C2C;\n\tMon, 21 Jun 2021 15:45:00 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"hyyShwdC\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1624283100;\n\tbh=0LBlha8KEcW9GBubDvXcixzjudG8APN6GisTjjjv+HY=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=hyyShwdCAdf5X9F7vsztGC8ZWcL5yx1idIAwnIDGXqFzd0TNI2Lu3dNMYf4MIq4gy\n\tzHHAkvmfExY5u7RfGZ2MgVp8gTr/X+5K7Weo1wv0hPuiBafPFdEOmnArHyH24Is/0u\n\ty9oGi0GEIRvLz99fPm/AiWbtbLe6Rrczyqg6Ch8M=","Date":"Mon, 21 Jun 2021 16:44:34 +0300","From":"Laurent Pinchart <laurent.pinchart@ideasonboard.com>","To":"Vedant Paranjape <vedantparanjape160201@gmail.com>","Message-ID":"<YNCXwu7OXfgbWTiU@pendragon.ideasonboard.com>","References":"<20210621133449.80586-1-vedantparanjape160201@gmail.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","In-Reply-To":"<20210621133449.80586-1-vedantparanjape160201@gmail.com>","Subject":"Re: [libcamera-devel] [PATCH v1] libcamera: Add OV5647 sensor\n\tproperties","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>","Cc":"libcamera-devel@lists.libcamera.org","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}},{"id":17660,"web_url":"https://patchwork.libcamera.org/comment/17660/","msgid":"<YNCYohzIddYIetdo@pendragon.ideasonboard.com>","date":"2021-06-21T13:48:18","subject":"Re: [libcamera-devel] [PATCH v1] libcamera: Add OV5647 sensor\n\tproperties","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"On Mon, Jun 21, 2021 at 02:44:56PM +0100, Kieran Bingham wrote:\n> Hi Vedant,\n> \n> On 21/06/2021 14:34, Vedant Paranjape wrote:\n> > Brief specifications available at\n> > https://cdn.sparkfun.com/datasheets/Dev/RaspberryPi/ov5647_full.pdf\n> > \n> \n> I presume this comes from the features on page 5 stating\n> \n> \"\"\"\n> 1.4 μm x 1.4 μm pixel with OmniBSI technology for\n> high performance (high sensitivity, low crosstalk, low\n> noise\n> \"\"\"\n> \n> It might be nice to reference that specifically rather than the whole\n> document, so readers can quickly and easily find the relevant source.\n> \n> \n> > Signed-off-by: Vedant Paranjape <vedantparanjape160201@gmail.com>\n> > ---\n> >  src/libcamera/camera_sensor_properties.cpp | 5 +++++\n> >  1 file changed, 5 insertions(+)\n> > \n> > diff --git a/src/libcamera/camera_sensor_properties.cpp b/src/libcamera/camera_sensor_properties.cpp\n> > index f660743a..43030e8b 100644\n> > --- a/src/libcamera/camera_sensor_properties.cpp\n> > +++ b/src/libcamera/camera_sensor_properties.cpp\n> > @@ -81,6 +81,11 @@ const CameraSensorProperties *CameraSensorProperties::get(const std::string &sen\n> >  \t\t\t\t{ 1, controls::draft::TestPatternModeColorBars },\n> >  \t\t\t},\n> >  \t\t} },\n> > +\t\t{ \"ov5647\", {\n> > +\t\t\t.unitCellSize = { 1400, 1400 },\n> > +\t\t\t/* \\todo fill test pattern modes for ov5647. */\n> \n> It seems this todo is quite involved, as it will require updating the\n> kernel driver to actually have test patterns :-S\n> \n> I can't see any available at:\n> \n> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/media/i2c/ov5647.c\n> \n> So the todo is likely sufficient, but I'd be more explicit saying there\n> are no current supported test modes in the kernel.\n\nI'd say there's no \\todo needed in that case, if the driver doesn't\nsupport test patterns, the implementation here is enough.\n\n> Other than those comments,\n> \n> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>\n> \n> > +\t\t\t.testPatternModes = {},\n> > +\t\t} },\n> >  \t\t{ \"ov5693\", {\n> >  \t\t\t.unitCellSize = { 1400, 1400 },\n> >  \t\t\t.testPatternModes = {\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 4A8EAC3218\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon, 21 Jun 2021 13:48:49 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id C0BB560296;\n\tMon, 21 Jun 2021 15:48:48 +0200 (CEST)","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 8334260295\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon, 21 Jun 2021 15:48:46 +0200 (CEST)","from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi\n\t[62.78.145.57])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id C87E85594;\n\tMon, 21 Jun 2021 15:48:44 +0200 (CEST)"],"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=\"qfTLd0BG\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1624283325;\n\tbh=081B3gdQnjNELsHLefeC4QGrnZOplfxt4Ij+7ENbQUo=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=qfTLd0BG2p7o1viqCCEipoGV2kAJqF2J5jr4LpsnKk99J+VLh+f2VmV/DRc7JSXPH\n\tCE0uRxVpxQ0E+Kr/Flvwezur77Lh3V10jrs9rTkdC9/PmjFNXu9cOMPHOYlXuoRcLr\n\tw4GZG4fPc2OTJe4gSwajURxIv/Ny4P6DwgLO3x5k=","Date":"Mon, 21 Jun 2021 16:48:18 +0300","From":"Laurent Pinchart <laurent.pinchart@ideasonboard.com>","To":"Kieran Bingham <kieran.bingham@ideasonboard.com>","Message-ID":"<YNCYohzIddYIetdo@pendragon.ideasonboard.com>","References":"<20210621133449.80586-1-vedantparanjape160201@gmail.com>\n\t<ef611e03-e6eb-013c-2926-075612dd163e@ideasonboard.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","Content-Transfer-Encoding":"8bit","In-Reply-To":"<ef611e03-e6eb-013c-2926-075612dd163e@ideasonboard.com>","Subject":"Re: [libcamera-devel] [PATCH v1] libcamera: Add OV5647 sensor\n\tproperties","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>","Cc":"libcamera-devel@lists.libcamera.org,\n\tVedant Paranjape <vedantparanjape160201@gmail.com>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}},{"id":17662,"web_url":"https://patchwork.libcamera.org/comment/17662/","msgid":"<CACGrz-MJnVZqLik+8r8gb+BMhO1_zP13W7ZxLrrCJThWiu5L+g@mail.gmail.com>","date":"2021-06-21T14:02:18","subject":"Re: [libcamera-devel] [PATCH v1] libcamera: Add OV5647 sensor\n\tproperties","submitter":{"id":85,"url":"https://patchwork.libcamera.org/api/people/85/","name":"Vedant Paranjape","email":"vedantparanjape160201@gmail.com"},"content":"Hi Laurent and Kieran,\nSent a v2 patch with requested changes.\n\nRegards,\n*Vedant Paranjape*\n\nOn Mon, Jun 21, 2021 at 7:18 PM Laurent Pinchart <\nlaurent.pinchart@ideasonboard.com> wrote:\n\n> On Mon, Jun 21, 2021 at 02:44:56PM +0100, Kieran Bingham wrote:\n> > Hi Vedant,\n> >\n> > On 21/06/2021 14:34, Vedant Paranjape wrote:\n> > > Brief specifications available at\n> > > https://cdn.sparkfun.com/datasheets/Dev/RaspberryPi/ov5647_full.pdf\n> > >\n> >\n> > I presume this comes from the features on page 5 stating\n> >\n> > \"\"\"\n> > 1.4 μm x 1.4 μm pixel with OmniBSI technology for\n> > high performance (high sensitivity, low crosstalk, low\n> > noise\n> > \"\"\"\n> >\n> > It might be nice to reference that specifically rather than the whole\n> > document, so readers can quickly and easily find the relevant source.\n> >\n> >\n> > > Signed-off-by: Vedant Paranjape <vedantparanjape160201@gmail.com>\n> > > ---\n> > >  src/libcamera/camera_sensor_properties.cpp | 5 +++++\n> > >  1 file changed, 5 insertions(+)\n> > >\n> > > diff --git a/src/libcamera/camera_sensor_properties.cpp\n> b/src/libcamera/camera_sensor_properties.cpp\n> > > index f660743a..43030e8b 100644\n> > > --- a/src/libcamera/camera_sensor_properties.cpp\n> > > +++ b/src/libcamera/camera_sensor_properties.cpp\n> > > @@ -81,6 +81,11 @@ const CameraSensorProperties\n> *CameraSensorProperties::get(const std::string &sen\n> > >                             { 1,\n> controls::draft::TestPatternModeColorBars },\n> > >                     },\n> > >             } },\n> > > +           { \"ov5647\", {\n> > > +                   .unitCellSize = { 1400, 1400 },\n> > > +                   /* \\todo fill test pattern modes for ov5647. */\n> >\n> > It seems this todo is quite involved, as it will require updating the\n> > kernel driver to actually have test patterns :-S\n> >\n> > I can't see any available at:\n> >\n> >\n> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/media/i2c/ov5647.c\n> >\n> > So the todo is likely sufficient, but I'd be more explicit saying there\n> > are no current supported test modes in the kernel.\n>\n> I'd say there's no \\todo needed in that case, if the driver doesn't\n> support test patterns, the implementation here is enough.\n>\n> > Other than those comments,\n> >\n> > Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>\n> >\n> > > +                   .testPatternModes = {},\n> > > +           } },\n> > >             { \"ov5693\", {\n> > >                     .unitCellSize = { 1400, 1400 },\n> > >                     .testPatternModes = {\n> > >\n>\n> --\n> Regards,\n>\n> Laurent Pinchart\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 202BEC3218\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon, 21 Jun 2021 14:02:33 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id D71D268935;\n\tMon, 21 Jun 2021 16:02:32 +0200 (CEST)","from mail-qk1-x72d.google.com (mail-qk1-x72d.google.com\n\t[IPv6:2607:f8b0:4864:20::72d])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 8555360295\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon, 21 Jun 2021 16:02:31 +0200 (CEST)","by mail-qk1-x72d.google.com with SMTP id g4so29424591qkl.1\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon, 21 Jun 2021 07:02:31 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"lL+CHJmC\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:references:in-reply-to:from:date:message-id:subject:to;\n\tbh=t11MxIDlAbpWTyE44vw5yOqG94pnrHzVZXfKFP0bBgg=;\n\tb=lL+CHJmCkcqylVU6t/LE0W8uf0T7SPiznWDvpWYy7FiX3K8xfA9krPZMjiRsjLL4R/\n\thCWsYwlIS0F3OmsuEcc8zNyMq2Y5+JyqzkvTzBXT6DlHw//XBHyfexxFsjeeg2OtCejG\n\tJbq/MaOM1Ak7/ucMNsjOKXxxLcK4lJBkkwPegSg7FFYSEmYJfdRmpmoWZywCe1qFEqjN\n\tZsrYVnK7MFOMBPA+WNgme98GHa2YoHFWrcgeK5FrpwV+VIamGW/5XYJWl9FURmEuETiJ\n\tfauXZ/XHUStA2/gAstmUgTu5q4kSWM8HAVjKHcM+5wf62vOPqnr9LJCiMAyolDuPdTG0\n\tUf7A==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:references:in-reply-to:from:date\n\t:message-id:subject:to;\n\tbh=t11MxIDlAbpWTyE44vw5yOqG94pnrHzVZXfKFP0bBgg=;\n\tb=p8WuwS33UB/TlKFUju7FfIyioFWQLXRD1A0cG/XF0D4rIGYWr9TIHoC8aBsD4WVVK1\n\tz37N5QPuBXZTOixbZSbdzBcKfwyZj71dfQ0fbqDqBUisTGry4SIeBNBCARVDl2F+VNzA\n\trhyKZuQsShp9uzj9ADMWSUitbNfyBa6n4lMA62CtmAFt9pmy+x2xnyWLc7QnViV/yINy\n\tLdhljBU2zSMtEZJTkPexgtu0NMC1jDD3BzSQB+4Yc+OJxV4gXkOwaQeUOF69IyXweQOJ\n\tNcdg2oXFYbVcpvwpI6seACPHDPykwgQpxzQR4SFAu5k4UwXGZevoSWFvyHzoxurmmRz2\n\tUNGw==","X-Gm-Message-State":"AOAM530Sj4IdqT7tqYpQXJE1bq6GoYmuhaToUFAtlsFYoanhJeWTajqn\n\tWSCvvI0EBJU02mRFuGSueqVgPgGmaMcFgHd1kyY=","X-Google-Smtp-Source":"ABdhPJxWpncXrsmD4Su+sW6Tf43jcjfu1IUr24WpIkQKAJH5VQBFpftdpNUkn9ra9mkDp3KZaxZD84mKQktn1n2rz/w=","X-Received":"by 2002:a05:6902:68a:: with SMTP id\n\ti10mr31165985ybt.175.1624284150129; \n\tMon, 21 Jun 2021 07:02:30 -0700 (PDT)","MIME-Version":"1.0","References":"<20210621133449.80586-1-vedantparanjape160201@gmail.com>\n\t<ef611e03-e6eb-013c-2926-075612dd163e@ideasonboard.com>\n\t<YNCYohzIddYIetdo@pendragon.ideasonboard.com>","In-Reply-To":"<YNCYohzIddYIetdo@pendragon.ideasonboard.com>","From":"Vedant Paranjape <vedantparanjape160201@gmail.com>","Date":"Mon, 21 Jun 2021 19:32:18 +0530","Message-ID":"<CACGrz-MJnVZqLik+8r8gb+BMhO1_zP13W7ZxLrrCJThWiu5L+g@mail.gmail.com>","To":"Laurent Pinchart <laurent.pinchart@ideasonboard.com>,\n\tkieran.bingham@ideasonboard.com, libcamera-devel@lists.libcamera.org","Content-Type":"multipart/alternative; boundary=\"000000000000c8676d05c5471f5a\"","Subject":"Re: [libcamera-devel] [PATCH v1] libcamera: Add OV5647 sensor\n\tproperties","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>"}}]