From dfaf09ac8555141b7311bb69b456bd96886fd90c Mon Sep 17 00:00:00 2001 From: Laxman Dewangan Date: Thu, 21 Feb 2013 16:44:36 -0800 Subject: [PATCH] drivers/rtc/rtc-tps65910.c: remove unnecessary irq stat save and restore The driver stores the interrupt enable register before going to suspend and restore in resume. Also it enables alarm before going to suspend. The driver only write the Interrupt enable register for enabling ALARM and does not enable any other bits. So it is not require to save complete register and enable ALARM interrupt before suspend and restore in resume. Also ALARM interrupt already enable if alarm is enabled before going to suspend and hence it is not require to enable explictly in suspend. Removing such above code. Signed-off-by: Laxman Dewangan Cc: Mark Brown Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds --- drivers/rtc/rtc-tps65910.c | 21 ++------------------- 1 file changed, 2 insertions(+), 19 deletions(-) diff --git a/drivers/rtc/rtc-tps65910.c b/drivers/rtc/rtc-tps65910.c index 932a655aa41c..7ef42c25358c 100644 --- a/drivers/rtc/rtc-tps65910.c +++ b/drivers/rtc/rtc-tps65910.c @@ -28,8 +28,6 @@ struct tps65910_rtc { struct rtc_device *rtc; int irq; - /* To store the list of enabled interrupts */ - u32 irqstat; }; /* Total number of RTC registers needed to set time*/ @@ -309,35 +307,20 @@ static int tps65910_rtc_remove(struct platform_device *pdev) static int tps65910_rtc_suspend(struct device *dev) { - struct tps65910 *tps = dev_get_drvdata(dev->parent); struct tps65910_rtc *tps_rtc = dev_get_drvdata(dev); - u8 alarm = TPS65910_RTC_INTERRUPTS_IT_ALARM; - int ret; if (device_may_wakeup(dev)) enable_irq_wake(tps_rtc->irq); - - /* Store current list of enabled interrupts*/ - ret = regmap_read(tps->regmap, TPS65910_RTC_INTERRUPTS, - &tps->rtc->irqstat); - if (ret < 0) - return ret; - - /* Enable RTC ALARM interrupt only */ - return regmap_write(tps->regmap, TPS65910_RTC_INTERRUPTS, alarm); + return 0; } static int tps65910_rtc_resume(struct device *dev) { - struct tps65910 *tps = dev_get_drvdata(dev->parent); struct tps65910_rtc *tps_rtc = dev_get_drvdata(dev); if (device_may_wakeup(dev)) disable_irq_wake(tps_rtc->irq); - - /* Restore list of enabled interrupts before suspend */ - return regmap_write(tps->regmap, TPS65910_RTC_INTERRUPTS, - tps->rtc->irqstat); + return 0; } static const struct dev_pm_ops tps65910_rtc_pm_ops = {