[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <f291cca2741f6ac994b2bde1fb9d21194fec4d3e.1765968841.git.ukleinek@kernel.org>
Date: Wed, 17 Dec 2025 12:09:24 +0100
From: Uwe Kleine-König <u.kleine-koenig@...libre.com>
To: Ulf Hansson <ulf.hansson@...aro.org>
Cc: linux-mmc@...r.kernel.org,
linux-wireless@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: [PATCH v1 2/4] wifi: rsi: sdio: Reduce use of #ifdef for CONFIG_PM
This drops two #ifdefs (which is good because they are ugly) without
changing semantics. This also improves compile coverage because all the
code in the first #ifdef block is now compiled even for configurations
without CONFIG_PM (and then thrown away).
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@...libre.com>
---
drivers/net/wireless/rsi/rsi_91x_sdio.c | 8 ++------
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/net/wireless/rsi/rsi_91x_sdio.c b/drivers/net/wireless/rsi/rsi_91x_sdio.c
index 1e578533e473..c2b2d09b616f 100644
--- a/drivers/net/wireless/rsi/rsi_91x_sdio.c
+++ b/drivers/net/wireless/rsi/rsi_91x_sdio.c
@@ -1243,7 +1243,6 @@ static void rsi_disconnect(struct sdio_func *pfunction)
}
-#ifdef CONFIG_PM
static int rsi_set_sdio_pm_caps(struct rsi_hw *adapter)
{
struct rsi_91x_sdiodev *dev = adapter->rsi_dev;
@@ -1499,7 +1498,6 @@ static const struct dev_pm_ops rsi_pm_ops = {
.thaw = rsi_thaw,
.restore = rsi_restore,
};
-#endif
static const struct sdio_device_id rsi_dev_table[] = {
{ SDIO_DEVICE(SDIO_VENDOR_ID_RSI, SDIO_DEVICE_ID_RSI_9113) },
@@ -1512,12 +1510,10 @@ static struct sdio_driver rsi_driver = {
.probe = rsi_probe,
.remove = rsi_disconnect,
.id_table = rsi_dev_table,
-#ifdef CONFIG_PM
.drv = {
- .pm = &rsi_pm_ops,
- .shutdown = rsi_shutdown,
+ .pm = pm_ptr(&rsi_pm_ops),
+ .shutdown = pm_ptr(rsi_shutdown),
}
-#endif
};
module_sdio_driver(rsi_driver);
--
2.47.3
Powered by blists - more mailing lists