commit e22cf21efdc5f25fb9ba136cba7acb71aa379ea7
author Blue Swirl <blauwirbel@gmail.com> Sat Feb 05 19:37:40 2011 +0000
committer Blue Swirl <blauwirbel@gmail.com> Sun Feb 20 09:32:50 2011 +0000
tree 779320539ee9d91fbaf98acfce00a6aace73232f
parent cd1b8a8b0d5951fb3a575a13bfd3aa3f7018ef1f

serial: refactor device creation

Turn serial_init into an inline function.

Signed-off-by: Blue Swirl <blauwirbel@gmail.com>

hw/pc.h

diff --git a/hw/pc.h b/hw/pc.h
index 50d9943..09f63f0 100644
--- a/hw/pc.h
+++ b/hw/pc.h
@@ -17,7 +17,19 @@
                              qemu_irq irq, int baudbase,
                              CharDriverState *chr, int ioregister,
                              int be);
-SerialState *serial_isa_init(int index, CharDriverState *chr);
+static inline bool serial_isa_init(int index, CharDriverState *chr)
+{
+    ISADevice *dev;
+
+    dev = isa_create("isa-serial");
+    qdev_prop_set_uint32(&dev->qdev, "index", index);
+    qdev_prop_set_chr(&dev->qdev, "chardev", chr);
+    if (qdev_init(&dev->qdev) < 0) {
+        return false;
+    }
+    return true;
+}
+
 void serial_set_frequency(SerialState *s, uint32_t frequency);
 
 /* parallel.c */

hw/serial.c

diff --git a/hw/serial.c b/hw/serial.c
index 2c4af61..879237f 100644
--- a/hw/serial.c
+++ b/hw/serial.c
@@ -782,18 +782,6 @@
     return 0;
 }
 
-SerialState *serial_isa_init(int index, CharDriverState *chr)
-{
-    ISADevice *dev;
-
-    dev = isa_create("isa-serial");
-    qdev_prop_set_uint32(&dev->qdev, "index", index);
-    qdev_prop_set_chr(&dev->qdev, "chardev", chr);
-    if (qdev_init(&dev->qdev) < 0)
-        return NULL;
-    return &DO_UPCAST(ISASerialState, dev, dev)->state;
-}
-
 static const VMStateDescription vmstate_isa_serial = {
     .name = "serial",
     .version_id = 3,