scsi: mac_esp: Fix PIO transfers for MESSAGE IN phase
authorFinn Thain <fthain@telegraphics.com.au>
Fri, 4 Aug 2017 05:43:19 +0000 (01:43 -0400)
committerBen Hutchings <ben@decadent.org.uk>
Sun, 26 Nov 2017 13:51:03 +0000 (13:51 +0000)
commitaf8bbadc62b289fd658e56b02ecdefd808c4ddff
tree7979b5bf598543e7407aec3bbfc0cf486df804d4
parentb467649e33ec97a1956c2038bb43e317f32a9911
scsi: mac_esp: Fix PIO transfers for MESSAGE IN phase

commit 7640d91d285893a5cf1e62b2cd00f0884c401d93 upstream.

When in MESSAGE IN phase, the ESP device does not automatically
acknowledge each byte that is transferred by PIO. The mac_esp driver
neglects to explicitly ack them, which causes a timeout during messages
larger than one byte (e.g. tag bytes during reconnect). Fix this with an
ESP_CMD_MOK command after each byte.

The MESSAGE IN phase is also different in that each byte transferred
raises ESP_INTR_FDONE. So don't exit the transfer loop for this interrupt,
for this phase.

That resolves the "Reconnect IRQ2 timeout" error on those Macs which use
PIO transfers instead of PDMA. This patch also improves on the weak tests
for unexpected interrupts and phase changes during PIO transfers.

Tested-by: Stan Johnson <userm57@yahoo.com>
Fixes: 02507a80b35e ("[PATCH] [SCSI] mac_esp: fix PIO mode, take 2")
Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
[bwh: Backported to 3.2: adjust context]
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
drivers/scsi/mac_esp.c