Re: [PATCH v2 1/1] vb2: Only requeue buffers immediately once streaming is started

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

 



Hi Hans,

Hans Verkuil wrote:
On 07/03/2015 03:28 PM, Sakari Ailus wrote:
Hi Hans,

Hans Verkuil wrote:
On 07/03/2015 02:47 PM, Sakari Ailus wrote:
Buffers can be returned back to videobuf2 in driver's streamon handler. In
this case vb2_buffer_done() with buffer state VB2_BUF_STATE_QUEUED will
cause the driver's buf_queue vb2 operation to be called, queueing the same
buffer again only to be returned to videobuf2 using vb2_buffer_done() and so
on.

Add a new buffer state VB2_BUF_STATE_REQUEUEING which, when used as the

It's spelled as requeuing (no e). The verb is 'to queue', but the -ing form is
queuing. Check the dictionary: http://dictionary.reference.com/browse/queuing

My dictionary disagrees with yours. :-)

http://dictionary.cambridge.org/dictionary/british/queue?q=queueing

$ git grep -i queueing|wc
     655    5660   54709
$ git grep -i queuing|wc
     650    5623   55249

That's not helpful either...

On the other hand:

$ git grep -i queuing drivers/media/|wc
      19     200    1846
$ git grep -i queueing drivers/media/|wc
       2      25     203

Within drivers/media there seems to be a clear preference for queuing :-)

The rest of the kernel apparently prefers "queueing" with a slight margin, if you don't consider V4L2. And who do you think might have added those lines containing "queuing" in V4L2? :-D

The matter was discussed long time ago and my understanding was in case of multiple possible spellings both should be allowed.

I'll post v3, replacing the if's at the end by a single switch. I think it's cleaner that way.

--
Kind regards,

Sakari Ailus
sakari.ailus@xxxxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[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