Re: [Bugme-new] [Bug 12301] New: Fingerprint reader doesn't respod after the first use

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Alan,

On Tue, 2008-12-30 at 14:09 -0500, Alan Stern wrote:

> Timo, pay attention to this: There may still be another problem
> lurking in the library source code.  _libthinkfinger_usb_hello()
> behaves very badly: It sends a direct Set-Config request without using
> usb_set_configuration().  You should remove the entire first call to
> usb_control_msg() from that function.  I know the comment says "should
> not be relevant", but the comment is wrong.

OK, will do.  Thanks for the pointer.  This code was taken from Pavel's
initial attempts on getting this device supported.

> If it turns out that the Set-Config really is needed (it might indeed
> be necessary for Windows systems), it shouldn't be done this way.  
> Instead, usb_set_configuration() should be called from within
> _libthinkfinger_usb_init(), just before usb_claim_interface().

Will do.  Out of curiosity: Would this bug have never shown up if
libthinkfinger would have called this as you propose?

As libthinkfinger was merged into libfprint I'll care to fix it there as
well, if required.

Thanks,

   Timo

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux