Re: [PATCH 02/22] ncr5380: Remove FLAG_NO_PSEUDO_DMA where possible

2016-03-14 Thread Hannes Reinecke
On 03/14/2016 05:27 AM, Finn Thain wrote:
> Drivers that define PSEUDO_DMA also define NCR5380_dma_xfer_len.
> The core driver must call NCR5380_dma_xfer_len which means
> FLAG_NO_PSEUDO_DMA can be eradicated from the core driver.
> 
> dmx3191d doesn't define PSEUDO_DMA and has no use for FLAG_NO_PSEUDO_DMA,
> so remove it there also.
> 
> Signed-off-by: Finn Thain 
> 
> ---
>  drivers/scsi/NCR5380.c   |3 +--
>  drivers/scsi/dmx3191d.c  |2 +-
>  drivers/scsi/g_NCR5380.c |7 ++-
>  drivers/scsi/g_NCR5380.h |2 +-
>  drivers/scsi/mac_scsi.c  |   15 ++-
>  5 files changed, 23 insertions(+), 6 deletions(-)
> 
Reviewed-by: Hannes Reinecke 

Cheers,

Hannes
-- 
Dr. Hannes ReineckeTeamlead Storage & Networking
h...@suse.de   +49 911 74053 688
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton
HRB 21284 (AG Nürnberg)


[PATCH 02/22] ncr5380: Remove FLAG_NO_PSEUDO_DMA where possible

2016-03-13 Thread Finn Thain
Drivers that define PSEUDO_DMA also define NCR5380_dma_xfer_len.
The core driver must call NCR5380_dma_xfer_len which means
FLAG_NO_PSEUDO_DMA can be eradicated from the core driver.

dmx3191d doesn't define PSEUDO_DMA and has no use for FLAG_NO_PSEUDO_DMA,
so remove it there also.

Signed-off-by: Finn Thain 

---
 drivers/scsi/NCR5380.c   |3 +--
 drivers/scsi/dmx3191d.c  |2 +-
 drivers/scsi/g_NCR5380.c |7 ++-
 drivers/scsi/g_NCR5380.h |2 +-
 drivers/scsi/mac_scsi.c  |   15 ++-
 5 files changed, 23 insertions(+), 6 deletions(-)

Index: linux/drivers/scsi/dmx3191d.c
===
--- linux.orig/drivers/scsi/dmx3191d.c  2016-03-14 15:17:25.0 +1100
+++ linux/drivers/scsi/dmx3191d.c   2016-03-14 15:26:22.0 +1100
@@ -93,7 +93,7 @@ static int dmx3191d_probe_one(struct pci
 */
shost->irq = NO_IRQ;
 
-   error = NCR5380_init(shost, FLAG_NO_PSEUDO_DMA);
+   error = NCR5380_init(shost, 0);
if (error)
goto out_host_put;
 
Index: linux/drivers/scsi/mac_scsi.c
===
--- linux.orig/drivers/scsi/mac_scsi.c  2016-03-14 15:17:25.0 +1100
+++ linux/drivers/scsi/mac_scsi.c   2016-03-14 15:26:22.0 +1100
@@ -37,7 +37,9 @@
 
 #define NCR5380_pread   macscsi_pread
 #define NCR5380_pwrite  macscsi_pwrite
-#define NCR5380_dma_xfer_len(instance, cmd, phase) (cmd->transfersize)
+
+#define NCR5380_dma_xfer_len(instance, cmd, phase) \
+macscsi_dma_xfer_len(instance, cmd)
 
 #define NCR5380_intrmacscsi_intr
 #define NCR5380_queue_command   macscsi_queue_command
@@ -303,6 +305,17 @@ static int macscsi_pwrite(struct Scsi_Ho
 }
 #endif
 
+static int macscsi_dma_xfer_len(struct Scsi_Host *instance,
+struct scsi_cmnd *cmd)
+{
+   struct NCR5380_hostdata *hostdata = shost_priv(instance);
+
+   if (hostdata->flags & FLAG_NO_PSEUDO_DMA)
+   return 0;
+
+   return cmd->transfersize;
+}
+
 #include "NCR5380.c"
 
 #define DRV_MODULE_NAME "mac_scsi"
Index: linux/drivers/scsi/g_NCR5380.c
===
--- linux.orig/drivers/scsi/g_NCR5380.c 2016-03-14 15:26:20.0 +1100
+++ linux/drivers/scsi/g_NCR5380.c  2016-03-14 15:26:22.0 +1100
@@ -712,10 +712,15 @@ static inline int NCR5380_pwrite(struct
return 0;
 }
 
-static int generic_NCR5380_dma_xfer_len(struct scsi_cmnd *cmd)
+static int generic_NCR5380_dma_xfer_len(struct Scsi_Host *instance,
+struct scsi_cmnd *cmd)
 {
+   struct NCR5380_hostdata *hostdata = shost_priv(instance);
int transfersize = cmd->transfersize;
 
+   if (hostdata->flags & FLAG_NO_PSEUDO_DMA)
+   return 0;
+
/* Limit transfers to 32K, for xx400 & xx406
 * pseudoDMA that transfers in 128 bytes blocks.
 */
Index: linux/drivers/scsi/NCR5380.c
===
--- linux.orig/drivers/scsi/NCR5380.c   2016-03-14 15:26:20.0 +1100
+++ linux/drivers/scsi/NCR5380.c2016-03-14 15:26:22.0 +1100
@@ -1833,8 +1833,7 @@ static void NCR5380_information_transfer
 
 #if defined(PSEUDO_DMA) || defined(REAL_DMA_POLL)
transfersize = 0;
-   if (!cmd->device->borken &&
-   !(hostdata->flags & FLAG_NO_PSEUDO_DMA))
+   if (!cmd->device->borken)
transfersize = 
NCR5380_dma_xfer_len(instance, cmd, phase);
 
if (transfersize) {
Index: linux/drivers/scsi/g_NCR5380.h
===
--- linux.orig/drivers/scsi/g_NCR5380.h 2016-03-14 15:26:20.0 +1100
+++ linux/drivers/scsi/g_NCR5380.h  2016-03-14 15:26:22.0 +1100
@@ -61,7 +61,7 @@
 #endif
 
 #define NCR5380_dma_xfer_len(instance, cmd, phase) \
-generic_NCR5380_dma_xfer_len(cmd)
+generic_NCR5380_dma_xfer_len(instance, cmd)
 
 #define NCR5380_intr generic_NCR5380_intr
 #define NCR5380_queue_command generic_NCR5380_queue_command