public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Adam Baker <linux@baker-net.org.uk>
To: Hans de Goede <j.w.r.degoede@hhs.nl>
Cc: Linux and Kernel Video <video4linux-list@redhat.com>,
	SPCA50x Linux Device Driver Development
	<spca50x-devs@lists.sourceforge.net>,
	Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: libv4l release: 0.5.97: the whitebalance release!
Date: Thu, 16 Apr 2009 21:46:59 +0100	[thread overview]
Message-ID: <200904162146.59742.linux@baker-net.org.uk> (raw)
In-Reply-To: <49E66787.2080301@hhs.nl>

On Thursday 16 Apr 2009, Hans de Goede wrote:
> On 04/16/2009 12:26 AM, Adam Baker wrote:
> > On Wednesday 15 Apr 2009, Hans de Goede wrote:
> >> Currently only whitebalancing is enabled and only on Pixarts (pac)
> >> webcams (which benefit tremendously from this). To test this with other
> >> webcams (after instaling this release) do:
> >>
> >> export LIBV4LCONTROL_CONTROLS=15
> >> LD_PRELOAD=/usr/lib/libv4l/v4l2convert.so v4l2ucp&
> >
> > Strangely while those instructions give me a whitebalance control for the
> > sq905 based camera I can't get it to appear for a pac207 based camera
> > regardless of whether LIBV4LCONTROL_CONTROLS is set.
>
> Thats weird, there is a small bug in the handling of pac207
> cams with usb id 093a:2476 causing libv4l to not automatically
> enable whitebalancing (and the control) for cams with that id,
> but if you have LIBV4LCONTROL_CONTROLS set (exported!) both
> when loading v4l2ucp (you must preload v4l2convert.so!) and
> when loading your viewer, then it should work.
>

I've tested it by plugging in the sq905 camera, verifying the whitebablance 
control is present and working, unplugging the sq905 and plugging in the 
pac207 and using up arrow to restart v4l2ucp and svv so I think I've 
eliminated most finger trouble possibilities. The pac207 is id 093a:2460 so 
not the problem id. I'll have to investigate more thoroughly later.

> >> Notice the whitebalance and normalize checkboxes in v4l2ucp,
> >> as well as low and high limits for normalize.
> >>
> >> Now start your favorite webcam viewing app and play around with the
> >> 2 checkboxes. Note normalize seems to be useless in most cases. If
> >> whitebalancing makes a *strongly noticable* difference for your webcam
> >> please mail me info about your cam (the usb id), then I can add it to
> >> the list of cams which will have the whitebalancing algorithm (and the
> >> v4l2 control to enable/disable it) enabled by default.
> >
> > The whitebalance works really well with the sq905 (ID 2770:9120). Without
> > it the image is very red when using artificial light, with auto
> > whitebalance the colours look slightly desaturated but otherwise fine.
>
> Good to hear it helps for the sq905 too, is that the only know id for sq905
> cams ?

Yes

Adam



  reply	other threads:[~2009-04-16 20:47 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-15 12:36 libv4l release: 0.5.97: the whitebalance release! Hans de Goede
2009-04-15 22:26 ` Adam Baker
2009-04-15 23:02   ` Hans de Goede
2009-04-16 20:46     ` Adam Baker [this message]
2009-04-17  8:54       ` Hans de Goede
2009-04-17 19:27         ` Erik Andrén
2009-04-18 11:29           ` Hans de Goede
2009-04-18 14:40             ` Erik Andrén
2009-04-19  7:45               ` Hans de Goede
2009-04-19 19:20                 ` Erik Andrén
2009-04-19 19:49                   ` Hans de Goede
2009-04-20  4:43                     ` Erik Andrén
2009-04-20  7:43                       ` Hans de Goede
2009-04-20  7:54                         ` Erik Andrén
2009-04-18 19:44             ` Adam Baker
2009-04-18 22:45               ` [PATCH][libv4l] Support V4L2_CTRL_FLAG_NEXT_CTRL for fake controls Adam Baker
2009-05-17 13:49                 ` Hans de Goede
2009-04-17 22:13         ` libv4l release: 0.5.97: the whitebalance release! Adam Baker
2009-04-18 11:27           ` Hans de Goede
2009-04-16  6:16 ` Gilles Gigan
2009-04-16  9:25   ` Hans de Goede
2009-04-16  9:38     ` Gilles Gigan
2009-04-17 19:31 ` Erik Andrén

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=200904162146.59742.linux@baker-net.org.uk \
    --to=linux@baker-net.org.uk \
    --cc=j.w.r.degoede@hhs.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=spca50x-devs@lists.sourceforge.net \
    --cc=video4linux-list@redhat.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