[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Fix usage of USB_DEV_FLAG_IS_HOST flag.
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH] Fix usage of USB_DEV_FLAG_IS_HOST flag. |
Date: |
Mon, 13 May 2013 17:51:54 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130329 Thunderbird/17.0.5 |
On 05/13/13 01:19, Michael Marineau wrote:
> USB_DEV_FLAG_IS_HOST is the bit number, not value. Booting with a
> "Fitbit Base Station" USB dongle was triggering this assert.
>
> Signed-off-by: Michael Marineau <address@hidden>
> ---
> hw/usb/core.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/usb/core.c b/hw/usb/core.c
> index 15a150a..05948ca 100644
> --- a/hw/usb/core.c
> +++ b/hw/usb/core.c
> @@ -410,7 +410,7 @@ void usb_handle_packet(USBDevice *dev, USBPacket *p)
> assert(p->ep->type != USB_ENDPOINT_XFER_ISOC);
> /* using async for interrupt packets breaks migration */
> assert(p->ep->type != USB_ENDPOINT_XFER_INT ||
> - (dev->flags & USB_DEV_FLAG_IS_HOST));
> + (dev->flags & (1 << USB_DEV_FLAG_IS_HOST)));
> usb_packet_set_state(p, USB_PACKET_ASYNC);
> QTAILQ_INSERT_TAIL(&p->ep->queue, p, queue);
> } else if (p->status == USB_RET_ADD_TO_QUEUE) {
Looks good.
Anthony? Can you pick it for 1.5 directly? I don't have any other
pending usb bits for 1.5 at the moment.
Reviewed-by: Gerd Hoffmann <address@hidden>
cheers,
Gerd