On 02/04/2012 05:54 AM, Blue Swirl wrote:
On Fri, Feb 3, 2012 at 23:41, Anthony Liguori<aligu...@us.ibm.com> wrote:
Untested but simple enough.
Signed-off-by: Anthony Liguori<aligu...@us.ibm.com>
There are still some problems, though build does not fail anymore:
CC usb-bsd.o
/src/qemu/usb-bsd.c:417: warning: initialization from incompatible pointer type
/src/qemu/usb-bsd.c:401: warning: 'usb_host_class_initfn' defined but not used
---
usb-bsd.c | 11 ++++++-----
1 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/usb-bsd.c b/usb-bsd.c
index 2c6afc8..43fd1a1 100644
--- a/usb-bsd.c
+++ b/usb-bsd.c
@@ -410,15 +410,16 @@ static void usb_host_class_initfn(ObjectClass *klass,
void *data)
uc->handle_destroy = usb_host_handle_destroy;
}
-static struct DeviceInfo usb_host_dev_info = {
- .name = "usb-host",
- .size = sizeof(USBHostDevice),
- .class_init= usb_host_initfn,
+static TypeInfo usb_host_dev_info = {
+ .name = "usb-host",
+ .parent = TYPE_USB_DEVICE,
+ .instance_size = sizeof(USBHostDevice),
+ .class_init = usb_host_initfn,
That's a typo. It should be usb_host_class_initfn.
We don't do -Werror on BSD which is a I guess why the build wasn't failing
before.
Can you make the change and commit? I can also provide another patch if you'd
prefer that approach.
Regards,
Anthony Liguori
};
static void usb_host_register_devices(void)
{
- usb_qdev_register(&usb_host_dev_info, NULL, NULL);
+ type_register_static(&usb_host_dev_info);
}
device_init(usb_host_register_devices)
--
1.7.4.1