mac80211: minstrel_ht: fix a crash in rate sorting
[cascardo/linux.git] / drivers / staging / vme / devices / vme_user.c
1 /*
2  * VMEbus User access driver
3  *
4  * Author: Martyn Welch <martyn.welch@ge.com>
5  * Copyright 2008 GE Intelligent Platforms Embedded Systems, Inc.
6  *
7  * Based on work by:
8  *   Tom Armistead and Ajit Prem
9  *     Copyright 2004 Motorola Inc.
10  *
11  *
12  * This program is free software; you can redistribute  it and/or modify it
13  * under  the terms of  the GNU General  Public License as published by the
14  * Free Software Foundation;  either version 2 of the  License, or (at your
15  * option) any later version.
16  */
17
18 #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
19
20 #include <linux/cdev.h>
21 #include <linux/delay.h>
22 #include <linux/device.h>
23 #include <linux/dma-mapping.h>
24 #include <linux/errno.h>
25 #include <linux/init.h>
26 #include <linux/ioctl.h>
27 #include <linux/kernel.h>
28 #include <linux/mm.h>
29 #include <linux/module.h>
30 #include <linux/pagemap.h>
31 #include <linux/pci.h>
32 #include <linux/mutex.h>
33 #include <linux/slab.h>
34 #include <linux/spinlock.h>
35 #include <linux/syscalls.h>
36 #include <linux/types.h>
37
38 #include <linux/io.h>
39 #include <linux/uaccess.h>
40 #include <linux/vme.h>
41
42 #include "vme_user.h"
43
44 static DEFINE_MUTEX(vme_user_mutex);
45 static const char driver_name[] = "vme_user";
46
47 static int bus[VME_USER_BUS_MAX];
48 static unsigned int bus_num;
49
50 /* Currently Documentation/devices.txt defines the following for VME:
51  *
52  * 221 char     VME bus
53  *                0 = /dev/bus/vme/m0           First master image
54  *                1 = /dev/bus/vme/m1           Second master image
55  *                2 = /dev/bus/vme/m2           Third master image
56  *                3 = /dev/bus/vme/m3           Fourth master image
57  *                4 = /dev/bus/vme/s0           First slave image
58  *                5 = /dev/bus/vme/s1           Second slave image
59  *                6 = /dev/bus/vme/s2           Third slave image
60  *                7 = /dev/bus/vme/s3           Fourth slave image
61  *                8 = /dev/bus/vme/ctl          Control
62  *
63  *              It is expected that all VME bus drivers will use the
64  *              same interface.  For interface documentation see
65  *              http://www.vmelinux.org/.
66  *
67  * However the VME driver at http://www.vmelinux.org/ is rather old and doesn't
68  * even support the tsi148 chipset (which has 8 master and 8 slave windows).
69  * We'll run with this for now as far as possible, however it probably makes
70  * sense to get rid of the old mappings and just do everything dynamically.
71  *
72  * So for now, we'll restrict the driver to providing 4 masters and 4 slaves as
73  * defined above and try to support at least some of the interface from
74  * http://www.vmelinux.org/ as an alternative the driver can be written
75  * providing a saner interface later.
76  *
77  * The vmelinux.org driver never supported slave images, the devices reserved
78  * for slaves were repurposed to support all 8 master images on the UniverseII!
79  * We shall support 4 masters and 4 slaves with this driver.
80  */
81 #define VME_MAJOR       221     /* VME Major Device Number */
82 #define VME_DEVS        9       /* Number of dev entries */
83
84 #define MASTER_MINOR    0
85 #define MASTER_MAX      3
86 #define SLAVE_MINOR     4
87 #define SLAVE_MAX       7
88 #define CONTROL_MINOR   8
89
90 #define PCI_BUF_SIZE  0x20000   /* Size of one slave image buffer */
91
92 /*
93  * Structure to handle image related parameters.
94  */
95 struct image_desc {
96         void *kern_buf; /* Buffer address in kernel space */
97         dma_addr_t pci_buf;     /* Buffer address in PCI address space */
98         unsigned long long size_buf;    /* Buffer size */
99         struct mutex mutex;     /* Mutex for locking image */
100         struct device *device;  /* Sysfs device */
101         struct vme_resource *resource;  /* VME resource */
102         int users;              /* Number of current users */
103 };
104 static struct image_desc image[VME_DEVS];
105
106 struct driver_stats {
107         unsigned long reads;
108         unsigned long writes;
109         unsigned long ioctls;
110         unsigned long irqs;
111         unsigned long berrs;
112         unsigned long dmaerrors;
113         unsigned long timeouts;
114         unsigned long external;
115 };
116 static struct driver_stats statistics;
117
118 static struct cdev *vme_user_cdev;              /* Character device */
119 static struct class *vme_user_sysfs_class;      /* Sysfs class */
120 static struct vme_dev *vme_user_bridge;         /* Pointer to user device */
121
122
123 static const int type[VME_DEVS] = {     MASTER_MINOR,   MASTER_MINOR,
124                                         MASTER_MINOR,   MASTER_MINOR,
125                                         SLAVE_MINOR,    SLAVE_MINOR,
126                                         SLAVE_MINOR,    SLAVE_MINOR,
127                                         CONTROL_MINOR
128                                 };
129
130
131 static int vme_user_open(struct inode *, struct file *);
132 static int vme_user_release(struct inode *, struct file *);
133 static ssize_t vme_user_read(struct file *, char __user *, size_t, loff_t *);
134 static ssize_t vme_user_write(struct file *, const char __user *, size_t,
135         loff_t *);
136 static loff_t vme_user_llseek(struct file *, loff_t, int);
137 static long vme_user_unlocked_ioctl(struct file *, unsigned int, unsigned long);
138
139 static int vme_user_match(struct vme_dev *);
140 static int vme_user_probe(struct vme_dev *);
141 static int vme_user_remove(struct vme_dev *);
142
143 static const struct file_operations vme_user_fops = {
144         .open = vme_user_open,
145         .release = vme_user_release,
146         .read = vme_user_read,
147         .write = vme_user_write,
148         .llseek = vme_user_llseek,
149         .unlocked_ioctl = vme_user_unlocked_ioctl,
150         .compat_ioctl = vme_user_unlocked_ioctl,
151 };
152
153
154 /*
155  * Reset all the statistic counters
156  */
157 static void reset_counters(void)
158 {
159         statistics.reads = 0;
160         statistics.writes = 0;
161         statistics.ioctls = 0;
162         statistics.irqs = 0;
163         statistics.berrs = 0;
164         statistics.dmaerrors = 0;
165         statistics.timeouts = 0;
166 }
167
168 static int vme_user_open(struct inode *inode, struct file *file)
169 {
170         int err;
171         unsigned int minor = MINOR(inode->i_rdev);
172
173         mutex_lock(&image[minor].mutex);
174         /* Allow device to be opened if a resource is needed and allocated. */
175         if (minor < CONTROL_MINOR && image[minor].resource == NULL) {
176                 pr_err("No resources allocated for device\n");
177                 err = -EINVAL;
178                 goto err_res;
179         }
180
181         /* Increment user count */
182         image[minor].users++;
183
184         mutex_unlock(&image[minor].mutex);
185
186         return 0;
187
188 err_res:
189         mutex_unlock(&image[minor].mutex);
190
191         return err;
192 }
193
194 static int vme_user_release(struct inode *inode, struct file *file)
195 {
196         unsigned int minor = MINOR(inode->i_rdev);
197
198         mutex_lock(&image[minor].mutex);
199
200         /* Decrement user count */
201         image[minor].users--;
202
203         mutex_unlock(&image[minor].mutex);
204
205         return 0;
206 }
207
208 /*
209  * We are going ot alloc a page during init per window for small transfers.
210  * Small transfers will go VME -> buffer -> user space. Larger (more than a
211  * page) transfers will lock the user space buffer into memory and then
212  * transfer the data directly into the user space buffers.
213  */
214 static ssize_t resource_to_user(int minor, char __user *buf, size_t count,
215         loff_t *ppos)
216 {
217         ssize_t retval;
218         ssize_t copied = 0;
219
220         if (count <= image[minor].size_buf) {
221                 /* We copy to kernel buffer */
222                 copied = vme_master_read(image[minor].resource,
223                         image[minor].kern_buf, count, *ppos);
224                 if (copied < 0)
225                         return (int)copied;
226
227                 retval = __copy_to_user(buf, image[minor].kern_buf,
228                         (unsigned long)copied);
229                 if (retval != 0) {
230                         copied = (copied - retval);
231                         pr_info("User copy failed\n");
232                         return -EINVAL;
233                 }
234
235         } else {
236                 /* XXX Need to write this */
237                 pr_info("Currently don't support large transfers\n");
238                 /* Map in pages from userspace */
239
240                 /* Call vme_master_read to do the transfer */
241                 return -EINVAL;
242         }
243
244         return copied;
245 }
246
247 /*
248  * We are going to alloc a page during init per window for small transfers.
249  * Small transfers will go user space -> buffer -> VME. Larger (more than a
250  * page) transfers will lock the user space buffer into memory and then
251  * transfer the data directly from the user space buffers out to VME.
252  */
253 static ssize_t resource_from_user(unsigned int minor, const char __user *buf,
254         size_t count, loff_t *ppos)
255 {
256         ssize_t retval;
257         ssize_t copied = 0;
258
259         if (count <= image[minor].size_buf) {
260                 retval = __copy_from_user(image[minor].kern_buf, buf,
261                         (unsigned long)count);
262                 if (retval != 0)
263                         copied = (copied - retval);
264                 else
265                         copied = count;
266
267                 copied = vme_master_write(image[minor].resource,
268                         image[minor].kern_buf, copied, *ppos);
269         } else {
270                 /* XXX Need to write this */
271                 pr_info("Currently don't support large transfers\n");
272                 /* Map in pages from userspace */
273
274                 /* Call vme_master_write to do the transfer */
275                 return -EINVAL;
276         }
277
278         return copied;
279 }
280
281 static ssize_t buffer_to_user(unsigned int minor, char __user *buf,
282         size_t count, loff_t *ppos)
283 {
284         void *image_ptr;
285         ssize_t retval;
286
287         image_ptr = image[minor].kern_buf + *ppos;
288
289         retval = __copy_to_user(buf, image_ptr, (unsigned long)count);
290         if (retval != 0) {
291                 retval = (count - retval);
292                 pr_warn("Partial copy to userspace\n");
293         } else
294                 retval = count;
295
296         /* Return number of bytes successfully read */
297         return retval;
298 }
299
300 static ssize_t buffer_from_user(unsigned int minor, const char __user *buf,
301         size_t count, loff_t *ppos)
302 {
303         void *image_ptr;
304         size_t retval;
305
306         image_ptr = image[minor].kern_buf + *ppos;
307
308         retval = __copy_from_user(image_ptr, buf, (unsigned long)count);
309         if (retval != 0) {
310                 retval = (count - retval);
311                 pr_warn("Partial copy to userspace\n");
312         } else
313                 retval = count;
314
315         /* Return number of bytes successfully read */
316         return retval;
317 }
318
319 static ssize_t vme_user_read(struct file *file, char __user *buf, size_t count,
320                         loff_t *ppos)
321 {
322         unsigned int minor = MINOR(file_inode(file)->i_rdev);
323         ssize_t retval;
324         size_t image_size;
325         size_t okcount;
326
327         if (minor == CONTROL_MINOR)
328                 return 0;
329
330         mutex_lock(&image[minor].mutex);
331
332         /* XXX Do we *really* want this helper - we can use vme_*_get ? */
333         image_size = vme_get_size(image[minor].resource);
334
335         /* Ensure we are starting at a valid location */
336         if ((*ppos < 0) || (*ppos > (image_size - 1))) {
337                 mutex_unlock(&image[minor].mutex);
338                 return 0;
339         }
340
341         /* Ensure not reading past end of the image */
342         if (*ppos + count > image_size)
343                 okcount = image_size - *ppos;
344         else
345                 okcount = count;
346
347         switch (type[minor]) {
348         case MASTER_MINOR:
349                 retval = resource_to_user(minor, buf, okcount, ppos);
350                 break;
351         case SLAVE_MINOR:
352                 retval = buffer_to_user(minor, buf, okcount, ppos);
353                 break;
354         default:
355                 retval = -EINVAL;
356         }
357
358         mutex_unlock(&image[minor].mutex);
359         if (retval > 0)
360                 *ppos += retval;
361
362         return retval;
363 }
364
365 static ssize_t vme_user_write(struct file *file, const char __user *buf,
366                         size_t count, loff_t *ppos)
367 {
368         unsigned int minor = MINOR(file_inode(file)->i_rdev);
369         ssize_t retval;
370         size_t image_size;
371         size_t okcount;
372
373         if (minor == CONTROL_MINOR)
374                 return 0;
375
376         mutex_lock(&image[minor].mutex);
377
378         image_size = vme_get_size(image[minor].resource);
379
380         /* Ensure we are starting at a valid location */
381         if ((*ppos < 0) || (*ppos > (image_size - 1))) {
382                 mutex_unlock(&image[minor].mutex);
383                 return 0;
384         }
385
386         /* Ensure not reading past end of the image */
387         if (*ppos + count > image_size)
388                 okcount = image_size - *ppos;
389         else
390                 okcount = count;
391
392         switch (type[minor]) {
393         case MASTER_MINOR:
394                 retval = resource_from_user(minor, buf, okcount, ppos);
395                 break;
396         case SLAVE_MINOR:
397                 retval = buffer_from_user(minor, buf, okcount, ppos);
398                 break;
399         default:
400                 retval = -EINVAL;
401         }
402
403         mutex_unlock(&image[minor].mutex);
404
405         if (retval > 0)
406                 *ppos += retval;
407
408         return retval;
409 }
410
411 static loff_t vme_user_llseek(struct file *file, loff_t off, int whence)
412 {
413         loff_t absolute = -1;
414         unsigned int minor = MINOR(file_inode(file)->i_rdev);
415         size_t image_size;
416
417         if (minor == CONTROL_MINOR)
418                 return -EINVAL;
419
420         mutex_lock(&image[minor].mutex);
421         image_size = vme_get_size(image[minor].resource);
422
423         switch (whence) {
424         case SEEK_SET:
425                 absolute = off;
426                 break;
427         case SEEK_CUR:
428                 absolute = file->f_pos + off;
429                 break;
430         case SEEK_END:
431                 absolute = image_size + off;
432                 break;
433         default:
434                 mutex_unlock(&image[minor].mutex);
435                 return -EINVAL;
436         }
437
438         if ((absolute < 0) || (absolute >= image_size)) {
439                 mutex_unlock(&image[minor].mutex);
440                 return -EINVAL;
441         }
442
443         file->f_pos = absolute;
444
445         mutex_unlock(&image[minor].mutex);
446
447         return absolute;
448 }
449
450 /*
451  * The ioctls provided by the old VME access method (the one at vmelinux.org)
452  * are most certainly wrong as the effectively push the registers layout
453  * through to user space. Given that the VME core can handle multiple bridges,
454  * with different register layouts this is most certainly not the way to go.
455  *
456  * We aren't using the structures defined in the Motorola driver either - these
457  * are also quite low level, however we should use the definitions that have
458  * already been defined.
459  */
460 static int vme_user_ioctl(struct inode *inode, struct file *file,
461         unsigned int cmd, unsigned long arg)
462 {
463         struct vme_master master;
464         struct vme_slave slave;
465         struct vme_irq_id irq_req;
466         unsigned long copied;
467         unsigned int minor = MINOR(inode->i_rdev);
468         int retval;
469         dma_addr_t pci_addr;
470         void __user *argp = (void __user *)arg;
471
472         statistics.ioctls++;
473
474         switch (type[minor]) {
475         case CONTROL_MINOR:
476                 switch (cmd) {
477                 case VME_IRQ_GEN:
478                         copied = copy_from_user(&irq_req, argp,
479                                                 sizeof(struct vme_irq_id));
480                         if (copied != 0) {
481                                 pr_warn("Partial copy from userspace\n");
482                                 return -EFAULT;
483                         }
484
485                         return vme_irq_generate(vme_user_bridge,
486                                                   irq_req.level,
487                                                   irq_req.statid);
488                 }
489                 break;
490         case MASTER_MINOR:
491                 switch (cmd) {
492                 case VME_GET_MASTER:
493                         memset(&master, 0, sizeof(struct vme_master));
494
495                         /* XXX  We do not want to push aspace, cycle and width
496                          *      to userspace as they are
497                          */
498                         retval = vme_master_get(image[minor].resource,
499                                 &master.enable, &master.vme_addr,
500                                 &master.size, &master.aspace,
501                                 &master.cycle, &master.dwidth);
502
503                         copied = copy_to_user(argp, &master,
504                                 sizeof(struct vme_master));
505                         if (copied != 0) {
506                                 pr_warn("Partial copy to userspace\n");
507                                 return -EFAULT;
508                         }
509
510                         return retval;
511
512                 case VME_SET_MASTER:
513
514                         copied = copy_from_user(&master, argp, sizeof(master));
515                         if (copied != 0) {
516                                 pr_warn("Partial copy from userspace\n");
517                                 return -EFAULT;
518                         }
519
520                         /* XXX  We do not want to push aspace, cycle and width
521                          *      to userspace as they are
522                          */
523                         return vme_master_set(image[minor].resource,
524                                 master.enable, master.vme_addr, master.size,
525                                 master.aspace, master.cycle, master.dwidth);
526
527                         break;
528                 }
529                 break;
530         case SLAVE_MINOR:
531                 switch (cmd) {
532                 case VME_GET_SLAVE:
533                         memset(&slave, 0, sizeof(struct vme_slave));
534
535                         /* XXX  We do not want to push aspace, cycle and width
536                          *      to userspace as they are
537                          */
538                         retval = vme_slave_get(image[minor].resource,
539                                 &slave.enable, &slave.vme_addr,
540                                 &slave.size, &pci_addr, &slave.aspace,
541                                 &slave.cycle);
542
543                         copied = copy_to_user(argp, &slave,
544                                 sizeof(struct vme_slave));
545                         if (copied != 0) {
546                                 pr_warn("Partial copy to userspace\n");
547                                 return -EFAULT;
548                         }
549
550                         return retval;
551
552                 case VME_SET_SLAVE:
553
554                         copied = copy_from_user(&slave, argp, sizeof(slave));
555                         if (copied != 0) {
556                                 pr_warn("Partial copy from userspace\n");
557                                 return -EFAULT;
558                         }
559
560                         /* XXX  We do not want to push aspace, cycle and width
561                          *      to userspace as they are
562                          */
563                         return vme_slave_set(image[minor].resource,
564                                 slave.enable, slave.vme_addr, slave.size,
565                                 image[minor].pci_buf, slave.aspace,
566                                 slave.cycle);
567
568                         break;
569                 }
570                 break;
571         }
572
573         return -EINVAL;
574 }
575
576 static long
577 vme_user_unlocked_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
578 {
579         int ret;
580
581         mutex_lock(&vme_user_mutex);
582         ret = vme_user_ioctl(file_inode(file), file, cmd, arg);
583         mutex_unlock(&vme_user_mutex);
584
585         return ret;
586 }
587
588
589 /*
590  * Unallocate a previously allocated buffer
591  */
592 static void buf_unalloc(int num)
593 {
594         if (image[num].kern_buf) {
595 #ifdef VME_DEBUG
596                 pr_debug("UniverseII:Releasing buffer at %p\n",
597                          image[num].pci_buf);
598 #endif
599
600                 vme_free_consistent(image[num].resource, image[num].size_buf,
601                         image[num].kern_buf, image[num].pci_buf);
602
603                 image[num].kern_buf = NULL;
604                 image[num].pci_buf = 0;
605                 image[num].size_buf = 0;
606
607 #ifdef VME_DEBUG
608         } else {
609                 pr_debug("UniverseII: Buffer not allocated\n");
610 #endif
611         }
612 }
613
614 static struct vme_driver vme_user_driver = {
615         .name = driver_name,
616         .match = vme_user_match,
617         .probe = vme_user_probe,
618         .remove = vme_user_remove,
619 };
620
621
622 static int __init vme_user_init(void)
623 {
624         int retval = 0;
625
626         pr_info("VME User Space Access Driver\n");
627
628         if (bus_num == 0) {
629                 pr_err("No cards, skipping registration\n");
630                 retval = -ENODEV;
631                 goto err_nocard;
632         }
633
634         /* Let's start by supporting one bus, we can support more than one
635          * in future revisions if that ever becomes necessary.
636          */
637         if (bus_num > VME_USER_BUS_MAX) {
638                 pr_err("Driver only able to handle %d buses\n",
639                        VME_USER_BUS_MAX);
640                 bus_num = VME_USER_BUS_MAX;
641         }
642
643         /*
644          * Here we just register the maximum number of devices we can and
645          * leave vme_user_match() to allow only 1 to go through to probe().
646          * This way, if we later want to allow multiple user access devices,
647          * we just change the code in vme_user_match().
648          */
649         retval = vme_register_driver(&vme_user_driver, VME_MAX_SLOTS);
650         if (retval != 0)
651                 goto err_reg;
652
653         return retval;
654
655 err_reg:
656 err_nocard:
657         return retval;
658 }
659
660 static int vme_user_match(struct vme_dev *vdev)
661 {
662         int i;
663
664         int cur_bus = vme_bus_num(vdev);
665         int cur_slot = vme_slot_num(vdev);
666
667         for (i = 0; i < bus_num; i++)
668                 if ((cur_bus == bus[i]) && (cur_slot == vdev->num))
669                         return 1;
670
671         return 0;
672 }
673
674 /*
675  * In this simple access driver, the old behaviour is being preserved as much
676  * as practical. We will therefore reserve the buffers and request the images
677  * here so that we don't have to do it later.
678  */
679 static int vme_user_probe(struct vme_dev *vdev)
680 {
681         int i, err;
682         char *name;
683
684         /* Save pointer to the bridge device */
685         if (vme_user_bridge != NULL) {
686                 dev_err(&vdev->dev, "Driver can only be loaded for 1 device\n");
687                 err = -EINVAL;
688                 goto err_dev;
689         }
690         vme_user_bridge = vdev;
691
692         /* Initialise descriptors */
693         for (i = 0; i < VME_DEVS; i++) {
694                 image[i].kern_buf = NULL;
695                 image[i].pci_buf = 0;
696                 mutex_init(&image[i].mutex);
697                 image[i].device = NULL;
698                 image[i].resource = NULL;
699                 image[i].users = 0;
700         }
701
702         /* Initialise statistics counters */
703         reset_counters();
704
705         /* Assign major and minor numbers for the driver */
706         err = register_chrdev_region(MKDEV(VME_MAJOR, 0), VME_DEVS,
707                 driver_name);
708         if (err) {
709                 dev_warn(&vdev->dev, "Error getting Major Number %d for driver.\n",
710                          VME_MAJOR);
711                 goto err_region;
712         }
713
714         /* Register the driver as a char device */
715         vme_user_cdev = cdev_alloc();
716         if (!vme_user_cdev) {
717                 err = -ENOMEM;
718                 goto err_char;
719         }
720         vme_user_cdev->ops = &vme_user_fops;
721         vme_user_cdev->owner = THIS_MODULE;
722         err = cdev_add(vme_user_cdev, MKDEV(VME_MAJOR, 0), VME_DEVS);
723         if (err) {
724                 dev_warn(&vdev->dev, "cdev_all failed\n");
725                 goto err_char;
726         }
727
728         /* Request slave resources and allocate buffers (128kB wide) */
729         for (i = SLAVE_MINOR; i < (SLAVE_MAX + 1); i++) {
730                 /* XXX Need to properly request attributes */
731                 /* For ca91cx42 bridge there are only two slave windows
732                  * supporting A16 addressing, so we request A24 supported
733                  * by all windows.
734                  */
735                 image[i].resource = vme_slave_request(vme_user_bridge,
736                         VME_A24, VME_SCT);
737                 if (image[i].resource == NULL) {
738                         dev_warn(&vdev->dev,
739                                  "Unable to allocate slave resource\n");
740                         err = -ENOMEM;
741                         goto err_slave;
742                 }
743                 image[i].size_buf = PCI_BUF_SIZE;
744                 image[i].kern_buf = vme_alloc_consistent(image[i].resource,
745                         image[i].size_buf, &image[i].pci_buf);
746                 if (image[i].kern_buf == NULL) {
747                         dev_warn(&vdev->dev,
748                                  "Unable to allocate memory for buffer\n");
749                         image[i].pci_buf = 0;
750                         vme_slave_free(image[i].resource);
751                         err = -ENOMEM;
752                         goto err_slave;
753                 }
754         }
755
756         /*
757          * Request master resources allocate page sized buffers for small
758          * reads and writes
759          */
760         for (i = MASTER_MINOR; i < (MASTER_MAX + 1); i++) {
761                 /* XXX Need to properly request attributes */
762                 image[i].resource = vme_master_request(vme_user_bridge,
763                         VME_A32, VME_SCT, VME_D32);
764                 if (image[i].resource == NULL) {
765                         dev_warn(&vdev->dev,
766                                  "Unable to allocate master resource\n");
767                         err = -ENOMEM;
768                         goto err_master;
769                 }
770                 image[i].size_buf = PCI_BUF_SIZE;
771                 image[i].kern_buf = kmalloc(image[i].size_buf, GFP_KERNEL);
772                 if (image[i].kern_buf == NULL) {
773                         err = -ENOMEM;
774                         vme_master_free(image[i].resource);
775                         goto err_master;
776                 }
777         }
778
779         /* Create sysfs entries - on udev systems this creates the dev files */
780         vme_user_sysfs_class = class_create(THIS_MODULE, driver_name);
781         if (IS_ERR(vme_user_sysfs_class)) {
782                 dev_err(&vdev->dev, "Error creating vme_user class.\n");
783                 err = PTR_ERR(vme_user_sysfs_class);
784                 goto err_class;
785         }
786
787         /* Add sysfs Entries */
788         for (i = 0; i < VME_DEVS; i++) {
789                 int num;
790
791                 switch (type[i]) {
792                 case MASTER_MINOR:
793                         name = "bus/vme/m%d";
794                         break;
795                 case CONTROL_MINOR:
796                         name = "bus/vme/ctl";
797                         break;
798                 case SLAVE_MINOR:
799                         name = "bus/vme/s%d";
800                         break;
801                 default:
802                         err = -EINVAL;
803                         goto err_sysfs;
804                 }
805
806                 num = (type[i] == SLAVE_MINOR) ? i - (MASTER_MAX + 1) : i;
807                 image[i].device = device_create(vme_user_sysfs_class, NULL,
808                                         MKDEV(VME_MAJOR, i), NULL, name, num);
809                 if (IS_ERR(image[i].device)) {
810                         dev_info(&vdev->dev, "Error creating sysfs device\n");
811                         err = PTR_ERR(image[i].device);
812                         goto err_sysfs;
813                 }
814         }
815
816         return 0;
817
818 err_sysfs:
819         while (i > 0) {
820                 i--;
821                 device_destroy(vme_user_sysfs_class, MKDEV(VME_MAJOR, i));
822         }
823         class_destroy(vme_user_sysfs_class);
824
825         /* Ensure counter set correcty to unalloc all master windows */
826         i = MASTER_MAX + 1;
827 err_master:
828         while (i > MASTER_MINOR) {
829                 i--;
830                 kfree(image[i].kern_buf);
831                 vme_master_free(image[i].resource);
832         }
833
834         /*
835          * Ensure counter set correcty to unalloc all slave windows and buffers
836          */
837         i = SLAVE_MAX + 1;
838 err_slave:
839         while (i > SLAVE_MINOR) {
840                 i--;
841                 buf_unalloc(i);
842                 vme_slave_free(image[i].resource);
843         }
844 err_class:
845         cdev_del(vme_user_cdev);
846 err_char:
847         unregister_chrdev_region(MKDEV(VME_MAJOR, 0), VME_DEVS);
848 err_region:
849 err_dev:
850         return err;
851 }
852
853 static int vme_user_remove(struct vme_dev *dev)
854 {
855         int i;
856
857         /* Remove sysfs Entries */
858         for (i = 0; i < VME_DEVS; i++) {
859                 mutex_destroy(&image[i].mutex);
860                 device_destroy(vme_user_sysfs_class, MKDEV(VME_MAJOR, i));
861         }
862         class_destroy(vme_user_sysfs_class);
863
864         for (i = MASTER_MINOR; i < (MASTER_MAX + 1); i++) {
865                 kfree(image[i].kern_buf);
866                 vme_master_free(image[i].resource);
867         }
868
869         for (i = SLAVE_MINOR; i < (SLAVE_MAX + 1); i++) {
870                 vme_slave_set(image[i].resource, 0, 0, 0, 0, VME_A32, 0);
871                 buf_unalloc(i);
872                 vme_slave_free(image[i].resource);
873         }
874
875         /* Unregister device driver */
876         cdev_del(vme_user_cdev);
877
878         /* Unregiser the major and minor device numbers */
879         unregister_chrdev_region(MKDEV(VME_MAJOR, 0), VME_DEVS);
880
881         return 0;
882 }
883
884 static void __exit vme_user_exit(void)
885 {
886         vme_unregister_driver(&vme_user_driver);
887 }
888
889
890 MODULE_PARM_DESC(bus, "Enumeration of VMEbus to which the driver is connected");
891 module_param_array(bus, int, &bus_num, 0);
892
893 MODULE_DESCRIPTION("VME User Space Access Driver");
894 MODULE_AUTHOR("Martyn Welch <martyn.welch@ge.com");
895 MODULE_LICENSE("GPL");
896
897 module_init(vme_user_init);
898 module_exit(vme_user_exit);