1 files changed, 17 insertions, 16 deletions

diff --git a/drivers/scsi/atari_NCR5380.c b/drivers/scsi/atari_NCR5380.c
index 81e4f566b6c8..c912733ac077 100644
--- a/drivers/scsi/atari_NCR5380.c
+++ b/drivers/scsi/atari_NCR5380.c
@@ -272,8 +272,9 @@ static struct scsi_host_template *the_template = NULL;
         (struct NCR5380_hostdata *)(in)->hostdata
 #define HOSTDATA(in) ((struct NCR5380_hostdata *)(in)->hostdata)
 
-#define NEXT(cmd)       ((cmd)->host_scribble)
-#define NEXTADDR(cmd)   ((Scsi_Cmnd **)&((cmd)->host_scribble))
+#define NEXT(cmd)               ((Scsi_Cmnd *)(cmd)->host_scribble)
+#define SET_NEXT(cmd,next)      ((cmd)->host_scribble = (void *)(next))
+#define NEXTADDR(cmd)           ((Scsi_Cmnd **)&(cmd)->host_scribble)
 
 #define HOSTNO          instance->host_no
 #define H_NO(cmd)       (cmd)->device->host->host_no
@@ -479,7 +480,7 @@ static void merge_contiguous_buffers(Scsi_Cmnd *cmd)
              virt_to_phys(page_address(cmd->SCp.buffer[1].page) +
                           cmd->SCp.buffer[1].offset) == endaddr;) {
                 MER_PRINTK("VTOP(%p) == %08lx -> merging\n",
-                           cmd->SCp.buffer[1].address, endaddr);
+                           page_address(cmd->SCp.buffer[1].page), endaddr);
 #if (NDEBUG & NDEBUG_MERGING)
                 ++cnt;
 #endif
@@ -1002,7 +1003,7 @@ static int NCR5380_queue_command(Scsi_Cmnd *cmd, void (*done)(Scsi_Cmnd *))
          * in a queue
          */
 
-        NEXT(cmd) = NULL;
+        SET_NEXT(cmd, NULL);
         cmd->scsi_done = done;
 
         cmd->result = 0;
@@ -1034,14 +1035,14 @@ static int NCR5380_queue_command(Scsi_Cmnd *cmd, void (*done)(Scsi_Cmnd *))
         }
         if (!(hostdata->issue_queue) || (cmd->cmnd[0] == REQUEST_SENSE)) {
                 LIST(cmd, hostdata->issue_queue);
-                NEXT(cmd) = hostdata->issue_queue;
+                SET_NEXT(cmd, hostdata->issue_queue);
                 hostdata->issue_queue = cmd;
         } else {
                 for (tmp = (Scsi_Cmnd *)hostdata->issue_queue;
                      NEXT(tmp); tmp = NEXT(tmp))
                         ;
                 LIST(cmd, tmp);
-                NEXT(tmp) = cmd;
+                SET_NEXT(tmp, cmd);
         }
         local_irq_restore(flags);
 
@@ -1149,12 +1150,12 @@ static void NCR5380_main(void *bl)
                                         local_irq_disable();
                                         if (prev) {
                                                 REMOVE(prev, NEXT(prev), tmp, NEXT(tmp));
-                                                NEXT(prev) = NEXT(tmp);
+                                                SET_NEXT(prev, NEXT(tmp));
                                         } else {
                                                 REMOVE(-1, hostdata->issue_queue, tmp, NEXT(tmp));
                                                 hostdata->issue_queue = NEXT(tmp);
                                         }
-                                        NEXT(tmp) = NULL;
+                                        SET_NEXT(tmp, NULL);
                                         falcon_dont_release++;
 
                                         /* reenable interrupts after finding one */
@@ -1192,7 +1193,7 @@ static void NCR5380_main(void *bl)
                                         } else {
                                                 local_irq_disable();
                                                 LIST(tmp, hostdata->issue_queue);
-                                                NEXT(tmp) = hostdata->issue_queue;
+                                                SET_NEXT(tmp, hostdata->issue_queue);
                                                 hostdata->issue_queue = tmp;
 #ifdef SUPPORT_TAGS
                                                 cmd_free_tag(tmp);
@@ -2295,7 +2296,7 @@ static void NCR5380_information_transfer(struct Scsi_Host *instance)
 
                                                 local_irq_save(flags);
                                                 LIST(cmd,hostdata->issue_queue);
-                                                NEXT(cmd) = hostdata->issue_queue;
+                                                SET_NEXT(cmd, hostdata->issue_queue);
                                                 hostdata->issue_queue = (Scsi_Cmnd *) cmd;
                                                 local_irq_restore(flags);
                                                 QU_PRINTK("scsi%d: REQUEST SENSE added to head of "
@@ -2357,7 +2358,7 @@ static void NCR5380_information_transfer(struct Scsi_Host *instance)
                                         local_irq_save(flags);
                                         cmd->device->disconnect = 1;
                                         LIST(cmd,hostdata->disconnected_queue);
-                                        NEXT(cmd) = hostdata->disconnected_queue;
+                                        SET_NEXT(cmd, hostdata->disconnected_queue);
                                         hostdata->connected = NULL;
                                         hostdata->disconnected_queue = cmd;
                                         local_irq_restore(flags);
@@ -2632,12 +2633,12 @@ static void NCR5380_reselect(struct Scsi_Host *instance)
                         falcon_dont_release++;
                         if (prev) {
                                 REMOVE(prev, NEXT(prev), tmp, NEXT(tmp));
-                                NEXT(prev) = NEXT(tmp);
+                                SET_NEXT(prev, NEXT(tmp));
                         } else {
                                 REMOVE(-1, hostdata->disconnected_queue, tmp, NEXT(tmp));
                                 hostdata->disconnected_queue = NEXT(tmp);
                         }
-                        NEXT(tmp) = NULL;
+                        SET_NEXT(tmp, NULL);
                         break;
                 }
         }
@@ -2769,7 +2770,7 @@ int NCR5380_abort(Scsi_Cmnd *cmd)
                 if (cmd == tmp) {
                         REMOVE(5, *prev, tmp, NEXT(tmp));
                         (*prev) = NEXT(tmp);
-                        NEXT(tmp) = NULL;
+                        SET_NEXT(tmp, NULL);
                         tmp->result = DID_ABORT << 16;
                         local_irq_restore(flags);
                         ABRT_PRINTK("scsi%d: abort removed command from issue queue.\n",
@@ -2844,7 +2845,7 @@ int NCR5380_abort(Scsi_Cmnd *cmd)
                                 if (cmd == tmp) {
                                         REMOVE(5, *prev, tmp, NEXT(tmp));
                                         *prev = NEXT(tmp);
-                                        NEXT(tmp) = NULL;
+                                        SET_NEXT(tmp, NULL);
                                         tmp->result = DID_ABORT << 16;
                                         /* We must unlock the tag/LUN immediately here, since the
                                          * target goes to BUS FREE and doesn't send us another
@@ -2965,7 +2966,7 @@ static int NCR5380_bus_reset(Scsi_Cmnd *cmd)
 
         for (i = 0; (cmd = disconnected_queue); ++i) {
                 disconnected_queue = NEXT(cmd);
-                NEXT(cmd) = NULL;
+                SET_NEXT(cmd, NULL);
                 cmd->result = (cmd->result & 0xffff) | (DID_RESET << 16);
                 cmd->scsi_done(cmd);
         }