[fprint] [PATCH] upeke2: Add one more stage to the enrollment

[Vasily Khoruzhick]

On Thu, Aug 15, 2013 at 12:08 PM, Ricardo Ribalda Delgado <
ricardo.ribalda at gmail.com> wrote:

> And what about in fp_enroll_finger_img:
>
> if n_stages < max_stages -> all OK :)
> else if n_stages < max_stages *2 -> show warning
> else error
>

That will break imaging drivers.



> ?
>
> On Thu, Aug 15, 2013 at 10:39 AM, Vasily Khoruzhick <anarsoul at gmail.com>
> wrote:
> > On Wed, Aug 14, 2013 at 8:49 PM, Ricardo Ribalda Delgado
> > <ricardo.ribalda at gmail.com> wrote:
> >>
> >> Hi Vasily
> >
> >
> > Hi Ricardo,
> >
> >>
> >> Maybe it is a different firmware on the device. I have tried on two
> >> different hardware, the sensor on the thinkpad w520, and a usb device
> >> that was around my office.
> >>
> >> I am not the only one with that problem though
> >> http://lists.freedesktop.org/archives/fprint/2012-January/000085.html
> >>
> >> Does the patch that I sent break the library for you? I think the only
> >> counterpart is one extra enrolling stage on error. Maybe it is a good
> >> idea to apply it if it supports more hardware than before :)
> >
> >
> > Well, with patch applied it claims that there's 6 stages of enrollment,
> but
> > it completes enrollment on 5th.
> > Will test tonight with another device just to check that it doesn't break
> > anything.
> >
> >
> >>
> >> I will try your branch as soon as I get hold of some hardware.
> >>
> >>
> >> Thanks!
> >
> >
> > Regards
> > Vasily
>
>
>
> --
> Ricardo Ribalda
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/fprint/attachments/20130815/fc45a66f/attachment.html>