<div dir="ltr">Ok, I'll try to find some time to prepare a patch against upstream in the coming days.</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jun 18, 2018 at 4:22 PM Hans de Goede <<a href="mailto:hdegoede@redhat.com">hdegoede@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
On 18-06-18 15:03, Bastien Nocera wrote:<br>
> On Mon, 2018-06-18 at 15:01 +0200, Hans de Goede wrote:<br>
>> Hi,<br>
>><br>
>> On 18-06-18 14:58, Igor Filatov wrote:<br>
>>> I'd like to but I'm worried that people will have bad experience<br>
>>> with these devices (esp. 96x96 ones) and blame libfprint. And I<br>
>>> still haven't had the chance to pull in latest upstream, so...<br>
>><br>
>> Well ATM people are just blindly adding new USB-ids without<br>
> <br>
> Who's blindly doing that?<br>
<br>
Ok, blindly is not the right term, sorry. What I meant to say<br>
is that they are being submitted with what seems to be<br>
less thorough testing then what has been done by the people<br>
working on the code which is now in Igor's branch.<br>
<br>
e.g. the code in Igor's branch reads back the reader<br>
firmware version and basis calibration behavior on that.<br>
<br>
IIRC some readers have the same USB-id but a different fw<br>
version and you need to talk slightly different to the device<br>
based on the firmware version. So even if a USB-id added to<br>
the current code works for the submitter it may not work<br>
for all devices with that USB-id.<br>
<br>
Which is why I believe it is best to get the changes from<br>
Igor's branch upstream even though these readers still<br>
could use some more work.<br>
<br>
>> even getting all the improvements which have been done, so<br>
>> although I agree that these devices need more work<br>
>> (specifically a better match algorithm more suited for<br>
>> low res devices), I still think it would be good to at least<br>
>> get what we have upstream.<br>
> <br>
> I agree that even if we choose to disable those smaller resolution<br>
> devices, we probably want the code to drive them to be upstreamed.<br>
<br>
Agreed.<br>
<br>
Regards,<br>
<br>
Hans<br>
<br>
_______________________________________________<br>
fprint mailing list<br>
<a href="mailto:fprint@lists.freedesktop.org" target="_blank">fprint@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/fprint" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/fprint</a><br>
</blockquote></div>