From: Stanimir Varbanov <stanimir.varbanov@linaro.org>
To: Guennadi Liakhovetski <g.liakhovetski@gmx.de>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>,
sakari.ailus@iki.fi, Hans Verkuil <hverkuil@xs4all.nl>
Subject: Re: metadata node
Date: Mon, 30 Jan 2017 19:26:19 +0200 [thread overview]
Message-ID: <b6c8267d-d18d-419e-bb2c-a21cfcbdd5bc@linaro.org> (raw)
In-Reply-To: <Pine.LNX.4.64.1701111007540.761@axis700.grange>
Hi Guennadi,
On 01/11/2017 11:42 AM, Guennadi Liakhovetski wrote:
> Hi Laurent,
>
> As you know, I'm working on a project, that involves streaming metadata,
> obtained from UVC payload headers to the userspace. Luckily, you have
> created "metadata node" patces a while ago. The core patch has also been
> acked by Hans, so, I decided it would be a safe enough bet to base my work
> on top of that.
>
> Your patch makes creating /dev/video* metadata device nodes possible, but
> it doesn't provide any means to associate metadata nodes to respective
> video (image) nodes. Another important aspect of using per-frame metadata
> is synchronisation between metadata and image buffers. The user is
> supposed to use buffer sequence numbers for that. That should be possible,
> but might be difficult if buffers lose synchronisation at some point. As a
> solution to the latter problem the use of requests with buffers for both
> nodes has been proposed, which should be possible once the request API is
> available.
>
> An alternative approach to metadata support, e.g. heterogeneous
> multi-plain buffers with one plain carrying image data and another plain
> carrying metadata would be possible. It could also have other uses. E.g.
> we have come across cameras, streaming buffers, containing multiple images
> (e.g. A and B). Possibly both images have supported fourcc format, but
> they cannot be re-used, because A+B now are transferred in a single
> buffer. Instead a new fourcc code has to be invented for such cases to
> describe A+B.
>
> As an important argument in favour of having a separate video node for
> metadata you named cases, when metadata has to be obtained before a
> complete image is available. Do you remember specifically what those cases
> are? Or have you got a link to that discussion?
>
> In any case, _if_ we do keep the current approach of separate /dev/video*
> nodes, we need a way to associate video and metadata nodes. Earlier I
> proposed using media controller links for that. In your implementation of
I don't think that media controller links is a good idea. This metadata
api could be used by mem2mem drivers which don't have media controller
links so we will need a generic v4l2 way to bound image buffer and its
metadata buffer.
--
regards,
Stan
next prev parent reply other threads:[~2017-01-30 17:26 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-11 9:42 metadata node Guennadi Liakhovetski
2017-01-30 17:26 ` Stanimir Varbanov [this message]
2017-02-02 18:35 ` Guennadi Liakhovetski
2017-02-10 12:09 ` Stanimir Varbanov
2017-02-11 20:38 ` Guennadi Liakhovetski
2017-02-11 22:07 ` 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=b6c8267d-d18d-419e-bb2c-a21cfcbdd5bc@linaro.org \
--to=stanimir.varbanov@linaro.org \
--cc=g.liakhovetski@gmx.de \
--cc=hverkuil@xs4all.nl \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=sakari.ailus@iki.fi \
/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;
as well as URLs for NNTP newsgroup(s).