Re: [SeaBIOS] [Qemu-devel] [PATCH] Fix qemu mouse Set_Protocol behavior

[Anthony Liguori]

On 05/01/2010 09:38 PM, Kevin O'Connor wrote:
> The QEMU USB mouse claims to support the "boot" protocol
> (bInterfaceSubClass is 1).  However, the mouse rejects the
> Set_Protocol command.
>
> The qemu mouse does support the "boot" protocol specification, so a
> simple fix is to just enable the Set_Portocol request.
>
> -Kevin

Could you included a Signed-of-by: and resubmit both patches?

Thanks,

Anthony Liguori
> --- a/hw/usb-hid.c
> +++ b/hw/usb-hid.c
> @@ -790,13 +790,13 @@ static int usb_hid_handle_control(USBDevice *dev, int request, int value,
>               goto fail;
>           break;
>       case GET_PROTOCOL:
>    if (s->kind != USB_KEYBOARD)
>    if (s->kind != USB_KEYBOARD&&  s->kind != USB_MOUSE)
>         goto fail;
>     ret = 1;
>     data[0] = s->protocol;
>     break;
> case SET_PROTOCOL:
>    if (s->kind != USB_KEYBOARD)
>    if (s->kind != USB_KEYBOARD&&  s->kind != USB_MOUSE)
>         goto fail;
>     ret = 0;
>     s->protocol = value;