Linux Media Controller development
 help / color / mirror / Atom feed
From: Hans Verkuil <hverkuil@xs4all.nl>
To: Mauro Carvalho Chehab <m.chehab@samsung.com>
Cc: Gregor Jasny <gjasny@googlemail.com>,
	Linux Media Mailing List <linux-media@vger.kernel.org>,
	Hans de Goede <hdegoede@redhat.com>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Subject: Re: Time for v4l-utils 1.2 release?
Date: Mon, 23 Jun 2014 10:22:18 +0200	[thread overview]
Message-ID: <53A7E3BA.1080100@xs4all.nl> (raw)
In-Reply-To: <20140621075348.50b8a47d.m.chehab@samsung.com>

On 06/21/2014 12:53 PM, Mauro Carvalho Chehab wrote:
> Em Sat, 21 Jun 2014 08:07:57 +0200
> Hans Verkuil <hverkuil@xs4all.nl> escreveu:
> 
>> On 06/21/2014 12:29 AM, Mauro Carvalho Chehab wrote:
>>> Em Sat, 21 Jun 2014 00:07:19 +0200
>>> Hans Verkuil <hverkuil@xs4all.nl> escreveu:
>>>
>>>> On 06/20/2014 10:31 PM, Gregor Jasny wrote:
>>>>> Hello,
>>>>>
>>>>> It's been 11 months since the 1.0.0 release. What do you think about
>>>>> releasing HEAD? Do you have any pending commits?
>>>>
>>>> I've got two patches from Laurent pending that ensure that the 'installed
>>>> kernel headers' are used. I plan on processing those on Monday. After that
>>>> I think it's OK to do a release.
>>>>
>>>> Mauro, did you look at my email where I suggest to remove three apps from
>>>> contrib? If you agree with that, then I can do that Monday as well.
>>>
>>> Well, I don't remember about such email, nor I was able to find on a quick
>>> look.
>>
>> https://www.mail-archive.com/linux-media@vger.kernel.org/msg76120.html
>>
>> Marked with ATTN as well!
> 
> Well, from my side, feel free to drop those 3 utilities. If you drop v4lgrab,
> you'll need to check the DocBook Makefile scripts, as it used to have some
> automation to include it at the media DocBook. 
> 
> I think that this was removed in the past, but it doesn't hurt to
> double-check.
> 
>>>
>>> What apps are you planning to remove?
>>>
>>> Btw, I think it could be a good idea to be able to install some of those
>>> stuff under contrib to a separate package. I had to do a quick hack
>>> in order to install v4l2grab on a Tizen package, in order to be able to
>>> test a card there (as was needing to do some tests via CLI).
>>
>> What does v4l2grab offer that v4l2-ctl doesn't? I would be much more inclined
>> to remove v4l2grab.
> 
> I never used v4l2-ctl for streaming (didn't even know/remember) that it was
> capable of doing that ;) 

It was added about a year ago or something like that.

> Looking at --help-streaming, though, one thing that it is not clear there
> is what's the format of the output, when --stream-to= is used. 

Completely raw output.

> Btw, I think that one big miss on v4l2-ctl is the lack of a man page witch
> would have an EXAMPLES section explaining things like that.

Absolutely. With a bit of luck I might have time for that fairly soon.

> One of the advantages of v4l2grab is that it takes per-frame snapshots, instead
> of writing a stream file. Those snapshots help to identify, for example, if
> there are interlacing issues on a frame, or if some frames have some other
> problems.

You can dump just a single frame as well with v4l2-ctl (--stream-count=1).

> Also, v4l2grab is a good example of how to use libv4l, and it is enclosed
> at DocBook (not sure if we're using automation to allow including
> the latest version of it):
> 	http://linuxtv.org/downloads/v4l-dvb-apis/v4l2grab-example.html

Isn't capture.c.xml much more useful as example code? I have to admit, I
didn't know v4l2grab.c was used as an example.

> 
> So, I don't think we should remove it.

No problem.

Regards,

	Hans


  reply	other threads:[~2014-06-23  8:22 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-20 20:31 Time for v4l-utils 1.2 release? Gregor Jasny
2014-06-20 20:44 ` Mauro Carvalho Chehab
2014-06-20 22:07 ` Hans Verkuil
2014-06-20 22:29   ` Mauro Carvalho Chehab
2014-06-21  6:07     ` Hans Verkuil
2014-06-21 10:53       ` Mauro Carvalho Chehab
2014-06-23  8:22         ` Hans Verkuil [this message]
2014-06-23 10:58           ` Mauro Carvalho Chehab
2014-06-23 11:20             ` Laurent Pinchart
2014-06-23  8:17   ` 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=53A7E3BA.1080100@xs4all.nl \
    --to=hverkuil@xs4all.nl \
    --cc=gjasny@googlemail.com \
    --cc=hdegoede@redhat.com \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=m.chehab@samsung.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