Re: Gspca USB driver zc3xx and STV06xx probe the same device ..

[Erik Andrén]

 2010/3/4 Hans de Goede <hdegoede@xxxxxxxxxx>:
> Hi,
>
> On 03/04/2010 12:36 PM, Erik Andrén wrote:
>>
>> 2010/3/4 Hans de Goede<hdegoede@xxxxxxxxxx>:
>>>
>>> Hi,
>>>
>>> On 03/03/2010 02:17 PM, Erik Andrén wrote:
>>>>
>>>> 2010/3/3 Gabriel C<nix.or.die@xxxxxxxxxxxxxx>:
>>>>>
>>>>> Hello,
>>>>>
>>>>> I own a QuickCam Messanger webcam.. I didn't used it in ages but today
>>>>> I plugged it in..
>>>>> ( Device 002: ID 046d:08da Logitech, Inc. QuickCam Messanger )
>>>>>
>>>>> Now zc3xx and stv06xx are starting both to probe the device .. In
>>>>> 2.6.33 that result in a not working webcam.
>>>>> ( rmmod both&&    modprobe zc3xx one seems to fix that )
>>>>>
>>>>> On current git head zc3xx works fine even when both are probing the
>>>>> device.
>>>>>
>>>>> Also I noticed stv06xx fails anyway for my webcam with this error:
>>>>> ....
>>>>>
>>>>> [  360.910243] STV06xx: Configuring camera
>>>>> [  360.910244] STV06xx: st6422 sensor detected
>>>>> [  360.910245] STV06xx: Initializing camera
>>>>> [  361.161948] STV06xx: probe of 6-1:1.0 failed with error -32
>>>>> [  361.161976] usbcore: registered new interface driver STV06xx
>>>>> [  361.161978] STV06xx: registered
>>>>> .....
>>>>>
>>>>> Next thing is stv06xx tells it is an st6422 sensor and does not work
>>>>> with it while zc3xx tells it is an HV7131R(c) sensor and works fine
>>>>> with it.
>>>>>
>>>>> What is right ?
>>>>
>>>> Hans,
>>>> As you added support for the st6422 sensor to the stv06xx subdriver I
>>>> imagine you best know what's going on.
>>>>
>>>
>>> I took the USB-ID in question from the out of tree v4l1 driver I was
>>> basing
>>> my
>>> st6422 work on. Looking at the other ID's (which are very close together)
>>> and
>>> combining that with this bug report, I think it is safe to say that the
>>> USB-ID
>>> in question should be removed from the stv06xx driver.
>>>
>>> Erik will you handle this, or shall I ?
>>>
>> Either way is fine by me.
>> I can try to do it tonight.
>>
>
> If you could take care of this that would be great!
>
> Thanks,
>
> Hans
>

Sorry for delaying this, real life came in the way.
I'm pasting in a patch that removes the usb id.
I'm also attaching it as an attachment as gmail probably will stomp on
the inline version.

Gabriel, could you please apply and test this patch and verify that it
works as intended, i. e. the stv06xx driver _doesn't_ bind to your
camera but the zx3xx driver instead does.
If it works as intended could you please reply to this mail with a
tested-by: your name <email> tag.

Best regards,
Erik