On Sat, 12 Aug 2006 10:06:29 -0700
Roland Dreier <[EMAIL PROTECTED]> wrote:
> > > 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).
>
OIC. Yes, that would be useful.
-
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
