Hi Daniel,
Thanks for fixing up and sending a v2.
Quoting Daniel Lundberg Pedersen (2022-01-12 12:28:09)
> Example should be comparing errno to EINTR instead of doing assignment.
>
> Signed-off-by: Daniel Lundberg Pedersen <dlp@xxxxxxxx>
I think my tag from the previous patch still holds here for the change
itself.
Reviewed-by: Kieran Bingham <kieran.bingham+renesas@xxxxxxxxxxxxxxxx>
> ---
> Documentation/userspace-api/media/v4l/v4l2grab.c.rst | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/userspace-api/media/v4l/v4l2grab.c.rst b/Documentation/userspace-api/media/v4l/v4l2grab.c.rst
> index eaa0f95048e7..c98a78d63a8b 100644
> --- a/Documentation/userspace-api/media/v4l/v4l2grab.c.rst
> +++ b/Documentation/userspace-api/media/v4l/v4l2grab.c.rst
> @@ -134,7 +134,7 @@ file: media/v4l/v4l2grab.c
> tv.tv_usec = 0;
>
> r = select(fd + 1, &fds, NULL, NULL, &tv);
> - } while ((r == -1 && (errno = EINTR)));
> + } while ((r == -1 && (errno == EINTR)));
> if (r == -1) {
> perror("select");
> return errno;
> --
> 2.33.1
>
