Re: [PATCH] media: Documentation: dev-subdev: Fix sentence conjunction

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Umang,

Quoting Umang Jain (2024-04-12 10:28:36)
> Fix sentence conjunction in the streams and routes section.
> 
> Signed-off-by: Umang Jain <umang.jain@xxxxxxxxxxxxxxxx>
> ---
>  Documentation/userspace-api/media/v4l/dev-subdev.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/userspace-api/media/v4l/dev-subdev.rst b/Documentation/userspace-api/media/v4l/dev-subdev.rst
> index 43988516acdd..7f6620cbdf78 100644
> --- a/Documentation/userspace-api/media/v4l/dev-subdev.rst
> +++ b/Documentation/userspace-api/media/v4l/dev-subdev.rst
> @@ -561,7 +561,7 @@ selections. The order of configuring formats and selections along a stream is
>  the same as without streams (see :ref:`format-propagation`).
>  
>  Instead of the sub-device wide merging of streams from all sink pads
> -towards all source pads, data flows for each route are separate from each
> +towards all source pads, data flows for each route that are separate from each

I'm not sure this is correct I'm afraid.

The original explains that there are multiple flows of data ('streams?')
and that they are each separate (and can flow in different routes?)

I would only expect a 'that' if there was some further expansion which
doesn't follow.

In fact, I just tried, and I can't make the little green 'grammar-ok'
light in my head illuminate when there is a 'that' added here ;-S


My grammar skills are not good enough to distinguish/explain precisely
except for that magic builtin-native-speaker-right-wrong check and so
ChatGPT (take with a pinch of salt) explained the following:

https://chat.openai.com/share/632bbcae-22f7-4b42-bbe0-6f9178a36d14

Between Sentence A and Sentence B, Sentence A is more grammatically
correct.

Sentence A: "Instead of the sub-device wide merging of streams from all
sink pads towards all source pads, data flows for each route are
separate from each other."

In this sentence, "data flows for each route are separate from each
other" is a clear and independent clause, indicating that the data flows
are separate for each route.

Sentence B: "Instead of the sub-device wide merging of streams from all
sink pads towards all source pads, data flows for each route that are
separate from each other."

In this sentence, the phrase "that are separate from each other" is a
restrictive relative clause modifying "each route." However, this
construction makes the sentence slightly confusing because it seems to
imply that the separation applies to the routes themselves, not to the
data flows. The sentence structure leads to ambiguity regarding what
exactly is separate from what.

Therefore, Sentence A is clearer and more grammatically sound in the
context provided.


--
Kieran


>  other. Any number of routes from streams on sink pads towards streams on
>  source pads is allowed, to the extent supported by drivers. For every
>  stream on a source pad, however, only a single route is allowed.
> -- 
> 2.44.0
>





[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux