On 11/29/2012 10:36 PM, Julia Lawall 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, >>> +-}; > > So it often happens that eg all of suspend, freeze, and poweroff have > the same value? > > julia I will send a patch to replace them with SIMPLE_DEV_PM_OPS() upstream for the wireless drivers, so this part of the patch will not be needed any more in the future. Hauke -- To unsubscribe from this list: send the line "unsubscribe backports" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html