On Tue, 2 Jul 2019 13:58:30 +0200 Marek Vasut <ma...@denx.de> wrote: > On 7/1/19 5:56 PM, Michal Suchanek wrote: > > Causes unbound key repeat on error otherwise. > > > > Signed-off-by: Michal Suchanek <msucha...@suse.de> > > --- > > common/usb_kbd.c | 7 +++---- > > 1 file changed, 3 insertions(+), 4 deletions(-) > > > > diff --git a/common/usb_kbd.c b/common/usb_kbd.c > > index cc99c6be0720..948f9fd68490 100644 > > --- a/common/usb_kbd.c > > +++ b/common/usb_kbd.c > > @@ -339,10 +339,9 @@ static inline void usb_kbd_poll_for_event(struct > > usb_device *dev) > > struct usb_kbd_pdata *data = dev->privptr; > > > > /* Submit a interrupt transfer request */ > > - usb_submit_int_msg(dev, data->intpipe, &data->new[0], data->intpktsize, > > - data->intinterval); > > - > > - usb_kbd_irq_worker(dev); > > + if (!usb_submit_int_msg(dev, data->intpipe, &data->new[0], > > Shouldn't you propagate return value from this function ? It can return > ENOTSUPP. >
If it did then probing keyboard would fail and we would not get here. Thanks Michal _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot