Subject: Re: [PATCH] CONFIG_PM=n slim: drivers/net/*

From: Roland Dreier <[email-masked]>
Date: Sat, 12 Aug 2006 10:06:29 -0700

> > 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.