* Re: [git:v4l-dvb/for_v3.3] [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT documentation
[not found] <E1Rmmdy-0002zt-5K@www.linuxtv.org>
@ 2012-01-16 22:16 ` Guennadi Liakhovetski
2012-01-17 10:13 ` Hans Verkuil
0 siblings, 1 reply; 6+ messages in thread
From: Guennadi Liakhovetski @ 2012-01-16 22:16 UTC (permalink / raw)
To: linux-media; +Cc: linuxtv-commits, Rupert Eibauer, Hans Verkuil
On Mon, 16 Jan 2012, Mauro Carvalho Chehab wrote:
> This is an automatic generated email to let you know that the following patch were queued at the
> http://git.linuxtv.org/media_tree.git tree:
>
> Subject: [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT documentation
> Author: Hans Verkuil <hans.verkuil@cisco.com>
> Date: Wed Jan 11 07:37:54 2012 -0300
[snip]
> diff --git a/Documentation/DocBook/media/v4l/vidioc-g-output.xml b/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> index fd45f1c..4533068 100644
> --- a/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> +++ b/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> @@ -61,8 +61,9 @@ desired output in an integer and call the
> <constant>VIDIOC_S_OUTPUT</constant> ioctl with a pointer to this integer.
> Side effects are possible. For example outputs may support different
> video standards, so the driver may implicitly switch the current
> -standard. It is good practice to select an output before querying or
> -negotiating any other parameters.</para>
> +standard.
> +standard. Because of these possible side effects applications
> +must select an output before querying or negotiating any other parameters.</para>
something seems to be wrong here.
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [git:v4l-dvb/for_v3.3] [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT documentation
2012-01-16 22:16 ` [git:v4l-dvb/for_v3.3] [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT documentation Guennadi Liakhovetski
@ 2012-01-17 10:13 ` Hans Verkuil
2012-01-17 10:16 ` Guennadi Liakhovetski
0 siblings, 1 reply; 6+ messages in thread
From: Hans Verkuil @ 2012-01-17 10:13 UTC (permalink / raw)
To: Guennadi Liakhovetski
Cc: linux-media, linuxtv-commits, Rupert Eibauer, Hans Verkuil
On Monday 16 January 2012 23:16:31 Guennadi Liakhovetski wrote:
> On Mon, 16 Jan 2012, Mauro Carvalho Chehab wrote:
> > This is an automatic generated email to let you know that the following
> > patch were queued at the http://git.linuxtv.org/media_tree.git tree:
> >
> > Subject: [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT documentation
> > Author: Hans Verkuil <hans.verkuil@cisco.com>
> > Date: Wed Jan 11 07:37:54 2012 -0300
>
> [snip]
>
> > diff --git a/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> > b/Documentation/DocBook/media/v4l/vidioc-g-output.xml index
> > fd45f1c..4533068 100644
> > --- a/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> > +++ b/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> > @@ -61,8 +61,9 @@ desired output in an integer and call the
> >
> > <constant>VIDIOC_S_OUTPUT</constant> ioctl with a pointer to this
> > integer. Side effects are possible. For example outputs may support
> > different video standards, so the driver may implicitly switch the
> > current
> >
> > -standard. It is good practice to select an output before querying or
> > -negotiating any other parameters.</para>
> > +standard.
> > +standard. Because of these possible side effects applications
> > +must select an output before querying or negotiating any other
> > parameters.</para>
>
> something seems to be wrong here.
Hi Guennadi!
What's wrong here? I've no idea what you mean.
Regards,
Hans
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [git:v4l-dvb/for_v3.3] [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT documentation
2012-01-17 10:13 ` Hans Verkuil
@ 2012-01-17 10:16 ` Guennadi Liakhovetski
2012-01-17 10:49 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 6+ messages in thread
From: Guennadi Liakhovetski @ 2012-01-17 10:16 UTC (permalink / raw)
To: Hans Verkuil; +Cc: Linux Media Mailing List, Rupert Eibauer, Hans Verkuil
Hi Hans
On Tue, 17 Jan 2012, Hans Verkuil wrote:
> On Monday 16 January 2012 23:16:31 Guennadi Liakhovetski wrote:
> > On Mon, 16 Jan 2012, Mauro Carvalho Chehab wrote:
> > > This is an automatic generated email to let you know that the following
> > > patch were queued at the http://git.linuxtv.org/media_tree.git tree:
> > >
> > > Subject: [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT documentation
> > > Author: Hans Verkuil <hans.verkuil@cisco.com>
> > > Date: Wed Jan 11 07:37:54 2012 -0300
> >
> > [snip]
> >
> > > diff --git a/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> > > b/Documentation/DocBook/media/v4l/vidioc-g-output.xml index
> > > fd45f1c..4533068 100644
> > > --- a/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> > > +++ b/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> > > @@ -61,8 +61,9 @@ desired output in an integer and call the
> > >
> > > <constant>VIDIOC_S_OUTPUT</constant> ioctl with a pointer to this
> > > integer. Side effects are possible. For example outputs may support
> > > different video standards, so the driver may implicitly switch the
> > > current
> > >
> > > -standard. It is good practice to select an output before querying or
> > > -negotiating any other parameters.</para>
> > > +standard.
> > > +standard. Because of these possible side effects applications
> > > +must select an output before querying or negotiating any other
> > > parameters.</para>
> >
> > something seems to be wrong here.
>
> Hi Guennadi!
>
> What's wrong here? I've no idea what you mean.
> > > +standard.
> > > +standard. Because of these possible side effects applications
doesn't seem to make much sense?
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [git:v4l-dvb/for_v3.3] [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT documentation
2012-01-17 10:16 ` Guennadi Liakhovetski
@ 2012-01-17 10:49 ` Mauro Carvalho Chehab
2012-01-17 10:54 ` Guennadi Liakhovetski
0 siblings, 1 reply; 6+ messages in thread
From: Mauro Carvalho Chehab @ 2012-01-17 10:49 UTC (permalink / raw)
To: Guennadi Liakhovetski
Cc: Hans Verkuil, Linux Media Mailing List, Rupert Eibauer,
Hans Verkuil
Em 17-01-2012 08:16, Guennadi Liakhovetski escreveu:
> Hi Hans
>
> On Tue, 17 Jan 2012, Hans Verkuil wrote:
>
>> On Monday 16 January 2012 23:16:31 Guennadi Liakhovetski wrote:
>>> On Mon, 16 Jan 2012, Mauro Carvalho Chehab wrote:
>>>> This is an automatic generated email to let you know that the following
>>>> patch were queued at the http://git.linuxtv.org/media_tree.git tree:
>>>>
>>>> Subject: [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT documentation
>>>> Author: Hans Verkuil <hans.verkuil@cisco.com>
>>>> Date: Wed Jan 11 07:37:54 2012 -0300
>>>
>>> [snip]
>>>
>>>> diff --git a/Documentation/DocBook/media/v4l/vidioc-g-output.xml
>>>> b/Documentation/DocBook/media/v4l/vidioc-g-output.xml index
>>>> fd45f1c..4533068 100644
>>>> --- a/Documentation/DocBook/media/v4l/vidioc-g-output.xml
>>>> +++ b/Documentation/DocBook/media/v4l/vidioc-g-output.xml
>>>> @@ -61,8 +61,9 @@ desired output in an integer and call the
>>>>
>>>> <constant>VIDIOC_S_OUTPUT</constant> ioctl with a pointer to this
>>>> integer. Side effects are possible. For example outputs may support
>>>> different video standards, so the driver may implicitly switch the
>>>> current
>>>>
>>>> -standard. It is good practice to select an output before querying or
>>>> -negotiating any other parameters.</para>
>>>> +standard.
>>>> +standard. Because of these possible side effects applications
>>>> +must select an output before querying or negotiating any other
>>>> parameters.</para>
>>>
>>> something seems to be wrong here.
>>
>> Hi Guennadi!
>>
>> What's wrong here? I've no idea what you mean.
>
>>>> +standard.
>>>> +standard. Because of these possible side effects applications
>
> doesn't seem to make much sense?
Are you referring to grammar?
I think that a comma is missing there [1]:
Because of these possible side effects, applications
must select an output before querying or negotiating any other
parameters.
IMO, reverting the order of the explanation clause is semantically better,
as it bolds the need to select an output, and put the phrase at the
cause-effect order:
Applications must select an output before querying or negotiating
any other parameters, because of these possible side effects.
But this is really a matter of choice. In any case, except for the
comma, that makes sense for me.
[1] Rule 4 of http://grammar.ccc.commnet.edu/grammar/commas.htm
Regards,
Mauro
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [git:v4l-dvb/for_v3.3] [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT documentation
2012-01-17 10:49 ` Mauro Carvalho Chehab
@ 2012-01-17 10:54 ` Guennadi Liakhovetski
2012-01-17 10:58 ` Hans Verkuil
0 siblings, 1 reply; 6+ messages in thread
From: Guennadi Liakhovetski @ 2012-01-17 10:54 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: Hans Verkuil, Linux Media Mailing List, Rupert Eibauer,
Hans Verkuil
On Tue, 17 Jan 2012, Mauro Carvalho Chehab wrote:
> Em 17-01-2012 08:16, Guennadi Liakhovetski escreveu:
> > Hi Hans
> >
> > On Tue, 17 Jan 2012, Hans Verkuil wrote:
> >
> >> On Monday 16 January 2012 23:16:31 Guennadi Liakhovetski wrote:
> >>> On Mon, 16 Jan 2012, Mauro Carvalho Chehab wrote:
> >>>> This is an automatic generated email to let you know that the following
> >>>> patch were queued at the http://git.linuxtv.org/media_tree.git tree:
> >>>>
> >>>> Subject: [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT documentation
> >>>> Author: Hans Verkuil <hans.verkuil@cisco.com>
> >>>> Date: Wed Jan 11 07:37:54 2012 -0300
> >>>
> >>> [snip]
> >>>
> >>>> diff --git a/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> >>>> b/Documentation/DocBook/media/v4l/vidioc-g-output.xml index
> >>>> fd45f1c..4533068 100644
> >>>> --- a/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> >>>> +++ b/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> >>>> @@ -61,8 +61,9 @@ desired output in an integer and call the
> >>>>
> >>>> <constant>VIDIOC_S_OUTPUT</constant> ioctl with a pointer to this
> >>>> integer. Side effects are possible. For example outputs may support
> >>>> different video standards, so the driver may implicitly switch the
> >>>> current
> >>>>
> >>>> -standard. It is good practice to select an output before querying or
> >>>> -negotiating any other parameters.</para>
> >>>> +standard.
> >>>> +standard. Because of these possible side effects applications
> >>>> +must select an output before querying or negotiating any other
> >>>> parameters.</para>
> >>>
> >>> something seems to be wrong here.
> >>
> >> Hi Guennadi!
> >>
> >> What's wrong here? I've no idea what you mean.
> >
> >>>> +standard.
> >>>> +standard. Because of these possible side effects applications
> >
> > doesn't seem to make much sense?
>
> Are you referring to grammar?
Sorry, am I being blind? The above seems to produce
<quote>
different video standards, so the driver may implicitly switch the
current standard. standard. Because of these possible side effects
applications
</quote>
i.e., double "standard." - one too many.
Thanks
Guennadi
> I think that a comma is missing there [1]:
>
> Because of these possible side effects, applications
> must select an output before querying or negotiating any other
> parameters.
>
> IMO, reverting the order of the explanation clause is semantically better,
> as it bolds the need to select an output, and put the phrase at the
> cause-effect order:
>
> Applications must select an output before querying or negotiating
> any other parameters, because of these possible side effects.
>
> But this is really a matter of choice. In any case, except for the
> comma, that makes sense for me.
>
> [1] Rule 4 of http://grammar.ccc.commnet.edu/grammar/commas.htm
>
>
> Regards,
> Mauro
>
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [git:v4l-dvb/for_v3.3] [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT documentation
2012-01-17 10:54 ` Guennadi Liakhovetski
@ 2012-01-17 10:58 ` Hans Verkuil
0 siblings, 0 replies; 6+ messages in thread
From: Hans Verkuil @ 2012-01-17 10:58 UTC (permalink / raw)
To: Guennadi Liakhovetski
Cc: Mauro Carvalho Chehab, Linux Media Mailing List, Rupert Eibauer,
Hans Verkuil
On Tuesday 17 January 2012 11:54:10 Guennadi Liakhovetski wrote:
> On Tue, 17 Jan 2012, Mauro Carvalho Chehab wrote:
> > Em 17-01-2012 08:16, Guennadi Liakhovetski escreveu:
> > > Hi Hans
> > >
> > > On Tue, 17 Jan 2012, Hans Verkuil wrote:
> > >> On Monday 16 January 2012 23:16:31 Guennadi Liakhovetski wrote:
> > >>> On Mon, 16 Jan 2012, Mauro Carvalho Chehab wrote:
> > >>>> This is an automatic generated email to let you know that the
> > >>>> following patch were queued at the
> > >>>> http://git.linuxtv.org/media_tree.git tree:
> > >>>>
> > >>>> Subject: [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT
> > >>>> documentation Author: Hans Verkuil <hans.verkuil@cisco.com>
> > >>>> Date: Wed Jan 11 07:37:54 2012 -0300
> > >>>
> > >>> [snip]
> > >>>
> > >>>> diff --git a/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> > >>>> b/Documentation/DocBook/media/v4l/vidioc-g-output.xml index
> > >>>> fd45f1c..4533068 100644
> > >>>> --- a/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> > >>>> +++ b/Documentation/DocBook/media/v4l/vidioc-g-output.xml
> > >>>> @@ -61,8 +61,9 @@ desired output in an integer and call the
> > >>>>
> > >>>> <constant>VIDIOC_S_OUTPUT</constant> ioctl with a pointer to this
> > >>>> integer. Side effects are possible. For example outputs may support
> > >>>> different video standards, so the driver may implicitly switch the
> > >>>> current
> > >>>>
> > >>>> -standard. It is good practice to select an output before querying
> > >>>> or -negotiating any other parameters.</para>
> > >>>> +standard.
> > >>>> +standard. Because of these possible side effects applications
> > >>>> +must select an output before querying or negotiating any other
> > >>>> parameters.</para>
> > >>>
> > >>> something seems to be wrong here.
> > >>
> > >> Hi Guennadi!
> > >>
> > >> What's wrong here? I've no idea what you mean.
> > >>
> > >>>> +standard.
> > >>>> +standard. Because of these possible side effects applications
> > >
> > > doesn't seem to make much sense?
> >
> > Are you referring to grammar?
>
> Sorry, am I being blind? The above seems to produce
>
> <quote>
> different video standards, so the driver may implicitly switch the
> current standard. standard. Because of these possible side effects
> applications
> </quote>
>
> i.e., double "standard." - one too many.
LOL! You are right, and I completely missed that :-)
I'm the one that's blind.
Mauro, can you fix this?
Thanks,
Hans
>
> Thanks
> Guennadi
>
> > I think that a comma is missing there [1]:
> > Because of these possible side effects, applications
> > must select an output before querying or negotiating any other
> > parameters.
> >
> > IMO, reverting the order of the explanation clause is semantically
> > better, as it bolds the need to select an output, and put the phrase at
> > the
> >
> > cause-effect order:
> > Applications must select an output before querying or negotiating
> > any other parameters, because of these possible side effects.
> >
> > But this is really a matter of choice. In any case, except for the
> > comma, that makes sense for me.
> >
> > [1] Rule 4 of http://grammar.ccc.commnet.edu/grammar/commas.htm
> >
> >
> > Regards,
> > Mauro
>
> ---
> Guennadi Liakhovetski, Ph.D.
> Freelance Open-Source Software Developer
> http://www.open-technology.de/
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-01-17 10:58 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <E1Rmmdy-0002zt-5K@www.linuxtv.org>
2012-01-16 22:16 ` [git:v4l-dvb/for_v3.3] [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT documentation Guennadi Liakhovetski
2012-01-17 10:13 ` Hans Verkuil
2012-01-17 10:16 ` Guennadi Liakhovetski
2012-01-17 10:49 ` Mauro Carvalho Chehab
2012-01-17 10:54 ` Guennadi Liakhovetski
2012-01-17 10:58 ` Hans Verkuil
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox