From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:57233) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V3t1f-0004rT-M1 for qemu-devel@nongnu.org; Mon, 29 Jul 2013 15:22:47 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1V3t1T-0002nv-Tu for qemu-devel@nongnu.org; Mon, 29 Jul 2013 15:22:35 -0400 Received: from cantor2.suse.de ([195.135.220.15]:53492 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V3t1T-0002nq-LQ for qemu-devel@nongnu.org; Mon, 29 Jul 2013 15:22:23 -0400 From: =?UTF-8?q?Andreas=20F=C3=A4rber?= Date: Mon, 29 Jul 2013 21:19:44 +0200 Message-Id: <1375125630-24869-128-git-send-email-afaerber@suse.de> In-Reply-To: <1375125630-24869-1-git-send-email-afaerber@suse.de> References: <1375125630-24869-1-git-send-email-afaerber@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] [PULL 127/173] xgmac: QOM cast cleanup List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: =?UTF-8?q?Andreas=20F=C3=A4rber?= , Mark Langsdorf Introduce type constant and use QOM casts and typedef consistently. Signed-off-by: Andreas F=C3=A4rber --- hw/net/xgmac.c | 46 ++++++++++++++++++++++++++-------------------- 1 file changed, 26 insertions(+), 20 deletions(-) diff --git a/hw/net/xgmac.c b/hw/net/xgmac.c index 997a5b5..9384fa0 100644 --- a/hw/net/xgmac.c +++ b/hw/net/xgmac.c @@ -135,8 +135,12 @@ typedef struct RxTxStats { uint64_t rx_mcast; } RxTxStats; =20 +#define TYPE_XGMAC "xgmac" +#define XGMAC(obj) OBJECT_CHECK(XgmacState, (obj), TYPE_XGMAC) + typedef struct XgmacState { - SysBusDevice busdev; + SysBusDevice parent_obj; + MemoryRegion iomem; qemu_irq sbd_irq; qemu_irq pmt_irq; @@ -173,14 +177,14 @@ static const VMStateDescription vmstate_xgmac =3D { } }; =20 -static void xgmac_read_desc(struct XgmacState *s, struct desc *d, int rx= ) +static void xgmac_read_desc(XgmacState *s, struct desc *d, int rx) { uint32_t addr =3D rx ? s->regs[DMA_CUR_RX_DESC_ADDR] : s->regs[DMA_CUR_TX_DESC_ADDR]; cpu_physical_memory_read(addr, d, sizeof(*d)); } =20 -static void xgmac_write_desc(struct XgmacState *s, struct desc *d, int r= x) +static void xgmac_write_desc(XgmacState *s, struct desc *d, int rx) { int reg =3D rx ? DMA_CUR_RX_DESC_ADDR : DMA_CUR_TX_DESC_ADDR; uint32_t addr =3D s->regs[reg]; @@ -195,7 +199,7 @@ static void xgmac_write_desc(struct XgmacState *s, st= ruct desc *d, int rx) cpu_physical_memory_write(addr, d, sizeof(*d)); } =20 -static void xgmac_enet_send(struct XgmacState *s) +static void xgmac_enet_send(XgmacState *s) { struct desc bd; int frame_size; @@ -246,7 +250,7 @@ static void xgmac_enet_send(struct XgmacState *s) } } =20 -static void enet_update_irq(struct XgmacState *s) +static void enet_update_irq(XgmacState *s) { int stat =3D s->regs[DMA_STATUS] & s->regs[DMA_INTR_ENA]; qemu_set_irq(s->sbd_irq, !!stat); @@ -254,7 +258,7 @@ static void enet_update_irq(struct XgmacState *s) =20 static uint64_t enet_read(void *opaque, hwaddr addr, unsigned size) { - struct XgmacState *s =3D opaque; + XgmacState *s =3D opaque; uint64_t r =3D 0; addr >>=3D 2; =20 @@ -274,7 +278,7 @@ static uint64_t enet_read(void *opaque, hwaddr addr, = unsigned size) static void enet_write(void *opaque, hwaddr addr, uint64_t value, unsigned size) { - struct XgmacState *s =3D opaque; + XgmacState *s =3D opaque; =20 addr >>=3D 2; switch (addr) { @@ -310,7 +314,7 @@ static const MemoryRegionOps enet_mem_ops =3D { =20 static int eth_can_rx(NetClientState *nc) { - struct XgmacState *s =3D qemu_get_nic_opaque(nc); + XgmacState *s =3D qemu_get_nic_opaque(nc); =20 /* RX enabled? */ return s->regs[DMA_CONTROL] & DMA_CONTROL_SR; @@ -318,7 +322,7 @@ static int eth_can_rx(NetClientState *nc) =20 static ssize_t eth_rx(NetClientState *nc, const uint8_t *buf, size_t siz= e) { - struct XgmacState *s =3D qemu_get_nic_opaque(nc); + XgmacState *s =3D qemu_get_nic_opaque(nc); static const unsigned char sa_bcast[6] =3D {0xff, 0xff, 0xff, 0xff, 0xff, 0xff}; int unicast, broadcast, multicast; @@ -366,7 +370,8 @@ out: =20 static void eth_cleanup(NetClientState *nc) { - struct XgmacState *s =3D qemu_get_nic_opaque(nc); + XgmacState *s =3D qemu_get_nic_opaque(nc); + s->nic =3D NULL; } =20 @@ -378,20 +383,21 @@ static NetClientInfo net_xgmac_enet_info =3D { .cleanup =3D eth_cleanup, }; =20 -static int xgmac_enet_init(SysBusDevice *dev) +static int xgmac_enet_init(SysBusDevice *sbd) { - struct XgmacState *s =3D FROM_SYSBUS(typeof(*s), dev); + DeviceState *dev =3D DEVICE(sbd); + XgmacState *s =3D XGMAC(dev); =20 memory_region_init_io(&s->iomem, OBJECT(s), &enet_mem_ops, s, "xgmac", 0x1000); - sysbus_init_mmio(dev, &s->iomem); - sysbus_init_irq(dev, &s->sbd_irq); - sysbus_init_irq(dev, &s->pmt_irq); - sysbus_init_irq(dev, &s->mci_irq); + sysbus_init_mmio(sbd, &s->iomem); + sysbus_init_irq(sbd, &s->sbd_irq); + sysbus_init_irq(sbd, &s->pmt_irq); + sysbus_init_irq(sbd, &s->mci_irq); =20 qemu_macaddr_default_if_unset(&s->conf.macaddr); s->nic =3D qemu_new_nic(&net_xgmac_enet_info, &s->conf, - object_get_typename(OBJECT(dev)), dev->qdev.id= , s); + object_get_typename(OBJECT(dev)), dev->id, s); qemu_format_nic_info_str(qemu_get_queue(s->nic), s->conf.macaddr.a); =20 s->regs[XGMAC_ADDR_HIGH(0)] =3D (s->conf.macaddr.a[5] << 8) | @@ -405,7 +411,7 @@ static int xgmac_enet_init(SysBusDevice *dev) } =20 static Property xgmac_properties[] =3D { - DEFINE_NIC_PROPERTIES(struct XgmacState, conf), + DEFINE_NIC_PROPERTIES(XgmacState, conf), DEFINE_PROP_END_OF_LIST(), }; =20 @@ -420,9 +426,9 @@ static void xgmac_enet_class_init(ObjectClass *klass,= void *data) } =20 static const TypeInfo xgmac_enet_info =3D { - .name =3D "xgmac", + .name =3D TYPE_XGMAC, .parent =3D TYPE_SYS_BUS_DEVICE, - .instance_size =3D sizeof(struct XgmacState), + .instance_size =3D sizeof(XgmacState), .class_init =3D xgmac_enet_class_init, }; =20 --=20 1.8.1.4