public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Hans Verkuil <hverkuil@xs4all.nl>
To: Ezequiel Garcia <elezegarcia@gmail.com>
Cc: Sakari Ailus <sakari.ailus@iki.fi>,
	linux-media@vger.kernel.org, laurent.pinchart@ideasonboard.com
Subject: Re: [PATCH v3 1/1] v4l: drop v4l2_buffer.input and V4L2_BUF_FLAG_INPUT
Date: Mon, 28 May 2012 20:00:52 +0200	[thread overview]
Message-ID: <201205282000.52971.hverkuil@xs4all.nl> (raw)
In-Reply-To: <CALF0-+X80fLz5XrP3dJwoKH+MZ+2ykTfAt9Mk-6mS50693FTHQ@mail.gmail.com>

On Mon May 28 2012 19:25:41 Ezequiel Garcia wrote:
> On Mon, May 28, 2012 at 2:10 PM, Hans Verkuil <hverkuil@xs4all.nl> wrote:
> >>
> >> I may be missing something, but I don't see any relation between
> >> video buffer queue and selected input.
> >> (Perhaps this is OT and we should discuss this in another thread)
> >
> > Well, this particular API was intended to let the hardware switch from one input
> > to another automatically: e.g. the first frame is from input 1, the second from
> > input 2, etc. until it has gone through all inputs and goes back to input 1.
> >
> > This requires hardware support and if the stk1160 can't do that, then you can
> > forget about all this.
> 
> I did some tests earlier this morning and apparently the input
> switching is done
> through some stk1160 GPIO port setting.
> However, I'll have to figure this out reading easycap staging driver
> and trying different values over and over.
> Plus, saa7115 driver supports "s_routing" function, but couldn't make
> it work: it looked like saa7115 input
> setting was fixed and stk1160 handles the input switching.
> 
> Also, I'm not sure if saa7115 "s_stream" to enable/disable capture is
> also working.
> Sounds a bit weird to me.
> 
> One last query: I found *no* way to distinguish between one device
> (one composite input)
> and the other (four composite input). What do you suggest? Just
> implement all of them always?

If you can't tell them apart, then that would be your only option. You
might also add a 'model' module option so users can tell the driver which model
it is.

Another might be (I don't know if that will work) to try all inputs and see
if you can detect a difference in the saa7115: however, I suspect you can't
tell the difference between 'I have only one input' and 'I have four but I
haven't hooked up anything to my inputs'.

I *hate* it when they don't bother to change the USB IDs.
 
> Looks like I'm on the right path anyway :)

Yup!

Regards,

	Hans

      reply	other threads:[~2012-05-28 18:00 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-30 12:40 [PATCH 1/1] v4l: drop v4l2_buffer.input and V4L2_BUF_FLAG_INPUT Sakari Ailus
2012-04-30 12:48 ` Laurent Pinchart
2012-04-30 13:04   ` Sakari Ailus
2012-04-30 13:34     ` Sakari Ailus
2012-04-30 13:48       ` Hans Verkuil
2012-04-30 14:06         ` Sakari Ailus
2012-04-30 14:15           ` Hans Verkuil
2012-04-30 14:27             ` Laurent Pinchart
2012-05-02 12:40               ` [PATCH v3 " Sakari Ailus
2012-05-28 10:27                 ` Hans Verkuil
2012-05-28 11:52                   ` Ezequiel Garcia
2012-05-28 16:29                     ` Ezequiel Garcia
2012-05-28 17:10                       ` Hans Verkuil
2012-05-28 17:25                         ` Ezequiel Garcia
2012-05-28 18:00                           ` Hans Verkuil [this message]

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=201205282000.52971.hverkuil@xs4all.nl \
    --to=hverkuil@xs4all.nl \
    --cc=elezegarcia@gmail.com \
    --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