The patch titled
Subject: drivers/rtc/rtc-twl.c: ensure all interrupts are disabled during probe
has been removed from the -mm tree. Its filename was
rtc-twl-ensure-all-interrupts-are-disabled-during-probe.patch
This patch was dropped because it was merged into mainline or a subsystem tree
------------------------------------------------------
From: Kevin Hilman <khilman@xxxxxx>
Subject: drivers/rtc/rtc-twl.c: ensure all interrupts are disabled during probe
On some platforms, bootloaders are known to do some interesting RTC
programming. Without going into the obscurities as to why this may be the
case, suffice it to say the the driver should not make any assumptions
about the state of the RTC when the driver loads. In particular, the
driver probe should be sure that all interrupts are disabled until
otherwise programmed.
This was discovered when finding bursty I2C traffic every second on Overo
platforms. This I2C overhead was keeping the SoC from hitting deep power
states. The cause was found to be the RTC firing every second on the
I2C-connected TWL PMIC.
Special thanks to Felipe Balbi for suggesting to look for a rogue driver
as the source of the I2C traffic rather than the I2C driver itself.
Special thanks to Steve Sakoman for helping track down the source of the
continuous RTC interrups on the Overo boards.
Signed-off-by: Kevin Hilman <khilman@xxxxxx>
Cc: Felipe Balbi <balbi@xxxxxx>
Tested-by: Steve Sakoman <steve@xxxxxxxxxxx>
Cc: Alessandro Zummo <a.zummo@xxxxxxxxxxxx>
Tested-by: Shubhrajyoti Datta <omaplinuxkernel@xxxxxxxxx>
Cc: <stable@xxxxxxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
drivers/rtc/rtc-twl.c | 5 +++++
1 file changed, 5 insertions(+)
diff -puN drivers/rtc/rtc-twl.c~rtc-twl-ensure-all-interrupts-are-disabled-during-probe drivers/rtc/rtc-twl.c
--- a/drivers/rtc/rtc-twl.c~rtc-twl-ensure-all-interrupts-are-disabled-during-probe
+++ a/drivers/rtc/rtc-twl.c
@@ -495,6 +495,11 @@ static int __devinit twl_rtc_probe(struc
if (ret < 0)
goto out1;
+ /* ensure interrupts are disabled, bootloaders can be strange */
+ ret = twl_rtc_write_u8(0, REG_RTC_INTERRUPTS_REG);
+ if (ret < 0)
+ dev_warn(&pdev->dev, "unable to disable interrupt\n");
+
/* init cached IRQ enable bits */
ret = twl_rtc_read_u8(&rtc_irq_bits, REG_RTC_INTERRUPTS_REG);
if (ret < 0)
_
Patches currently in -mm which might be from khilman@xxxxxx are
origin.patch
linux-next.patch
--
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
