spi: rpc-if: Remove CONFIG_PM_SLEEP ifdefery
authorLad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Wed, 30 Dec 2020 14:57:08 +0000 (14:57 +0000)
committerMark Brown <broonie@kernel.org>
Thu, 31 Dec 2020 12:46:31 +0000 (12:46 +0000)
Use __maybe_unused for the suspend()/resume() hooks and get rid of
the CONFIG_PM_SLEEP ifdefery to improve the code.

Suggested-by: Pavel Machek <pavel@denx.de>
Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Acked-by: Pavel Machek <pavel@denx.de>
Link: https://lore.kernel.org/r/20201230145708.28544-3-prabhakar.mahadev-lad.rj@bp.renesas.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-rpc-if.c

index 3579675..c313dbe 100644 (file)
@@ -176,15 +176,14 @@ static int rpcif_spi_remove(struct platform_device *pdev)
        return 0;
 }
 
-#ifdef CONFIG_PM_SLEEP
-static int rpcif_spi_suspend(struct device *dev)
+static int __maybe_unused rpcif_spi_suspend(struct device *dev)
 {
        struct spi_controller *ctlr = dev_get_drvdata(dev);
 
        return spi_controller_suspend(ctlr);
 }
 
-static int rpcif_spi_resume(struct device *dev)
+static int __maybe_unused rpcif_spi_resume(struct device *dev)
 {
        struct spi_controller *ctlr = dev_get_drvdata(dev);
 
@@ -192,17 +191,13 @@ static int rpcif_spi_resume(struct device *dev)
 }
 
 static SIMPLE_DEV_PM_OPS(rpcif_spi_pm_ops, rpcif_spi_suspend, rpcif_spi_resume);
-#define DEV_PM_OPS     (&rpcif_spi_pm_ops)
-#else
-#define DEV_PM_OPS     NULL
-#endif
 
 static struct platform_driver rpcif_spi_driver = {
        .probe  = rpcif_spi_probe,
        .remove = rpcif_spi_remove,
        .driver = {
                .name   = "rpc-if-spi",
-               .pm     = DEV_PM_OPS,
+               .pm     = &rpcif_spi_pm_ops,
        },
 };
 module_platform_driver(rpcif_spi_driver);