ide: use lock bitops for ports serialization (v2)
[linux-2.6] / drivers / ide / ide.c
1 /*
2  *  Copyright (C) 1994-1998         Linus Torvalds & authors (see below)
3  *  Copyright (C) 2003-2005, 2007   Bartlomiej Zolnierkiewicz
4  */
5
6 /*
7  *  Mostly written by Mark Lord  <mlord@pobox.com>
8  *                and Gadi Oxman <gadio@netvision.net.il>
9  *                and Andre Hedrick <andre@linux-ide.org>
10  *
11  *  See linux/MAINTAINERS for address of current maintainer.
12  *
13  * This is the multiple IDE interface driver, as evolved from hd.c.
14  * It supports up to MAX_HWIFS IDE interfaces, on one or more IRQs
15  *   (usually 14 & 15).
16  * There can be up to two drives per interface, as per the ATA-2 spec.
17  *
18  * ...
19  *
20  *  From hd.c:
21  *  |
22  *  | It traverses the request-list, using interrupts to jump between functions.
23  *  | As nearly all functions can be called within interrupts, we may not sleep.
24  *  | Special care is recommended.  Have Fun!
25  *  |
26  *  | modified by Drew Eckhardt to check nr of hd's from the CMOS.
27  *  |
28  *  | Thanks to Branko Lankester, lankeste@fwi.uva.nl, who found a bug
29  *  | in the early extended-partition checks and added DM partitions.
30  *  |
31  *  | Early work on error handling by Mika Liljeberg (liljeber@cs.Helsinki.FI).
32  *  |
33  *  | IRQ-unmask, drive-id, multiple-mode, support for ">16 heads",
34  *  | and general streamlining by Mark Lord (mlord@pobox.com).
35  *
36  *  October, 1994 -- Complete line-by-line overhaul for linux 1.1.x, by:
37  *
38  *      Mark Lord       (mlord@pobox.com)               (IDE Perf.Pkg)
39  *      Delman Lee      (delman@ieee.org)               ("Mr. atdisk2")
40  *      Scott Snyder    (snyder@fnald0.fnal.gov)        (ATAPI IDE cd-rom)
41  *
42  *  This was a rewrite of just about everything from hd.c, though some original
43  *  code is still sprinkled about.  Think of it as a major evolution, with
44  *  inspiration from lots of linux users, esp.  hamish@zot.apana.org.au
45  */
46
47 #include <linux/module.h>
48 #include <linux/types.h>
49 #include <linux/string.h>
50 #include <linux/kernel.h>
51 #include <linux/interrupt.h>
52 #include <linux/major.h>
53 #include <linux/errno.h>
54 #include <linux/genhd.h>
55 #include <linux/slab.h>
56 #include <linux/init.h>
57 #include <linux/pci.h>
58 #include <linux/ide.h>
59 #include <linux/hdreg.h>
60 #include <linux/completion.h>
61 #include <linux/device.h>
62
63
64 /* default maximum number of failures */
65 #define IDE_DEFAULT_MAX_FAILURES        1
66
67 struct class *ide_port_class;
68
69 static const u8 ide_hwif_to_major[] = { IDE0_MAJOR, IDE1_MAJOR,
70                                         IDE2_MAJOR, IDE3_MAJOR,
71                                         IDE4_MAJOR, IDE5_MAJOR,
72                                         IDE6_MAJOR, IDE7_MAJOR,
73                                         IDE8_MAJOR, IDE9_MAJOR };
74
75 DEFINE_MUTEX(ide_cfg_mtx);
76
77 static void ide_port_init_devices_data(ide_hwif_t *);
78
79 /*
80  * Do not even *think* about calling this!
81  */
82 void ide_init_port_data(ide_hwif_t *hwif, unsigned int index)
83 {
84         /* bulk initialize hwif & drive info with zeros */
85         memset(hwif, 0, sizeof(ide_hwif_t));
86
87         /* fill in any non-zero initial values */
88         hwif->index     = index;
89         hwif->major     = ide_hwif_to_major[index];
90
91         hwif->name[0]   = 'i';
92         hwif->name[1]   = 'd';
93         hwif->name[2]   = 'e';
94         hwif->name[3]   = '0' + index;
95
96         init_completion(&hwif->gendev_rel_comp);
97
98         hwif->tp_ops = &default_tp_ops;
99
100         ide_port_init_devices_data(hwif);
101 }
102
103 static void ide_port_init_devices_data(ide_hwif_t *hwif)
104 {
105         int unit;
106
107         for (unit = 0; unit < MAX_DRIVES; ++unit) {
108                 ide_drive_t *drive = &hwif->drives[unit];
109                 u8 j = (hwif->index * MAX_DRIVES) + unit;
110
111                 memset(drive, 0, sizeof(*drive));
112
113                 drive->media                    = ide_disk;
114                 drive->select                   = (unit << 4) | ATA_DEVICE_OBS;
115                 drive->hwif                     = hwif;
116                 drive->ready_stat               = ATA_DRDY;
117                 drive->bad_wstat                = BAD_W_STAT;
118                 drive->special.b.recalibrate    = 1;
119                 drive->special.b.set_geometry   = 1;
120                 drive->name[0]                  = 'h';
121                 drive->name[1]                  = 'd';
122                 drive->name[2]                  = 'a' + j;
123                 drive->max_failures             = IDE_DEFAULT_MAX_FAILURES;
124
125                 INIT_LIST_HEAD(&drive->list);
126                 init_completion(&drive->gendev_rel_comp);
127         }
128 }
129
130 static void __ide_port_unregister_devices(ide_hwif_t *hwif)
131 {
132         int i;
133
134         for (i = 0; i < MAX_DRIVES; i++) {
135                 ide_drive_t *drive = &hwif->drives[i];
136
137                 if (drive->dev_flags & IDE_DFLAG_PRESENT) {
138                         device_unregister(&drive->gendev);
139                         wait_for_completion(&drive->gendev_rel_comp);
140                 }
141         }
142 }
143
144 void ide_port_unregister_devices(ide_hwif_t *hwif)
145 {
146         mutex_lock(&ide_cfg_mtx);
147         __ide_port_unregister_devices(hwif);
148         hwif->present = 0;
149         ide_port_init_devices_data(hwif);
150         mutex_unlock(&ide_cfg_mtx);
151 }
152 EXPORT_SYMBOL_GPL(ide_port_unregister_devices);
153
154 /**
155  *      ide_unregister          -       free an IDE interface
156  *      @hwif: IDE interface
157  *
158  *      Perform the final unregister of an IDE interface. At the moment
159  *      we don't refcount interfaces so this will also get split up.
160  *
161  *      Locking:
162  *      The caller must not hold the IDE locks
163  *      The drive present/vanishing is not yet properly locked
164  *      Take care with the callbacks. These have been split to avoid
165  *      deadlocking the IDE layer. The shutdown callback is called
166  *      before we take the lock and free resources. It is up to the
167  *      caller to be sure there is no pending I/O here, and that
168  *      the interface will not be reopened (present/vanishing locking
169  *      isn't yet done BTW). After we commit to the final kill we
170  *      call the cleanup callback with the ide locks held.
171  *
172  *      Unregister restores the hwif structures to the default state.
173  *      This is raving bonkers.
174  */
175
176 void ide_unregister(ide_hwif_t *hwif)
177 {
178         BUG_ON(in_interrupt());
179         BUG_ON(irqs_disabled());
180
181         mutex_lock(&ide_cfg_mtx);
182
183         if (hwif->present) {
184                 __ide_port_unregister_devices(hwif);
185                 hwif->present = 0;
186         }
187
188         ide_proc_unregister_port(hwif);
189
190         free_irq(hwif->irq, hwif);
191
192         ide_remove_port_from_hwgroup(hwif);
193
194         device_unregister(hwif->portdev);
195         device_unregister(&hwif->gendev);
196         wait_for_completion(&hwif->gendev_rel_comp);
197
198         /*
199          * Remove us from the kernel's knowledge
200          */
201         blk_unregister_region(MKDEV(hwif->major, 0), MAX_DRIVES<<PARTN_BITS);
202         kfree(hwif->sg_table);
203         unregister_blkdev(hwif->major, hwif->name);
204
205         ide_release_dma_engine(hwif);
206
207         mutex_unlock(&ide_cfg_mtx);
208 }
209
210 void ide_init_port_hw(ide_hwif_t *hwif, hw_regs_t *hw)
211 {
212         memcpy(&hwif->io_ports, &hw->io_ports, sizeof(hwif->io_ports));
213         hwif->irq = hw->irq;
214         hwif->chipset = hw->chipset;
215         hwif->dev = hw->dev;
216         hwif->gendev.parent = hw->parent ? hw->parent : hw->dev;
217         hwif->ack_intr = hw->ack_intr;
218         hwif->config_data = hw->config;
219 }
220
221 /*
222  *      Locks for IDE setting functionality
223  */
224
225 DEFINE_MUTEX(ide_setting_mtx);
226
227 ide_devset_get(io_32bit, io_32bit);
228
229 static int set_io_32bit(ide_drive_t *drive, int arg)
230 {
231         if (drive->dev_flags & IDE_DFLAG_NO_IO_32BIT)
232                 return -EPERM;
233
234         if (arg < 0 || arg > 1 + (SUPPORT_VLB_SYNC << 1))
235                 return -EINVAL;
236
237         drive->io_32bit = arg;
238
239         return 0;
240 }
241
242 ide_devset_get_flag(ksettings, IDE_DFLAG_KEEP_SETTINGS);
243
244 static int set_ksettings(ide_drive_t *drive, int arg)
245 {
246         if (arg < 0 || arg > 1)
247                 return -EINVAL;
248
249         if (arg)
250                 drive->dev_flags |= IDE_DFLAG_KEEP_SETTINGS;
251         else
252                 drive->dev_flags &= ~IDE_DFLAG_KEEP_SETTINGS;
253
254         return 0;
255 }
256
257 ide_devset_get_flag(using_dma, IDE_DFLAG_USING_DMA);
258
259 static int set_using_dma(ide_drive_t *drive, int arg)
260 {
261 #ifdef CONFIG_BLK_DEV_IDEDMA
262         int err = -EPERM;
263
264         if (arg < 0 || arg > 1)
265                 return -EINVAL;
266
267         if (ata_id_has_dma(drive->id) == 0)
268                 goto out;
269
270         if (drive->hwif->dma_ops == NULL)
271                 goto out;
272
273         err = 0;
274
275         if (arg) {
276                 if (ide_set_dma(drive))
277                         err = -EIO;
278         } else
279                 ide_dma_off(drive);
280
281 out:
282         return err;
283 #else
284         if (arg < 0 || arg > 1)
285                 return -EINVAL;
286
287         return -EPERM;
288 #endif
289 }
290
291 /*
292  * handle HDIO_SET_PIO_MODE ioctl abusers here, eventually it will go away
293  */
294 static int set_pio_mode_abuse(ide_hwif_t *hwif, u8 req_pio)
295 {
296         switch (req_pio) {
297         case 202:
298         case 201:
299         case 200:
300         case 102:
301         case 101:
302         case 100:
303                 return (hwif->host_flags & IDE_HFLAG_ABUSE_DMA_MODES) ? 1 : 0;
304         case 9:
305         case 8:
306                 return (hwif->host_flags & IDE_HFLAG_ABUSE_PREFETCH) ? 1 : 0;
307         case 7:
308         case 6:
309                 return (hwif->host_flags & IDE_HFLAG_ABUSE_FAST_DEVSEL) ? 1 : 0;
310         default:
311                 return 0;
312         }
313 }
314
315 static int set_pio_mode(ide_drive_t *drive, int arg)
316 {
317         ide_hwif_t *hwif = drive->hwif;
318         ide_hwgroup_t *hwgroup = hwif->hwgroup;
319         const struct ide_port_ops *port_ops = hwif->port_ops;
320
321         if (arg < 0 || arg > 255)
322                 return -EINVAL;
323
324         if (port_ops == NULL || port_ops->set_pio_mode == NULL ||
325             (hwif->host_flags & IDE_HFLAG_NO_SET_MODE))
326                 return -ENOSYS;
327
328         if (set_pio_mode_abuse(drive->hwif, arg)) {
329                 if (arg == 8 || arg == 9) {
330                         unsigned long flags;
331
332                         /* take lock for IDE_DFLAG_[NO_]UNMASK/[NO_]IO_32BIT */
333                         spin_lock_irqsave(&hwgroup->lock, flags);
334                         port_ops->set_pio_mode(drive, arg);
335                         spin_unlock_irqrestore(&hwgroup->lock, flags);
336                 } else
337                         port_ops->set_pio_mode(drive, arg);
338         } else {
339                 int keep_dma = !!(drive->dev_flags & IDE_DFLAG_USING_DMA);
340
341                 ide_set_pio(drive, arg);
342
343                 if (hwif->host_flags & IDE_HFLAG_SET_PIO_MODE_KEEP_DMA) {
344                         if (keep_dma)
345                                 ide_dma_on(drive);
346                 }
347         }
348
349         return 0;
350 }
351
352 ide_devset_get_flag(unmaskirq, IDE_DFLAG_UNMASK);
353
354 static int set_unmaskirq(ide_drive_t *drive, int arg)
355 {
356         if (drive->dev_flags & IDE_DFLAG_NO_UNMASK)
357                 return -EPERM;
358
359         if (arg < 0 || arg > 1)
360                 return -EINVAL;
361
362         if (arg)
363                 drive->dev_flags |= IDE_DFLAG_UNMASK;
364         else
365                 drive->dev_flags &= ~IDE_DFLAG_UNMASK;
366
367         return 0;
368 }
369
370 ide_ext_devset_rw_sync(io_32bit, io_32bit);
371 ide_ext_devset_rw_sync(keepsettings, ksettings);
372 ide_ext_devset_rw_sync(unmaskirq, unmaskirq);
373 ide_ext_devset_rw_sync(using_dma, using_dma);
374 __IDE_DEVSET(pio_mode, DS_SYNC, NULL, set_pio_mode);
375
376 /**
377  * ide_device_get       -       get an additional reference to a ide_drive_t
378  * @drive:      device to get a reference to
379  *
380  * Gets a reference to the ide_drive_t and increments the use count of the
381  * underlying LLDD module.
382  */
383 int ide_device_get(ide_drive_t *drive)
384 {
385         struct device *host_dev;
386         struct module *module;
387
388         if (!get_device(&drive->gendev))
389                 return -ENXIO;
390
391         host_dev = drive->hwif->host->dev[0];
392         module = host_dev ? host_dev->driver->owner : NULL;
393
394         if (module && !try_module_get(module)) {
395                 put_device(&drive->gendev);
396                 return -ENXIO;
397         }
398
399         return 0;
400 }
401 EXPORT_SYMBOL_GPL(ide_device_get);
402
403 /**
404  * ide_device_put       -       release a reference to a ide_drive_t
405  * @drive:      device to release a reference on
406  *
407  * Release a reference to the ide_drive_t and decrements the use count of
408  * the underlying LLDD module.
409  */
410 void ide_device_put(ide_drive_t *drive)
411 {
412 #ifdef CONFIG_MODULE_UNLOAD
413         struct device *host_dev = drive->hwif->host->dev[0];
414         struct module *module = host_dev ? host_dev->driver->owner : NULL;
415
416         if (module)
417                 module_put(module);
418 #endif
419         put_device(&drive->gendev);
420 }
421 EXPORT_SYMBOL_GPL(ide_device_put);
422
423 static int ide_bus_match(struct device *dev, struct device_driver *drv)
424 {
425         return 1;
426 }
427
428 static int ide_uevent(struct device *dev, struct kobj_uevent_env *env)
429 {
430         ide_drive_t *drive = to_ide_device(dev);
431
432         add_uevent_var(env, "MEDIA=%s", ide_media_string(drive));
433         add_uevent_var(env, "DRIVENAME=%s", drive->name);
434         add_uevent_var(env, "MODALIAS=ide:m-%s", ide_media_string(drive));
435         return 0;
436 }
437
438 static int generic_ide_probe(struct device *dev)
439 {
440         ide_drive_t *drive = to_ide_device(dev);
441         ide_driver_t *drv = to_ide_driver(dev->driver);
442
443         return drv->probe ? drv->probe(drive) : -ENODEV;
444 }
445
446 static int generic_ide_remove(struct device *dev)
447 {
448         ide_drive_t *drive = to_ide_device(dev);
449         ide_driver_t *drv = to_ide_driver(dev->driver);
450
451         if (drv->remove)
452                 drv->remove(drive);
453
454         return 0;
455 }
456
457 static void generic_ide_shutdown(struct device *dev)
458 {
459         ide_drive_t *drive = to_ide_device(dev);
460         ide_driver_t *drv = to_ide_driver(dev->driver);
461
462         if (dev->driver && drv->shutdown)
463                 drv->shutdown(drive);
464 }
465
466 struct bus_type ide_bus_type = {
467         .name           = "ide",
468         .match          = ide_bus_match,
469         .uevent         = ide_uevent,
470         .probe          = generic_ide_probe,
471         .remove         = generic_ide_remove,
472         .shutdown       = generic_ide_shutdown,
473         .dev_attrs      = ide_dev_attrs,
474         .suspend        = generic_ide_suspend,
475         .resume         = generic_ide_resume,
476 };
477
478 EXPORT_SYMBOL_GPL(ide_bus_type);
479
480 int ide_vlb_clk;
481 EXPORT_SYMBOL_GPL(ide_vlb_clk);
482
483 module_param_named(vlb_clock, ide_vlb_clk, int, 0);
484 MODULE_PARM_DESC(vlb_clock, "VLB clock frequency (in MHz)");
485
486 int ide_pci_clk;
487 EXPORT_SYMBOL_GPL(ide_pci_clk);
488
489 module_param_named(pci_clock, ide_pci_clk, int, 0);
490 MODULE_PARM_DESC(pci_clock, "PCI bus clock frequency (in MHz)");
491
492 static int ide_set_dev_param_mask(const char *s, struct kernel_param *kp)
493 {
494         int a, b, i, j = 1;
495         unsigned int *dev_param_mask = (unsigned int *)kp->arg;
496
497         if (sscanf(s, "%d.%d:%d", &a, &b, &j) != 3 &&
498             sscanf(s, "%d.%d", &a, &b) != 2)
499                 return -EINVAL;
500
501         i = a * MAX_DRIVES + b;
502
503         if (i >= MAX_HWIFS * MAX_DRIVES || j < 0 || j > 1)
504                 return -EINVAL;
505
506         if (j)
507                 *dev_param_mask |= (1 << i);
508         else
509                 *dev_param_mask &= (1 << i);
510
511         return 0;
512 }
513
514 static unsigned int ide_nodma;
515
516 module_param_call(nodma, ide_set_dev_param_mask, NULL, &ide_nodma, 0);
517 MODULE_PARM_DESC(nodma, "disallow DMA for a device");
518
519 static unsigned int ide_noflush;
520
521 module_param_call(noflush, ide_set_dev_param_mask, NULL, &ide_noflush, 0);
522 MODULE_PARM_DESC(noflush, "disable flush requests for a device");
523
524 static unsigned int ide_noprobe;
525
526 module_param_call(noprobe, ide_set_dev_param_mask, NULL, &ide_noprobe, 0);
527 MODULE_PARM_DESC(noprobe, "skip probing for a device");
528
529 static unsigned int ide_nowerr;
530
531 module_param_call(nowerr, ide_set_dev_param_mask, NULL, &ide_nowerr, 0);
532 MODULE_PARM_DESC(nowerr, "ignore the ATA_DF bit for a device");
533
534 static unsigned int ide_cdroms;
535
536 module_param_call(cdrom, ide_set_dev_param_mask, NULL, &ide_cdroms, 0);
537 MODULE_PARM_DESC(cdrom, "force device as a CD-ROM");
538
539 struct chs_geom {
540         unsigned int    cyl;
541         u8              head;
542         u8              sect;
543 };
544
545 static unsigned int ide_disks;
546 static struct chs_geom ide_disks_chs[MAX_HWIFS * MAX_DRIVES];
547
548 static int ide_set_disk_chs(const char *str, struct kernel_param *kp)
549 {
550         int a, b, c = 0, h = 0, s = 0, i, j = 1;
551
552         if (sscanf(str, "%d.%d:%d,%d,%d", &a, &b, &c, &h, &s) != 5 &&
553             sscanf(str, "%d.%d:%d", &a, &b, &j) != 3)
554                 return -EINVAL;
555
556         i = a * MAX_DRIVES + b;
557
558         if (i >= MAX_HWIFS * MAX_DRIVES || j < 0 || j > 1)
559                 return -EINVAL;
560
561         if (c > INT_MAX || h > 255 || s > 255)
562                 return -EINVAL;
563
564         if (j)
565                 ide_disks |= (1 << i);
566         else
567                 ide_disks &= (1 << i);
568
569         ide_disks_chs[i].cyl  = c;
570         ide_disks_chs[i].head = h;
571         ide_disks_chs[i].sect = s;
572
573         return 0;
574 }
575
576 module_param_call(chs, ide_set_disk_chs, NULL, NULL, 0);
577 MODULE_PARM_DESC(chs, "force device as a disk (using CHS)");
578
579 static void ide_dev_apply_params(ide_drive_t *drive, u8 unit)
580 {
581         int i = drive->hwif->index * MAX_DRIVES + unit;
582
583         if (ide_nodma & (1 << i)) {
584                 printk(KERN_INFO "ide: disallowing DMA for %s\n", drive->name);
585                 drive->dev_flags |= IDE_DFLAG_NODMA;
586         }
587         if (ide_noflush & (1 << i)) {
588                 printk(KERN_INFO "ide: disabling flush requests for %s\n",
589                                  drive->name);
590                 drive->dev_flags |= IDE_DFLAG_NOFLUSH;
591         }
592         if (ide_noprobe & (1 << i)) {
593                 printk(KERN_INFO "ide: skipping probe for %s\n", drive->name);
594                 drive->dev_flags |= IDE_DFLAG_NOPROBE;
595         }
596         if (ide_nowerr & (1 << i)) {
597                 printk(KERN_INFO "ide: ignoring the ATA_DF bit for %s\n",
598                                  drive->name);
599                 drive->bad_wstat = BAD_R_STAT;
600         }
601         if (ide_cdroms & (1 << i)) {
602                 printk(KERN_INFO "ide: forcing %s as a CD-ROM\n", drive->name);
603                 drive->dev_flags |= IDE_DFLAG_PRESENT;
604                 drive->media = ide_cdrom;
605                 /* an ATAPI device ignores DRDY */
606                 drive->ready_stat = 0;
607         }
608         if (ide_disks & (1 << i)) {
609                 drive->cyl  = drive->bios_cyl  = ide_disks_chs[i].cyl;
610                 drive->head = drive->bios_head = ide_disks_chs[i].head;
611                 drive->sect = drive->bios_sect = ide_disks_chs[i].sect;
612
613                 printk(KERN_INFO "ide: forcing %s as a disk (%d/%d/%d)\n",
614                                  drive->name,
615                                  drive->cyl, drive->head, drive->sect);
616
617                 drive->dev_flags |= IDE_DFLAG_FORCED_GEOM | IDE_DFLAG_PRESENT;
618                 drive->media = ide_disk;
619                 drive->ready_stat = ATA_DRDY;
620         }
621 }
622
623 static unsigned int ide_ignore_cable;
624
625 static int ide_set_ignore_cable(const char *s, struct kernel_param *kp)
626 {
627         int i, j = 1;
628
629         if (sscanf(s, "%d:%d", &i, &j) != 2 && sscanf(s, "%d", &i) != 1)
630                 return -EINVAL;
631
632         if (i >= MAX_HWIFS || j < 0 || j > 1)
633                 return -EINVAL;
634
635         if (j)
636                 ide_ignore_cable |= (1 << i);
637         else
638                 ide_ignore_cable &= (1 << i);
639
640         return 0;
641 }
642
643 module_param_call(ignore_cable, ide_set_ignore_cable, NULL, NULL, 0);
644 MODULE_PARM_DESC(ignore_cable, "ignore cable detection");
645
646 void ide_port_apply_params(ide_hwif_t *hwif)
647 {
648         int i;
649
650         if (ide_ignore_cable & (1 << hwif->index)) {
651                 printk(KERN_INFO "ide: ignoring cable detection for %s\n",
652                                  hwif->name);
653                 hwif->cbl = ATA_CBL_PATA40_SHORT;
654         }
655
656         for (i = 0; i < MAX_DRIVES; i++)
657                 ide_dev_apply_params(&hwif->drives[i], i);
658 }
659
660 /*
661  * This is gets invoked once during initialization, to set *everything* up
662  */
663 static int __init ide_init(void)
664 {
665         int ret;
666
667         printk(KERN_INFO "Uniform Multi-Platform E-IDE driver\n");
668
669         ret = bus_register(&ide_bus_type);
670         if (ret < 0) {
671                 printk(KERN_WARNING "IDE: bus_register error: %d\n", ret);
672                 return ret;
673         }
674
675         ide_port_class = class_create(THIS_MODULE, "ide_port");
676         if (IS_ERR(ide_port_class)) {
677                 ret = PTR_ERR(ide_port_class);
678                 goto out_port_class;
679         }
680
681         proc_ide_create();
682
683         return 0;
684
685 out_port_class:
686         bus_unregister(&ide_bus_type);
687
688         return ret;
689 }
690
691 static void __exit ide_exit(void)
692 {
693         proc_ide_destroy();
694
695         class_destroy(ide_port_class);
696
697         bus_unregister(&ide_bus_type);
698 }
699
700 module_init(ide_init);
701 module_exit(ide_exit);
702
703 MODULE_LICENSE("GPL");