From: Babu Moger <babu.moger@xxxxxxxxxx> Date: Wed, 23 Mar 2016 15:37:56 -0700 > +#ifdef CONFIG_PCI_IOV > +int pcibios_add_device(struct pci_dev *dev) > +{ > + struct pci_dev *pdev; > + /* > + * Add sriov arch specific initialization here. > + * Copy dev_archdata from PF to VF > + */ Always have an empty line after a set of local variable declarations. Sparc code is to have comments formatted: /* Like * this. */ > + memcpy(&dev->dev.archdata, &pdev->dev.archdata, > + sizeof(struct dev_archdata)); Sparc code should indent multi-line expressions by having the second and subsequent line be indented exactly to the column following the openning parenthesis on the first line. You must use the appropriate number of TAB then SPACE characters necessary to do so. Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html