diff options
| author | Finn Thain <fthain@telegraphics.com.au> | 2014-05-29 00:43:43 +1000 |
|---|---|---|
| committer | Christoph Hellwig <hch@lst.de> | 2014-05-28 18:14:16 +0200 |
| commit | 16b9d870a0f19f6c1061927a623d9c51c09f2bed (patch) | |
| tree | 9ac58f343fd79d23b62a6ea3e1fd033525dad54a /drivers/scsi/sun3_NCR5380.c | |
| parent | a48ac9e5d4f3555bdec2fd61e49fc71b26810d68 (diff) | |
| download | op-kernel-dev-16b9d870a0f19f6c1061927a623d9c51c09f2bed.zip op-kernel-dev-16b9d870a0f19f6c1061927a623d9c51c09f2bed.tar.gz | |
scsi/NCR5380: dprintk macro
This is the delta between the two submissions:
[PATCH 00/12] scsi/NCR5380: fix debugging macros and #include structure
and
[PATCH v2 00/12] scsi/NCR5380: fix debugging macros and #include structure
The macro definition changes were discussed on the mailing list during
review. The idea is to get the compiler to check the parameters of
disabled printk() calls so that the debugging code doesn't rot again.
Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/scsi/sun3_NCR5380.c')
| -rw-r--r-- | drivers/scsi/sun3_NCR5380.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/scsi/sun3_NCR5380.c b/drivers/scsi/sun3_NCR5380.c index 61a2c10..8822079 100644 --- a/drivers/scsi/sun3_NCR5380.c +++ b/drivers/scsi/sun3_NCR5380.c @@ -1006,12 +1006,8 @@ static void NCR5380_main (struct work_struct *bl) for (tmp = (struct scsi_cmnd *) hostdata->issue_queue, prev = NULL; tmp; prev = tmp, tmp = NEXT(tmp) ) { -#if (NDEBUG & NDEBUG_LISTS) if (prev != tmp) - printk("MAIN tmp=%p target=%d busy=%d lun=%d\n", - tmp, tmp->target, hostdata->busy[tmp->target], - tmp->lun); -#endif + dprintk(NDEBUG_LISTS, "MAIN tmp=%p target=%d busy=%d lun=%d\n", tmp, tmp->device->id, hostdata->busy[tmp->device->id], tmp->device->lun); /* When we find one, remove it from the issue queue. */ /* ++guenther: possible race with Falcon locking */ if ( |
