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-- - 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 Oct 31 2002 - 22:00:31 EST