All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sylwester Nawrocki <s.nawrocki@samsung.com>
To: Sakari Ailus <sakari.ailus@iki.fi>
Cc: "Sylwester Nawrocki" <snjw23@gmail.com>,
	linux-media <linux-media@vger.kernel.org>,
	"Hans Verkuil" <hverkuil@xs4all.nl>,
	"Rémi Denis-Courmont" <remi@remlab.net>
Subject: Re: [Query] V4L2 Integer (?) menu control
Date: Thu, 24 Nov 2011 10:34:29 +0100	[thread overview]
Message-ID: <4ECE0FA5.1040205@samsung.com> (raw)
In-Reply-To: <20111124085018.GF27136@valkosipuli.localdomain>

Thank you all for the comments.

On 11/24/2011 09:50 AM, Sakari Ailus wrote:
> Hi Sylwester,
> 
> There is not currently, but I have patches for it. The issue is that I need
> them myself but the driver I need them for isn't ready to be released yet.
> And as usual, I assume others than vivo is required to show they're really
> useful so I haven't sent them.

That's great news. Then I might not need to do all the work on my own;)

> 
> Good that you asked so we won't end up writing essentially the same code
> again. I'll try to send the patches today.

All right, there is no rush. I was just looking around how to support the
camera scene mode with m5mols sort of sensors. The scene mode is essentially
a compilation of several different parameters, for some of which there are
standard controls in V4L2 but for many there are not.

I've got a feeling the best way to handle this would be to create controls
for each single parameter and then do a batch set from user space, and keep
the scene mode mappings in user space. The only concern is there is a couple
of ISP-specific parameters involved with that scene mode thing. Perhaps they
just could be set initially to fixed values.

-- 
Regards,
Sylwester


  reply	other threads:[~2011-11-24  9:34 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-23 22:26 [Query] V4L2 Integer (?) menu control Sylwester Nawrocki
2011-11-24  6:24 ` Rémi Denis-Courmont
2011-11-24  9:17   ` Sylwester Nawrocki
2011-11-24  8:47 ` Hans Verkuil
2011-11-24  8:50 ` Sakari Ailus
2011-11-24  9:34   ` Sylwester Nawrocki [this message]
2011-11-24 20:57     ` Sakari Ailus
2011-11-24 23:53       ` Sylwester Nawrocki
2011-11-26  9:36         ` Sakari Ailus
2011-11-26 19:27           ` Sylwester Nawrocki
2011-12-12 21:08             ` 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=4ECE0FA5.1040205@samsung.com \
    --to=s.nawrocki@samsung.com \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=remi@remlab.net \
    --cc=sakari.ailus@iki.fi \
    --cc=snjw23@gmail.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.