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
-
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/
This archive was generated by hypermail 2b29 : Thu Nov 07 2002 - 22:00:30 EST