All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Devin Heitmueller <dheitmueller@kernellabs.com>
Cc: Andy Walls <awalls@radix.net>,
	Sakari Ailus <sakari.ailus@maxwell.research.nokia.com>,
	hverkuil@xs4all.nl,
	Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: Chroma gain configuration
Date: Mon, 22 Feb 2010 18:32:36 -0300	[thread overview]
Message-ID: <4B82F7F4.3090802@redhat.com> (raw)
In-Reply-To: <829197381002221317p42dda715lbd7ea1193c40d45c@mail.gmail.com>

Devin Heitmueller wrote:
> On Mon, Feb 22, 2010 at 8:58 AM, Mauro Carvalho Chehab
> <mchehab@redhat.com> wrote:
>>> Ok then.  I'll add the 15-20 lines of code which add the extended
>>> controls mechanism to the 7115, which just operates as a passthrough
>>> for the older control interface.
>> The better is to do the opposite: extended being the control interface and
>> the old calls as a passthrough, since the idea is to remove the old interface
>> after having all drivers converted.
> 
> I gave this a bit of thought, and I'm not sure what you are proposing
> is actually possible.  Because the extended controls provides a
> superset of the functionality of the older user controls interface, it
> is possible to create a extended control callback which just passes
> through the request (since any user control can be converted into a
> extended control).  However, you cannot convert the extended control
> results into the older user control format, since not all the
> information could be provided.
> 
> In fact, I would be in favor of taking the basic logic found in
> cx18_g_ext_ctrls(), and making that generic to the videodev interface,
> such that any driver which provides a user control interface but
> doesn't provide an extended control function will work if the calling
> application makes an extended control call.  This will allow userland
> applications to always use the extended controls API, even if the
> driver didn't explicitly add support for it.

That's exactly the idea: convert all driverst o use ext_ctrl's and let the
V4L2 core to handle the calls to the non-extended interface. 


-- 

Cheers,
Mauro

  reply	other threads:[~2010-02-22 21:33 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-22  4:07 Chroma gain configuration Devin Heitmueller
2010-02-22 11:04 ` Frej Drejhammar
2010-02-22 11:40 ` Andy Walls
2010-02-22 12:15   ` Mauro Carvalho Chehab
2010-02-22 13:10     ` Devin Heitmueller
2010-02-22 13:40       ` Mauro Carvalho Chehab
2010-02-22 13:47         ` Devin Heitmueller
2010-02-22 13:58           ` Mauro Carvalho Chehab
2010-02-22 21:17             ` Devin Heitmueller
2010-02-22 21:32               ` Mauro Carvalho Chehab [this message]
2010-02-22 21:38                 ` Devin Heitmueller
2010-02-22 21:47                   ` Hans Verkuil
2010-02-22 21:41               ` Hans Verkuil
2010-02-22 21:43                 ` Devin Heitmueller
2010-02-22 21:54                   ` Hans Verkuil
2010-02-22 22:00                     ` Devin Heitmueller
2010-02-23  7:53                       ` Hans Verkuil
2010-02-23 14:20                         ` Andy Walls
2010-02-23 14:41                           ` Hans Verkuil
2010-02-23 15:33                             ` Andy Walls
2010-02-23 15:41                               ` Andy Walls
2010-02-23 18:11                               ` My TV's standard video controls (Re: Chroma gain configuration) Andy Walls
2010-02-23 15:15                         ` Chroma gain configuration Devin Heitmueller
2010-02-25  4:39                         ` Devin Heitmueller
2010-02-22 23:50   ` 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=4B82F7F4.3090802@redhat.com \
    --to=mchehab@redhat.com \
    --cc=awalls@radix.net \
    --cc=dheitmueller@kernellabs.com \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=sakari.ailus@maxwell.research.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 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.