All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans Verkuil <hverkuil@xs4all.nl>
To: "Rémi Denis-Courmont" <remi@remlab.net>
Cc: linux-media@vger.kernel.org
Subject: Re: [RFC] Timestamps and V4L2
Date: Tue, 25 Sep 2012 08:50:35 +0200	[thread overview]
Message-ID: <201209250850.35473.hverkuil@xs4all.nl> (raw)
In-Reply-To: <201209242311.51003@leon.remlab.net>

On Mon September 24 2012 22:11:50 Rémi Denis-Courmont wrote:
> Le dimanche 23 septembre 2012 14:43:42, Sakari Ailus a écrit :
> > > I think I like this idea best, it's relatively simple (even with adding
> > > support for reporting flags in VIDIOC_QUERYBUF) for the purpose.
> > > 
> > > If we ever need the clock selection API I would vote for an IOCTL.
> > > The controls API is a bad choice for something such fundamental as
> > > type of clock for buffer timestamping IMHO. Let's stop making the
> > > controls API a dumping ground for almost everything in V4L2! ;)
> > 
> > Why would the control API be worse than an IOCTL for choosing the type of
> > the timestamp? The control API after all has functionality for exactly for
> > this: this is an obvious menu control.
> > 
> > What comes to the nature of things that can be configured using controls
> > and what can be done using IOCTLs I see no difference. It's just a
> > mechanism. That's what traditional Unix APIs do in general: provide
> > mechanism, not a policy.
> 
> Seriously? Timestamp is _not_ a controllable hardware feature like brightness 
> or flash. Controls are meant to build user interface controls for interaction 
> with the user. Timestamp is _not_ something the user should control directly. 
> The application should figure out what it gets and what it needs.

Exactly. I agree completely.

Regards,

	Hans

> 
> Or why do you use STREAMON/STREAMOFF instead of a STREAM boolean control, eh?
> 
> 

  reply	other threads:[~2012-09-25  6:51 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-20 20:21 [RFC] Timestamps and V4L2 Sakari Ailus
2012-09-20 21:08 ` Rémi Denis-Courmont
2012-09-21  8:47 ` Christian Gmeiner
2012-09-21  9:33 ` Hans Verkuil
2012-09-22 12:38   ` Sakari Ailus
2012-09-22 17:12     ` Sylwester Nawrocki
2012-09-22 20:28       ` Daniel Glöckner
2012-09-23 18:40         ` Sylwester Nawrocki
2012-09-25  0:35           ` Laurent Pinchart
     [not found]             ` <5061DAE3.2080808@samsung.com>
2012-09-25 17:17               ` Kamil Debski
2012-09-26 22:30             ` Sylwester Nawrocki
2012-09-27 10:41               ` Laurent Pinchart
2012-09-23 11:43       ` Sakari Ailus
2012-09-24 20:11         ` Rémi Denis-Courmont
2012-09-25  6:50           ` Hans Verkuil [this message]
2012-09-25  0:34       ` Laurent Pinchart
2012-09-25 22:48         ` Sylwester Nawrocki
2012-09-23  9:18     ` Hans Verkuil
2012-09-23 13:07       ` Sakari Ailus
2012-09-24  8:30         ` Hans Verkuil
2012-09-25  0:21       ` Laurent Pinchart
2012-09-24 23:42   ` Laurent Pinchart
2012-09-25  0:00   ` Laurent Pinchart
2012-09-25  6:47     ` Hans Verkuil
2012-09-25 10:48       ` Laurent Pinchart
2012-09-25 10:54         ` Hans Verkuil
2012-09-25 11:09           ` Laurent Pinchart
2012-09-25 20:12           ` Sakari Ailus
2012-09-26  9:13             ` Laurent Pinchart
2012-09-26 19:17               ` Sakari Ailus
2012-09-27 10:55                 ` Laurent Pinchart
2012-09-25 20:05       ` Sakari Ailus
2012-10-15 16:05 ` Sakari Ailus
2012-10-15 18:45   ` Laurent Pinchart
2012-10-15 18:53     ` Chris MacGregor
2012-10-15 19:59       ` Sakari Ailus
2012-10-15 20:10         ` Rémi Denis-Courmont
2012-10-16  1:25         ` Chris MacGregor
2012-10-25  0:47           ` Laurent Pinchart
2012-10-16  6:13     ` Hans Verkuil

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201209250850.35473.hverkuil@xs4all.nl \
    --to=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=remi@remlab.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.