linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] libata-core.c: restore configuration boot messages in ata_dev_configure(), v2
@ 2006-06-27 12:51 Borislav Petkov
  2006-06-27 12:57 ` Tejun Heo
  2006-07-06  2:17 ` Jeff Garzik
  0 siblings, 2 replies; 3+ messages in thread
From: Borislav Petkov @ 2006-06-27 12:51 UTC (permalink / raw)
  To: linux IDE ML; +Cc: Tejun Heo, Jeff Garzik

This one looks better, IMHO.

This restores the default libata configuration messages printed during booting.

Signed-off-by: <petkov@math.uni-muenster.de>


--- 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 14:47:58.000000000 +0200
@@ -1405,7 +1405,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) && print_info)
 				ata_dev_printk(dev, KERN_INFO, "ATA-%d, "
 					"max %s, %Lu sectors: %s %s\n",
 					ata_id_major_version(id),
@@ -1428,7 +1428,7 @@ int ata_dev_configure(struct ata_device 
 			}
 
 			/* print device info to dmesg */
-			if (ata_msg_info(ap))
+			if (ata_msg_drv(ap) && print_info)
 				ata_dev_printk(dev, KERN_INFO, "ATA-%d, "
 					"max %s, %Lu sectors: CHS %u/%u/%u\n",
 					ata_id_major_version(id),
@@ -1440,7 +1440,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) && print_info)
 				ata_dev_printk(dev, KERN_INFO,
 					"ata%u: dev %u multi count %u\n",
 					ap->id, dev->devno, dev->multi_count);
@@ -1469,7 +1469,7 @@ int ata_dev_configure(struct ata_device 
 		}
 
 		/* print device info to dmesg */
-		if (ata_msg_info(ap))
+		if (ata_msg_drv(ap) && print_info)
 			ata_dev_printk(dev, KERN_INFO, "ATAPI, max %s%s\n",
 				       ata_mode_string(xfer_mask),
 				       cdb_intr_string);
@@ -1483,7 +1483,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) && print_info)
 			ata_dev_printk(dev, KERN_INFO,
 				       "applying bridge limits\n");
 		dev->udma_mask &= ATA_UDMA5;

	

	
		
___________________________________________________________ 
Gesendet von Yahoo! Mail - Jetzt mit 1GB Speicher kostenlos - Hier anmelden: http://mail.yahoo.de


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] libata-core.c: restore configuration boot messages in ata_dev_configure(), v2
  2006-06-27 12:51 [PATCH] libata-core.c: restore configuration boot messages in ata_dev_configure(), v2 Borislav Petkov
@ 2006-06-27 12:57 ` Tejun Heo
  2006-07-06  2:17 ` Jeff Garzik
  1 sibling, 0 replies; 3+ messages in thread
From: Tejun Heo @ 2006-06-27 12:57 UTC (permalink / raw)
  To: Borislav Petkov; +Cc: linux IDE ML, Jeff Garzik

Borislav Petkov wrote:
> This one looks better, IMHO.
> 
> This restores the default libata configuration messages printed during booting.
> 
> Signed-off-by: <petkov@math.uni-muenster.de>
Signed-off-by: Tejun Heo <htejun@gmail.com>

-- 
tejun

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] libata-core.c: restore configuration boot messages in ata_dev_configure(), v2
  2006-06-27 12:51 [PATCH] libata-core.c: restore configuration boot messages in ata_dev_configure(), v2 Borislav Petkov
  2006-06-27 12:57 ` Tejun Heo
@ 2006-07-06  2:17 ` Jeff Garzik
  1 sibling, 0 replies; 3+ messages in thread
From: Jeff Garzik @ 2006-07-06  2:17 UTC (permalink / raw)
  To: Borislav Petkov; +Cc: linux IDE ML, Tejun Heo

applied


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2006-07-06  2:18 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-06-27 12:51 [PATCH] libata-core.c: restore configuration boot messages in ata_dev_configure(), v2 Borislav Petkov
2006-06-27 12:57 ` Tejun Heo
2006-07-06  2:17 ` Jeff Garzik

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).