[0/6] mac_esp, zorro_esp, esp_scsi: Various improvements

Message

Finn Thain Oct. 13, 2018, 12:51 a.m. UTC

This series has fixes and cleanup for mac_esp, zorro_esp and the
core esp_scsi driver.

The improvements include elimination of duplicated code temporarily
introduced for zorro_esp.

Michael, would you please regression test this series on elgar,
if convenient? So far, only mac_esp has been tested with this code.


Finn Thain (6):
  zorro_esp: Limit DMA transfers to 65535 bytes
  esp_scsi: Track residual for PIO transfers
  esp_scsi: Grant disconnect privilege for untagged commands
  esp_scsi: Eliminate ESP_FLAG_DOING_SLOWCMD
  esp_scsi: De-duplicate PIO routines
  esp_scsi: Optimize PIO loops

 drivers/scsi/esp_scsi.c  | 192 +++++++++++++++++++++++++++++--------
 drivers/scsi/esp_scsi.h  |  10 +-
 drivers/scsi/mac_esp.c   | 171 ++-------------------------------
 drivers/scsi/zorro_esp.c | 240 +++++++----------------------------------------
 4 files changed, 201 insertions(+), 412 deletions(-)

Comments

Michael Schmitz Oct. 14, 2018, 1:36 a.m. UTC | #1

Hi Finn,

Am 13.10.2018 um 13:51 schrieb Finn Thain:
> This series has fixes and cleanup for mac_esp, zorro_esp and the
> core esp_scsi driver.
>
> The improvements include elimination of duplicated code temporarily
> introduced for zorro_esp.
>
> Michael, would you please regression test this series on elgar,
> if convenient? So far, only mac_esp has been tested with this code.
>
>
> Finn Thain (6):
>   zorro_esp: Limit DMA transfers to 65535 bytes
>   esp_scsi: Track residual for PIO transfers
>   esp_scsi: Grant disconnect privilege for untagged commands
>   esp_scsi: Eliminate ESP_FLAG_DOING_SLOWCMD
>   esp_scsi: De-duplicate PIO routines
>   esp_scsi: Optimize PIO loops
>
>  drivers/scsi/esp_scsi.c  | 192 +++++++++++++++++++++++++++++--------
>  drivers/scsi/esp_scsi.h  |  10 +-
>  drivers/scsi/mac_esp.c   | 171 ++-------------------------------
>  drivers/scsi/zorro_esp.c | 240 +++++++----------------------------------------
>  4 files changed, 201 insertions(+), 412 deletions(-)
>

For Amiga zorro-esp:

Tested-by: Michael Schmitz <[e-mail]>