diff options
| author | Cédric Le Goater | 2020-09-01 14:21:50 +0200 |
|---|---|---|
| committer | Cédric Le Goater | 2020-09-01 14:21:50 +0200 |
| commit | 9c30f092a0e2a05aa37a4f22d3109b9913a6caea (patch) | |
| tree | 29176de747117ae2eb7f70a203c617739aa9f4eb | |
| parent | ftgmac100: Fix interrupt status "Packet moved to RX FIFO" (diff) | |
| download | qemu-9c30f092a0e2a05aa37a4f22d3109b9913a6caea.tar.gz qemu-9c30f092a0e2a05aa37a4f22d3109b9913a6caea.tar.xz qemu-9c30f092a0e2a05aa37a4f22d3109b9913a6caea.zip | |
ftgmac100: Change interrupt status when a DMA error occurs
The model uses today the "Normal priority transmit buffer unavailable"
interrupt status which it is not appropriate. According to the Aspeed
specs, no interrupts are raised in that case. An "AHB error" status
seems like a better modeling choice for all implementations since it
is covered by the Linux kernel.
Cc: Frederic Konrad <konrad.frederic@yahoo.fr>
Reviewed-by: Joel Stanley <joel@jms.id.au>
Message-Id: <20200819100956.2216690-13-clg@kaod.org>
Signed-off-by: Cédric Le Goater <clg@kaod.org>
| -rw-r--r-- | hw/net/ftgmac100.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/net/ftgmac100.c b/hw/net/ftgmac100.c index 5c0fe2d8cb..014980d30a 100644 --- a/hw/net/ftgmac100.c +++ b/hw/net/ftgmac100.c @@ -517,7 +517,7 @@ static void ftgmac100_do_tx(FTGMAC100State *s, uint32_t tx_ring, if (dma_memory_read(&address_space_memory, bd.des3, ptr, len)) { qemu_log_mask(LOG_GUEST_ERROR, "%s: failed to read packet @ 0x%x\n", __func__, bd.des3); - s->isr |= FTGMAC100_INT_NO_NPTXBUF; + s->isr |= FTGMAC100_INT_AHB_ERR; break; } |