[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC 04/10] pcnet: pcnet_common_init() always ret
From: |
Gonglei |
Subject: |
Re: [Qemu-devel] [PATCH RFC 04/10] pcnet: pcnet_common_init() always returns 0, change to void |
Date: |
Tue, 28 Oct 2014 16:42:19 +0800 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; rv:11.0) Gecko/20120327 Thunderbird/11.0.1 |
On 2014/10/28 15:35, Markus Armbruster wrote:
> The next commit will exploit the fact it never fails. This one makes
> it obvious.
>
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
> hw/net/lance.c | 3 ++-
> hw/net/pcnet-pci.c | 3 ++-
> hw/net/pcnet.c | 4 +---
> hw/net/pcnet.h | 2 +-
> 4 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/hw/net/lance.c b/hw/net/lance.c
> index a1c49f1..3663340 100644
> --- a/hw/net/lance.c
> +++ b/hw/net/lance.c
> @@ -134,7 +134,8 @@ static int lance_init(SysBusDevice *sbd)
>
> s->phys_mem_read = ledma_memory_read;
> s->phys_mem_write = ledma_memory_write;
> - return pcnet_common_init(dev, s, &net_lance_info);
> + pcnet_common_init(dev, s, &net_lance_info);
> + return 0;
> }
>
> static void lance_reset(DeviceState *dev)
> diff --git a/hw/net/pcnet-pci.c b/hw/net/pcnet-pci.c
> index fb5f5d6..50eb069 100644
> --- a/hw/net/pcnet-pci.c
> +++ b/hw/net/pcnet-pci.c
> @@ -335,7 +335,8 @@ static int pci_pcnet_init(PCIDevice *pci_dev)
> s->phys_mem_write = pci_physical_memory_write;
> s->dma_opaque = pci_dev;
>
> - return pcnet_common_init(DEVICE(pci_dev), s, &net_pci_pcnet_info);
> + pcnet_common_init(DEVICE(pci_dev), s, &net_pci_pcnet_info);
> + return 0;
> }
>
> static void pci_reset(DeviceState *dev)
> diff --git a/hw/net/pcnet.c b/hw/net/pcnet.c
> index d344c15..5a081c4 100644
> --- a/hw/net/pcnet.c
> +++ b/hw/net/pcnet.c
> @@ -1724,7 +1724,7 @@ void pcnet_common_cleanup(PCNetState *d)
> d->nic = NULL;
> }
>
> -int pcnet_common_init(DeviceState *dev, PCNetState *s, NetClientInfo *info)
> +void pcnet_common_init(DeviceState *dev, PCNetState *s, NetClientInfo *info)
Do we need consider to pass an Error **errp argument to it?
Best regards,
-Gonglei
> {
> int i;
> uint16_t checksum;
> @@ -1763,6 +1763,4 @@ int pcnet_common_init(DeviceState *dev, PCNetState *s,
> NetClientInfo *info)
> *(uint16_t *)&s->prom[12] = cpu_to_le16(checksum);
>
> s->lnkst = 0x40; /* initial link state: up */
> -
> - return 0;
> }
> diff --git a/hw/net/pcnet.h b/hw/net/pcnet.h
> index f8e8a6f..cfc196e 100644
> --- a/hw/net/pcnet.h
> +++ b/hw/net/pcnet.h
> @@ -64,6 +64,6 @@ int pcnet_can_receive(NetClientState *nc);
> ssize_t pcnet_receive(NetClientState *nc, const uint8_t *buf, size_t size_);
> void pcnet_set_link_status(NetClientState *nc);
> void pcnet_common_cleanup(PCNetState *d);
> -int pcnet_common_init(DeviceState *dev, PCNetState *s, NetClientInfo *info);
> +void pcnet_common_init(DeviceState *dev, PCNetState *s, NetClientInfo *info);
> extern const VMStateDescription vmstate_pcnet;
> #endif
- [Qemu-devel] [PATCH RFC 00/10] pci: Partial conversion to realize, Markus Armbruster, 2014/10/28
- [Qemu-devel] [PATCH RFC 10/10] pci-assign: Convert to realize, Markus Armbruster, 2014/10/28
- [Qemu-devel] [PATCH RFC 06/10] serial-pci: Convert to realize, Markus Armbruster, 2014/10/28
- [Qemu-devel] [PATCH RFC 02/10] pci: Permit incremental conversion of device models to realize, Markus Armbruster, 2014/10/28
- [Qemu-devel] [PATCH RFC 05/10] pcnet: Convert to realize, Markus Armbruster, 2014/10/28
- [Qemu-devel] [PATCH RFC 07/10] ide/ich: Convert to realize, Markus Armbruster, 2014/10/28