* 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