ide: delete filenames/versions from comments
[safe/jmp/linux-2.6] / drivers / ide / legacy / qd65xx.c
index ab482c8..37534bb 100644 (file)
@@ -1,6 +1,4 @@
 /*
- *  linux/drivers/ide/legacy/qd65xx.c          Version 0.07    Sep 30, 2001
- *
  *  Copyright (C) 1996-2001  Linus Torvalds & author (see below)
  */
 
 
 static int timings[4]={-1,-1,-1,-1}; /* stores current timing for each timer */
 
-static void qd_write_reg (u8 content, unsigned long reg)
-{
-       unsigned long flags;
-
-       spin_lock_irqsave(&ide_lock, flags);
-       outb(content,reg);
-       spin_unlock_irqrestore(&ide_lock, flags);
-}
-
-static u8 __init qd_read_reg (unsigned long reg)
-{
-       unsigned long flags;
-       u8 read;
-
-       spin_lock_irqsave(&ide_lock, flags);
-       read = inb(reg);
-       spin_unlock_irqrestore(&ide_lock, flags);
-       return read;
-}
-
 /*
  * qd_select:
  *
@@ -121,7 +99,7 @@ static void qd_select (ide_drive_t *drive)
                        (QD_TIMREG(drive) & 0x02);
 
        if (timings[index] != QD_TIMING(drive))
-               qd_write_reg(timings[index] = QD_TIMING(drive), QD_TIMREG(drive));
+               outb(timings[index] = QD_TIMING(drive), QD_TIMREG(drive));
 }
 
 /*
@@ -224,15 +202,14 @@ static void qd_set_timing (ide_drive_t *drive, u8 timing)
        printk(KERN_DEBUG "%s: %#x\n", drive->name, timing);
 }
 
-/*
- * qd6500_tune_drive
- */
-
-static void qd6500_tune_drive (ide_drive_t *drive, u8 pio)
+static void qd6500_set_pio_mode(ide_drive_t *drive, const u8 pio)
 {
        int active_time   = 175;
        int recovery_time = 415; /* worst case values from the dos driver */
 
+       /*
+        * FIXME: use "pio" value
+        */
        if (drive->id && !qd_find_disk_type(drive, &active_time, &recovery_time)
                && drive->id->tPIO && (drive->id->field_valid & 0x02)
                && drive->id->eide_pio >= 240) {
@@ -246,52 +223,46 @@ static void qd6500_tune_drive (ide_drive_t *drive, u8 pio)
        qd_set_timing(drive, qd6500_compute_timing(HWIF(drive), active_time, recovery_time));
 }
 
-/*
- * qd6580_tune_drive
- */
-
-static void qd6580_tune_drive (ide_drive_t *drive, u8 pio)
+static void qd6580_set_pio_mode(ide_drive_t *drive, const u8 pio)
 {
-       ide_pio_data_t d;
        int base = HWIF(drive)->select_data;
+       unsigned int cycle_time;
        int active_time   = 175;
        int recovery_time = 415; /* worst case values from the dos driver */
 
        if (drive->id && !qd_find_disk_type(drive, &active_time, &recovery_time)) {
-               pio = ide_get_best_pio_mode(drive, pio, 255, &d);
-               pio = min_t(u8, pio, 4);
+               cycle_time = ide_pio_cycle_time(drive, pio);
 
                switch (pio) {
                        case 0: break;
                        case 3:
-                               if (d.cycle_time >= 110) {
+                               if (cycle_time >= 110) {
                                        active_time = 86;
-                                       recovery_time = d.cycle_time - 102;
+                                       recovery_time = cycle_time - 102;
                                } else
                                        printk(KERN_WARNING "%s: Strange recovery time !\n",drive->name);
                                break;
                        case 4:
-                               if (d.cycle_time >= 69) {
+                               if (cycle_time >= 69) {
                                        active_time = 70;
-                                       recovery_time = d.cycle_time - 61;
+                                       recovery_time = cycle_time - 61;
                                } else
                                        printk(KERN_WARNING "%s: Strange recovery time !\n",drive->name);
                                break;
                        default:
-                               if (d.cycle_time >= 180) {
+                               if (cycle_time >= 180) {
                                        active_time = 110;
-                                       recovery_time = d.cycle_time - 120;
+                                       recovery_time = cycle_time - 120;
                                } else {
                                        active_time = ide_pio_timings[pio].active_time;
-                                       recovery_time = d.cycle_time
-                                                       -active_time;
+                                       recovery_time = cycle_time - active_time;
                                }
                }
                printk(KERN_INFO "%s: PIO mode%d\n", drive->name,pio);
        }
 
        if (!HWIF(drive)->channel && drive->media != ide_disk) {
-               qd_write_reg(0x5f, QD_CONTROL_PORT);
+               outb(0x5f, QD_CONTROL_PORT);
                printk(KERN_WARNING "%s: ATAPI: disabled read-ahead FIFO "
                        "and post-write buffer on %s.\n",
                        drive->name, HWIF(drive)->name);
@@ -308,16 +279,15 @@ static void qd6580_tune_drive (ide_drive_t *drive, u8 pio)
 
 static int __init qd_testreg(int port)
 {
-       u8 savereg;
-       u8 readreg;
        unsigned long flags;
+       u8 savereg, readreg;
 
-       spin_lock_irqsave(&ide_lock, flags);
+       local_irq_save(flags);
        savereg = inb_p(port);
        outb_p(QD_TESTVAL, port);       /* safe value */
        readreg = inb_p(port);
        outb(savereg, port);
-       spin_unlock_irqrestore(&ide_lock, flags);
+       local_irq_restore(flags);
 
        if (savereg == QD_TESTVAL) {
                printk(KERN_ERR "Outch ! the probe for qd65xx isn't reliable !\n");
@@ -336,8 +306,7 @@ static int __init qd_testreg(int port)
  */
 
 static void __init qd_setup(ide_hwif_t *hwif, int base, int config,
-                           unsigned int data0, unsigned int data1,
-                           void (*tuneproc) (ide_drive_t *, u8 pio))
+                           unsigned int data0, unsigned int data1)
 {
        hwif->chipset = ide_qd65xx;
        hwif->channel = hwif->index;
@@ -347,8 +316,7 @@ static void __init qd_setup(ide_hwif_t *hwif, int base, int config,
        hwif->drives[1].drive_data = data1;
        hwif->drives[0].io_32bit =
        hwif->drives[1].io_32bit = 1;
-       hwif->tuneproc = tuneproc;
-       probe_hwif_init(hwif);
+       hwif->pio_mask = ATA_PIO4;
 }
 
 /*
@@ -361,7 +329,7 @@ static void __exit qd_unsetup(ide_hwif_t *hwif)
 {
        u8 config = hwif->config_data;
        int base = hwif->select_data;
-       void *tuneproc = (void *) hwif->tuneproc;
+       void *set_pio_mode = (void *)hwif->set_pio_mode;
 
        if (hwif->chipset != ide_qd65xx)
                return;
@@ -369,17 +337,17 @@ static void __exit qd_unsetup(ide_hwif_t *hwif)
        printk(KERN_NOTICE "%s: back to defaults\n", hwif->name);
 
        hwif->selectproc = NULL;
-       hwif->tuneproc = NULL;
+       hwif->set_pio_mode = NULL;
 
-       if (tuneproc == (void *) qd6500_tune_drive) {
+       if (set_pio_mode == (void *)qd6500_set_pio_mode) {
                // will do it for both
-               qd_write_reg(QD6500_DEF_DATA, QD_TIMREG(&hwif->drives[0]));
-       } else if (tuneproc == (void *) qd6580_tune_drive) {
+               outb(QD6500_DEF_DATA, QD_TIMREG(&hwif->drives[0]));
+       } else if (set_pio_mode == (void *)qd6580_set_pio_mode) {
                if (QD_CONTROL(hwif) & QD_CONTR_SEC_DISABLED) {
-                       qd_write_reg(QD6580_DEF_DATA, QD_TIMREG(&hwif->drives[0]));
-                       qd_write_reg(QD6580_DEF_DATA2, QD_TIMREG(&hwif->drives[1]));
+                       outb(QD6580_DEF_DATA, QD_TIMREG(&hwif->drives[0]));
+                       outb(QD6580_DEF_DATA2, QD_TIMREG(&hwif->drives[1]));
                } else {
-                       qd_write_reg(hwif->channel ? QD6580_DEF_DATA2 : QD6580_DEF_DATA, QD_TIMREG(&hwif->drives[0]));
+                       outb(hwif->channel ? QD6580_DEF_DATA2 : QD6580_DEF_DATA, QD_TIMREG(&hwif->drives[0]));
                }
        } else {
                printk(KERN_WARNING "Unknown qd65xx tuning fonction !\n");
@@ -398,10 +366,11 @@ static void __exit qd_unsetup(ide_hwif_t *hwif)
 static int __init qd_probe(int base)
 {
        ide_hwif_t *hwif;
+       u8 idx[4] = { 0xff, 0xff, 0xff, 0xff };
        u8 config;
        u8 unit;
 
-       config = qd_read_reg(QD_CONFIG_PORT);
+       config = inb(QD_CONFIG_PORT);
 
        if (! ((config & QD_CONFIG_BASEPORT) >> 1 == (base == 0xb0)) )
                return 1;
@@ -424,10 +393,13 @@ static int __init qd_probe(int base)
                        return 1;
                }
 
-               qd_setup(hwif, base, config, QD6500_DEF_DATA, QD6500_DEF_DATA,
-                        &qd6500_tune_drive);
+               qd_setup(hwif, base, config, QD6500_DEF_DATA, QD6500_DEF_DATA);
+
+               hwif->set_pio_mode = &qd6500_set_pio_mode;
 
-               create_proc_ide_interfaces();
+               idx[0] = unit;
+
+               ide_device_add(idx);
 
                return 1;
        }
@@ -442,7 +414,7 @@ static int __init qd_probe(int base)
 
                /* qd6580 found */
 
-               control = qd_read_reg(QD_CONTROL_PORT);
+               control = inb(QD_CONTROL_PORT);
 
                printk(KERN_NOTICE "qd6580 at %#x\n", base);
                printk(KERN_DEBUG "qd6580: config=%#x, control=%#x, ID3=%u\n",
@@ -455,11 +427,15 @@ static int __init qd_probe(int base)
                        printk(KERN_INFO "%s: qd6580: single IDE board\n",
                                         hwif->name);
                        qd_setup(hwif, base, config | (control << 8),
-                                QD6580_DEF_DATA, QD6580_DEF_DATA2,
-                                &qd6580_tune_drive);
-                       qd_write_reg(QD_DEF_CONTR,QD_CONTROL_PORT);
+                                QD6580_DEF_DATA, QD6580_DEF_DATA2);
+
+                       hwif->set_pio_mode = &qd6580_set_pio_mode;
 
-                       create_proc_ide_interfaces();
+                       idx[0] = unit;
+
+                       ide_device_add(idx);
+
+                       outb(QD_DEF_CONTR, QD_CONTROL_PORT);
 
                        return 1;
                } else {
@@ -472,14 +448,21 @@ static int __init qd_probe(int base)
                                        hwif->name, mate->name);
 
                        qd_setup(hwif, base, config | (control << 8),
-                                QD6580_DEF_DATA, QD6580_DEF_DATA,
-                                &qd6580_tune_drive);
+                                QD6580_DEF_DATA, QD6580_DEF_DATA);
+
+                       hwif->set_pio_mode = &qd6580_set_pio_mode;
+
                        qd_setup(mate, base, config | (control << 8),
-                                QD6580_DEF_DATA2, QD6580_DEF_DATA2,
-                                &qd6580_tune_drive);
-                       qd_write_reg(QD_DEF_CONTR,QD_CONTROL_PORT);
+                                QD6580_DEF_DATA2, QD6580_DEF_DATA2);
 
-                       create_proc_ide_interfaces();
+                       mate->set_pio_mode = &qd6580_set_pio_mode;
+
+                       idx[0] = 0;
+                       idx[1] = 1;
+
+                       ide_device_add(idx);
+
+                       outb(QD_DEF_CONTR, QD_CONTROL_PORT);
 
                        return 0; /* no other qd65xx possible */
                }
@@ -488,9 +471,16 @@ static int __init qd_probe(int base)
        return 1;
 }
 
-/* Can be called directly from ide.c. */
-int __init qd65xx_init(void)
+int probe_qd65xx = 0;
+
+module_param_named(probe, probe_qd65xx, bool, 0);
+MODULE_PARM_DESC(probe, "probe for QD65xx chipsets");
+
+static int __init qd65xx_init(void)
 {
+       if (probe_qd65xx == 0)
+               return -ENODEV;
+
        if (qd_probe(0x30))
                qd_probe(0xb0);
        if (ide_hwifs[0].chipset != ide_qd65xx &&
@@ -499,9 +489,7 @@ int __init qd65xx_init(void)
        return 0;
 }
 
-#ifdef MODULE
 module_init(qd65xx_init);
-#endif
 
 MODULE_AUTHOR("Samuel Thibault");
 MODULE_DESCRIPTION("support of qd65xx vlb ide chipset");