On Tue, Feb 01, 2011 at 02:50:24AM +0100, Timo von Holtz wrote: > static struct file_operations bcm_fops = { > .owner = THIS_MODULE, > - .open = bcm_char_open, > + .open = bcm_char_open,, ^^ Only one comma needed. > .release = bcm_char_release, > .read = bcm_char_read, regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel