On 12 October 2011 03:28, Larry Finger <Larry.Finger@xxxxxxxxxxxx> wrote: > > +static const struct dev_pm_ops rtlwifi_pm_ops = { > + .suspend = rtl_pci_suspend, > + .resume = rtl_pci_resume, > + .freeze = rtl_pci_suspend, > + .thaw = rtl_pci_resume, > + .poweroff = rtl_pci_suspend, > + .restore = rtl_pci_resume, > +}; > + Hi Larry, You can save a few lines of code here, and in the other files where it is done - there is a convenience macro for simplifying creating a pm_ops struct like this, see SIMPLE_DEV_PM_OPS() in pm.h. Cheers, Mark -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html