lkml.org 
[lkml]   [2003]   [May]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[2.4 patch] 2.4.21-rc1 pointless IDE noise reduction
The patch forwarded below still applies against 2.4.21-rc2. Please apply 
or explain why you reject it.

TIA
Adrian


----- Forwarded message from Adrian Bunk <bunk@fs.tum.de> -----

Date: Sun, 27 Apr 2003 14:21:04 +0200
From: Adrian Bunk <bunk@fs.tum.de>
To: Erik Andersen <andersen@codepoet.org>,
Marcelo Tosatti <marcelo@conectiva.com.br>,
linux-kernel <linux-kernel@vger.kernel.org>,
alan@redhat.com
Subject: Re: [PATCH] 2.4.21-rc1 pointless IDE noise reduction

On Thu, Apr 24, 2003 at 03:34:43AM -0600, Erik Andersen wrote:
> The ide driver does not list whether drives support things like
> write cache, SMART, SECURITY ERASE UNIT. But for some silly
> reason it tells us at boot whether each drive is capable of
> supporting the Host Protected Area feature set. If people want
> to know the capabilites of their drive, they can run 'hdparm'
> and find out.
>
> This patch removes this pointless noise. Please apply,
>
>
> --- linux/drivers/ide/ide-disk.c.orig 2003-04-24 03:23:53.000000000 -0600
> +++ linux/drivers/ide/ide-disk.c 2003-04-24 03:24:54.000000000 -0600
> @@ -1133,10 +1133,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);
> }
>
> /*


Looking at the only user of this function it seems we can completely
remove it (patch below).

Alan:
Is the patch below OK or are there any future plans for more uses of
idedisk_supports_host_protected_area?

> -Erik

cu
Adrian


--- linux-2.4.21-rc1-full/drivers/ide/ide-disk.c.old 2003-04-27 13:26:17.000000000 +0200
+++ linux-2.4.21-rc1-full/drivers/ide/ide-disk.c 2003-04-27 13:30:48.000000000 +0200
@@ -1128,18 +1128,6 @@
#endif /* CONFIG_IDEDISK_STROKE */

/*
- * Tests if the drive supports Host Protected Area feature.
- * Returns true if supported, false otherwise.
- */
-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;
-}
-
-/*
* Compute drive->capacity, the full capacity of the drive
* Called with drive->id != NULL.
*
@@ -1165,8 +1153,6 @@
drive->capacity48 = 0;
drive->select.b.lba = 0;

- (void) idedisk_supports_host_protected_area(drive);
-
if (id->cfs_enable_2 & 0x0400) {
capacity_2 = id->lba_capacity_2;
drive->head = drive->bios_head = 255;
-
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/
----- End forwarded message -----
-
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/

\
 
 \ /
  Last update: 2005-03-22 13:35    [W:0.030 / U:0.332 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site