* [PATCH] de-cryptify ide-disk host protected area output
@ 2002-10-26 13:07 bert hubert
2002-10-26 13:24 ` Erik Andersen
0 siblings, 1 reply; 4+ messages in thread
From: bert hubert @ 2002-10-26 13:07 UTC (permalink / raw)
To: andre, linux-kernel
Useless number '1' being printed leading to operator confusion.
--- linux-2.5.44/drivers/ide/ide-disk.c~orig Sat Oct 26 14:59:35 2002
+++ linux-2.5.44/drivers/ide/ide-disk.c Sat Oct 26 15:00:40 2002
@@ -1128,7 +1128,7 @@
{
int flag = (drive->id->cfs_enable_1 & 0x0400) ? 1 : 0;
if (flag)
- printk("%s: host protected area => %d\n", drive->name, flag);
+ printk("%s: supports host protected area", drive->name);
return flag;
}
--
http://www.PowerDNS.com Versatile DNS Software & Services
http://lartc.org Linux Advanced Routing & Traffic Control HOWTO
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] de-cryptify ide-disk host protected area output
2002-10-26 13:07 [PATCH] de-cryptify ide-disk host protected area output bert hubert
@ 2002-10-26 13:24 ` Erik Andersen
2002-10-26 14:25 ` Alan Cox
2002-11-03 21:00 ` Andre Hedrick
0 siblings, 2 replies; 4+ messages in thread
From: Erik Andersen @ 2002-10-26 13:24 UTC (permalink / raw)
To: bert hubert, andre, linux-kernel
On Sat Oct 26, 2002 at 03:07:01PM +0200, bert hubert wrote:
> Useless number '1' being printed leading to operator confusion.
>
> --- linux-2.5.44/drivers/ide/ide-disk.c~orig Sat Oct 26 14:59:35 2002
> +++ linux-2.5.44/drivers/ide/ide-disk.c Sat Oct 26 15:00:40 2002
> @@ -1128,7 +1128,7 @@
> {
> int flag = (drive->id->cfs_enable_1 & 0x0400) ? 1 : 0;
> if (flag)
> - printk("%s: host protected area => %d\n", drive->name, flag);
> + printk("%s: supports host protected area", drive->name);
> return flag;
> }
Even better -- kill the prink entirely. If anyone really
cares, they can run 'hdparm -I <drivename>' and get the
exhaustive list of everything the drive supports....
-Erik
--
Erik B. Andersen http://codepoet-consulting.com/
--This message was written using 73% post-consumer electrons--
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] de-cryptify ide-disk host protected area output
2002-10-26 13:24 ` Erik Andersen
@ 2002-10-26 14:25 ` Alan Cox
2002-11-03 21:00 ` Andre Hedrick
1 sibling, 0 replies; 4+ messages in thread
From: Alan Cox @ 2002-10-26 14:25 UTC (permalink / raw)
To: andersen; +Cc: bert hubert, Andre Hedrick, Linux Kernel Mailing List
On Sat, 2002-10-26 at 14:24, Erik Andersen wrote:
> Even better -- kill the prink entirely. If anyone really
> cares, they can run 'hdparm -I <drivename>' and get the
> exhaustive list of everything the drive supports....
Please leave it alone for now. There are a whole collection of extremely
verbose and pointless IDE messages left, and I want them left until we
are close to release, so we can debug stuff
In fact I will probably add more very soon
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] de-cryptify ide-disk host protected area output
2002-10-26 13:24 ` Erik Andersen
2002-10-26 14:25 ` Alan Cox
@ 2002-11-03 21:00 ` Andre Hedrick
1 sibling, 0 replies; 4+ messages in thread
From: Andre Hedrick @ 2002-11-03 21:00 UTC (permalink / raw)
To: Erik Andersen; +Cc: bert hubert, linux-kernel
Agreed w/ Erik.
On Sat, 26 Oct 2002, Erik Andersen wrote:
> On Sat Oct 26, 2002 at 03:07:01PM +0200, bert hubert wrote:
> > Useless number '1' being printed leading to operator confusion.
> >
> > --- linux-2.5.44/drivers/ide/ide-disk.c~orig Sat Oct 26 14:59:35 2002
> > +++ linux-2.5.44/drivers/ide/ide-disk.c Sat Oct 26 15:00:40 2002
> > @@ -1128,7 +1128,7 @@
> > {
> > int flag = (drive->id->cfs_enable_1 & 0x0400) ? 1 : 0;
> > if (flag)
> > - printk("%s: host protected area => %d\n", drive->name, flag);
> > + printk("%s: supports host protected area", drive->name);
> > return flag;
> > }
>
> Even better -- kill the prink entirely. If anyone really
> cares, they can run 'hdparm -I <drivename>' and get the
> exhaustive list of everything the drive supports....
>
> -Erik
>
> --
> Erik B. Andersen http://codepoet-consulting.com/
> --This message was written using 73% post-consumer electrons--
>
Andre Hedrick
LAD Storage Consulting Group
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2002-11-03 20:55 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2002-10-26 13:07 [PATCH] de-cryptify ide-disk host protected area output bert hubert
2002-10-26 13:24 ` Erik Andersen
2002-10-26 14:25 ` Alan Cox
2002-11-03 21:00 ` Andre Hedrick
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox