[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v5 06/11] esp: add pseudo-DMA as us
From: |
Mark Cave-Ayland |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH v5 06/11] esp: add pseudo-DMA as used by Macintosh |
Date: |
Tue, 30 Oct 2018 10:09:53 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.2.1 |
On 30/10/2018 06:47, Hervé Poussineau wrote:
> Le 29/10/2018 à 14:39, Mark Cave-Ayland a écrit :
>> From: Laurent Vivier <address@hidden>
>>
>> Co-developed-by: Mark Cave-Ayland <address@hidden>
>> Signed-off-by: Mark Cave-Ayland <address@hidden>
>> Signed-off-by: Laurent Vivier <address@hidden>
>> ---
>> hw/scsi/esp.c | 292
>> +++++++++++++++++++++++++++++++++++++++++++++-----
>> include/hw/scsi/esp.h | 7 ++
>> 2 files changed, 270 insertions(+), 29 deletions(-)
>>
>
> [...]
>
>> @@ -409,6 +564,7 @@ uint64_t esp_reg_read(ESPState *s, uint32_t saddr)
>> s->ti_size--;
>> s->rregs[ESP_FIFO] = s->ti_buf[s->ti_rptr++];
>> }
>> + esp_raise_irq(s);
>> if (s->ti_rptr == s->ti_wptr) {
>> s->ti_rptr = 0;
>> s->ti_wptr = 0;
>
> You're now raising irq (not drq) when reading registers. In my opinion, this
> deserves
> its own patch with an explanation.
Oooh good spot. Laurent, can you explain a bit more as to how the pseudo-DMA is
wired
up for ESP?
ATB,
Mark.
[Qemu-block] [PATCH v5 07/11] hw/m68k: add Nubus support, Mark Cave-Ayland, 2018/10/29
[Qemu-block] [PATCH v5 08/11] hw/m68k: add Nubus support for macfb video card, Mark Cave-Ayland, 2018/10/29
[Qemu-block] [PATCH v5 09/11] hw/m68k: add a dummy SWIM floppy controller, Mark Cave-Ayland, 2018/10/29