[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 14/25] esp: move pdma_len and TC logic into esp_pdma_read()/esp_p
From: |
Mark Cave-Ayland |
Subject: |
[PATCH 14/25] esp: move pdma_len and TC logic into esp_pdma_read()/esp_pdma_write() |
Date: |
Wed, 30 Dec 2020 15:37:34 +0000 |
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
---
hw/scsi/esp.c | 50 ++++++++++++++++++++++++++++++++------------------
1 file changed, 32 insertions(+), 18 deletions(-)
diff --git a/hw/scsi/esp.c b/hw/scsi/esp.c
index 6a3a4cbc80..2b307341ad 100644
--- a/hw/scsi/esp.c
+++ b/hw/scsi/esp.c
@@ -153,22 +153,45 @@ static uint8_t *get_pdma_buf(ESPState *s)
static uint8_t esp_pdma_read(ESPState *s)
{
+ uint32_t dmalen = esp_get_tc(s);
+ uint8_t val;
+
+ if (dmalen == 0 || s->pdma_len == 0) {
+ return 0;
+ }
+
switch (s->pdma_origin) {
case PDMA:
- return s->pdma_buf[s->pdma_cur++];
+ val = s->pdma_buf[s->pdma_cur++];
+ break;
case TI:
- return s->ti_buf[s->pdma_cur++];
+ val = s->ti_buf[s->pdma_cur++];
+ break;
case CMD:
- return s->cmdbuf[s->pdma_cur++];
+ val = s->cmdbuf[s->pdma_cur++];
+ break;
case ASYNC:
- return s->async_buf[s->pdma_cur++];
+ val = s->async_buf[s->pdma_cur++];
+ break;
default:
g_assert_not_reached();
}
+
+ s->pdma_len--;
+ dmalen--;
+ esp_set_tc(s, dmalen);
+
+ return val;
}
static void esp_pdma_write(ESPState *s, uint8_t val)
{
+ uint32_t dmalen = esp_get_tc(s);
+
+ if (dmalen == 0 || s->pdma_len == 0) {
+ return;
+ }
+
switch (s->pdma_origin) {
case PDMA:
s->pdma_buf[s->pdma_cur++] = val;
@@ -185,6 +208,10 @@ static void esp_pdma_write(ESPState *s, uint8_t val)
default:
g_assert_not_reached();
}
+
+ s->pdma_len--;
+ dmalen--;
+ esp_set_tc(s, dmalen);
}
static int get_cmd_cb(ESPState *s)
@@ -914,27 +941,18 @@ static void sysbus_esp_pdma_write(void *opaque, hwaddr
addr,
{
SysBusESPState *sysbus = opaque;
ESPState *s = &sysbus->esp;
- uint32_t dmalen = esp_get_tc(s);
trace_esp_pdma_write(size);
- if (dmalen == 0 || s->pdma_len == 0) {
- return;
- }
switch (size) {
case 1:
esp_pdma_write(s, val);
- s->pdma_len--;
- dmalen--;
break;
case 2:
esp_pdma_write(s, val >> 8);
esp_pdma_write(s, val);
- s->pdma_len -= 2;
- dmalen -= 2;
break;
}
- esp_set_tc(s, dmalen);
if (s->pdma_len == 0 && s->pdma_cb) {
esp_lower_drq(s);
s->pdma_cb(s);
@@ -958,17 +976,13 @@ static uint64_t sysbus_esp_pdma_read(void *opaque, hwaddr
addr,
switch (size) {
case 1:
val = esp_pdma_read(s);
- s->pdma_len--;
- dmalen--;
break;
case 2:
val = esp_pdma_read(s);
val = (val << 8) | esp_pdma_read(s);
- s->pdma_len -= 2;
- dmalen -= 2;
break;
}
- esp_set_tc(s, dmalen);
+ dmalen = esp_get_tc(s);
if (dmalen == 0 || (s->pdma_len == 0 && s->pdma_cb)) {
esp_lower_drq(s);
s->pdma_cb(s);
--
2.20.1
- [PATCH 03/25] esp: fix esp_reg_read() trace event, (continued)
- [PATCH 03/25] esp: fix esp_reg_read() trace event, Mark Cave-Ayland, 2020/12/30
- [PATCH 04/25] esp: add PDMA trace events, Mark Cave-Ayland, 2020/12/30
- [PATCH 07/25] esp: introduce esp_get_stc(), Mark Cave-Ayland, 2020/12/30
- [PATCH 05/25] esp: determine transfer direction directly from SCSI phase, Mark Cave-Ayland, 2020/12/30
- [PATCH 06/25] esp: introduce esp_get_tc() and esp_set_tc(), Mark Cave-Ayland, 2020/12/30
- [PATCH 08/25] esp: apply transfer length adjustment when STC is zero at TC load time, Mark Cave-Ayland, 2020/12/30
- [PATCH 09/25] esp: remove dma_counter from ESPState, Mark Cave-Ayland, 2020/12/30
- [PATCH 10/25] esp: remove dma_left from ESPState, Mark Cave-Ayland, 2020/12/30
- [PATCH 11/25] esp: remove minlen restriction in handle_ti, Mark Cave-Ayland, 2020/12/30
- [PATCH 12/25] esp: introduce esp_pdma_read() and esp_pdma_write() functions, Mark Cave-Ayland, 2020/12/30
- [PATCH 14/25] esp: move pdma_len and TC logic into esp_pdma_read()/esp_pdma_write(),
Mark Cave-Ayland <=
- [PATCH 13/25] esp: use pdma_origin directly in esp_pdma_read()/esp_pdma_write(), Mark Cave-Ayland, 2020/12/30
- [PATCH 16/25] esp: remove redundant pdma_start from ESPState, Mark Cave-Ayland, 2020/12/30
- [PATCH 15/25] esp: accumulate SCSI commands for PDMA transfers in cmdbuf instead of pdma_buf, Mark Cave-Ayland, 2020/12/30
- [PATCH 17/25] esp: move PDMA length adjustments into esp_pdma_read()/esp_pdma_write(), Mark Cave-Ayland, 2020/12/30
- [PATCH 18/25] esp: use ti_wptr/ti_rptr to manage the current FIFO position for PDMA, Mark Cave-Ayland, 2020/12/30
- [PATCH 19/25] esp: use in-built TC to determine PDMA transfer length, Mark Cave-Ayland, 2020/12/30
- [PATCH 20/25] esp: remove CMD pdma_origin, Mark Cave-Ayland, 2020/12/30
- [PATCH 21/25] esp: rename get_cmd_cb() to esp_select(), Mark Cave-Ayland, 2020/12/30
- [PATCH 22/25] esp: fix PDMA target selection, Mark Cave-Ayland, 2020/12/30
- [PATCH 23/25] esp: use FIFO for PDMA transfers between initiator and device, Mark Cave-Ayland, 2020/12/30