Re: [PATCH] Staging: vme: devices: vme_user.c: fixed ERRORS and WARNINGS

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

Thanks for spending some time looking at this, I guess these are
checkpatch.pl errors? I few minor issues below.

Martyn

nanakos@xxxxxxxxxxxx wrote:
> From: Nanakos Chrysostomos <nanakos@xxxxxxxxxxxx>
>
> Fixed ERRORS and WARNINGS
>
> Signed-off-by: Nanakos Chrysostomos <nanakos@xxxxxxxxxxxx>
> ---
>  drivers/staging/vme/devices/vme_user.c |   90 +++++++++++++++----------------
>  1 files changed, 44 insertions(+), 46 deletions(-)
>
> diff --git a/drivers/staging/vme/devices/vme_user.c b/drivers/staging/vme/devices/vme_user.c
> index bc16fc0..727dc0c 100644
> --- a/drivers/staging/vme/devices/vme_user.c
> +++ b/drivers/staging/vme/devices/vme_user.c
> @@ -34,8 +34,8 @@
>  #include <linux/smp_lock.h>
>  #include <linux/types.h>
>  
> -#include <asm/io.h>
> -#include <asm/uaccess.h>
> +#include <linux/io.h>
> +#include <linux/uaccess.h>
>  
>  #include "../vme.h"
>  #include "vme_user.h"
> @@ -48,19 +48,19 @@ static int bus_num;
>  /* Currently Documentation/devices.txt defines the following for VME:
>   *
>   * 221 char	VME bus
> - * 		  0 = /dev/bus/vme/m0		First master image
> - * 		  1 = /dev/bus/vme/m1		Second master image
> - * 		  2 = /dev/bus/vme/m2		Third master image
> - * 		  3 = /dev/bus/vme/m3		Fourth master image
> - * 		  4 = /dev/bus/vme/s0		First slave image
> - * 		  5 = /dev/bus/vme/s1		Second slave image
> - * 		  6 = /dev/bus/vme/s2		Third slave image
> - * 		  7 = /dev/bus/vme/s3		Fourth slave image
> - * 		  8 = /dev/bus/vme/ctl		Control
> + *		0 = /dev/bus/vme/m0		First master image
> + *		1 = /dev/bus/vme/m1		Second master image
> + *		2 = /dev/bus/vme/m2		Third master image
> + *		3 = /dev/bus/vme/m3		Fourth master image
> + *		4 = /dev/bus/vme/s0		First slave image
> + *		5 = /dev/bus/vme/s1		Second slave image
> + *		6 = /dev/bus/vme/s2		Third slave image
> + *		7 = /dev/bus/vme/s3		Fourth slave image
> + *		8 = /dev/bus/vme/ctl		Control
>   

Whilst the space before the tabbing is definitely incorrect, the double
space after it is intentional and shouldn't cause an error or warning,
please can you leave that in.

>   *
> - * 		It is expected that all VME bus drivers will use the
> - * 		same interface.  For interface documentation see
> - * 		http://www.vmelinux.org/.
> + *		It is expected that all VME bus drivers will use the
> + *		same interface.  For interface documentation see
> + *		http://www.vmelinux.org/.
>   *
>   * However the VME driver at http://www.vmelinux.org/ is rather old and doesn't
>   * even support the tsi148 chipset (which has 8 master and 8 slave windows).
> @@ -136,13 +136,13 @@ static long vme_user_unlocked_ioctl(struct file *, unsigned int, unsigned long);
>  static int __init vme_user_probe(struct device *, int, int);
>  static int __exit vme_user_remove(struct device *, int, int);
>  
> -static struct file_operations vme_user_fops = {
> -        .open = vme_user_open,
> -        .release = vme_user_release,
> -        .read = vme_user_read,
> -        .write = vme_user_write,
> -        .llseek = vme_user_llseek,
> -        .unlocked_ioctl = vme_user_unlocked_ioctl,
> +const static struct file_operations vme_user_fops = {
> +	.open = vme_user_open,
> +	.release = vme_user_release,
> +	.read = vme_user_read,
> +	.write = vme_user_write,
> +	.llseek = vme_user_llseek,
> +	.unlocked_ioctl = vme_user_unlocked_ioctl,
>  };
>  
>  
> @@ -151,13 +151,13 @@ static struct file_operations vme_user_fops = {
>   */
>  static void reset_counters(void)
>  {
> -        statistics.reads = 0;
> -        statistics.writes = 0;
> -        statistics.ioctls = 0;
> -        statistics.irqs = 0;
> -        statistics.berrs = 0;
> -        statistics.dmaErrors = 0;
> -        statistics.timeouts = 0;
> +	statistics.reads = 0;
> +	statistics.writes = 0;
> +	statistics.ioctls = 0;
> +	statistics.irqs = 0;
> +	statistics.berrs = 0;
> +	statistics.dmaErrors = 0;
> +	statistics.timeouts = 0;
>  }
>  
>  static int vme_user_open(struct inode *inode, struct file *file)
> @@ -216,9 +216,8 @@ static ssize_t resource_to_user(int minor, char __user *buf, size_t count,
>  		/* We copy to kernel buffer */
>  		copied = vme_master_read(image[minor].resource,
>  			image[minor].kern_buf, count, *ppos);
> -		if (copied < 0) {
> +		if (copied < 0)
>  			return (int)copied;
> -		}
>  
>  		retval = __copy_to_user(buf, image[minor].kern_buf,
>  			(unsigned long)copied);
> @@ -313,7 +312,7 @@ static ssize_t buffer_from_user(unsigned int minor, const char *buf,
>  }
>  
>  static ssize_t vme_user_read(struct file *file, char *buf, size_t count,
> -			loff_t * ppos)
> +			loff_t *ppos)
>  {
>  	unsigned int minor = MINOR(file->f_dentry->d_inode->i_rdev);
>  	ssize_t retval;
> @@ -337,7 +336,7 @@ static ssize_t vme_user_read(struct file *file, char *buf, size_t count,
>  	else
>  		okcount = count;
>  
> -	switch (type[minor]){
> +	switch (type[minor]) {
>  	case MASTER_MINOR:
>  		retval = resource_to_user(minor, buf, okcount, ppos);
>  		break;
> @@ -380,7 +379,7 @@ static ssize_t vme_user_write(struct file *file, const char *buf, size_t count,
>  	else
>  		okcount = count;
>  
> -	switch (type[minor]){
> +	switch (type[minor]) {
>  	case MASTER_MINOR:
>  		retval = resource_from_user(minor, buf, okcount, ppos);
>  		break;
> @@ -571,7 +570,7 @@ vme_user_unlocked_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
>  /*
>   * Unallocate a previously allocated buffer
>   */
> -static void buf_unalloc (int num)
> +static void buf_unalloc(int num)
>  {
>  	if (image[num].kern_buf) {
>  #ifdef VME_DEBUG
> @@ -594,8 +593,8 @@ static void buf_unalloc (int num)
>  }
>  
>  static struct vme_driver vme_user_driver = {
> -        .name = driver_name,
> -        .probe = vme_user_probe,
> +	.name = driver_name,
> +	.probe = vme_user_probe,
>  	.remove = vme_user_remove,
>  };
>  
> @@ -770,16 +769,16 @@ static int __init vme_user_probe(struct device *dev, int cur_bus, int cur_slot)
>  	}
>  
>  	/* Add sysfs Entries */
> -	for (i=0; i<VME_DEVS; i++) {
> +	for (i = 0; i < VME_DEVS; i++) {
>  		switch (type[i]) {
>  		case MASTER_MINOR:
> -			sprintf(name,"bus/vme/m%%d");
> +			sprintf(name, "bus/vme/m%%d");
>  			break;
>  		case CONTROL_MINOR:
> -			sprintf(name,"bus/vme/ctl");
> +			sprintf(name, "bus/vme/ctl");
>  			break;
>  		case SLAVE_MINOR:
> -			sprintf(name,"bus/vme/s%%d");
> +			sprintf(name, "bus/vme/s%%d");
>  			break;
>  		default:
>  			err = -EINVAL;
> @@ -788,9 +787,9 @@ static int __init vme_user_probe(struct device *dev, int cur_bus, int cur_slot)
>  		}
>  
>  		image[i].device =
> -			device_create(vme_user_sysfs_class, NULL,
> -				MKDEV(VME_MAJOR, i), NULL, name,
> -				(type[i] == SLAVE_MINOR)? i - (MASTER_MAX + 1) : i);
> +		   device_create(vme_user_sysfs_class, NULL,
> +			MKDEV(VME_MAJOR, i), NULL, name,
> +			(type[i] == SLAVE_MINOR) ? i - (MASTER_MAX + 1) : i);
>   

It looks like you have compacted the tabbing here to try and avoid the
80 character width limit, but in doing so have replaced some clean
tabbing at the beginning of the "device_create" line for spaces. I'm not
sure this is overly constructive.

>  		if (IS_ERR(image[i].device)) {
>  			printk("%s: Error creating sysfs device\n",
>  				driver_name);
> @@ -804,7 +803,7 @@ static int __init vme_user_probe(struct device *dev, int cur_bus, int cur_slot)
>  	/* Ensure counter set correcty to destroy all sysfs devices */
>  	i = VME_DEVS;
>  err_sysfs:
> -	while (i > 0){
> +	while (i > 0) {
>  		i--;
>  		device_destroy(vme_user_sysfs_class, MKDEV(VME_MAJOR, i));
>  	}
> @@ -845,9 +844,8 @@ static int __exit vme_user_remove(struct device *dev, int cur_bus, int cur_slot)
>  	int i;
>  
>  	/* Remove sysfs Entries */
> -	for(i=0; i<VME_DEVS; i++) {
> +	for (i = 0; i < VME_DEVS; i++)
>  		device_destroy(vme_user_sysfs_class, MKDEV(VME_MAJOR, i));
> -	}
>  	class_destroy(vme_user_sysfs_class);
>  
>  	for (i = MASTER_MINOR; i < (MASTER_MAX + 1); i++)
>   


-- 
Martyn Welch (Principal Software Engineer)   |   Registered in England and
GE Intelligent Platforms                     |   Wales (3828642) at 100
T +44(0)127322748                            |   Barbirolli Square, Manchester,
E martyn.welch@xxxxxx                        |   M2 3AB  VAT:GB 927559189

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux