On Tue, Jan 25, 2011 at 12:31:20PM +0100, Marek Belisko wrote: > static long get_request_value(struct ft1000_device *ft1000dev) > { > - u32 value; > - u16 tempword; > - u32 status; > + u32 value; > + u16 tempword; > + u32 status; ^^^ Whacky. > struct ft1000_info *pft1000info = netdev_priv(ft1000dev->net); > > > - if ( pft1000info->bootmode == 1) Only one blank line is needed here between the declarations and the code. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel