<div dir="ltr"><div dir="ltr">Hi Laurent,<div><br></div><div>Thanks for the feedback!</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 28 Mar 2022 at 00:13, Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com">laurent.pinchart@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">Hi Naush,<br>
<br>
On Fri, Mar 25, 2022 at 09:08:55AM +0000, Naushir Patuck via libcamera-devel wrote:<br>
> Hi,<br>
> <br>
> Version 5 of this series changes the following:<br>
> <br>
> - In patch 6/8 use -ESHUTDOWN instead of -EPERM.<br>
> - In patch 6/8 reference the function directly in the doxygen comments.<br>
> <br>
> Patch 6/8 needs one more R-B tag, then this series should be ready for merging.<br>
<br>
The series is now fully reviewed. If you agree with my review of 5/8 I<br>
can update that patch when applying.<br></blockquote><div><br></div><div>Oops, sorry I forgot to add those changes to my last revision.</div><div>I'm happy with your suggestions, so feel free to update when applying :-)</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>
> Naushir Patuck (8):<br>
>   pipeline: raspberrypi: Avoid over-allocation for ISP Output 1<br>
>   pipeline: raspberrypi: Move freeBuffers() to the RPiCameraData class<br>
>   pipeline: raspberrypi: Free buffers in the RPiCamera destructor and<br>
>     re-configure<br>
>   pipeline: raspberrypi: Repurpose RPi::Stream::reset()<br>
>   libcamera: v4l2_videodevice: Better tracking of the device state<br>
>   libcamera: v4l2_videodevice: Do not allow buffer queueing in stopping<br>
>     state<br>
>   libcamera: v4l2_videodevice: Empty the V4L2 buffer cache on<br>
>     streamOff()<br>
>   test: Test V4L2BufferCache::isEmpty() member function<br>
> <br>
>  include/libcamera/internal/v4l2_videodevice.h |  4 +-<br>
>  .../pipeline/raspberrypi/raspberrypi.cpp      | 56 ++++++++++++-------<br>
>  .../pipeline/raspberrypi/rpi_stream.cpp       | 13 ++---<br>
>  .../pipeline/raspberrypi/rpi_stream.h         |  2 +-<br>
>  src/libcamera/v4l2_videodevice.cpp            | 34 +++++++++--<br>
>  test/v4l2_videodevice/buffer_cache.cpp        | 36 ++++++++++++<br>
>  6 files changed, 112 insertions(+), 33 deletions(-)<br>
> <br>
<br>
-- <br>
Regards,<br>
<br>
Laurent Pinchart<br>
</blockquote></div></div>