From: Eino-Ville Talvala <talvala@stanford.edu>
To: Sakari Ailus <sakari.ailus@maxwell.research.nokia.com>
Cc: Hans Verkuil <hverkuil@xs4all.nl>,
Mauro Carvalho Chehab <mchehab@infradead.org>,
"Ivan T. Ivanov" <iivanov@mm-sol.com>,
"linux-media@vger.kernel.org" <linux-media@vger.kernel.org>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Zutshi Vimarsh <vimarsh.zutshi@nokia.com>,
Cohen David Abraham <david.cohen@nokia.com>,
Guru Raj <gururaj.nagendra@intel.com>,
Mike Krufky <mkrufky@linuxtv.org>,
Devin Heitmueller <dheitmueller@kernellabs.com>
Subject: Re: [RFC] Video events, version 2.2
Date: Sat, 14 Nov 2009 12:11:15 -0800 [thread overview]
Message-ID: <4AFF0EE3.8060300@stanford.edu> (raw)
In-Reply-To: <4AFDADFF.2030503@maxwell.research.nokia.com>
On 11/13/2009 11:05 AM, Sakari Ailus wrote:
> Eino-Ville Talvala wrote:
>> I think we have a use case for events that would require correlating
>> with frames, although I agree that the buffer index would be far
>> simpler to match with than a timestamp. The specific feature is
>> letting the application know exactly what sensor settings were used
>> with a given frame, which is essential for our slowly-developing
>> computational camera API, which will be changing sensor parameters on
>> nearly every frame boundary.
>>
>> I think one event is probably sufficient to encode the relevant
>> register values of our sensor. Would you expect there to be any
>> issue with having an event happen per frame?
>
> I do expect several events per frame from the AEWB, AF and histogram
> statistics and no problems. :-)
>
> But if I understand correctly, the registers are some kind of metadata
> associated to the frame? That perhaps includes exposure time, gain
> etc. The events interface would be good for this if the metadata fits
> to a single v4l2_event structure. A new ioctl could be an alternative,
> perhaps it could be a private ioctl first.
>
> This is more or less comparable to the H3A statistics IMO. So the user
> space gets an event and can query the H3A data.
>
> Associating events to a single frame is slightly troublesome since a
> succesful frame reception is only certain when it already has
> happened. There could be a metadata event and after that a receive
> buffer overflow that spoils the frame. In that case the field_count
> could be just incremented without dequeueing any buffers, though.
>
Right, all of the sensor settings that applied to that particular
frame. We're changing the sensor settings on nearly every frame, so
it's fairly key to keep track of them in some way, and events seem to be
far nicer solution than what we currently do (which involves abusing the
frame input field, as it was the fastest thing I saw to hack in).
Of course, the event queue and frame queue would have to be kept in
sync, or just let the app discard events that apply to frames it never
saw - as long as the event queue is a bit bigger than the frame queue, I
don't think there'd be a problem in practice.
Eino-Ville Talvala
Camera 2.0 Project
Stanford University
next prev parent reply other threads:[~2009-11-14 20:11 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-10-23 10:18 [RFC] Video events, version 2.2 Sakari Ailus
2009-10-23 12:59 ` Ivan T. Ivanov
2009-10-24 21:56 ` Sakari Ailus
2009-11-11 7:19 ` Hans Verkuil
2009-11-11 17:29 ` Sakari Ailus
2009-11-11 17:59 ` Hans Verkuil
2009-11-13 15:29 ` Mauro Carvalho Chehab
2009-11-13 16:00 ` Hans Verkuil
2009-11-13 17:30 ` Eino-Ville Talvala
2009-11-13 19:05 ` Sakari Ailus
2009-11-14 20:11 ` Eino-Ville Talvala [this message]
2009-11-13 17:44 ` Sakari Ailus
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=4AFF0EE3.8060300@stanford.edu \
--to=talvala@stanford.edu \
--cc=david.cohen@nokia.com \
--cc=dheitmueller@kernellabs.com \
--cc=gururaj.nagendra@intel.com \
--cc=hverkuil@xs4all.nl \
--cc=iivanov@mm-sol.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@infradead.org \
--cc=mkrufky@linuxtv.org \
--cc=sakari.ailus@maxwell.research.nokia.com \
--cc=vimarsh.zutshi@nokia.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox