Re: [PATCH v2 1/4] docs: fix repeated word 'that' in userspace-api documentation

From: Hans Verkuil

Date: Tue May 12 2026 - 03:18:53 EST


On 09/05/2026 16:30, Adrien Reynard wrote:
> Remove duplicated word 'that' in two places in the V4L2
> documentation files vidioc-g-ext-ctrls.rst and
> vidioc-subscribe-event.rst.
>
> Signed-off-by: Adrien Reynard <reynard.adrien.08@xxxxxxxxx>
> ---
> Documentation/userspace-api/media/v4l/vidioc-g-ext-ctrls.rst | 2 +-
> .../userspace-api/media/v4l/vidioc-subscribe-event.rst | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/userspace-api/media/v4l/vidioc-g-ext-ctrls.rst b/Documentation/userspace-api/media/v4l/vidioc-g-ext-ctrls.rst
> index b8698b85bd80..8d5db414f8fa 100644
> --- a/Documentation/userspace-api/media/v4l/vidioc-g-ext-ctrls.rst
> +++ b/Documentation/userspace-api/media/v4l/vidioc-g-ext-ctrls.rst
> @@ -64,7 +64,7 @@ should re-allocate the memory to this new size and try again. For the
> string type it is possible that the same issue occurs again if the
> string has grown in the meantime. It is recommended to call
> :ref:`VIDIOC_QUERYCTRL` first and use
> -``maximum``\ +1 as the new ``size`` value. It is guaranteed that that is
> +``maximum``\ +1 as the new ``size`` value. It is guaranteed that is
> sufficient memory.
>
> N-dimensional arrays are set and retrieved row-by-row. You cannot set a
> diff --git a/Documentation/userspace-api/media/v4l/vidioc-subscribe-event.rst b/Documentation/userspace-api/media/v4l/vidioc-subscribe-event.rst
> index a6fc3c5fe99d..845a90573e54 100644
> --- a/Documentation/userspace-api/media/v4l/vidioc-subscribe-event.rst
> +++ b/Documentation/userspace-api/media/v4l/vidioc-subscribe-event.rst
> @@ -94,7 +94,7 @@ using the :ref:`VIDIOC_DQEVENT` ioctl.
> a V4L2_EVENT_CTRL to be sent back to that same filehandle.
> Normally such events are suppressed to prevent feedback loops
> where an application changes a control to a one value and then
> - another, and then receives an event telling it that that control
> + another, and then receives an event telling it that control
> has changed to the first value.
>
> Since it can't tell whether that event was caused by another

This is actually correct English:

https://englishcolege.com/is-that-that-grammatically-correct/#:~:text=Yes.,specific%20(as%20a%20pronoun).

Rejecting this patch.

Regards,

Hans