This increases build coverage and removes two ugly #ifdefs. Signed-off-by: Uwe Kleine-König --- Hello, I found the patch opportunity while looking in this driver for a different reason. Best regards Uwe drivers/net/wireless/ti/wlcore/sdio.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/net/wireless/ti/wlcore/sdio.c b/drivers/net/wireless/ti/wlcore/sdio.c index a73207bbe5d7..421d688ae58b 100644 --- a/drivers/net/wireless/ti/wlcore/sdio.c +++ b/drivers/net/wireless/ti/wlcore/sdio.c @@ -370,7 +370,6 @@ static void wl1271_remove(struct sdio_func *func) platform_device_unregister(glue->core); } -#ifdef CONFIG_PM static int wl1271_suspend(struct device *dev) { /* Tell MMC/SDIO core it's OK to power down the card @@ -422,18 +421,15 @@ static const struct dev_pm_ops wl1271_sdio_pm_ops = { .suspend = wl1271_suspend, .resume = wl1271_resume, }; -#endif static struct sdio_driver wl1271_sdio_driver = { .name = "wl1271_sdio", .id_table = wl1271_devices, .probe = wl1271_probe, .remove = wl1271_remove, -#ifdef CONFIG_PM .drv = { - .pm = &wl1271_sdio_pm_ops, + .pm = pm_ptr(&wl1271_sdio_pm_ops), }, -#endif }; module_sdio_driver(wl1271_sdio_driver); base-commit: 8f0b4cce4481fb22653697cced8d0d04027cb1e8 -- 2.47.3