Hi Jérôme,

On Friday 29 May 2009 12:30:36 Jérôme Pouiller wrote:
> Hi Laurent,
>
> On Thursday 28 May 2009 15:59:58 Laurent Pinchart wrote:
> > Hi Jérôme,
> >
> > On Thursday 28 May 2009 15:21:52 Jérôme Pouiller wrote:
> > > On Wednesday 27 May 2009 17:11:03 Laurent Pinchart wrote:
> > > > On Tuesday 26 May 2009 17:49:14 Jérôme Pouiller wrote:
>
> [...]
>
> > > > Could you try to read the value (VIDIOC_G_CTRL) of the hue
> > > > control ?
> > >
> > > It   return  EIO   and   uvcvideo  driver   print   same  message
> > > (see test-BisonCam.c and test-output).
> >
> > I expected the uvcvideo driver to print a slightly different message
> > (129 instead of 135). I'm pretty sure it did, you might have missed
> > that.
>
> Yes, you're right.
>
> [...]
>
> > > VIDIOC_QUERYCTRL  success only  if  I remove  GET_DEF, GET_MIN,
> > > GET_MAX and  GET_RES flags.  In  all  other cases,  my  test
> > > program give  same output as  in test-output. (Except:  if I remove
> > > UVC_CONTROL_GET* flags, VIDIOC_G_CTRL return EINVAL).
> >
> > It seems that the camera doesn't support the hue control at all, even
> > though it reports it does.
> >
> > The proper way to fix this is to blacklist the hue control for your
> > model. Setting the UVC_QUIRK_PRUNE_CONTROLS and modifying
> > uvc_ctrl_prune_controls() should do the job. Could you please try the
> > attached patch ?
>
> Your patch apply right on 2.6.29. Nevertheless, with 2.6.28, I had to
> do some changes in order to use it. I also add UVC_QUIRK_PRUNE_CONTROLS
> flag to 5986:0241 in uvc_ids to automatically apply the quirk.
>
> Else, this patch works very well. Thank you.

You're welcome. I've reworked the patch a bit and I'll submit it soon.

Could you please tell me in what monitor/laptop/whatever the 5986:0241 camera 
can be found ? I'll then update the supported devices list.

Best regards,

Laurent Pinchart

_______________________________________________
Linux-uvc-devel mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/linux-uvc-devel

Reply via email to