Re: [PATCH v3 09/14] media: platform: pxa_camera: add buffer sequencing

From	Robert Jarzmik <robert.jarzmik@free.fr>
Newsgroups	linux.kernel
Subject	Re: [PATCH v3 09/14] media: platform: pxa_camera: add buffer sequencing
Date	2016-08-15 16:50 +0200
Message-ID	<s6rnb-6Or-19@gated-at.bofh.it> (permalink)
References	<s3YyZ-5Ay-7@gated-at.bofh.it> <s3Yz0-5Ay-33@gated-at.bofh.it> <s6q7L-68h-11@gated-at.bofh.it> <s6qhs-6cc-19@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hans Verkuil <hverkuil@xs4all.nl> writes:

> On 08/15/2016 03:26 PM, Robert Jarzmik wrote:
>> Robert Jarzmik <robert.jarzmik@free.fr> writes:
>> I've seen no documentation on the rules applicable to this sequence number:
>>  - should it be reset if a "start streaming" operation occurs ?
>
> start_streaming is the recommended place for setting the counter to 0. It's what
> v4l2-compliance expects.
>
> It is not documented since 1) not every driver does this and 2) there may
> be cases where this behavior is not desired.
>
> That said, I have yet to see a driver where zeroing this in start_streaming
> is not appropriate.

Thanks for the information Hans, I'll modify this patch accordingly.

Cheers.

--
Robert

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH v3 09/14] media: platform: pxa_camera: add buffer sequencing Robert Jarzmik <robert.jarzmik@free.fr> - 2016-08-15 15:30 +0200
  Re: [PATCH v3 09/14] media: platform: pxa_camera: add buffer  sequencing Hans Verkuil <hverkuil@xs4all.nl> - 2016-08-15 15:40 +0200
    Re: [PATCH v3 09/14] media: platform: pxa_camera: add buffer sequencing Robert Jarzmik <robert.jarzmik@free.fr> - 2016-08-15 16:50 +0200

csiph-web