Re: [PATCH] Staging: comedi: fix bracing and 80 character style issues in ni_670x.c This is a patch to the ni_670x.c file that fixes up bracing and 80 character line issues found by checkpatch.pl tool Signed-off-by Graham M Howe <gman.1352@xxxxxxxxxxxxxx>

[Jiri Slaby <jirislaby@xxxxxxxxx>]

On 02/12/2010 03:20 PM, gman.1352@xxxxxxxxxxxxxx wrote:
> From: root <root@graham-eeepc.(none)>

Hm :).

> @@ -219,9 +221,9 @@ static int ni_670x_detach(struct comedi_device *dev)
>  {
>  	printk("comedi%d: ni_670x: remove\n", dev->minor);
> 
> -	if (dev->subdevices[0].range_table_list) {
> +	if (dev->subdevices[0].range_table_list)
>  		kfree(dev->subdevices[0].range_table_list);

Actually kfree(NULL) is fine, so the test can go away completely.

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