It is not necessary to call device_init_wakeup(dev, false) in .remove as
device_del will take care of that. It is also not necessary to
devm_free_irq. Finally, dev_pm_clear_wake_irq can be called
unconditionally.

Signed-off-by: Alexandre Belloni <alexandre.bell...@bootlin.com>
---
 drivers/rtc/rtc-ds1343.c | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/drivers/rtc/rtc-ds1343.c b/drivers/rtc/rtc-ds1343.c
index 3e2957983703..73513688f6f7 100644
--- a/drivers/rtc/rtc-ds1343.c
+++ b/drivers/rtc/rtc-ds1343.c
@@ -437,15 +437,7 @@ static int ds1343_probe(struct spi_device *spi)
 
 static int ds1343_remove(struct spi_device *spi)
 {
-       struct ds1343_priv *priv = spi_get_drvdata(spi);
-
-       if (spi->irq) {
-               dev_pm_clear_wake_irq(&spi->dev);
-               device_init_wakeup(&spi->dev, false);
-               devm_free_irq(&spi->dev, spi->irq, priv);
-       }
-
-       spi_set_drvdata(spi, NULL);
+       dev_pm_clear_wake_irq(&spi->dev);
 
        return 0;
 }
-- 
2.21.0

Reply via email to