[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 02/19] usb-bus: convert USBDeviceClass init to r
From: |
Gonglei (Arei) |
Subject: |
Re: [Qemu-devel] [PATCH 02/19] usb-bus: convert USBDeviceClass init to realize |
Date: |
Thu, 18 Sep 2014 10:19:18 +0000 |
> From: Paolo Bonzini [mailto:address@hidden
> Sent: Thursday, September 18, 2014 6:08 PM
> Subject: Re: [PATCH 02/19] usb-bus: convert USBDeviceClass init to realize
>
> Il 18/09/2014 11:32, address@hidden ha scritto:
> > @@ -460,7 +460,7 @@ static void usb_serial_event(void *opaque, int event)
> > break;
> > case CHR_EVENT_OPENED:
> > if (!s->dev.attached) {
> > - usb_device_attach(&s->dev);
> > + usb_device_attach(&s->dev, NULL);
>
> After this patch, this stops printing the error if the attach fails.
> Please add qerror_report_err.
>
Good catch. Will fix it in v2.
Thanks a lot for your fast reviewing! :)
> > }
> > break;
> > case CHR_EVENT_CLOSED:
> > @@ -489,7 +489,7 @@ static int usb_serial_initfn(USBDevice *dev)
> > usb_serial_handle_reset(dev);
> >
> > if (s->cs->be_open && !dev->attached) {
> > - usb_device_attach(dev);
> > + usb_device_attach(dev, NULL);
>
> This too. Please add qerror_report_err here too and, in this case,
> please make patch 12 pass the errp instead of using qerror_report_err.
>
OK. Got it.
Best regards,
-Gonglei
> Paolo
>
> > }
> > return 0;
- [Qemu-devel] [PATCH 00/19] usb: convert device init to realize, arei.gonglei, 2014/09/18
- [Qemu-devel] [PATCH 02/19] usb-bus: convert USBDeviceClass init to realize, arei.gonglei, 2014/09/18
- [Qemu-devel] [PATCH 06/19] usb-hub: convert init to realize, arei.gonglei, 2014/09/18
- [Qemu-devel] [PATCH 01/19] usb-storage: fix possible memory leak and missing error message, arei.gonglei, 2014/09/18
- [Qemu-devel] [PATCH 04/19] libusb: convert init to realize, arei.gonglei, 2014/09/18
- [Qemu-devel] [PATCH 08/19] dev-storage: usring error_report instead of fprintf/printf, arei.gonglei, 2014/09/18