Hi Tejun, Jeff, is this one like what you had in mind? I left out the ENTER/EXIT printk's since they are debug level. By the way, should I convert the dbg message levels to the scheme you both proposed now while not everything is converted or wait first. However, i'd rather do it now since it is less work than later :)? This restores the default libata configuration messages printed during booting. Signed-off-by: <petkov@xxxxxxxxxxxxxxxxxxxx> --- libata-dev/drivers/scsi/libata-core.c.orig2 2006-06-27 09:21:44.000000000 +0200 +++ libata-dev/drivers/scsi/libata-core.c 2006-06-27 09:23:54.000000000 +0200 @@ -1359,11 +1359,10 @@ int ata_dev_configure(struct ata_device __FUNCTION__, ap->id, dev->devno); /* print device capabilities */ - if (ata_msg_probe(ap)) - ata_dev_printk(dev, KERN_DEBUG, - "%s: cfg 49:%04x 82:%04x 83:%04x 84:%04x " + if (ata_msg_drv(ap)) + ata_dev_printk(dev, KERN_INFO, + "cfg 49:%04x 82:%04x 83:%04x 84:%04x " "85:%04x 86:%04x 87:%04x 88:%04x\n", - __FUNCTION__, id[49], id[82], id[83], id[84], id[85], id[86], id[87], id[88]); @@ -1405,7 +1404,7 @@ int ata_dev_configure(struct ata_device ata_dev_config_ncq(dev, ncq_desc, sizeof(ncq_desc)); /* print device info to dmesg */ - if (ata_msg_info(ap)) + if (ata_msg_drv(ap)) ata_dev_printk(dev, KERN_INFO, "ATA-%d, " "max %s, %Lu sectors: %s %s\n", ata_id_major_version(id), @@ -1428,7 +1427,7 @@ int ata_dev_configure(struct ata_device } /* print device info to dmesg */ - if (ata_msg_info(ap)) + if (ata_msg_drv(ap)) ata_dev_printk(dev, KERN_INFO, "ATA-%d, " "max %s, %Lu sectors: CHS %u/%u/%u\n", ata_id_major_version(id), @@ -1440,7 +1439,7 @@ int ata_dev_configure(struct ata_device if (dev->id[59] & 0x100) { dev->multi_count = dev->id[59] & 0xff; - if (ata_msg_info(ap)) + if (ata_msg_drv(ap)) ata_dev_printk(dev, KERN_INFO, "ata%u: dev %u multi count %u\n", ap->id, dev->devno, dev->multi_count); @@ -1469,7 +1468,7 @@ int ata_dev_configure(struct ata_device } /* print device info to dmesg */ - if (ata_msg_info(ap)) + if (ata_msg_drv(ap)) ata_dev_printk(dev, KERN_INFO, "ATAPI, max %s%s\n", ata_mode_string(xfer_mask), cdb_intr_string); @@ -1483,7 +1482,7 @@ int ata_dev_configure(struct ata_device /* limit bridge transfers to udma5, 200 sectors */ if (ata_dev_knobble(dev)) { - if (ata_msg_info(ap)) + if (ata_msg_drv(ap)) ata_dev_printk(dev, KERN_INFO, "applying bridge limits\n"); dev->udma_mask &= ATA_UDMA5; ___________________________________________________________ Telefonate ohne weitere Kosten vom PC zum PC: http://messenger.yahoo.de - : send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html