From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx1.redhat.com ([209.132.183.28]:47486 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752918Ab2AQKuK (ORCPT ); Tue, 17 Jan 2012 05:50:10 -0500 Message-ID: <4F155252.2060705@redhat.com> Date: Tue, 17 Jan 2012 08:49:54 -0200 From: Mauro Carvalho Chehab MIME-Version: 1.0 To: Guennadi Liakhovetski CC: Hans Verkuil , Linux Media Mailing List , Rupert Eibauer , Hans Verkuil Subject: Re: [git:v4l-dvb/for_v3.3] [media] V4L2 Spec: improve the G/S_INPUT/OUTPUT documentation References: <201201171113.14927.hansverk@cisco.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: 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 >>>> 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 >>>> >>>> VIDIOC_S_OUTPUT 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. >>>> +standard. >>>> +standard. Because of these possible side effects applications >>>> +must select an output before querying or negotiating any other >>>> parameters. >>> >>> 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