On Tue, Mar 07, 2017 at 05:33:06PM +1100, Tobin C. Harding wrote: > @@ -419,17 +411,14 @@ static int dgnc_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) > brd->dpastatus = BD_RUNNING; > > dgnc_board[dgnc_num_boards++] = brd; > - > return 0; > There's nothing wrong with putting a blank before a return 0. The blank sort of makes it stand out nicely. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel