Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751303AbaDZBw0 (ORCPT ); Fri, 25 Apr 2014 21:52:26 -0400 Received: from mail-pd0-f178.google.com ([209.85.192.178]:38483 "EHLO mail-pd0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750840AbaDZBwY (ORCPT ); Fri, 25 Apr 2014 21:52:24 -0400 Message-ID: <535B1150.6040802@gmail.com> Date: Sat, 26 Apr 2014 13:52:16 +1200 From: Michael Schmitz User-Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130330 Thunderbird/17.0.5 MIME-Version: 1.0 To: Finn Thain CC: "James E.J. Bottomley" , linux-scsi@vger.kernel.org, Sam Creasey , Russell King , Michael Schmitz , Joe Perches , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-m68k@vger.kernel.org Subject: Re: [PATCH v2 08/12] scsi/NCR5380: adopt dprintk() References: <20140319123516.542623278@telegraphics.com.au> <20140319123518.942420067@telegraphics.com.au> In-Reply-To: <20140319123518.942420067@telegraphics.com.au> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Acked-by: Michael Schmitz > All NCR5380 drivers already include the NCR5380.h header. Better to > adopt those macros rather than have three variations on them. > > Moreover, the macros in NCR5380.h are preferable because the atari_NCR5380 > and sun3_NCR5380 versions are inflexible. For example, they can't accomodate > dprintk(NDEBUG_MAIN | NDEBUG_QUEUES, ...) > > Replace the *_PRINTK macros from atari_NCR5380.h and sun3_NCR5380.h with > the equivalent macros from NCR5380.h. > > Signed-off-by: Finn Thain > > --- > drivers/scsi/atari_NCR5380.c | 130 +++++++++++++++++++++---------------------- > drivers/scsi/atari_scsi.c | 16 ++--- > drivers/scsi/sun3_NCR5380.c | 122 ++++++++++++++++++++-------------------- > 3 files changed, 134 insertions(+), 134 deletions(-) > > Index: linux-m68k/drivers/scsi/atari_NCR5380.c > =================================================================== > --- linux-m68k.orig/drivers/scsi/atari_NCR5380.c 2014-03-19 23:34:43.000000000 +1100 > +++ linux-m68k/drivers/scsi/atari_NCR5380.c 2014-03-19 23:34:44.000000000 +1100 > @@ -370,7 +370,7 @@ static int is_lun_busy(Scsi_Cmnd *cmd, i > return 0; > if (TagAlloc[cmd->device->id][cmd->device->lun].nr_allocated >= > TagAlloc[cmd->device->id][cmd->device->lun].queue_size) { > - TAG_PRINTK("scsi%d: target %d lun %d: no free tags\n", > + dprintk(NDEBUG_TAGS, "scsi%d: target %d lun %d: no free tags\n", > H_NO(cmd), cmd->device->id, cmd->device->lun); > return 1; > } > @@ -394,7 +394,7 @@ static void cmd_get_tag(Scsi_Cmnd *cmd, > !setup_use_tagged_queuing || !cmd->device->tagged_supported) { > cmd->tag = TAG_NONE; > hostdata->busy[cmd->device->id] |= (1 << cmd->device->lun); > - TAG_PRINTK("scsi%d: target %d lun %d now allocated by untagged " > + dprintk(NDEBUG_TAGS, "scsi%d: target %d lun %d now allocated by untagged " > "command\n", H_NO(cmd), cmd->device->id, cmd->device->lun); > } else { > TAG_ALLOC *ta = &TagAlloc[cmd->device->id][cmd->device->lun]; > @@ -402,7 +402,7 @@ static void cmd_get_tag(Scsi_Cmnd *cmd, > cmd->tag = find_first_zero_bit(ta->allocated, MAX_TAGS); > set_bit(cmd->tag, ta->allocated); > ta->nr_allocated++; > - TAG_PRINTK("scsi%d: using tag %d for target %d lun %d " > + dprintk(NDEBUG_TAGS, "scsi%d: using tag %d for target %d lun %d " > "(now %d tags in use)\n", > H_NO(cmd), cmd->tag, cmd->device->id, > cmd->device->lun, ta->nr_allocated); > @@ -420,7 +420,7 @@ static void cmd_free_tag(Scsi_Cmnd *cmd) > > if (cmd->tag == TAG_NONE) { > hostdata->busy[cmd->device->id] &= ~(1 << cmd->device->lun); > - TAG_PRINTK("scsi%d: target %d lun %d untagged cmd finished\n", > + dprintk(NDEBUG_TAGS, "scsi%d: target %d lun %d untagged cmd finished\n", > H_NO(cmd), cmd->device->id, cmd->device->lun); > } else if (cmd->tag >= MAX_TAGS) { > printk(KERN_NOTICE "scsi%d: trying to free bad tag %d!\n", > @@ -429,7 +429,7 @@ static void cmd_free_tag(Scsi_Cmnd *cmd) > TAG_ALLOC *ta = &TagAlloc[cmd->device->id][cmd->device->lun]; > clear_bit(cmd->tag, ta->allocated); > ta->nr_allocated--; > - TAG_PRINTK("scsi%d: freed tag %d for target %d lun %d\n", > + dprintk(NDEBUG_TAGS, "scsi%d: freed tag %d for target %d lun %d\n", > H_NO(cmd), cmd->tag, cmd->device->id, cmd->device->lun); > } > } > @@ -478,7 +478,7 @@ static void merge_contiguous_buffers(Scs > for (endaddr = virt_to_phys(cmd->SCp.ptr + cmd->SCp.this_residual - 1) + 1; > cmd->SCp.buffers_residual && > virt_to_phys(sg_virt(&cmd->SCp.buffer[1])) == endaddr;) { > - MER_PRINTK("VTOP(%p) == %08lx -> merging\n", > + dprintk(NDEBUG_MERGING, "VTOP(%p) == %08lx -> merging\n", > page_address(sg_page(&cmd->SCp.buffer[1])), endaddr); > #if (NDEBUG & NDEBUG_MERGING) > ++cnt; > @@ -490,7 +490,7 @@ static void merge_contiguous_buffers(Scs > } > #if (NDEBUG & NDEBUG_MERGING) > if (oldlen != cmd->SCp.this_residual) > - MER_PRINTK("merged %d buffers from %p, new length %08x\n", > + dprintk(NDEBUG_MERGING, "merged %d buffers from %p, new length %08x\n", > cnt, cmd->SCp.ptr, cmd->SCp.this_residual); > #endif > } > @@ -676,7 +676,7 @@ static inline void NCR5380_all_init(void > { > static int done = 0; > if (!done) { > - INI_PRINTK("scsi : NCR5380_all_init()\n"); > + dprintk(NDEBUG_INIT, "scsi : NCR5380_all_init()\n"); > done = 1; > } > } > @@ -984,7 +984,7 @@ static int NCR5380_queue_command_lck(Scs > } > local_irq_restore(flags); > > - QU_PRINTK("scsi%d: command added to %s of queue\n", H_NO(cmd), > + dprintk(NDEBUG_QUEUES, "scsi%d: command added to %s of queue\n", H_NO(cmd), > (cmd->cmnd[0] == REQUEST_SENSE) ? "head" : "tail"); > > /* If queue_command() is called from an interrupt (real one or bottom > @@ -1054,7 +1054,7 @@ static void NCR5380_main(struct work_str > done = 1; > > if (!hostdata->connected) { > - MAIN_PRINTK("scsi%d: not connected\n", HOSTNO); > + dprintk(NDEBUG_MAIN, "scsi%d: not connected\n", HOSTNO); > /* > * Search through the issue_queue for a command destined > * for a target that's not busy. > @@ -1107,7 +1107,7 @@ static void NCR5380_main(struct work_str > * On failure, we must add the command back to the > * issue queue so we can keep trying. > */ > - MAIN_PRINTK("scsi%d: main(): command for target %d " > + dprintk(NDEBUG_MAIN, "scsi%d: main(): command for target %d " > "lun %d removed from issue_queue\n", > HOSTNO, tmp->device->id, tmp->device->lun); > /* > @@ -1140,7 +1140,7 @@ static void NCR5380_main(struct work_str > #endif > falcon_dont_release--; > local_irq_restore(flags); > - MAIN_PRINTK("scsi%d: main(): select() failed, " > + dprintk(NDEBUG_MAIN, "scsi%d: main(): select() failed, " > "returned to issue_queue\n", HOSTNO); > if (hostdata->connected) > break; > @@ -1155,10 +1155,10 @@ static void NCR5380_main(struct work_str > #endif > ) { > local_irq_restore(flags); > - MAIN_PRINTK("scsi%d: main: performing information transfer\n", > + dprintk(NDEBUG_MAIN, "scsi%d: main: performing information transfer\n", > HOSTNO); > NCR5380_information_transfer(instance); > - MAIN_PRINTK("scsi%d: main: done set false\n", HOSTNO); > + dprintk(NDEBUG_MAIN, "scsi%d: main: done set false\n", HOSTNO); > done = 0; > } > } while (!done); > @@ -1204,12 +1204,12 @@ static void NCR5380_dma_complete(struct > (BASR_PHASE_MATCH|BASR_ACK)) { > saved_data = NCR5380_read(INPUT_DATA_REG); > overrun = 1; > - DMA_PRINTK("scsi%d: read overrun handled\n", HOSTNO); > + dprintk(NDEBUG_DMA, "scsi%d: read overrun handled\n", HOSTNO); > } > } > } > > - DMA_PRINTK("scsi%d: real DMA transfer complete, basr 0x%X, sr 0x%X\n", > + dprintk(NDEBUG_DMA, "scsi%d: real DMA transfer complete, basr 0x%X, sr 0x%X\n", > HOSTNO, NCR5380_read(BUS_AND_STATUS_REG), > NCR5380_read(STATUS_REG)); > > @@ -1229,13 +1229,13 @@ static void NCR5380_dma_complete(struct > if ((NCR5380_read(STATUS_REG) & PHASE_MASK) == p && (p & SR_IO)) { > cnt = toPIO = atari_read_overruns; > if (overrun) { > - DMA_PRINTK("Got an input overrun, using saved byte\n"); > + dprintk(NDEBUG_DMA, "Got an input overrun, using saved byte\n"); > *(*data)++ = saved_data; > (*count)--; > cnt--; > toPIO--; > } > - DMA_PRINTK("Doing %d-byte PIO to 0x%08lx\n", cnt, (long)*data); > + dprintk(NDEBUG_DMA, "Doing %d-byte PIO to 0x%08lx\n", cnt, (long)*data); > NCR5380_transfer_pio(instance, &p, &cnt, data); > *count -= toPIO - cnt; > } > @@ -1261,25 +1261,25 @@ static irqreturn_t NCR5380_intr(int irq, > int done = 1, handled = 0; > unsigned char basr; > > - INT_PRINTK("scsi%d: NCR5380 irq triggered\n", HOSTNO); > + dprintk(NDEBUG_INTR, "scsi%d: NCR5380 irq triggered\n", HOSTNO); > > /* Look for pending interrupts */ > basr = NCR5380_read(BUS_AND_STATUS_REG); > - INT_PRINTK("scsi%d: BASR=%02x\n", HOSTNO, basr); > + dprintk(NDEBUG_INTR, "scsi%d: BASR=%02x\n", HOSTNO, basr); > /* dispatch to appropriate routine if found and done=0 */ > if (basr & BASR_IRQ) { > NCR5380_dprint(NDEBUG_INTR, instance); > if ((NCR5380_read(STATUS_REG) & (SR_SEL|SR_IO)) == (SR_SEL|SR_IO)) { > done = 0; > ENABLE_IRQ(); > - INT_PRINTK("scsi%d: SEL interrupt\n", HOSTNO); > + dprintk(NDEBUG_INTR, "scsi%d: SEL interrupt\n", HOSTNO); > NCR5380_reselect(instance); > (void)NCR5380_read(RESET_PARITY_INTERRUPT_REG); > } else if (basr & BASR_PARITY_ERROR) { > - INT_PRINTK("scsi%d: PARITY interrupt\n", HOSTNO); > + dprintk(NDEBUG_INTR, "scsi%d: PARITY interrupt\n", HOSTNO); > (void)NCR5380_read(RESET_PARITY_INTERRUPT_REG); > } else if ((NCR5380_read(STATUS_REG) & SR_RST) == SR_RST) { > - INT_PRINTK("scsi%d: RESET interrupt\n", HOSTNO); > + dprintk(NDEBUG_INTR, "scsi%d: RESET interrupt\n", HOSTNO); > (void)NCR5380_read(RESET_PARITY_INTERRUPT_REG); > } else { > /* > @@ -1298,7 +1298,7 @@ static irqreturn_t NCR5380_intr(int irq, > ((basr & BASR_END_DMA_TRANSFER) || > !(basr & BASR_PHASE_MATCH))) { > > - INT_PRINTK("scsi%d: PHASE MISM or EOP interrupt\n", HOSTNO); > + dprintk(NDEBUG_INTR, "scsi%d: PHASE MISM or EOP interrupt\n", HOSTNO); > NCR5380_dma_complete( instance ); > done = 0; > ENABLE_IRQ(); > @@ -1323,7 +1323,7 @@ static irqreturn_t NCR5380_intr(int irq, > } > > if (!done) { > - INT_PRINTK("scsi%d: in int routine, calling main\n", HOSTNO); > + dprintk(NDEBUG_INTR, "scsi%d: in int routine, calling main\n", HOSTNO); > /* Put a call to NCR5380_main() on the queue... */ > queue_main(); > } > @@ -1397,7 +1397,7 @@ static int NCR5380_select(struct Scsi_Ho > > hostdata->restart_select = 0; > NCR5380_dprint(NDEBUG_ARBITRATION, instance); > - ARB_PRINTK("scsi%d: starting arbitration, id = %d\n", HOSTNO, > + dprintk(NDEBUG_ARBITRATION, "scsi%d: starting arbitration, id = %d\n", HOSTNO, > instance->this_id); > > /* > @@ -1442,7 +1442,7 @@ static int NCR5380_select(struct Scsi_Ho > ; > #endif > > - ARB_PRINTK("scsi%d: arbitration complete\n", HOSTNO); > + dprintk(NDEBUG_ARBITRATION, "scsi%d: arbitration complete\n", HOSTNO); > > if (hostdata->connected) { > NCR5380_write(MODE_REG, MR_BASE); > @@ -1463,7 +1463,7 @@ static int NCR5380_select(struct Scsi_Ho > (NCR5380_read(INITIATOR_COMMAND_REG) & ICR_ARBITRATION_LOST) || > hostdata->connected) { > NCR5380_write(MODE_REG, MR_BASE); > - ARB_PRINTK("scsi%d: lost arbitration, deasserting MR_ARBITRATE\n", > + dprintk(NDEBUG_ARBITRATION, "scsi%d: lost arbitration, deasserting MR_ARBITRATE\n", > HOSTNO); > return -1; > } > @@ -1478,7 +1478,7 @@ static int NCR5380_select(struct Scsi_Ho > hostdata->connected) { > NCR5380_write(MODE_REG, MR_BASE); > NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE); > - ARB_PRINTK("scsi%d: lost arbitration, deasserting ICR_ASSERT_SEL\n", > + dprintk(NDEBUG_ARBITRATION, "scsi%d: lost arbitration, deasserting ICR_ASSERT_SEL\n", > HOSTNO); > return -1; > } > @@ -1501,7 +1501,7 @@ static int NCR5380_select(struct Scsi_Ho > return -1; > } > > - ARB_PRINTK("scsi%d: won arbitration\n", HOSTNO); > + dprintk(NDEBUG_ARBITRATION, "scsi%d: won arbitration\n", HOSTNO); > > /* > * Now that we have won arbitration, start Selection process, asserting > @@ -1561,7 +1561,7 @@ static int NCR5380_select(struct Scsi_Ho > > udelay(1); > > - SEL_PRINTK("scsi%d: selecting target %d\n", HOSTNO, cmd->device->id); > + dprintk(NDEBUG_SELECTION, "scsi%d: selecting target %d\n", HOSTNO, cmd->device->id); > > /* > * The SCSI specification calls for a 250 ms timeout for the actual > @@ -1630,7 +1630,7 @@ static int NCR5380_select(struct Scsi_Ho > #endif > cmd->scsi_done(cmd); > NCR5380_write(SELECT_ENABLE_REG, hostdata->id_mask); > - SEL_PRINTK("scsi%d: target did not respond within 250ms\n", HOSTNO); > + dprintk(NDEBUG_SELECTION, "scsi%d: target did not respond within 250ms\n", HOSTNO); > NCR5380_write(SELECT_ENABLE_REG, hostdata->id_mask); > return 0; > } > @@ -1656,7 +1656,7 @@ static int NCR5380_select(struct Scsi_Ho > while (!(NCR5380_read(STATUS_REG) & SR_REQ)) > ; > > - SEL_PRINTK("scsi%d: target %d selected, going into MESSAGE OUT phase.\n", > + dprintk(NDEBUG_SELECTION, "scsi%d: target %d selected, going into MESSAGE OUT phase.\n", > HOSTNO, cmd->device->id); > tmp[0] = IDENTIFY(1, cmd->device->lun); > > @@ -1676,7 +1676,7 @@ static int NCR5380_select(struct Scsi_Ho > data = tmp; > phase = PHASE_MSGOUT; > NCR5380_transfer_pio(instance, &phase, &len, &data); > - SEL_PRINTK("scsi%d: nexus established.\n", HOSTNO); > + dprintk(NDEBUG_SELECTION, "scsi%d: nexus established.\n", HOSTNO); > /* XXX need to handle errors here */ > hostdata->connected = cmd; > #ifndef SUPPORT_TAGS > @@ -1737,11 +1737,11 @@ static int NCR5380_transfer_pio(struct S > while (!((tmp = NCR5380_read(STATUS_REG)) & SR_REQ)) > ; > > - HSH_PRINTK("scsi%d: REQ detected\n", HOSTNO); > + dprintk(NDEBUG_HANDSHAKE, "scsi%d: REQ detected\n", HOSTNO); > > /* Check for phase mismatch */ > if ((tmp & PHASE_MASK) != p) { > - PIO_PRINTK("scsi%d: phase mismatch\n", HOSTNO); > + dprintk(NDEBUG_PIO, "scsi%d: phase mismatch\n", HOSTNO); > NCR5380_dprint_phase(NDEBUG_PIO, instance); > break; > } > @@ -1782,7 +1782,7 @@ static int NCR5380_transfer_pio(struct S > while (NCR5380_read(STATUS_REG) & SR_REQ) > ; > > - HSH_PRINTK("scsi%d: req false, handshake complete\n", HOSTNO); > + dprintk(NDEBUG_HANDSHAKE, "scsi%d: req false, handshake complete\n", HOSTNO); > > /* > * We have several special cases to consider during REQ/ACK handshaking : > @@ -1803,7 +1803,7 @@ static int NCR5380_transfer_pio(struct S > } > } while (--c); > > - PIO_PRINTK("scsi%d: residual %d\n", HOSTNO, c); > + dprintk(NDEBUG_PIO, "scsi%d: residual %d\n", HOSTNO, c); > > *count = c; > *data = d; > @@ -1917,7 +1917,7 @@ static int NCR5380_transfer_dma(struct S > if (atari_read_overruns && (p & SR_IO)) > c -= atari_read_overruns; > > - DMA_PRINTK("scsi%d: initializing DMA for %s, %d bytes %s %p\n", > + dprintk(NDEBUG_DMA, "scsi%d: initializing DMA for %s, %d bytes %s %p\n", > HOSTNO, (p & SR_IO) ? "reading" : "writing", > c, (p & SR_IO) ? "to" : "from", d); > > @@ -2039,7 +2039,7 @@ static void NCR5380_information_transfer > * they are at contiguous physical addresses. > */ > merge_contiguous_buffers(cmd); > - INF_PRINTK("scsi%d: %d bytes and %d buffers left\n", > + dprintk(NDEBUG_INFORMATION, "scsi%d: %d bytes and %d buffers left\n", > HOSTNO, cmd->SCp.this_residual, > cmd->SCp.buffers_residual); > } > @@ -2123,7 +2123,7 @@ static void NCR5380_information_transfer > /* Accept message by clearing ACK */ > NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE); > > - LNK_PRINTK("scsi%d: target %d lun %d linked command " > + dprintk(NDEBUG_LINKED, "scsi%d: target %d lun %d linked command " > "complete.\n", HOSTNO, cmd->device->id, cmd->device->lun); > > /* Enable reselect interrupts */ > @@ -2148,7 +2148,7 @@ static void NCR5380_information_transfer > * and don't free it! */ > cmd->next_link->tag = cmd->tag; > cmd->result = cmd->SCp.Status | (cmd->SCp.Message << 8); > - LNK_PRINTK("scsi%d: target %d lun %d linked request " > + dprintk(NDEBUG_LINKED, "scsi%d: target %d lun %d linked request " > "done, calling scsi_done().\n", > HOSTNO, cmd->device->id, cmd->device->lun); > #ifdef NCR5380_STATS > @@ -2165,7 +2165,7 @@ static void NCR5380_information_transfer > /* ++guenther: possible race with Falcon locking */ > falcon_dont_release++; > hostdata->connected = NULL; > - QU_PRINTK("scsi%d: command for target %d, lun %d " > + dprintk(NDEBUG_QUEUES, "scsi%d: command for target %d, lun %d " > "completed\n", HOSTNO, cmd->device->id, cmd->device->lun); > #ifdef SUPPORT_TAGS > cmd_free_tag(cmd); > @@ -2179,7 +2179,7 @@ static void NCR5380_information_transfer > /* ++Andreas: the mid level code knows about > QUEUE_FULL now. */ > TAG_ALLOC *ta = &TagAlloc[cmd->device->id][cmd->device->lun]; > - TAG_PRINTK("scsi%d: target %d lun %d returned " > + dprintk(NDEBUG_TAGS, "scsi%d: target %d lun %d returned " > "QUEUE_FULL after %d commands\n", > HOSTNO, cmd->device->id, cmd->device->lun, > ta->nr_allocated); > @@ -2224,14 +2224,14 @@ static void NCR5380_information_transfer > (status_byte(cmd->SCp.Status) == CHECK_CONDITION)) { > scsi_eh_prep_cmnd(cmd, &hostdata->ses, NULL, 0, ~0); > > - ASEN_PRINTK("scsi%d: performing request sense\n", HOSTNO); > + dprintk(NDEBUG_AUTOSENSE, "scsi%d: performing request sense\n", HOSTNO); > > local_irq_save(flags); > LIST(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 " > + dprintk(NDEBUG_QUEUES, "scsi%d: REQUEST SENSE added to head of " > "issue queue\n", H_NO(cmd)); > } else > #endif /* def AUTOSENSE */ > @@ -2277,7 +2277,7 @@ static void NCR5380_information_transfer > cmd->device->tagged_supported = 0; > hostdata->busy[cmd->device->id] |= (1 << cmd->device->lun); > cmd->tag = TAG_NONE; > - TAG_PRINTK("scsi%d: target %d lun %d rejected " > + dprintk(NDEBUG_TAGS, "scsi%d: target %d lun %d rejected " > "QUEUE_TAG message; tagged queuing " > "disabled\n", > HOSTNO, cmd->device->id, cmd->device->lun); > @@ -2294,7 +2294,7 @@ static void NCR5380_information_transfer > hostdata->connected = NULL; > hostdata->disconnected_queue = cmd; > local_irq_restore(flags); > - QU_PRINTK("scsi%d: command for target %d lun %d was " > + dprintk(NDEBUG_QUEUES, "scsi%d: command for target %d lun %d was " > "moved from connected to the " > "disconnected_queue\n", HOSTNO, > cmd->device->id, cmd->device->lun); > @@ -2344,13 +2344,13 @@ static void NCR5380_information_transfer > /* Accept first byte by clearing ACK */ > NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE); > > - EXT_PRINTK("scsi%d: receiving extended message\n", HOSTNO); > + dprintk(NDEBUG_EXTENDED, "scsi%d: receiving extended message\n", HOSTNO); > > len = 2; > data = extended_msg + 1; > phase = PHASE_MSGIN; > NCR5380_transfer_pio(instance, &phase, &len, &data); > - EXT_PRINTK("scsi%d: length=%d, code=0x%02x\n", HOSTNO, > + dprintk(NDEBUG_EXTENDED, "scsi%d: length=%d, code=0x%02x\n", HOSTNO, > (int)extended_msg[1], (int)extended_msg[2]); > > if (!len && extended_msg[1] <= > @@ -2362,7 +2362,7 @@ static void NCR5380_information_transfer > phase = PHASE_MSGIN; > > NCR5380_transfer_pio(instance, &phase, &len, &data); > - EXT_PRINTK("scsi%d: message received, residual %d\n", > + dprintk(NDEBUG_EXTENDED, "scsi%d: message received, residual %d\n", > HOSTNO, len); > > switch (extended_msg[2]) { > @@ -2493,7 +2493,7 @@ static void NCR5380_reselect(struct Scsi > > target_mask = NCR5380_read(CURRENT_SCSI_DATA_REG) & ~(hostdata->id_mask); > > - RSL_PRINTK("scsi%d: reselect\n", HOSTNO); > + dprintk(NDEBUG_RESELECTION, "scsi%d: reselect\n", HOSTNO); > > /* > * At this point, we have detected that our SCSI ID is on the bus, > @@ -2544,7 +2544,7 @@ static void NCR5380_reselect(struct Scsi > if (!NCR5380_transfer_pio(instance, &phase, &len, &data) && > msg[1] == SIMPLE_QUEUE_TAG) > tag = msg[2]; > - TAG_PRINTK("scsi%d: target mask %02x, lun %d sent tag %d at " > + dprintk(NDEBUG_TAGS, "scsi%d: target mask %02x, lun %d sent tag %d at " > "reselection\n", HOSTNO, target_mask, lun, tag); > } > #endif > @@ -2598,7 +2598,7 @@ static void NCR5380_reselect(struct Scsi > NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE); > > hostdata->connected = tmp; > - RSL_PRINTK("scsi%d: nexus established, target = %d, lun = %d, tag = %d\n", > + dprintk(NDEBUG_RESELECTION, "scsi%d: nexus established, target = %d, lun = %d, tag = %d\n", > HOSTNO, tmp->device->id, tmp->device->lun, tmp->tag); > falcon_dont_release--; > } > @@ -2640,7 +2640,7 @@ int NCR5380_abort(Scsi_Cmnd *cmd) > printk(KERN_ERR "scsi%d: !!BINGO!! Falcon has no lock in NCR5380_abort\n", > HOSTNO); > > - ABRT_PRINTK("scsi%d: abort called basr 0x%02x, sr 0x%02x\n", HOSTNO, > + dprintk(NDEBUG_ABORT, "scsi%d: abort called basr 0x%02x, sr 0x%02x\n", HOSTNO, > NCR5380_read(BUS_AND_STATUS_REG), > NCR5380_read(STATUS_REG)); > > @@ -2653,7 +2653,7 @@ int NCR5380_abort(Scsi_Cmnd *cmd) > > if (hostdata->connected == cmd) { > > - ABRT_PRINTK("scsi%d: aborting connected command\n", HOSTNO); > + dprintk(NDEBUG_ABORT, "scsi%d: aborting connected command\n", HOSTNO); > /* > * We should perform BSY checking, and make sure we haven't slipped > * into BUS FREE. > @@ -2705,7 +2705,7 @@ int NCR5380_abort(Scsi_Cmnd *cmd) > SET_NEXT(tmp, NULL); > tmp->result = DID_ABORT << 16; > local_irq_restore(flags); > - ABRT_PRINTK("scsi%d: abort removed command from issue queue.\n", > + dprintk(NDEBUG_ABORT, "scsi%d: abort removed command from issue queue.\n", > HOSTNO); > /* Tagged queuing note: no tag to free here, hasn't been assigned > * yet... */ > @@ -2728,7 +2728,7 @@ int NCR5380_abort(Scsi_Cmnd *cmd) > > if (hostdata->connected) { > local_irq_restore(flags); > - ABRT_PRINTK("scsi%d: abort failed, command connected.\n", HOSTNO); > + dprintk(NDEBUG_ABORT, "scsi%d: abort failed, command connected.\n", HOSTNO); > return SCSI_ABORT_SNOOZE; > } > > @@ -2761,12 +2761,12 @@ int NCR5380_abort(Scsi_Cmnd *cmd) > tmp = NEXT(tmp)) { > if (cmd == tmp) { > local_irq_restore(flags); > - ABRT_PRINTK("scsi%d: aborting disconnected command.\n", HOSTNO); > + dprintk(NDEBUG_ABORT, "scsi%d: aborting disconnected command.\n", HOSTNO); > > if (NCR5380_select(instance, cmd, (int)cmd->tag)) > return SCSI_ABORT_BUSY; > > - ABRT_PRINTK("scsi%d: nexus reestablished.\n", HOSTNO); > + dprintk(NDEBUG_ABORT, "scsi%d: nexus reestablished.\n", HOSTNO); > > do_abort(instance); > > @@ -2890,7 +2890,7 @@ static int NCR5380_bus_reset(Scsi_Cmnd * > */ > > if ((cmd = connected)) { > - ABRT_PRINTK("scsi%d: reset aborted a connected command\n", H_NO(cmd)); > + dprintk(NDEBUG_ABORT, "scsi%d: reset aborted a connected command\n", H_NO(cmd)); > cmd->result = (cmd->result & 0xffff) | (DID_RESET << 16); > cmd->scsi_done(cmd); > } > @@ -2902,7 +2902,7 @@ static int NCR5380_bus_reset(Scsi_Cmnd * > cmd->scsi_done(cmd); > } > if (i > 0) > - ABRT_PRINTK("scsi: reset aborted %d disconnected command(s)\n", i); > + dprintk(NDEBUG_ABORT, "scsi: reset aborted %d disconnected command(s)\n", i); > > /* The Falcon lock should be released after a reset... > */ > @@ -2942,11 +2942,11 @@ static int NCR5380_bus_reset(Scsi_Cmnd * > */ > > if (hostdata->issue_queue) > - ABRT_PRINTK("scsi%d: reset aborted issued command(s)\n", H_NO(cmd)); > + dprintk(NDEBUG_ABORT, "scsi%d: reset aborted issued command(s)\n", H_NO(cmd)); > if (hostdata->connected) > - ABRT_PRINTK("scsi%d: reset aborted a connected command\n", H_NO(cmd)); > + dprintk(NDEBUG_ABORT, "scsi%d: reset aborted a connected command\n", H_NO(cmd)); > if (hostdata->disconnected_queue) > - ABRT_PRINTK("scsi%d: reset aborted disconnected command(s)\n", H_NO(cmd)); > + dprintk(NDEBUG_ABORT, "scsi%d: reset aborted disconnected command(s)\n", H_NO(cmd)); > > local_irq_save(flags); > hostdata->issue_queue = NULL; > Index: linux-m68k/drivers/scsi/atari_scsi.c > =================================================================== > --- linux-m68k.orig/drivers/scsi/atari_scsi.c 2014-03-19 23:23:02.000000000 +1100 > +++ linux-m68k/drivers/scsi/atari_scsi.c 2014-03-19 23:34:44.000000000 +1100 > @@ -313,7 +313,7 @@ static irqreturn_t scsi_tt_intr(int irq, > > dma_stat = tt_scsi_dma.dma_ctrl; > > - INT_PRINTK("scsi%d: NCR5380 interrupt, DMA status = %02x\n", > + dprintk(NDEBUG_INTR, "scsi%d: NCR5380 interrupt, DMA status = %02x\n", > atari_scsi_host->host_no, dma_stat & 0xff); > > /* Look if it was the DMA that has interrupted: First possibility > @@ -339,7 +339,7 @@ static irqreturn_t scsi_tt_intr(int irq, > if ((dma_stat & 0x02) && !(dma_stat & 0x40)) { > atari_dma_residual = HOSTDATA_DMALEN - (SCSI_DMA_READ_P(dma_addr) - atari_dma_startaddr); > > - DMA_PRINTK("SCSI DMA: There are %ld residual bytes.\n", > + dprintk(NDEBUG_DMA, "SCSI DMA: There are %ld residual bytes.\n", > atari_dma_residual); > > if ((signed int)atari_dma_residual < 0) > @@ -370,7 +370,7 @@ static irqreturn_t scsi_tt_intr(int irq, > * other command. These shouldn't disconnect anyway. > */ > if (atari_dma_residual & 0x1ff) { > - DMA_PRINTK("SCSI DMA: DMA bug corrected, " > + dprintk(NDEBUG_DMA, "SCSI DMA: DMA bug corrected, " > "difference %ld bytes\n", > 512 - (atari_dma_residual & 0x1ff)); > atari_dma_residual = (atari_dma_residual + 511) & ~0x1ff; > @@ -437,7 +437,7 @@ static irqreturn_t scsi_falcon_intr(int > "ST-DMA fifo\n", transferred & 15); > > atari_dma_residual = HOSTDATA_DMALEN - transferred; > - DMA_PRINTK("SCSI DMA: There are %ld residual bytes.\n", > + dprintk(NDEBUG_DMA, "SCSI DMA: There are %ld residual bytes.\n", > atari_dma_residual); > } else > atari_dma_residual = 0; > @@ -473,11 +473,11 @@ static void atari_scsi_fetch_restbytes(v > /* there are 'nr' bytes left for the last long address > before the DMA pointer */ > phys_dst ^= nr; > - DMA_PRINTK("SCSI DMA: there are %d rest bytes for phys addr 0x%08lx", > + dprintk(NDEBUG_DMA, "SCSI DMA: there are %d rest bytes for phys addr 0x%08lx", > nr, phys_dst); > /* The content of the DMA pointer is a physical address! */ > dst = phys_to_virt(phys_dst); > - DMA_PRINTK(" = virt addr %p\n", dst); > + dprintk(NDEBUG_DMA, " = virt addr %p\n", dst); > for (src = (char *)&tt_scsi_dma.dma_restdata; nr != 0; --nr) > *dst++ = *src++; > } > @@ -877,7 +877,7 @@ static unsigned long atari_scsi_dma_setu > { > unsigned long addr = virt_to_phys(data); > > - DMA_PRINTK("scsi%d: setting up dma, data = %p, phys = %lx, count = %ld, " > + dprintk(NDEBUG_DMA, "scsi%d: setting up dma, data = %p, phys = %lx, count = %ld, " > "dir = %d\n", instance->host_no, data, addr, count, dir); > > if (!IS_A_TT() && !STRAM_ADDR(addr)) { > @@ -1057,7 +1057,7 @@ static unsigned long atari_dma_xfer_len( > possible_len = limit; > > if (possible_len != wanted_len) > - DMA_PRINTK("Sorry, must cut DMA transfer size to %ld bytes " > + dprintk(NDEBUG_DMA, "Sorry, must cut DMA transfer size to %ld bytes " > "instead of %ld\n", possible_len, wanted_len); > > return possible_len; > Index: linux-m68k/drivers/scsi/sun3_NCR5380.c > =================================================================== > --- linux-m68k.orig/drivers/scsi/sun3_NCR5380.c 2014-03-19 23:34:43.000000000 +1100 > +++ linux-m68k/drivers/scsi/sun3_NCR5380.c 2014-03-19 23:34:44.000000000 +1100 > @@ -364,7 +364,7 @@ static int is_lun_busy(struct scsi_cmnd > return( 0 ); > if (TagAlloc[cmd->device->id][cmd->device->lun].nr_allocated >= > TagAlloc[cmd->device->id][cmd->device->lun].queue_size ) { > - TAG_PRINTK( "scsi%d: target %d lun %d: no free tags\n", > + dprintk(NDEBUG_TAGS, "scsi%d: target %d lun %d: no free tags\n", > H_NO(cmd), cmd->device->id, cmd->device->lun ); > return( 1 ); > } > @@ -388,7 +388,7 @@ static void cmd_get_tag(struct scsi_cmnd > !setup_use_tagged_queuing || !cmd->device->tagged_supported) { > cmd->tag = TAG_NONE; > hostdata->busy[cmd->device->id] |= (1 << cmd->device->lun); > - TAG_PRINTK( "scsi%d: target %d lun %d now allocated by untagged " > + dprintk(NDEBUG_TAGS, "scsi%d: target %d lun %d now allocated by untagged " > "command\n", H_NO(cmd), cmd->device->id, cmd->device->lun ); > } > else { > @@ -397,7 +397,7 @@ static void cmd_get_tag(struct scsi_cmnd > cmd->tag = find_first_zero_bit( &ta->allocated, MAX_TAGS ); > set_bit( cmd->tag, &ta->allocated ); > ta->nr_allocated++; > - TAG_PRINTK( "scsi%d: using tag %d for target %d lun %d " > + dprintk(NDEBUG_TAGS, "scsi%d: using tag %d for target %d lun %d " > "(now %d tags in use)\n", > H_NO(cmd), cmd->tag, cmd->device->id, cmd->device->lun, > ta->nr_allocated ); > @@ -415,7 +415,7 @@ static void cmd_free_tag(struct scsi_cmn > > if (cmd->tag == TAG_NONE) { > hostdata->busy[cmd->device->id] &= ~(1 << cmd->device->lun); > - TAG_PRINTK( "scsi%d: target %d lun %d untagged cmd finished\n", > + dprintk(NDEBUG_TAGS, "scsi%d: target %d lun %d untagged cmd finished\n", > H_NO(cmd), cmd->device->id, cmd->device->lun ); > } > else if (cmd->tag >= MAX_TAGS) { > @@ -426,7 +426,7 @@ static void cmd_free_tag(struct scsi_cmn > TAG_ALLOC *ta = &TagAlloc[cmd->device->id][cmd->device->lun]; > clear_bit( cmd->tag, &ta->allocated ); > ta->nr_allocated--; > - TAG_PRINTK( "scsi%d: freed tag %d for target %d lun %d\n", > + dprintk(NDEBUG_TAGS, "scsi%d: freed tag %d for target %d lun %d\n", > H_NO(cmd), cmd->tag, cmd->device->id, cmd->device->lun ); > } > } > @@ -618,7 +618,7 @@ static inline void NCR5380_all_init (voi > { > static int done = 0; > if (!done) { > - INI_PRINTK("scsi : NCR5380_all_init()\n"); > + dprintk(NDEBUG_INIT, "scsi : NCR5380_all_init()\n"); > done = 1; > } > } > @@ -928,7 +928,7 @@ static int NCR5380_queue_command_lck(str > > local_irq_restore(flags); > > - QU_PRINTK("scsi%d: command added to %s of queue\n", H_NO(cmd), > + dprintk(NDEBUG_QUEUES, "scsi%d: command added to %s of queue\n", H_NO(cmd), > (cmd->cmnd[0] == REQUEST_SENSE) ? "head" : "tail"); > > /* If queue_command() is called from an interrupt (real one or bottom > @@ -998,7 +998,7 @@ static void NCR5380_main (struct work_st > done = 1; > > if (!hostdata->connected) { > - MAIN_PRINTK( "scsi%d: not connected\n", HOSTNO ); > + dprintk(NDEBUG_MAIN, "scsi%d: not connected\n", HOSTNO ); > /* > * Search through the issue_queue for a command destined > * for a target that's not busy. > @@ -1043,7 +1043,7 @@ static void NCR5380_main (struct work_st > * On failure, we must add the command back to the > * issue queue so we can keep trying. > */ > - MAIN_PRINTK("scsi%d: main(): command for target %d " > + dprintk(NDEBUG_MAIN, "scsi%d: main(): command for target %d " > "lun %d removed from issue_queue\n", > HOSTNO, tmp->device->id, tmp->device->lun); > /* > @@ -1072,7 +1072,7 @@ static void NCR5380_main (struct work_st > cmd_free_tag( tmp ); > #endif > local_irq_restore(flags); > - MAIN_PRINTK("scsi%d: main(): select() failed, " > + dprintk(NDEBUG_MAIN, "scsi%d: main(): select() failed, " > "returned to issue_queue\n", HOSTNO); > if (hostdata->connected) > break; > @@ -1086,10 +1086,10 @@ static void NCR5380_main (struct work_st > #endif > ) { > local_irq_restore(flags); > - MAIN_PRINTK("scsi%d: main: performing information transfer\n", > + dprintk(NDEBUG_MAIN, "scsi%d: main: performing information transfer\n", > HOSTNO); > NCR5380_information_transfer(instance); > - MAIN_PRINTK("scsi%d: main: done set false\n", HOSTNO); > + dprintk(NDEBUG_MAIN, "scsi%d: main: done set false\n", HOSTNO); > done = 0; > } > } while (!done); > @@ -1126,7 +1126,7 @@ static void NCR5380_dma_complete( struct > return; > } > > - DMA_PRINTK("scsi%d: real DMA transfer complete, basr 0x%X, sr 0x%X\n", > + dprintk(NDEBUG_DMA, "scsi%d: real DMA transfer complete, basr 0x%X, sr 0x%X\n", > HOSTNO, NCR5380_read(BUS_AND_STATUS_REG), > NCR5380_read(STATUS_REG)); > > @@ -1185,27 +1185,27 @@ static irqreturn_t NCR5380_intr (int irq > int done = 1, handled = 0; > unsigned char basr; > > - INT_PRINTK("scsi%d: NCR5380 irq triggered\n", HOSTNO); > + dprintk(NDEBUG_INTR, "scsi%d: NCR5380 irq triggered\n", HOSTNO); > > /* Look for pending interrupts */ > basr = NCR5380_read(BUS_AND_STATUS_REG); > - INT_PRINTK("scsi%d: BASR=%02x\n", HOSTNO, basr); > + dprintk(NDEBUG_INTR, "scsi%d: BASR=%02x\n", HOSTNO, basr); > /* dispatch to appropriate routine if found and done=0 */ > if (basr & BASR_IRQ) { > NCR5380_dprint(NDEBUG_INTR, instance); > if ((NCR5380_read(STATUS_REG) & (SR_SEL|SR_IO)) == (SR_SEL|SR_IO)) { > done = 0; > // ENABLE_IRQ(); > - INT_PRINTK("scsi%d: SEL interrupt\n", HOSTNO); > + dprintk(NDEBUG_INTR, "scsi%d: SEL interrupt\n", HOSTNO); > NCR5380_reselect(instance); > (void) NCR5380_read(RESET_PARITY_INTERRUPT_REG); > } > else if (basr & BASR_PARITY_ERROR) { > - INT_PRINTK("scsi%d: PARITY interrupt\n", HOSTNO); > + dprintk(NDEBUG_INTR, "scsi%d: PARITY interrupt\n", HOSTNO); > (void) NCR5380_read(RESET_PARITY_INTERRUPT_REG); > } > else if ((NCR5380_read(STATUS_REG) & SR_RST) == SR_RST) { > - INT_PRINTK("scsi%d: RESET interrupt\n", HOSTNO); > + dprintk(NDEBUG_INTR, "scsi%d: RESET interrupt\n", HOSTNO); > (void)NCR5380_read(RESET_PARITY_INTERRUPT_REG); > } > else { > @@ -1225,7 +1225,7 @@ static irqreturn_t NCR5380_intr (int irq > ((basr & BASR_END_DMA_TRANSFER) || > !(basr & BASR_PHASE_MATCH))) { > > - INT_PRINTK("scsi%d: PHASE MISM or EOP interrupt\n", HOSTNO); > + dprintk(NDEBUG_INTR, "scsi%d: PHASE MISM or EOP interrupt\n", HOSTNO); > NCR5380_dma_complete( instance ); > done = 0; > // ENABLE_IRQ(); > @@ -1234,7 +1234,7 @@ static irqreturn_t NCR5380_intr (int irq > { > /* MS: Ignore unknown phase mismatch interrupts (caused by EOP interrupt) */ > if (basr & BASR_PHASE_MATCH) > - INT_PRINTK("scsi%d: unknown interrupt, " > + dprintk(NDEBUG_INTR, "scsi%d: unknown interrupt, " > "BASR 0x%x, MR 0x%x, SR 0x%x\n", > HOSTNO, basr, NCR5380_read(MODE_REG), > NCR5380_read(STATUS_REG)); > @@ -1258,7 +1258,7 @@ static irqreturn_t NCR5380_intr (int irq > } > > if (!done) { > - INT_PRINTK("scsi%d: in int routine, calling main\n", HOSTNO); > + dprintk(NDEBUG_INTR, "scsi%d: in int routine, calling main\n", HOSTNO); > /* Put a call to NCR5380_main() on the queue... */ > queue_main(); > } > @@ -1335,7 +1335,7 @@ static int NCR5380_select(struct Scsi_Ho > > hostdata->restart_select = 0; > NCR5380_dprint(NDEBUG_ARBITRATION, instance); > - ARB_PRINTK("scsi%d: starting arbitration, id = %d\n", HOSTNO, > + dprintk(NDEBUG_ARBITRATION, "scsi%d: starting arbitration, id = %d\n", HOSTNO, > instance->this_id); > > /* > @@ -1381,7 +1381,7 @@ static int NCR5380_select(struct Scsi_Ho > && !hostdata->connected); > #endif > > - ARB_PRINTK("scsi%d: arbitration complete\n", HOSTNO); > + dprintk(NDEBUG_ARBITRATION, "scsi%d: arbitration complete\n", HOSTNO); > > if (hostdata->connected) { > NCR5380_write(MODE_REG, MR_BASE); > @@ -1402,7 +1402,7 @@ static int NCR5380_select(struct Scsi_Ho > (NCR5380_read(INITIATOR_COMMAND_REG) & ICR_ARBITRATION_LOST) || > hostdata->connected) { > NCR5380_write(MODE_REG, MR_BASE); > - ARB_PRINTK("scsi%d: lost arbitration, deasserting MR_ARBITRATE\n", > + dprintk(NDEBUG_ARBITRATION, "scsi%d: lost arbitration, deasserting MR_ARBITRATE\n", > HOSTNO); > return -1; > } > @@ -1417,7 +1417,7 @@ static int NCR5380_select(struct Scsi_Ho > hostdata->connected) { > NCR5380_write(MODE_REG, MR_BASE); > NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE); > - ARB_PRINTK("scsi%d: lost arbitration, deasserting ICR_ASSERT_SEL\n", > + dprintk(NDEBUG_ARBITRATION, "scsi%d: lost arbitration, deasserting ICR_ASSERT_SEL\n", > HOSTNO); > return -1; > } > @@ -1440,7 +1440,7 @@ static int NCR5380_select(struct Scsi_Ho > return -1; > } > > - ARB_PRINTK("scsi%d: won arbitration\n", HOSTNO); > + dprintk(NDEBUG_ARBITRATION, "scsi%d: won arbitration\n", HOSTNO); > > /* > * Now that we have won arbitration, start Selection process, asserting > @@ -1500,7 +1500,7 @@ static int NCR5380_select(struct Scsi_Ho > > udelay(1); > > - SEL_PRINTK("scsi%d: selecting target %d\n", HOSTNO, cmd->device->id); > + dprintk(NDEBUG_SELECTION, "scsi%d: selecting target %d\n", HOSTNO, cmd->device->id); > > /* > * The SCSI specification calls for a 250 ms timeout for the actual > @@ -1568,7 +1568,7 @@ static int NCR5380_select(struct Scsi_Ho > #endif > cmd->scsi_done(cmd); > NCR5380_write(SELECT_ENABLE_REG, hostdata->id_mask); > - SEL_PRINTK("scsi%d: target did not respond within 250ms\n", HOSTNO); > + dprintk(NDEBUG_SELECTION, "scsi%d: target did not respond within 250ms\n", HOSTNO); > NCR5380_write(SELECT_ENABLE_REG, hostdata->id_mask); > return 0; > } > @@ -1593,7 +1593,7 @@ static int NCR5380_select(struct Scsi_Ho > /* Wait for start of REQ/ACK handshake */ > while (!(NCR5380_read(STATUS_REG) & SR_REQ)); > > - SEL_PRINTK("scsi%d: target %d selected, going into MESSAGE OUT phase.\n", > + dprintk(NDEBUG_SELECTION, "scsi%d: target %d selected, going into MESSAGE OUT phase.\n", > HOSTNO, cmd->device->id); > tmp[0] = IDENTIFY(1, cmd->device->lun); > > @@ -1613,7 +1613,7 @@ static int NCR5380_select(struct Scsi_Ho > data = tmp; > phase = PHASE_MSGOUT; > NCR5380_transfer_pio(instance, &phase, &len, &data); > - SEL_PRINTK("scsi%d: nexus established.\n", HOSTNO); > + dprintk(NDEBUG_SELECTION, "scsi%d: nexus established.\n", HOSTNO); > /* XXX need to handle errors here */ > hostdata->connected = cmd; > #ifndef SUPPORT_TAGS > @@ -1676,11 +1676,11 @@ static int NCR5380_transfer_pio( struct > */ > while (!((tmp = NCR5380_read(STATUS_REG)) & SR_REQ)); > > - HSH_PRINTK("scsi%d: REQ detected\n", HOSTNO); > + dprintk(NDEBUG_HANDSHAKE, "scsi%d: REQ detected\n", HOSTNO); > > /* Check for phase mismatch */ > if ((tmp & PHASE_MASK) != p) { > - PIO_PRINTK("scsi%d: phase mismatch\n", HOSTNO); > + dprintk(NDEBUG_PIO, "scsi%d: phase mismatch\n", HOSTNO); > NCR5380_dprint_phase(NDEBUG_PIO, instance); > break; > } > @@ -1721,7 +1721,7 @@ static int NCR5380_transfer_pio( struct > > while (NCR5380_read(STATUS_REG) & SR_REQ); > > - HSH_PRINTK("scsi%d: req false, handshake complete\n", HOSTNO); > + dprintk(NDEBUG_HANDSHAKE, "scsi%d: req false, handshake complete\n", HOSTNO); > > /* > * We have several special cases to consider during REQ/ACK handshaking : > @@ -1742,7 +1742,7 @@ static int NCR5380_transfer_pio( struct > } > } while (--c); > > - PIO_PRINTK("scsi%d: residual %d\n", HOSTNO, c); > + dprintk(NDEBUG_PIO, "scsi%d: residual %d\n", HOSTNO, c); > > *count = c; > *data = d; > @@ -1850,7 +1850,7 @@ static int NCR5380_transfer_dma( struct > } > hostdata->dma_len = c; > > - DMA_PRINTK("scsi%d: initializing DMA for %s, %d bytes %s %p\n", > + dprintk(NDEBUG_DMA, "scsi%d: initializing DMA for %s, %d bytes %s %p\n", > HOSTNO, (p & SR_IO) ? "reading" : "writing", > c, (p & SR_IO) ? "to" : "from", *data); > > @@ -1992,7 +1992,7 @@ static void NCR5380_information_transfer > --cmd->SCp.buffers_residual; > cmd->SCp.this_residual = cmd->SCp.buffer->length; > cmd->SCp.ptr = SGADDR(cmd->SCp.buffer); > - INF_PRINTK("scsi%d: %d bytes and %d buffers left\n", > + dprintk(NDEBUG_INFORMATION, "scsi%d: %d bytes and %d buffers left\n", > HOSTNO, cmd->SCp.this_residual, > cmd->SCp.buffers_residual); > } > @@ -2084,7 +2084,7 @@ static void NCR5380_information_transfer > /* Accept message by clearing ACK */ > NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE); > > - LNK_PRINTK("scsi%d: target %d lun %d linked command " > + dprintk(NDEBUG_LINKED, "scsi%d: target %d lun %d linked command " > "complete.\n", HOSTNO, cmd->device->id, cmd->device->lun); > > /* Enable reselect interrupts */ > @@ -2109,7 +2109,7 @@ static void NCR5380_information_transfer > * and don't free it! */ > cmd->next_link->tag = cmd->tag; > cmd->result = cmd->SCp.Status | (cmd->SCp.Message << 8); > - LNK_PRINTK("scsi%d: target %d lun %d linked request " > + dprintk(NDEBUG_LINKED, "scsi%d: target %d lun %d linked request " > "done, calling scsi_done().\n", > HOSTNO, cmd->device->id, cmd->device->lun); > #ifdef NCR5380_STATS > @@ -2124,7 +2124,7 @@ static void NCR5380_information_transfer > /* Accept message by clearing ACK */ > NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE); > hostdata->connected = NULL; > - QU_PRINTK("scsi%d: command for target %d, lun %d " > + dprintk(NDEBUG_QUEUES, "scsi%d: command for target %d, lun %d " > "completed\n", HOSTNO, cmd->device->id, cmd->device->lun); > #ifdef SUPPORT_TAGS > cmd_free_tag( cmd ); > @@ -2138,7 +2138,7 @@ static void NCR5380_information_transfer > /* ++Andreas: the mid level code knows about > QUEUE_FULL now. */ > TAG_ALLOC *ta = &TagAlloc[cmd->device->id][cmd->device->lun]; > - TAG_PRINTK("scsi%d: target %d lun %d returned " > + dprintk(NDEBUG_TAGS, "scsi%d: target %d lun %d returned " > "QUEUE_FULL after %d commands\n", > HOSTNO, cmd->device->id, cmd->device->lun, > ta->nr_allocated); > @@ -2182,7 +2182,7 @@ static void NCR5380_information_transfer > if ((cmd->cmnd[0] != REQUEST_SENSE) && > (status_byte(cmd->SCp.Status) == CHECK_CONDITION)) { > scsi_eh_prep_cmnd(cmd, &hostdata->ses, NULL, 0, ~0); > - ASEN_PRINTK("scsi%d: performing request sense\n", > + dprintk(NDEBUG_AUTOSENSE, "scsi%d: performing request sense\n", > HOSTNO); > /* this is initialized from initialize_SCp > cmd->SCp.buffer = NULL; > @@ -2194,7 +2194,7 @@ static void NCR5380_information_transfer > SET_NEXT(cmd, hostdata->issue_queue); > hostdata->issue_queue = (struct scsi_cmnd *) cmd; > local_irq_restore(flags); > - QU_PRINTK("scsi%d: REQUEST SENSE added to head of " > + dprintk(NDEBUG_QUEUES, "scsi%d: REQUEST SENSE added to head of " > "issue queue\n", H_NO(cmd)); > } else > #endif /* def AUTOSENSE */ > @@ -2234,7 +2234,7 @@ static void NCR5380_information_transfer > cmd->device->tagged_supported = 0; > hostdata->busy[cmd->device->id] |= (1 << cmd->device->lun); > cmd->tag = TAG_NONE; > - TAG_PRINTK("scsi%d: target %d lun %d rejected " > + dprintk(NDEBUG_TAGS, "scsi%d: target %d lun %d rejected " > "QUEUE_TAG message; tagged queuing " > "disabled\n", > HOSTNO, cmd->device->id, cmd->device->lun); > @@ -2251,7 +2251,7 @@ static void NCR5380_information_transfer > hostdata->connected = NULL; > hostdata->disconnected_queue = cmd; > local_irq_restore(flags); > - QU_PRINTK("scsi%d: command for target %d lun %d was " > + dprintk(NDEBUG_QUEUES, "scsi%d: command for target %d lun %d was " > "moved from connected to the " > "disconnected_queue\n", HOSTNO, > cmd->device->id, cmd->device->lun); > @@ -2304,13 +2304,13 @@ static void NCR5380_information_transfer > /* Accept first byte by clearing ACK */ > NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE); > > - EXT_PRINTK("scsi%d: receiving extended message\n", HOSTNO); > + dprintk(NDEBUG_EXTENDED, "scsi%d: receiving extended message\n", HOSTNO); > > len = 2; > data = extended_msg + 1; > phase = PHASE_MSGIN; > NCR5380_transfer_pio(instance, &phase, &len, &data); > - EXT_PRINTK("scsi%d: length=%d, code=0x%02x\n", HOSTNO, > + dprintk(NDEBUG_EXTENDED, "scsi%d: length=%d, code=0x%02x\n", HOSTNO, > (int)extended_msg[1], (int)extended_msg[2]); > > if (!len && extended_msg[1] <= > @@ -2322,7 +2322,7 @@ static void NCR5380_information_transfer > phase = PHASE_MSGIN; > > NCR5380_transfer_pio(instance, &phase, &len, &data); > - EXT_PRINTK("scsi%d: message received, residual %d\n", > + dprintk(NDEBUG_EXTENDED, "scsi%d: message received, residual %d\n", > HOSTNO, len); > > switch (extended_msg[2]) { > @@ -2454,7 +2454,7 @@ static void NCR5380_reselect (struct Scs > > target_mask = NCR5380_read(CURRENT_SCSI_DATA_REG) & ~(hostdata->id_mask); > > - RSL_PRINTK("scsi%d: reselect\n", HOSTNO); > + dprintk(NDEBUG_RESELECTION, "scsi%d: reselect\n", HOSTNO); > > /* > * At this point, we have detected that our SCSI ID is on the bus, > @@ -2576,13 +2576,13 @@ static void NCR5380_reselect (struct Scs > if (!NCR5380_transfer_pio(instance, &phase, &len, &data) && > msg[1] == SIMPLE_QUEUE_TAG) > tag = msg[2]; > - TAG_PRINTK("scsi%d: target mask %02x, lun %d sent tag %d at " > + dprintk(NDEBUG_TAGS, "scsi%d: target mask %02x, lun %d sent tag %d at " > "reselection\n", HOSTNO, target_mask, lun, tag); > } > #endif > > hostdata->connected = tmp; > - RSL_PRINTK("scsi%d: nexus established, target = %d, lun = %d, tag = %d\n", > + dprintk(NDEBUG_RESELECTION, "scsi%d: nexus established, target = %d, lun = %d, tag = %d\n", > HOSTNO, tmp->device->id, tmp->device->lun, tmp->tag); > } > > @@ -2618,7 +2618,7 @@ static int NCR5380_abort(struct scsi_cmn > > local_irq_save(flags); > > - ABRT_PRINTK("scsi%d: abort called basr 0x%02x, sr 0x%02x\n", HOSTNO, > + dprintk(NDEBUG_ABORT, "scsi%d: abort called basr 0x%02x, sr 0x%02x\n", HOSTNO, > NCR5380_read(BUS_AND_STATUS_REG), > NCR5380_read(STATUS_REG)); > > @@ -2631,7 +2631,7 @@ static int NCR5380_abort(struct scsi_cmn > > if (hostdata->connected == cmd) { > > - ABRT_PRINTK("scsi%d: aborting connected command\n", HOSTNO); > + dprintk(NDEBUG_ABORT, "scsi%d: aborting connected command\n", HOSTNO); > /* > * We should perform BSY checking, and make sure we haven't slipped > * into BUS FREE. > @@ -2682,7 +2682,7 @@ static int NCR5380_abort(struct scsi_cmn > SET_NEXT(tmp, NULL); > tmp->result = DID_ABORT << 16; > local_irq_restore(flags); > - ABRT_PRINTK("scsi%d: abort removed command from issue queue.\n", > + dprintk(NDEBUG_ABORT, "scsi%d: abort removed command from issue queue.\n", > HOSTNO); > /* Tagged queuing note: no tag to free here, hasn't been assigned > * yet... */ > @@ -2703,7 +2703,7 @@ static int NCR5380_abort(struct scsi_cmn > > if (hostdata->connected) { > local_irq_restore(flags); > - ABRT_PRINTK("scsi%d: abort failed, command connected.\n", HOSTNO); > + dprintk(NDEBUG_ABORT, "scsi%d: abort failed, command connected.\n", HOSTNO); > return SCSI_ABORT_SNOOZE; > } > > @@ -2736,12 +2736,12 @@ static int NCR5380_abort(struct scsi_cmn > tmp = NEXT(tmp)) > if (cmd == tmp) { > local_irq_restore(flags); > - ABRT_PRINTK("scsi%d: aborting disconnected command.\n", HOSTNO); > + dprintk(NDEBUG_ABORT, "scsi%d: aborting disconnected command.\n", HOSTNO); > > if (NCR5380_select (instance, cmd, (int) cmd->tag)) > return SCSI_ABORT_BUSY; > > - ABRT_PRINTK("scsi%d: nexus reestablished.\n", HOSTNO); > + dprintk(NDEBUG_ABORT, "scsi%d: nexus reestablished.\n", HOSTNO); > > do_abort (instance); > > @@ -2853,7 +2853,7 @@ static int NCR5380_bus_reset(struct scsi > */ > > if ((cmd = connected)) { > - ABRT_PRINTK("scsi%d: reset aborted a connected command\n", H_NO(cmd)); > + dprintk(NDEBUG_ABORT, "scsi%d: reset aborted a connected command\n", H_NO(cmd)); > cmd->result = (cmd->result & 0xffff) | (DID_RESET << 16); > cmd->scsi_done( cmd ); > } > @@ -2865,7 +2865,7 @@ static int NCR5380_bus_reset(struct scsi > cmd->scsi_done( cmd ); > } > if (i > 0) > - ABRT_PRINTK("scsi: reset aborted %d disconnected command(s)\n", i); > + dprintk(NDEBUG_ABORT, "scsi: reset aborted %d disconnected command(s)\n", i); > > > /* since all commands have been explicitly terminated, we need to tell > @@ -2899,11 +2899,11 @@ static int NCR5380_bus_reset(struct scsi > */ > > if (hostdata->issue_queue) > - ABRT_PRINTK("scsi%d: reset aborted issued command(s)\n", H_NO(cmd)); > + dprintk(NDEBUG_ABORT, "scsi%d: reset aborted issued command(s)\n", H_NO(cmd)); > if (hostdata->connected) > - ABRT_PRINTK("scsi%d: reset aborted a connected command\n", H_NO(cmd)); > + dprintk(NDEBUG_ABORT, "scsi%d: reset aborted a connected command\n", H_NO(cmd)); > if (hostdata->disconnected_queue) > - ABRT_PRINTK("scsi%d: reset aborted disconnected command(s)\n", H_NO(cmd)); > + dprintk(NDEBUG_ABORT, "scsi%d: reset aborted disconnected command(s)\n", H_NO(cmd)); > > local_irq_save(flags); > hostdata->issue_queue = NULL; > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/