public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH][2.4] Remove ide bootup noise
@ 2004-01-17 19:47 Nuno Monteiro
  2004-01-17 19:57 ` Erik Andersen
  0 siblings, 1 reply; 2+ messages in thread
From: Nuno Monteiro @ 2004-01-17 19:47 UTC (permalink / raw)
  To: marcelo.tosatti; +Cc: linux-kernel

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);
 }
 
 /*



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

* Re: [PATCH][2.4] Remove ide bootup noise
  2004-01-17 19:47 [PATCH][2.4] Remove ide bootup noise Nuno Monteiro
@ 2004-01-17 19:57 ` Erik Andersen
  0 siblings, 0 replies; 2+ messages in thread
From: Erik Andersen @ 2004-01-17 19:57 UTC (permalink / raw)
  To: Nuno Monteiro; +Cc: marcelo.tosatti, linux-kernel

On Sat Jan 17, 2004 at 07:47:07PM +0000, Nuno Monteiro wrote:
> 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 

Yup, that was me.  I later submitted a follow-on patch that
implemented proper hpa detection, which was accepted into
2.6.x, but never made it into 2.4,

 -Erik

--
Erik B. Andersen             http://codepoet-consulting.com/
--This message was written using 73% post-consumer electrons--

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

end of thread, other threads:[~2004-01-17 19:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-01-17 19:47 [PATCH][2.4] Remove ide bootup noise Nuno Monteiro
2004-01-17 19:57 ` Erik Andersen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox