[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 24/25] esp: remove pdma_origin from ESPState
From: |
Mark Cave-Ayland |
Subject: |
[PATCH 24/25] esp: remove pdma_origin from ESPState |
Date: |
Wed, 30 Dec 2020 15:37:44 +0000 |
Now that all data is transferred via the FIFO (ti_buf) there is no need to track
the source buffer being used for the data transfer. This also eliminates the
need for a separate subsection for PDMA state migration.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
---
hw/scsi/esp.c | 74 +++++--------------------------------------
include/hw/scsi/esp.h | 6 ----
2 files changed, 8 insertions(+), 72 deletions(-)
diff --git a/hw/scsi/esp.c b/hw/scsi/esp.c
index d2a70998aa..773fe09c49 100644
--- a/hw/scsi/esp.c
+++ b/hw/scsi/esp.c
@@ -127,32 +127,14 @@ static uint32_t esp_get_stc(ESPState *s)
return dmalen;
}
-static void set_pdma(ESPState *s, enum pdma_origin_id origin)
-{
- s->pdma_origin = origin;
-}
-
static uint8_t esp_pdma_read(ESPState *s)
{
uint8_t val;
- switch (s->pdma_origin) {
- case TI:
- if (s->do_cmd) {
- val = s->cmdbuf[s->cmdlen++];
- } else {
- val = s->ti_buf[s->ti_rptr++];
- }
- break;
- case ASYNC:
- val = s->async_buf[0];
- if (s->async_len > 0) {
- s->async_len--;
- s->async_buf++;
- }
- break;
- default:
- g_assert_not_reached();
+ if (s->do_cmd) {
+ val = s->cmdbuf[s->cmdlen++];
+ } else {
+ val = s->ti_buf[s->ti_rptr++];
}
s->ti_size--;
@@ -168,23 +150,10 @@ static void esp_pdma_write(ESPState *s, uint8_t val)
return;
}
- switch (s->pdma_origin) {
- case TI:
- if (s->do_cmd) {
- s->cmdbuf[s->cmdlen++] = val;
- } else {
- s->ti_buf[s->ti_wptr++] = val;
- }
- break;
- case ASYNC:
- s->async_buf[0] = val;
- if (s->async_len > 0) {
- s->async_len--;
- s->async_buf++;
- }
- break;
- default:
- g_assert_not_reached();
+ if (s->do_cmd) {
+ s->cmdbuf[s->cmdlen++] = val;
+ } else {
+ s->ti_buf[s->ti_wptr++] = val;
}
s->ti_size++;
@@ -234,7 +203,6 @@ static int32_t get_cmd(ESPState *s, uint8_t *buf, uint8_t
buflen)
if (s->dma_memory_read) {
s->dma_memory_read(s->dma_opaque, buf, dmalen);
} else {
- set_pdma(s, TI);
if (esp_select(s) < 0) {
return -1;
}
@@ -411,7 +379,6 @@ static void write_response(ESPState *s)
s->rregs[ESP_RINTR] = INTR_BS | INTR_FC;
s->rregs[ESP_RSEQ] = SEQ_CD;
} else {
- set_pdma(s, TI);
s->pdma_cb = write_response_pdma_cb;
esp_raise_drq(s);
return;
@@ -524,7 +491,6 @@ static void esp_do_dma(ESPState *s)
if (s->dma_memory_read) {
s->dma_memory_read(s->dma_opaque, &s->cmdbuf[s->cmdlen], len);
} else {
- set_pdma(s, TI);
s->pdma_cb = do_dma_pdma_cb;
esp_raise_drq(s);
return;
@@ -547,7 +513,6 @@ static void esp_do_dma(ESPState *s)
if (s->dma_memory_read) {
s->dma_memory_read(s->dma_opaque, s->async_buf, len);
} else {
- set_pdma(s, TI);
s->pdma_cb = do_dma_pdma_cb;
esp_raise_drq(s);
return;
@@ -563,7 +528,6 @@ static void esp_do_dma(ESPState *s)
s->async_buf += len;
s->async_len -= len;
esp_set_tc(s, esp_get_tc(s) - len);
- set_pdma(s, TI);
s->pdma_cb = do_dma_pdma_cb;
esp_raise_drq(s);
@@ -901,24 +865,6 @@ static bool esp_mem_accepts(void *opaque, hwaddr addr,
return (size == 1) || (is_write && size == 4);
}
-static bool esp_pdma_needed(void *opaque)
-{
- ESPState *s = opaque;
- return s->dma_memory_read == NULL && s->dma_memory_write == NULL &&
- s->dma_enabled;
-}
-
-static const VMStateDescription vmstate_esp_pdma = {
- .name = "esp/pdma",
- .version_id = 2,
- .minimum_version_id = 2,
- .needed = esp_pdma_needed,
- .fields = (VMStateField[]) {
- VMSTATE_INT32(pdma_origin, ESPState),
- VMSTATE_END_OF_LIST()
- }
-};
-
const VMStateDescription vmstate_esp = {
.name = "esp",
.version_id = 4,
@@ -941,10 +887,6 @@ const VMStateDescription vmstate_esp = {
VMSTATE_UINT32(mig_dma_left, ESPState),
VMSTATE_END_OF_LIST()
},
- .subsections = (const VMStateDescription * []) {
- &vmstate_esp_pdma,
- NULL
- }
};
static void sysbus_esp_mem_write(void *opaque, hwaddr addr,
diff --git a/include/hw/scsi/esp.h b/include/hw/scsi/esp.h
index b8170e58ca..45c15e0650 100644
--- a/include/hw/scsi/esp.h
+++ b/include/hw/scsi/esp.h
@@ -15,11 +15,6 @@ typedef void (*ESPDMAMemoryReadWriteFunc)(void *opaque,
uint8_t *buf, int len);
typedef struct ESPState ESPState;
-enum pdma_origin_id {
- TI,
- ASYNC,
-};
-
struct ESPState {
uint8_t rregs[ESP_REGS];
uint8_t wregs[ESP_REGS];
@@ -50,7 +45,6 @@ struct ESPState {
ESPDMAMemoryReadWriteFunc dma_memory_write;
void *dma_opaque;
void (*dma_cb)(ESPState *s);
- int pdma_origin;
void (*pdma_cb)(ESPState *s);
/* Legacy field for vmstate_sysbus_esp_scsi version < 2 */
--
2.20.1
- [PATCH 13/25] esp: use pdma_origin directly in esp_pdma_read()/esp_pdma_write(), (continued)
- [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
- [PATCH 24/25] esp: remove pdma_origin from ESPState,
Mark Cave-Ayland <=
- [PATCH 25/25] esp: add 4 byte PDMA read and write transfers, Mark Cave-Ayland, 2020/12/30