Skip to content

Commit

Permalink
pl050: QOM'ify pl050_keyboard and pl050_mouse
Browse files Browse the repository at this point in the history
Introduce an abstract type pl050 and let pl050_keyboar and pl050_mouse
inherit from it, using different instance_init functions.
Introduce a type constant and use QOM casts.

Signed-off-by: Andreas Färber <afaerber@suse.de>
  • Loading branch information
afaerber committed Jul 29, 2013
1 parent e607f25 commit 3e5dd36
Showing 1 changed file with 34 additions and 28 deletions.
62 changes: 34 additions & 28 deletions hw/input/pl050.c
Expand Up @@ -10,16 +10,20 @@
#include "hw/sysbus.h"
#include "hw/input/ps2.h"

#define TYPE_PL050 "pl050"
#define PL050(obj) OBJECT_CHECK(PL050State, (obj), TYPE_PL050)

typedef struct PL050State {
SysBusDevice busdev;
SysBusDevice parent_obj;

MemoryRegion iomem;
void *dev;
uint32_t cr;
uint32_t clk;
uint32_t last;
int pending;
qemu_irq irq;
int is_mouse;
bool is_mouse;
} PL050State;

static const VMStateDescription vmstate_pl050 = {
Expand Down Expand Up @@ -133,65 +137,67 @@ static const MemoryRegionOps pl050_ops = {
.endianness = DEVICE_NATIVE_ENDIAN,
};

static int pl050_init(SysBusDevice *dev, int is_mouse)
static int pl050_initfn(SysBusDevice *dev)
{
PL050State *s = FROM_SYSBUS(PL050State, dev);
PL050State *s = PL050(dev);

memory_region_init_io(&s->iomem, OBJECT(s), &pl050_ops, s, "pl050", 0x1000);
sysbus_init_mmio(dev, &s->iomem);
sysbus_init_irq(dev, &s->irq);
s->is_mouse = is_mouse;
if (s->is_mouse)
if (s->is_mouse) {
s->dev = ps2_mouse_init(pl050_update, s);
else
} else {
s->dev = ps2_kbd_init(pl050_update, s);
}
return 0;
}

static int pl050_init_keyboard(SysBusDevice *dev)
static void pl050_keyboard_init(Object *obj)
{
return pl050_init(dev, 0);
}
PL050State *s = PL050(obj);

static int pl050_init_mouse(SysBusDevice *dev)
{
return pl050_init(dev, 1);
s->is_mouse = false;
}

static void pl050_kbd_class_init(ObjectClass *klass, void *data)
static void pl050_mouse_init(Object *obj)
{
DeviceClass *dc = DEVICE_CLASS(klass);
SysBusDeviceClass *k = SYS_BUS_DEVICE_CLASS(klass);
PL050State *s = PL050(obj);

k->init = pl050_init_keyboard;
dc->vmsd = &vmstate_pl050;
s->is_mouse = true;
}

static const TypeInfo pl050_kbd_info = {
.name = "pl050_keyboard",
.parent = TYPE_SYS_BUS_DEVICE,
.instance_size = sizeof(PL050State),
.class_init = pl050_kbd_class_init,
.parent = TYPE_PL050,
.instance_init = pl050_keyboard_init,
};

static void pl050_mouse_class_init(ObjectClass *klass, void *data)
static const TypeInfo pl050_mouse_info = {
.name = "pl050_mouse",
.parent = TYPE_PL050,
.instance_init = pl050_mouse_init,
};

static void pl050_class_init(ObjectClass *oc, void *data)
{
DeviceClass *dc = DEVICE_CLASS(klass);
SysBusDeviceClass *k = SYS_BUS_DEVICE_CLASS(klass);
DeviceClass *dc = DEVICE_CLASS(oc);
SysBusDeviceClass *sdc = SYS_BUS_DEVICE_CLASS(oc);

k->init = pl050_init_mouse;
sdc->init = pl050_initfn;
dc->vmsd = &vmstate_pl050;
}

static const TypeInfo pl050_mouse_info = {
.name = "pl050_mouse",
static const TypeInfo pl050_type_info = {
.name = TYPE_PL050,
.parent = TYPE_SYS_BUS_DEVICE,
.instance_size = sizeof(PL050State),
.class_init = pl050_mouse_class_init,
.abstract = true,
.class_init = pl050_class_init,
};

static void pl050_register_types(void)
{
type_register_static(&pl050_type_info);
type_register_static(&pl050_kbd_info);
type_register_static(&pl050_mouse_info);
}
Expand Down

0 comments on commit 3e5dd36

Please sign in to comment.