summaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorTejun Heo <htejun@gmail.com>2007-06-25 21:31:05 +0900
committerJeff Garzik <jeff@garzik.org>2007-06-27 02:44:18 -0400
commit55f3952d45a439cecc36fd845a87026d04c82931 (patch)
tree12c8a8836a1e127a1f7db2e7633c6a3f4d475b79 /include
parent62877f6b0c4943c2231b84b49182a078eb02a777 (diff)
downloadlinux-55f3952d45a439cecc36fd845a87026d04c82931.tar.gz
linux-55f3952d45a439cecc36fd845a87026d04c82931.tar.bz2
linux-55f3952d45a439cecc36fd845a87026d04c82931.zip
libata: kill the infamous abnormal status message
The infamous abnormal status message triggers on not so abnormal cases including empty port and even when it's being triggered on actual errors the info it provides is redundant and out of context - higher level functions will print the info in better safe later anyway. Also, by being triggered all the time, it leads people to think that the abnormality is somehow related to all ATA and system problems they're experiencing and gives owners of healthy systems unfounded doubts about the integrity of the universe. Make it a DPRINTK and save the universe. Signed-off-by: Tejun Heo <htejun@gmail.com> Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'include')
-rw-r--r--include/linux/libata.h8
1 files changed, 3 insertions, 5 deletions
diff --git a/include/linux/libata.h b/include/linux/libata.h
index 745c4f9b4caa..a3380f808630 100644
--- a/include/linux/libata.h
+++ b/include/linux/libata.h
@@ -1088,11 +1088,9 @@ static inline u8 ata_wait_idle(struct ata_port *ap)
{
u8 status = ata_busy_wait(ap, ATA_BUSY | ATA_DRQ, 1000);
- if (status != 0xff && (status & (ATA_BUSY | ATA_DRQ))) {
- if (ata_msg_warn(ap))
- printk(KERN_WARNING "ATA: abnormal status 0x%X on port 0x%p\n",
- status, ap->ioaddr.status_addr);
- }
+ if (status != 0xff && (status & (ATA_BUSY | ATA_DRQ)))
+ DPRINTK("ATA: abnormal status 0x%X on port 0x%p\n",
+ status, ap->ioaddr.status_addr);
return status;
}