> > Also this won't allow > > > > struct pci_driver { > > ... > > #ifdef CONFIG_PM > > int (*suspend)(...); > > int (*resume)(...); > > #endif > > ... > > }; > > > > which is good for a) space savings in CONFIG_PM=n case, and > > b) making drivers care about CONFIG_PM=n users hard way aka compilation > > failure. > > eh? Both versions will generate identical code.
I think his point was that by #ifdef-ing out the assignment to .suspend/.resume, then the actual suspend/resume members of struct pci_driver could be removed in the CONFIG_PM=n case -- which would shrink sizeof (struct pci_driver). - R. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html