ide: remove obsoleted "hdx=autotune" kernel parameter
Bartlomiej Zolnierkiewicz [Sun, 27 Apr 2008 13:38:29 +0000 (15:38 +0200)]
* Remove obsoleted "hdx=autotune" kernel parameter
  (we always auto-tune PIO if possible nowadays).

* Remove no longer needed ide_drive_t.autotune flag.

Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>

Documentation/ide/ide.txt
drivers/ide/ide-probe.c
drivers/ide/ide.c
include/linux/ide.h

index eb4d3b7..601e79e 100644 (file)
@@ -196,13 +196,6 @@ Summary of ide driver parameters for kernel command line
 
  "hdx=cyl,head,sect"   : disk drive is present, with specified geometry
 
- "hdx=autotune"                : driver will attempt to tune interface speed
-                         to the fastest PIO mode supported,
-                         if possible for this drive only.
-                         Not fully supported by all chipset types,
-                         and quite likely to cause trouble with
-                         older/odd IDE drives.
-
  "hdx=nodma"           : disallow DMA
 
  "ide=doubler"         : probe/support IDE doublers on Amiga
index 99972fe..ace826f 100644 (file)
@@ -827,8 +827,7 @@ static void ide_port_tune_devices(ide_hwif_t *hwif)
                ide_drive_t *drive = &hwif->drives[unit];
 
                if (drive->present) {
-                       if (drive->autotune)
-                               ide_set_max_pio(drive);
+                       ide_set_max_pio(drive);
 
                        drive->nice1 = 1;
 
@@ -1325,7 +1324,6 @@ static void ide_port_init_devices(ide_hwif_t *hwif)
                        drive->unmask = 1;
                if (hwif->host_flags & IDE_HFLAG_NO_UNMASK_IRQS)
                        drive->no_unmask = 1;
-               drive->autotune = 1;
        }
 
        if (port_ops && port_ops->port_init_devs)
index ad34fe9..71fa379 100644 (file)
@@ -900,7 +900,7 @@ static int __init ide_setup(char *s)
        if (s[0] == 'h' && s[1] == 'd' && s[2] >= 'a' && s[2] <= max_drive) {
                const char *hd_words[] = {
                        "none", "noprobe", "nowerr", "cdrom", "nodma",
-                       "autotune", "-7", "-8", "-9", "-10",
+                       "-6", "-7", "-8", "-9", "-10",
                        "noflush", "remap", "remap63", "scsi", NULL };
                unit = s[2] - 'a';
                hw   = unit / MAX_DRIVES;
@@ -928,9 +928,6 @@ static int __init ide_setup(char *s)
                        case -5: /* nodma */
                                drive->nodma = 1;
                                goto done;
-                       case -6: /* "autotune" */
-                               drive->autotune = 1;
-                               goto obsolete_option;
                        case -11: /* noflush */
                                drive->noflush = 1;
                                goto done;
index 0e7d44a..65dcbcf 100644 (file)
@@ -321,7 +321,6 @@ typedef struct ide_drive_s {
        unsigned atapi_overlap  : 1;    /* ATAPI overlap (not supported) */
        unsigned doorlocking    : 1;    /* for removable only: door lock/unlock works */
        unsigned nodma          : 1;    /* disallow DMA */
-       unsigned autotune       : 1;    /* 0=default, 1=autotune */
        unsigned remap_0_to_1   : 1;    /* 0=noremap, 1=remap 0->1 (for EZDrive) */
        unsigned blocked        : 1;    /* 1=powermanagment told us not to do anything, so sleep nicely */
        unsigned vdma           : 1;    /* 1=doing PIO over DMA 0=doing normal DMA */