diff options
| author | Markus Armbruster <armbru@redhat.com> | 2015-12-17 17:35:12 +0100 |
|---|---|---|
| committer | Timothy Pearson <tpearson@raptorengineering.com> | 2019-11-29 19:28:24 -0600 |
| commit | d54c98c69d4dcf545e516f2ba878cf7848c03c65 (patch) | |
| tree | d6f05df2716f4d76cffbadaaa1c10ef12c4b5294 /hw/net | |
| parent | ccdf36a50b93b0997e20950a2945662611feb5cf (diff) | |
| download | hqemu-d54c98c69d4dcf545e516f2ba878cf7848c03c65.zip hqemu-d54c98c69d4dcf545e516f2ba878cf7848c03c65.tar.gz | |
etraxfs_eth: Don't use hw_error() in init() method
Device init() methods aren't supposed to call hw_error(), they should
report the error and fail cleanly. Do that.
Cc: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>
Signed-off-by: Markus Armbruster <armbru@pond.sub.org>
Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Message-Id: <1450370121-5768-5-git-send-email-armbru@redhat.com>
Diffstat (limited to 'hw/net')
| -rw-r--r-- | hw/net/etraxfs_eth.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/hw/net/etraxfs_eth.c b/hw/net/etraxfs_eth.c index d600275..b562ac9 100644 --- a/hw/net/etraxfs_eth.c +++ b/hw/net/etraxfs_eth.c @@ -26,6 +26,7 @@ #include "hw/sysbus.h" #include "net/net.h" #include "hw/cris/etraxfs.h" +#include "qemu/error-report.h" #define D(x) @@ -589,7 +590,8 @@ static int fs_eth_init(SysBusDevice *sbd) ETRAXFSEthState *s = ETRAX_FS_ETH(dev); if (!s->dma_out || !s->dma_in) { - hw_error("Unconnected ETRAX-FS Ethernet MAC.\n"); + error_report("Unconnected ETRAX-FS Ethernet MAC"); + return -1; } s->dma_out->client.push = eth_tx_push; |
