diff options
author | Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk> | 2023-09-13 21:44:08 +0100 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2023-10-03 10:29:39 +0200 |
commit | b86dc5cb0b4105fa8ad29e822ab5d21c589c5ec5 (patch) | |
tree | a4e9afdfbc59110ab99995ba7b88904a1f5231c5 /hw/scsi | |
parent | ce43e84260a7d6250bf212aac958d73cfa1ca704 (diff) | |
download | qemu-b86dc5cb0b4105fa8ad29e822ab5d21c589c5ec5.zip qemu-b86dc5cb0b4105fa8ad29e822ab5d21c589c5ec5.tar.gz qemu-b86dc5cb0b4105fa8ad29e822ab5d21c589c5ec5.tar.bz2 |
esp: use correct type for esp_dma_enable() in sysbus_esp_gpio_demux()
The call to esp_dma_enable() was being made with the SYSBUS_ESP type instead of
the ESP type. This meant that when GPIO 1 was being used to trigger a DMA
request from an external DMA controller, the setting of ESPState's dma_enabled
field would clobber unknown memory whilst the dma_cb callback pointer would
typically return NULL so the DMA request would never start.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-ID: <20230913204410.65650-2-mark.cave-ayland@ilande.co.uk>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'hw/scsi')
-rw-r--r-- | hw/scsi/esp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/scsi/esp.c b/hw/scsi/esp.c index e52188d..4218a6a 100644 --- a/hw/scsi/esp.c +++ b/hw/scsi/esp.c @@ -1395,7 +1395,7 @@ static void sysbus_esp_gpio_demux(void *opaque, int irq, int level) parent_esp_reset(s, irq, level); break; case 1: - esp_dma_enable(opaque, irq, level); + esp_dma_enable(s, irq, level); break; } } |