* [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