diff options
author | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2008-12-29 14:27:37 -0500 |
---|---|---|
committer | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2008-12-29 14:27:37 -0500 |
commit | d688b72c90a7faf13c070d4ebb63cacdf481de32 (patch) | |
tree | edc34f234827fe074b76ab101e892749e95ad99f /drivers/ide | |
parent | 7f92b11c1cc0e865fc64f3148eda87fff74920e1 (diff) |
ide: remove superfluous local_irq_{save,restore}() from ide_dump_status()
No reason to run this function with local IRQs off.
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Diffstat (limited to 'drivers/ide')
-rw-r--r-- | drivers/ide/ide-lib.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/ide/ide-lib.c b/drivers/ide/ide-lib.c index 7913ad5d4979..9f6e33d8a8b2 100644 --- a/drivers/ide/ide-lib.c +++ b/drivers/ide/ide-lib.c | |||
@@ -384,10 +384,8 @@ static void ide_dump_atapi_error(ide_drive_t *drive, u8 err) | |||
384 | 384 | ||
385 | u8 ide_dump_status(ide_drive_t *drive, const char *msg, u8 stat) | 385 | u8 ide_dump_status(ide_drive_t *drive, const char *msg, u8 stat) |
386 | { | 386 | { |
387 | unsigned long flags; | ||
388 | u8 err = 0; | 387 | u8 err = 0; |
389 | 388 | ||
390 | local_irq_save(flags); | ||
391 | printk(KERN_ERR "%s: %s: status=0x%02x { ", drive->name, msg, stat); | 389 | printk(KERN_ERR "%s: %s: status=0x%02x { ", drive->name, msg, stat); |
392 | if (stat & ATA_BUSY) | 390 | if (stat & ATA_BUSY) |
393 | printk(KERN_CONT "Busy "); | 391 | printk(KERN_CONT "Busy "); |
@@ -417,7 +415,6 @@ u8 ide_dump_status(ide_drive_t *drive, const char *msg, u8 stat) | |||
417 | ide_dump_atapi_error(drive, err); | 415 | ide_dump_atapi_error(drive, err); |
418 | } | 416 | } |
419 | ide_dump_opcode(drive); | 417 | ide_dump_opcode(drive); |
420 | local_irq_restore(flags); | ||
421 | return err; | 418 | return err; |
422 | } | 419 | } |
423 | EXPORT_SYMBOL(ide_dump_status); | 420 | EXPORT_SYMBOL(ide_dump_status); |