[{"id":15666,"web_url":"https://patchwork.libcamera.org/comment/15666/","msgid":"","date":"2021-03-14T01:43:18","subject":"Re: [libcamera-devel] [PATCH 3/8] libcamera: pipeline: ipu3: Reset\n\tsequence counts to zero on stop","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 Fri, Mar 12, 2021 at 06:11:26AM +0000, Kieran Bingham wrote:\n> Reset the request sequence counter back to zero when a camera has\n> stopped, giving each camera run an independent sequence stream.\n> \n> Signed-off-by: Kieran Bingham \n> \n> ---\n> RFC: This again would be perhaps better handled by the base class to\n> ensure all pipeline handlers handle this in a consistent manner.\n\nYes, if we want to do this, I think it should go in the base class. I'm\nnot sure we should reset the sequence number though, as not doing so may\nhelp identifying requests from a previous capture session that leak to a\nnew capture session.\n\n> src/libcamera/pipeline/ipu3/ipu3.cpp | 2 ++\n> 1 file changed, 2 insertions(+)\n> \n> diff --git a/src/libcamera/pipeline/ipu3/ipu3.cpp b/src/libcamera/pipeline/ipu3/ipu3.cpp\n> index a61e2b51ef9e..3d3bd4a43c23 100644\n> --- a/src/libcamera/pipeline/ipu3/ipu3.cpp\n> +++ b/src/libcamera/pipeline/ipu3/ipu3.cpp\n> @@ -780,6 +780,8 @@ void PipelineHandlerIPU3::stop(Camera *camera)\n> \t\tLOG(IPU3, Fatal) << \"There are still requests to complete.\";\n> \n> \tfreeBuffers(camera);\n> +\n> +\tdata->requestSequence_ = 0;\n> }\n> \n> int PipelineHandlerIPU3::queueRequestDevice(Camera *camera, Request *request)","headers":{"Return-Path":"","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 8E28FBD1F1\n\tfor ;\n\tSun, 14 Mar 2021 01:43:56 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 1274460850;\n\tSun, 14 Mar 2021 02:43:56 +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 34BF16084D\n\tfor ;\n\tSun, 14 Mar 2021 02:43:55 +0100 (CET)","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 9215F55C;\n\tSun, 14 Mar 2021 02:43:54 +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=\"mXNxSUIB\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1615686234;\n\tbh=mTklqORFXcEZWUF7TMTe4qB31fNiYOJE76Pby01BXqQ=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=mXNxSUIB9b80FFrvAH0aqv58uYvdf/7I+lclvmM3KkbyBkUAJ53fiVlfU4/uKIRJd\n\tkox/hvAY+K3L/2NDEIgZN/9ddY0rRzLxtvR9Q7iBNAysyFutT8e+itwUwlRmmfmlpA\n\tND6ZLsCsamJk0lXIS0/e5dRVyTBX1sSEpLFTVMx0=","Date":"Sun, 14 Mar 2021 03:43:18 +0200","From":"Laurent Pinchart ","To":"Kieran Bingham ","Message-ID":"","References":"<20210312061131.854849-1-kieran.bingham@ideasonboard.com>\n\t<20210312061131.854849-4-kieran.bingham@ideasonboard.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20210312061131.854849-4-kieran.bingham@ideasonboard.com>","Subject":"Re: [libcamera-devel] [PATCH 3/8] libcamera: pipeline: ipu3: Reset\n\tsequence counts to zero on stop","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"libcamera devel ","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":15699,"web_url":"https://patchwork.libcamera.org/comment/15699/","msgid":"<7bcf84d6-e014-29e9-c561-7cd5e97e8eba@ideasonboard.com>","date":"2021-03-15T12:13:15","subject":"Re: [libcamera-devel] [PATCH 3/8] libcamera: pipeline: ipu3: Reset\n\tsequence counts to zero on stop","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Hi Laurent,\n\nOn 14/03/2021 01:43, Laurent Pinchart wrote:\n> Hi Kieran,\n> \n> Thank you for the patch.\n> \n> On Fri, Mar 12, 2021 at 06:11:26AM +0000, Kieran Bingham wrote:\n>> Reset the request sequence counter back to zero when a camera has\n>> stopped, giving each camera run an independent sequence stream.\n>>\n>> Signed-off-by: Kieran Bingham \n>>\n>> ---\n>> RFC: This again would be perhaps better handled by the base class to\n>> ensure all pipeline handlers handle this in a consistent manner.\n> \n> Yes, if we want to do this, I think it should go in the base class. I'm\n> not sure we should reset the sequence number though, as not doing so may\n> help identifying requests from a previous capture session that leak to a\n> new capture session.\n\nI hope that lives in the world of things that now shouldn't be able to\nhappen - but given the exercise that caused this series was entirely\nabout requests from previous streams being used late ...\n\nBut wouldn't it be more obvious that such an event has happened if the\nstream starts running and there is a request involved with a high\nsequence value compared to the expectation of starting from zero?\n\nIf we expect streams to start at some arbitrary high value each time, it\nmight be harder to notice (without requiring a direct comparison against\nthe 'last known' sequence value).\n\n\n\n> \n>> src/libcamera/pipeline/ipu3/ipu3.cpp | 2 ++\n>> 1 file changed, 2 insertions(+)\n>>\n>> diff --git a/src/libcamera/pipeline/ipu3/ipu3.cpp b/src/libcamera/pipeline/ipu3/ipu3.cpp\n>> index a61e2b51ef9e..3d3bd4a43c23 100644\n>> --- a/src/libcamera/pipeline/ipu3/ipu3.cpp\n>> +++ b/src/libcamera/pipeline/ipu3/ipu3.cpp\n>> @@ -780,6 +780,8 @@ void PipelineHandlerIPU3::stop(Camera *camera)\n>> \t\tLOG(IPU3, Fatal) << \"There are still requests to complete.\";\n>> \n>> \tfreeBuffers(camera);\n>> +\n>> +\tdata->requestSequence_ = 0;\n>> }\n>> \n>> int PipelineHandlerIPU3::queueRequestDevice(Camera *camera, Request *request)\n>","headers":{"Return-Path":"","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 9A0D9BD80E\n\tfor ;\n\tMon, 15 Mar 2021 12:13:20 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id EAB0868D40;\n\tMon, 15 Mar 2021 13:13:19 +0100 (CET)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 5077C60106\n\tfor ;\n\tMon, 15 Mar 2021 13:13:18 +0100 (CET)","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 B6BC2556;\n\tMon, 15 Mar 2021 13:13:17 +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=\"I6wPX4b1\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1615810397;\n\tbh=d/DrSyetA/Hkit6eoplT3CpT+f08UQM+RawaKPVIhWo=;\n\th=Reply-To:Subject:To:Cc:References:From:Date:In-Reply-To:From;\n\tb=I6wPX4b1IxilngsRrFYAtrSVh133ckot3Q96l5KuiW/fAF9Q2J6pE4Zi4qthCYCdT\n\tzSsXw1u1LOaNS8QHnnZQXk9+crtfoPE4PYSGAquSRoKtdfbfakemrQwPCMYgtZASii\n\tog7lrONZchHD+NX43JlKbDe3iAcWjyIVwiUVqh60=","To":"Laurent Pinchart ","References":"<20210312061131.854849-1-kieran.bingham@ideasonboard.com>\n\t<20210312061131.854849-4-kieran.bingham@ideasonboard.com>\n\t","From":"Kieran Bingham ","Autocrypt":"addr=kieran.bingham@ideasonboard.com; keydata=\n\tmQINBFYE/WYBEACs1PwjMD9rgCu1hlIiUA1AXR4rv2v+BCLUq//vrX5S5bjzxKAryRf0uHat\n\tV/zwz6hiDrZuHUACDB7X8OaQcwhLaVlq6byfoBr25+hbZG7G3+5EUl9cQ7dQEdvNj6V6y/SC\n\trRanWfelwQThCHckbobWiQJfK9n7rYNcPMq9B8e9F020LFH7Kj6YmO95ewJGgLm+idg1Kb3C\n\tpotzWkXc1xmPzcQ1fvQMOfMwdS+4SNw4rY9f07Xb2K99rjMwZVDgESKIzhsDB5GY465sCsiQ\n\tcSAZRxqE49RTBq2+EQsbrQpIc8XiffAB8qexh5/QPzCmR4kJgCGeHIXBtgRj+nIkCJPZvZtf\n\tKr2EAbc6tgg6DkAEHJb+1okosV09+0+TXywYvtEop/WUOWQ+zo+Y/OBd+8Ptgt1pDRyOBzL8\n\tRXa8ZqRf0Mwg75D+dKntZeJHzPRJyrlfQokngAAs4PaFt6UfS+ypMAF37T6CeDArQC41V3ko\n\tlPn1yMsVD0p+6i3DPvA/GPIksDC4owjnzVX9kM8Zc5Cx+XoAN0w5Eqo4t6qEVbuettxx55gq\n\t8K8FieAjgjMSxngo/HST8TpFeqI5nVeq0/lqtBRQKumuIqDg+Bkr4L1V/PSB6XgQcOdhtd36\n\tOe9X9dXB8YSNt7VjOcO7BTmFn/Z8r92mSAfHXpb07YJWJosQOQARAQABtDBLaWVyYW4gQmlu\n\tZ2hhbSA8a2llcmFuLmJpbmdoYW1AaWRlYXNvbmJvYXJkLmNvbT6JAlcEEwEKAEECGwMFCwkI\n\tBwIGFQgJCgsCBBYCAwECHgECF4ACGQEWIQSQLdeYP70o/eNy1HqhHkZyEKRh/QUCXWTtygUJ\n\tCyJXZAAKCRChHkZyEKRh/f8dEACTDsbLN2nioNZMwyLuQRUAFcXNolDX48xcUXsWS2QjxaPm\n\tVsJx8Uy8aYkS85mdPBh0C83OovQR/OVbr8AxhGvYqBs3nQvbWuTl/+4od7DfK2VZOoKBAu5S\n\tQK2FYuUcikDqYcFWJ8DQnubxfE8dvzojHEkXw0sA4igINHDDFX3HJGZtLio+WpEFQtCbfTAG\n\tYZslasz1YZRbwEdSsmO3/kqy5eMnczlm8a21A3fKUo3g8oAZEFM+f4DUNzqIltg31OAB/kZS\n\tenKZQ/SWC8PmLg/ZXBrReYakxXtkP6w3FwMlzOlhGxqhIRNiAJfXJBaRhuUWzPOpEDE9q5YJ\n\tBmqQL2WJm1VSNNVxbXJHpaWMH1sA2R00vmvRrPXGwyIO0IPYeUYQa3gsy6k+En/aMQJd27dp\n\taScf9am9PFICPY5T4ppneeJLif2lyLojo0mcHOV+uyrds9XkLpp14GfTkeKPdPMrLLTsHRfH\n\tfA4I4OBpRrEPiGIZB/0im98MkGY/Mu6qxeZmYLCcgD6qz4idOvfgVOrNh+aA8HzIVR+RMW8H\n\tQGBN9f0E3kfwxuhl3omo6V7lDw8XOdmuWZNC9zPq1UfryVHANYbLGz9KJ4Aw6M+OgBC2JpkD\n\thXMdHUkC+d20dwXrwHTlrJi1YNp6rBc+xald3wsUPOZ5z8moTHUX/uPA/qhGsbkCDQRWBP1m\n\tARAAzijkb+Sau4hAncr1JjOY+KyFEdUNxRy+hqTJdJfaYihxyaj0Ee0P0zEi35CbE6lgU0Uz\n\ttih9fiUbSV3wfsWqg1Ut3/5rTKu7kLFp15kF7eqvV4uezXRD3Qu4yjv/rMmEJbbD4cTvGCYI\n\td6MDC417f7vK3hCbCVIZSp3GXxyC1LU+UQr3fFcOyCwmP9vDUR9JV0BSqHHxRDdpUXE26Dk6\n\tmhf0V1YkspE5St814ETXpEus2urZE5yJIUROlWPIL+hm3NEWfAP06vsQUyLvr/GtbOT79vXl\n\tEn1aulcYyu20dRRxhkQ6iILaURcxIAVJJKPi8dsoMnS8pB0QW12AHWuirPF0g6DiuUfPmrA5\n\tPKe56IGlpkjc8cO51lIxHkWTpCMWigRdPDexKX+Sb+W9QWK/0JjIc4t3KBaiG8O4yRX8ml2R\n\t+rxfAVKM6V769P/hWoRGdgUMgYHFpHGSgEt80OKK5HeUPy2cngDUXzwrqiM5Sz6Od0qw5pCk\n\tNlXqI0W/who0iSVM+8+RmyY0OEkxEcci7rRLsGnM15B5PjLJjh1f2ULYkv8s4SnDwMZ/kE04\n\t/UqCMK/KnX8pwXEMCjz0h6qWNpGwJ0/tYIgQJZh6bqkvBrDogAvuhf60Sogw+mH8b+PBlx1L\n\toeTK396wc+4c3BfiC6pNtUS5GpsPMMjYMk7kVvEAEQEAAYkCPAQYAQoAJgIbDBYhBJAt15g/\n\tvSj943LUeqEeRnIQpGH9BQJdizzIBQkLSKZiAAoJEKEeRnIQpGH9eYgQAJpjaWNgqNOnMTmD\n\tMJggbwjIotypzIXfhHNCeTkG7+qCDlSaBPclcPGYrTwCt0YWPU2TgGgJrVhYT20ierN8LUvj\n\t6qOPTd+Uk7NFzL65qkh80ZKNBFddx1AabQpSVQKbdcLb8OFs85kuSvFdgqZwgxA1vl4TFhNz\n\tPZ79NAmXLackAx3sOVFhk4WQaKRshCB7cSl+RIng5S/ThOBlwNlcKG7j7W2MC06BlTbdEkUp\n\tECzuuRBv8wX4OQl+hbWbB/VKIx5HKlLu1eypen/5lNVzSqMMIYkkZcjV2SWQyUGxSwq0O/sx\n\tS0A8/atCHUXOboUsn54qdxrVDaK+6jIAuo8JiRWctP16KjzUM7MO0/+4zllM8EY57rXrj48j\n\tsbEYX0YQnzaj+jO6kJtoZsIaYR7rMMq9aUAjyiaEZpmP1qF/2sYenDx0Fg2BSlLvLvXM0vU8\n\tpQk3kgDu7kb/7PRYrZvBsr21EIQoIjXbZxDz/o7z95frkP71EaICttZ6k9q5oxxA5WC6sTXc\n\tMW8zs8avFNuA9VpXt0YupJd2ijtZy2mpZNG02fFVXhIn4G807G7+9mhuC4XG5rKlBBUXTvPU\n\tAfYnB4JBDLmLzBFavQfvonSfbitgXwCG3vS+9HEwAjU30Bar1PEOmIbiAoMzuKeRm2LVpmq4\n\tWZw01QYHU/GUV/zHJSFk","Organization":"Ideas on Board","Message-ID":"<7bcf84d6-e014-29e9-c561-7cd5e97e8eba@ideasonboard.com>","Date":"Mon, 15 Mar 2021 12:13:15 +0000","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101\n\tThunderbird/68.10.0","MIME-Version":"1.0","In-Reply-To":"","Content-Language":"en-GB","Subject":"Re: [libcamera-devel] [PATCH 3/8] libcamera: pipeline: ipu3: Reset\n\tsequence counts to zero on stop","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Reply-To":"kieran.bingham@ideasonboard.com","Cc":"libcamera devel ","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":15706,"web_url":"https://patchwork.libcamera.org/comment/15706/","msgid":"","date":"2021-03-15T16:40:07","subject":"Re: [libcamera-devel] [PATCH 3/8] libcamera: pipeline: ipu3: Reset\n\tsequence counts to zero on stop","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Kieran,\n\nOn Mon, Mar 15, 2021 at 12:13:15PM +0000, Kieran Bingham wrote:\n> On 14/03/2021 01:43, Laurent Pinchart wrote:\n> > On Fri, Mar 12, 2021 at 06:11:26AM +0000, Kieran Bingham wrote:\n> >> Reset the request sequence counter back to zero when a camera has\n> >> stopped, giving each camera run an independent sequence stream.\n> >>\n> >> Signed-off-by: Kieran Bingham \n> >>\n> >> ---\n> >> RFC: This again would be perhaps better handled by the base class to\n> >> ensure all pipeline handlers handle this in a consistent manner.\n> > \n> > Yes, if we want to do this, I think it should go in the base class. I'm\n> > not sure we should reset the sequence number though, as not doing so may\n> > help identifying requests from a previous capture session that leak to a\n> > new capture session.\n> \n> I hope that lives in the world of things that now shouldn't be able to\n> happen - but given the exercise that caused this series was entirely\n> about requests from previous streams being used late ...\n> \n> But wouldn't it be more obvious that such an event has happened if the\n> stream starts running and there is a request involved with a high\n> sequence value compared to the expectation of starting from zero?\n> \n> If we expect streams to start at some arbitrary high value each time, it\n> might be harder to notice (without requiring a direct comparison against\n> the 'last known' sequence value).\n\nYou're right, but there's a potential case of the first capture session\nbeing very short (let's say just one request in the worst case),\nresetting the counter would not allow us to distinguish two different\n\"request 0\". That's a pathological case though.\n\n> >> src/libcamera/pipeline/ipu3/ipu3.cpp | 2 ++\n> >> 1 file changed, 2 insertions(+)\n> >>\n> >> diff --git a/src/libcamera/pipeline/ipu3/ipu3.cpp b/src/libcamera/pipeline/ipu3/ipu3.cpp\n> >> index a61e2b51ef9e..3d3bd4a43c23 100644\n> >> --- a/src/libcamera/pipeline/ipu3/ipu3.cpp\n> >> +++ b/src/libcamera/pipeline/ipu3/ipu3.cpp\n> >> @@ -780,6 +780,8 @@ void PipelineHandlerIPU3::stop(Camera *camera)\n> >> \t\tLOG(IPU3, Fatal) << \"There are still requests to complete.\";\n> >> \n> >> \tfreeBuffers(camera);\n> >> +\n> >> +\tdata->requestSequence_ = 0;\n> >> }\n> >> \n> >> int PipelineHandlerIPU3::queueRequestDevice(Camera *camera, Request *request)","headers":{"Return-Path":"","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 A2209BD80C\n\tfor ;\n\tMon, 15 Mar 2021 16:40:44 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id DF9CD68D48;\n\tMon, 15 Mar 2021 17:40:43 +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 3B94568D3E\n\tfor ;\n\tMon, 15 Mar 2021 17:40:43 +0100 (CET)","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 A0948316;\n\tMon, 15 Mar 2021 17:40:42 +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=\"Z6gTNmfx\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1615826442;\n\tbh=N1kBlyOTg6ivXrONXsl17srUd4z/fzjPGk6URRzibWk=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=Z6gTNmfxADsbeiCUbdyO/VMDldyL/NOLKNZ3RBf9aIxrvd/tMACbXLTPQ4OaCWnEC\n\t+LJ968FW0DuhZb4PwZ/iDhoEH2E+PUtGljNdx8HB1BSVdXGMVorsC9VPCWRmfpl9/S\n\tcBXp8k92yKaTKw0d0Qxy/H//Zfi96iK88rwgKG1M=","Date":"Mon, 15 Mar 2021 18:40:07 +0200","From":"Laurent Pinchart ","To":"Kieran Bingham ","Message-ID":"","References":"<20210312061131.854849-1-kieran.bingham@ideasonboard.com>\n\t<20210312061131.854849-4-kieran.bingham@ideasonboard.com>\n\t\n\t<7bcf84d6-e014-29e9-c561-7cd5e97e8eba@ideasonboard.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<7bcf84d6-e014-29e9-c561-7cd5e97e8eba@ideasonboard.com>","Subject":"Re: [libcamera-devel] [PATCH 3/8] libcamera: pipeline: ipu3: Reset\n\tsequence counts to zero on stop","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"libcamera devel ","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}}]