Re: [2.6.33.4 PATCH] V4L/uvcvideo: Add support for Suyin Corp. Lenovo Webcam

[Nils Radtke <Nils.Radtke@Think-Future.de>]

  Hi Laurent,

On ven 2010-06-25 @ 09-29-28 +0200, Laurent Pinchart wrote: 
# Hi Nils,
# 
# On Thursday 24 June 2010 20:19:41 Nils Radtke wrote:
# > On Wed 2010-06-23 @ 02-45-53PM +0200, Laurent Pinchart wrote:
# > # On Wednesday 23 June 2010 11:23:16 Nils Radtke wrote:
# > # > From: Nils Radtke <lkml@Think-Future.de>
# > # >
# > # > This patch adds support for the Suyin Corp. Lenovo Webcam.
# > # > lsusb: ID 064e:a102 Suyin Corp. Lenovo Webcam
# > # >
# > # > It is available as built-in webcam i.e. in ACER timeline 1810t
# > # > notebooks.
# > # >
# > # > The note in uvc_driver.c about Logitech cameras applies the same
# > # > to the Suyin web cam: it doesn't announce itself as UVC devices
# > # > but is compliant.
# > # >
# > # > Signed-off-by: Nils Radtke <lkml@Think-Future.de>
# > #
# > # Thanks for the patch. Could you please send me the output of lsusb -v for
# > your
# > 
# > Bus 002 Device 002: ID 064e:a102 Suyin Corp. Lenovo Webcam
# 
# [snip]
# 
# >     Interface Descriptor:
# >       bLength                 9
# >       bDescriptorType         4
# >       bInterfaceNumber        0
# >       bAlternateSetting       0
# >       bNumEndpoints           1
# >       bInterfaceClass        14 Video
# >       bInterfaceSubClass      1 Video Control
# >       bInterfaceProtocol      0
# >       iInterface              5 Webcam
# 
# This looks like a pretty standard UVC device. Are you sure your patch is 
# needed ?
Indeed. Hm, about whether the patch is needed.. What happened here was: the module
didn't get loaded w/o the patch. But then maybe something else made the change? 
After subsequent reboot the module did get loaded. Hm..

Isn't the vendor/product ID a point of reference that tells the kernel which module
to load when supported hw is found? That was the idea behind adding the ID and
submitting the patch. 

Thanks for clarification.

        Cheers, 

                  Nils