Commit 5ef11c56 authored by Paul Cercueil's avatar Paul Cercueil Committed by Rafael J. Wysocki
Browse files

r8169: Avoid misuse of pm_ptr() macro 



The pm_ptr() macro should be used when the suspend and resume functions
can be compiled independently of the CONFIG_PM Kconfig option.

In the case of this driver, the suspend and resume functions are inside
a section protected by a #ifdef CONFIG_PM guard. Therefore pm_ptr()
should not be used.

Signed-off-by: default avatarPaul Cercueil <paul@crapouillou.net>
Reviewed-by: default avatarArnd Bergmann <arnd@arndb.de>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent 2585cf9d

drivers/net/ethernet/realtek/r8169_main.c

+3 −1
Original line number Diff line number Diff line
@@ -5441,7 +5441,9 @@ static struct pci_driver rtl8169_pci_driver = { 
	.probe		= rtl_init_one,

	.remove		= rtl_remove_one,

	.shutdown	= rtl_shutdown,

	.driver.pm	= pm_ptr(&rtl8169_pm_ops),

#ifdef CONFIG_PM

	.driver.pm	= &rtl8169_pm_ops,

#endif

};



module_pci_driver(rtl8169_pci_driver);