<div dir="ltr"><div dir="ltr">Hi Kieran,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 17 Nov 2021 at 16:47, Kieran Bingham <<a href="mailto:kieran.bingham@ideasonboard.com">kieran.bingham@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Quoting Naushir Patuck (2021-11-12 10:03:02)<br>
> Hi,<br>
> <br>
> The second version of this series changes the following over v1:<br>
> <br>
> - Apply Laurent's suggestions in patch 2/3.<br>
> - Allocate 2 internal buffers for Unicam instead of one in patch 2/3.<br>
<br>
I'm getting curious about that VIDEO_MAX_FRAME and whether we should<br>
just always allocate that in the V4L2VideoDevice class, and then do away<br>
with the importBuffers(count); function altogether, as it would be<br>
redundant ... But that's not necessarily a requriement for this series -<br>
just thinking out loud.<br></blockquote><div><br></div><div>Yes, that could be an option, buffer slots are fairly small storage elements,</div><div>so it should not increase the memory footprint too much.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Would you like this series merged? Or are we waiting on confirmation of<br>
the symptoms/issues from Roman before we commit to these patches?<br></blockquote><div><br></div><div>I'm inclined to say we merge this series now, as it does fix overallocations.</div><div>Given we don't know exactly what the cause of Roman's issues are, I would</div><div>prefer to fix time on top of these patches.</div><div><br></div><div>Regards,</div><div>Naush</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
--<br>
Kieran<br>
<br>
<br>
> <br>
> Thanks,<br>
> Naush<br>
> <br>
> Naushir Patuck (3):<br>
>   pipeline: raspberrypi: Add const qualifer in isRaw()<br>
>   pipeline: raspberrypi: Rework the internal buffer allocation scheme<br>
>   pipeline: raspberrypi: Increase the V4L2BufferCache slot allocations<br>
> <br>
>  .../pipeline/raspberrypi/raspberrypi.cpp      | 45 ++++++++++++++-----<br>
>  .../pipeline/raspberrypi/rpi_stream.cpp       |  8 ++++<br>
>  2 files changed, 41 insertions(+), 12 deletions(-)<br>
> <br>
> -- <br>
> 2.25.1<br>
><br>
</blockquote></div></div>