Date: Sat, 17 Jan 2004 19:47:07 +0000
From: Nuno Monteiro <nuno@itsari.org>
To: marcelo.tosatti@cyclades.com
Cc: linux-kernel@vger.kernel.org
Subject: [PATCH][2.4] Remove ide bootup noise
Message-ID: <20040117194707.GC19667@hobbes.itsari.int>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Disposition: inline
Content-Transfer-Encoding: 7BIT
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1506
Lines: 47

Hi Marcelo,


Recently, while syncing my local tree from 2.4.22 to 2.4.25-pre I found 
this little patchlet (back from the 2.4.20 days, AFAIR) that still 
applies and looks pertinent. I believe the original author is Erik 
Andersen, although I can't say for certain since I don't have the 
original email around anymore. Also, I seem to recall that Jens Axboe 
ok'ed it, but for whatever reason it never found its way into mainline.

This will remove the pointless boot message about wether or not the drive 
supports host protected area. Since we don't report any other drive 
capabilities this should go, as it' is just pointless noise. In 2.6 this 
was killed too. Also, this information can be obtained using 'hdparm'.

Please review and apply.


Regards,


		Nuno


--- linux-2.4.25-pre5/drivers/ide/ide-disk.c.orig	2004-01-15 22:20:25.355342064 +0000
+++ linux-2.4.25-pre5/drivers/ide/ide-disk.c	2004-01-15 22:25:48.189263848 +0000
@@ -1136,10 +1136,7 @@
  */
 static inline int idedisk_supports_host_protected_area(ide_drive_t *drive)
 {
-	int flag = (drive->id->cfs_enable_1 & 0x0400) ? 1 : 0;
-	if (flag)
-		printk("%s: host protected area => %d\n", drive->name, flag);
-	return flag;
+	return ((drive->id->cfs_enable_1 & 0x0400) ? 1 : 0);
 }
 
 /*


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/