Commit b6877e8c authored by Bartlomiej Zolnierkiewicz's avatar Bartlomiej Zolnierkiewicz Committed by Linus Torvalds

[PATCH] ide: remove never changing FANCY_STATUS_DUMPS define

From: Alan Cox <alan@redhat.com>

Extracted from 2.6.8.1-ac1 by me.
Signed-off-by: default avatarBartlomiej Zolnierkiewicz <bzolnier@elka.pw.edu.pl>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent ee1311f7
...@@ -638,7 +638,6 @@ static u8 idedisk_dump_status (ide_drive_t *drive, const char *msg, u8 stat) ...@@ -638,7 +638,6 @@ static u8 idedisk_dump_status (ide_drive_t *drive, const char *msg, u8 stat)
local_irq_set(flags); local_irq_set(flags);
printk("%s: %s: status=0x%02x", drive->name, msg, stat); printk("%s: %s: status=0x%02x", drive->name, msg, stat);
#if FANCY_STATUS_DUMPS
printk(" { "); printk(" { ");
if (stat & BUSY_STAT) if (stat & BUSY_STAT)
printk("Busy "); printk("Busy ");
...@@ -652,12 +651,10 @@ static u8 idedisk_dump_status (ide_drive_t *drive, const char *msg, u8 stat) ...@@ -652,12 +651,10 @@ static u8 idedisk_dump_status (ide_drive_t *drive, const char *msg, u8 stat)
if (stat & ERR_STAT) printk("Error "); if (stat & ERR_STAT) printk("Error ");
} }
printk("}"); printk("}");
#endif /* FANCY_STATUS_DUMPS */
printk("\n"); printk("\n");
if ((stat & (BUSY_STAT|ERR_STAT)) == ERR_STAT) { if ((stat & (BUSY_STAT|ERR_STAT)) == ERR_STAT) {
err = hwif->INB(IDE_ERROR_REG); err = hwif->INB(IDE_ERROR_REG);
printk("%s: %s: error=0x%02x", drive->name, msg, err); printk("%s: %s: error=0x%02x", drive->name, msg, err);
#if FANCY_STATUS_DUMPS
printk(" { "); printk(" { ");
if (err & ABRT_ERR) printk("DriveStatusError "); if (err & ABRT_ERR) printk("DriveStatusError ");
if (err & ICRC_ERR) if (err & ICRC_ERR)
...@@ -700,7 +697,6 @@ static u8 idedisk_dump_status (ide_drive_t *drive, const char *msg, u8 stat) ...@@ -700,7 +697,6 @@ static u8 idedisk_dump_status (ide_drive_t *drive, const char *msg, u8 stat)
(unsigned long long)HWGROUP(drive)->rq->sector); (unsigned long long)HWGROUP(drive)->rq->sector);
} }
} }
#endif /* FANCY_STATUS_DUMPS */
printk("\n"); printk("\n");
{ {
struct request *rq; struct request *rq;
......
...@@ -1096,7 +1096,6 @@ static ide_startstop_t reset_pollfunc (ide_drive_t *drive) ...@@ -1096,7 +1096,6 @@ static ide_startstop_t reset_pollfunc (ide_drive_t *drive)
drive->failures = 0; drive->failures = 0;
} else { } else {
drive->failures++; drive->failures++;
#if FANCY_STATUS_DUMPS
printk("master: "); printk("master: ");
switch (tmp & 0x7f) { switch (tmp & 0x7f) {
case 1: printk("passed"); case 1: printk("passed");
...@@ -1114,9 +1113,6 @@ static ide_startstop_t reset_pollfunc (ide_drive_t *drive) ...@@ -1114,9 +1113,6 @@ static ide_startstop_t reset_pollfunc (ide_drive_t *drive)
if (tmp & 0x80) if (tmp & 0x80)
printk("; slave: failed"); printk("; slave: failed");
printk("\n"); printk("\n");
#else
printk("failed\n");
#endif /* FANCY_STATUS_DUMPS */
} }
} }
hwgroup->poll_timeout = 0; /* done polling */ hwgroup->poll_timeout = 0; /* done polling */
......
...@@ -465,7 +465,6 @@ byte ide_dump_atapi_status (ide_drive_t *drive, const char *msg, byte stat) ...@@ -465,7 +465,6 @@ byte ide_dump_atapi_status (ide_drive_t *drive, const char *msg, byte stat)
status.all = stat; status.all = stat;
local_irq_set(flags); local_irq_set(flags);
printk("%s: %s: status=0x%02x", drive->name, msg, stat); printk("%s: %s: status=0x%02x", drive->name, msg, stat);
#if FANCY_STATUS_DUMPS
printk(" { "); printk(" { ");
if (status.b.bsy) if (status.b.bsy)
printk("Busy "); printk("Busy ");
...@@ -479,19 +478,16 @@ byte ide_dump_atapi_status (ide_drive_t *drive, const char *msg, byte stat) ...@@ -479,19 +478,16 @@ byte ide_dump_atapi_status (ide_drive_t *drive, const char *msg, byte stat)
if (status.b.check) printk("Error "); if (status.b.check) printk("Error ");
} }
printk("}"); printk("}");
#endif /* FANCY_STATUS_DUMPS */
printk("\n"); printk("\n");
if ((status.all & (status.b.bsy|status.b.check)) == status.b.check) { if ((status.all & (status.b.bsy|status.b.check)) == status.b.check) {
error.all = HWIF(drive)->INB(IDE_ERROR_REG); error.all = HWIF(drive)->INB(IDE_ERROR_REG);
printk("%s: %s: error=0x%02x", drive->name, msg, error.all); printk("%s: %s: error=0x%02x", drive->name, msg, error.all);
#if FANCY_STATUS_DUMPS
if (error.b.ili) printk("IllegalLengthIndication "); if (error.b.ili) printk("IllegalLengthIndication ");
if (error.b.eom) printk("EndOfMedia "); if (error.b.eom) printk("EndOfMedia ");
if (error.b.abrt) printk("Aborted Command "); if (error.b.abrt) printk("Aborted Command ");
if (error.b.mcr) printk("MediaChangeRequested "); if (error.b.mcr) printk("MediaChangeRequested ");
if (error.b.sense_key) printk("LastFailedSense 0x%02x ", if (error.b.sense_key) printk("LastFailedSense 0x%02x ",
error.b.sense_key); error.b.sense_key);
#endif /* FANCY_STATUS_DUMPS */
printk("\n"); printk("\n");
} }
local_irq_restore(flags); local_irq_restore(flags);
......
...@@ -390,7 +390,6 @@ u8 ide_dump_status (ide_drive_t *drive, const char *msg, u8 stat) ...@@ -390,7 +390,6 @@ u8 ide_dump_status (ide_drive_t *drive, const char *msg, u8 stat)
local_irq_set(flags); local_irq_set(flags);
printk(KERN_WARNING "%s: %s: status=0x%02x", drive->name, msg, stat); printk(KERN_WARNING "%s: %s: status=0x%02x", drive->name, msg, stat);
#if FANCY_STATUS_DUMPS
printk(" { "); printk(" { ");
if (stat & BUSY_STAT) { if (stat & BUSY_STAT) {
printk("Busy "); printk("Busy ");
...@@ -404,12 +403,10 @@ u8 ide_dump_status (ide_drive_t *drive, const char *msg, u8 stat) ...@@ -404,12 +403,10 @@ u8 ide_dump_status (ide_drive_t *drive, const char *msg, u8 stat)
if (stat & ERR_STAT) printk("Error "); if (stat & ERR_STAT) printk("Error ");
} }
printk("}"); printk("}");
#endif /* FANCY_STATUS_DUMPS */
printk("\n"); printk("\n");
if ((stat & (BUSY_STAT|ERR_STAT)) == ERR_STAT) { if ((stat & (BUSY_STAT|ERR_STAT)) == ERR_STAT) {
err = hwif->INB(IDE_ERROR_REG); err = hwif->INB(IDE_ERROR_REG);
printk("%s: %s: error=0x%02x", drive->name, msg, err); printk("%s: %s: error=0x%02x", drive->name, msg, err);
#if FANCY_STATUS_DUMPS
if (drive->media == ide_disk) { if (drive->media == ide_disk) {
printk(" { "); printk(" { ");
if (err & ABRT_ERR) printk("DriveStatusError "); if (err & ABRT_ERR) printk("DriveStatusError ");
...@@ -453,7 +450,6 @@ u8 ide_dump_status (ide_drive_t *drive, const char *msg, u8 stat) ...@@ -453,7 +450,6 @@ u8 ide_dump_status (ide_drive_t *drive, const char *msg, u8 stat)
printk(", sector=%llu", (unsigned long long)HWGROUP(drive)->rq->sector); printk(", sector=%llu", (unsigned long long)HWGROUP(drive)->rq->sector);
} }
} }
#endif /* FANCY_STATUS_DUMPS */
printk("\n"); printk("\n");
} }
{ {
......
...@@ -51,9 +51,6 @@ ...@@ -51,9 +51,6 @@
#ifndef OK_TO_RESET_CONTROLLER /* 1 needed for good error recovery */ #ifndef OK_TO_RESET_CONTROLLER /* 1 needed for good error recovery */
#define OK_TO_RESET_CONTROLLER 1 /* 0 for use with AH2372A/B interface */ #define OK_TO_RESET_CONTROLLER 1 /* 0 for use with AH2372A/B interface */
#endif #endif
#ifndef FANCY_STATUS_DUMPS /* 1 for human-readable drive errors */
#define FANCY_STATUS_DUMPS 1 /* 0 to reduce kernel size */
#endif
#ifdef CONFIG_BLK_DEV_CMD640 #ifdef CONFIG_BLK_DEV_CMD640
#if 0 /* change to 1 when debugging cmd640 problems */ #if 0 /* change to 1 when debugging cmd640 problems */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment