<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 6 Apr 2022 at 13:42, 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>
Thank you for the patch.<br>
<br>
On Wed, Apr 06, 2022 at 01:38:05PM +0100, Naushir Patuck via libcamera-devel wrote:<br>
> s/prevetn/prevent/<br>
> <br>
> Signed-off-by: Naushir Patuck <<a href="mailto:naush@raspberrypi.com" target="_blank">naush@raspberrypi.com</a>><br>
> ---<br>
>  src/libcamera/v4l2_videodevice.cpp | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> <br>
> diff --git a/src/libcamera/v4l2_videodevice.cpp b/src/libcamera/v4l2_videodevice.cpp<br>
> index 0830be80c553..98671509fe02 100644<br>
> --- a/src/libcamera/v4l2_videodevice.cpp<br>
> +++ b/src/libcamera/v4l2_videodevice.cpp<br>
> @@ -1715,7 +1715,7 @@ FrameBuffer *V4L2VideoDevice::dequeueBuffer()<br>
>        * not find it in the queuedBuffers_ list.<br>
>        *<br>
>        * Whilst this kernel bug has been fixed in mainline, ensure that we<br>
> -      * safely ingore buffers which are unexpected to prevetn crashes on<br>
> +      * safely ingore buffers which are unexpected to prevent crashes on<br>
<br>
While at it, s/ingore/ignore/ ? :-) I'll fix that locally.<br></blockquote><div><br></div><div>Haha, I can't believe I missed that!</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>
Reviewed-by: Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com" target="_blank">laurent.pinchart@ideasonboard.com</a>><br>
<br>
>        * older kernels.<br>
>        */<br>
>       auto it = queuedBuffers_.find(buf.index);<br>
<br>
-- <br>
Regards,<br>
<br>
Laurent Pinchart<br>
</blockquote></div></div>