Commit 584721ca authored by Vincent Bossier's avatar Vincent Bossier Committed by Greg Kroah-Hartman

Staging VME: Fix remaining checkpatch.pl errors.

This patch solves all the existing issues reported by checkpatch.pl in the VME
sub-system.
Signed-off-by: default avatarVincent Bossier <vincent.bossier@gmail.com>
Acked-by: default avatarMartyn Welch <martyn.welch@ge.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent ed7c221c
...@@ -27,9 +27,9 @@ static void __exit vmic_exit(void); ...@@ -27,9 +27,9 @@ static void __exit vmic_exit(void);
/** Base address to access FPGA register */ /** Base address to access FPGA register */
static void *vmic_base; static void *vmic_base;
static char driver_name[] = "vmivme_7805"; static const char driver_name[] = "vmivme_7805";
static struct pci_device_id vmic_ids[] = { static DEFINE_PCI_DEVICE_TABLE(vmic_ids) = {
{ PCI_DEVICE(PCI_VENDOR_ID_VMIC, PCI_DEVICE_ID_VTIMR) }, { PCI_DEVICE(PCI_VENDOR_ID_VMIC, PCI_DEVICE_ID_VTIMR) },
{ }, { },
}; };
......
...@@ -42,7 +42,7 @@ static void __exit ca91cx42_exit(void); ...@@ -42,7 +42,7 @@ static void __exit ca91cx42_exit(void);
/* Module parameters */ /* Module parameters */
static int geoid; static int geoid;
static char driver_name[] = "vme_ca91cx42"; static const char driver_name[] = "vme_ca91cx42";
static DEFINE_PCI_DEVICE_TABLE(ca91cx42_ids) = { static DEFINE_PCI_DEVICE_TABLE(ca91cx42_ids) = {
{ PCI_DEVICE(PCI_VENDOR_ID_TUNDRA, PCI_DEVICE_ID_TUNDRA_CA91C142) }, { PCI_DEVICE(PCI_VENDOR_ID_TUNDRA, PCI_DEVICE_ID_TUNDRA_CA91C142) },
......
...@@ -44,7 +44,7 @@ static void __exit tsi148_exit(void); ...@@ -44,7 +44,7 @@ static void __exit tsi148_exit(void);
static int err_chk; static int err_chk;
static int geoid; static int geoid;
static char driver_name[] = "vme_tsi148"; static const char driver_name[] = "vme_tsi148";
static DEFINE_PCI_DEVICE_TABLE(tsi148_ids) = { static DEFINE_PCI_DEVICE_TABLE(tsi148_ids) = {
{ PCI_DEVICE(PCI_VENDOR_ID_TUNDRA, PCI_DEVICE_ID_TUNDRA_TSI148) }, { PCI_DEVICE(PCI_VENDOR_ID_TUNDRA, PCI_DEVICE_ID_TUNDRA_TSI148) },
......
...@@ -41,7 +41,7 @@ ...@@ -41,7 +41,7 @@
#include "vme_user.h" #include "vme_user.h"
static DEFINE_MUTEX(vme_user_mutex); static DEFINE_MUTEX(vme_user_mutex);
static char driver_name[] = "vme_user"; static const char driver_name[] = "vme_user";
static int bus[USER_BUS_MAX]; static int bus[USER_BUS_MAX];
static unsigned int bus_num; static unsigned int bus_num;
...@@ -91,7 +91,7 @@ static unsigned int bus_num; ...@@ -91,7 +91,7 @@ static unsigned int bus_num;
/* /*
* Structure to handle image related parameters. * Structure to handle image related parameters.
*/ */
typedef struct { struct image_desc {
void *kern_buf; /* Buffer address in kernel space */ void *kern_buf; /* Buffer address in kernel space */
dma_addr_t pci_buf; /* Buffer address in PCI address space */ dma_addr_t pci_buf; /* Buffer address in PCI address space */
unsigned long long size_buf; /* Buffer size */ unsigned long long size_buf; /* Buffer size */
...@@ -99,10 +99,10 @@ typedef struct { ...@@ -99,10 +99,10 @@ typedef struct {
struct device *device; /* Sysfs device */ struct device *device; /* Sysfs device */
struct vme_resource *resource; /* VME resource */ struct vme_resource *resource; /* VME resource */
int users; /* Number of current users */ int users; /* Number of current users */
} image_desc_t; };
static image_desc_t image[VME_DEVS]; static struct image_desc image[VME_DEVS];
typedef struct { struct driver_stats {
unsigned long reads; unsigned long reads;
unsigned long writes; unsigned long writes;
unsigned long ioctls; unsigned long ioctls;
...@@ -111,8 +111,8 @@ typedef struct { ...@@ -111,8 +111,8 @@ typedef struct {
unsigned long dmaErrors; unsigned long dmaErrors;
unsigned long timeouts; unsigned long timeouts;
unsigned long external; unsigned long external;
} driver_stats_t; };
static driver_stats_t statistics; static struct driver_stats statistics;
static struct cdev *vme_user_cdev; /* Character device */ static struct cdev *vme_user_cdev; /* Character device */
static struct class *vme_user_sysfs_class; /* Sysfs class */ static struct class *vme_user_sysfs_class; /* Sysfs class */
...@@ -138,7 +138,7 @@ static long vme_user_unlocked_ioctl(struct file *, unsigned int, unsigned long); ...@@ -138,7 +138,7 @@ static long vme_user_unlocked_ioctl(struct file *, unsigned int, unsigned long);
static int __devinit vme_user_probe(struct device *, int, int); static int __devinit vme_user_probe(struct device *, int, int);
static int __devexit vme_user_remove(struct device *, int, int); static int __devexit vme_user_remove(struct device *, int, int);
static struct file_operations vme_user_fops = { static const struct file_operations vme_user_fops = {
.open = vme_user_open, .open = vme_user_open,
.release = vme_user_release, .release = vme_user_release,
.read = vme_user_read, .read = vme_user_read,
...@@ -773,6 +773,7 @@ static int __devinit vme_user_probe(struct device *dev, int cur_bus, ...@@ -773,6 +773,7 @@ static int __devinit vme_user_probe(struct device *dev, int cur_bus,
/* Add sysfs Entries */ /* Add sysfs Entries */
for (i = 0; i < VME_DEVS; i++) { for (i = 0; i < VME_DEVS; i++) {
int num;
switch (type[i]) { switch (type[i]) {
case MASTER_MINOR: case MASTER_MINOR:
sprintf(name, "bus/vme/m%%d"); sprintf(name, "bus/vme/m%%d");
...@@ -789,10 +790,9 @@ static int __devinit vme_user_probe(struct device *dev, int cur_bus, ...@@ -789,10 +790,9 @@ static int __devinit vme_user_probe(struct device *dev, int cur_bus,
break; break;
} }
image[i].device = num = (type[i] == SLAVE_MINOR) ? i - (MASTER_MAX + 1) : i;
device_create(vme_user_sysfs_class, NULL, image[i].device = device_create(vme_user_sysfs_class, NULL,
MKDEV(VME_MAJOR, i), NULL, name, MKDEV(VME_MAJOR, i), NULL, name, num);
(type[i] == SLAVE_MINOR) ? i - (MASTER_MAX + 1) : i);
if (IS_ERR(image[i].device)) { if (IS_ERR(image[i].device)) {
printk(KERN_INFO "%s: Error creating sysfs device\n", printk(KERN_INFO "%s: Error creating sysfs device\n",
driver_name); driver_name);
......
...@@ -98,7 +98,7 @@ struct vme_device_id { ...@@ -98,7 +98,7 @@ struct vme_device_id {
struct vme_driver { struct vme_driver {
struct list_head node; struct list_head node;
char *name; const char *name;
const struct vme_device_id *bind_table; const struct vme_device_id *bind_table;
int (*probe) (struct device *, int, int); int (*probe) (struct device *, int, int);
int (*remove) (struct device *, int, int); int (*remove) (struct device *, int, int);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment