The patch titled Subject: rtc: omap: add support for pmic_power_en v4 has been added to the -mm tree. Its filename is rtc-omap-add-support-for-pmic_power_en-v4.patch This patch should soon appear at http://ozlabs.org/~akpm/mmots/broken-out/rtc-omap-add-support-for-pmic_power_en-v4.patch and later at http://ozlabs.org/~akpm/mmotm/broken-out/rtc-omap-add-support-for-pmic_power_en-v4.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** The -mm tree is included into linux-next and is updated there every 3-4 working days ------------------------------------------------------ From: Johan Hovold <johan@xxxxxxxxxx> Subject: rtc: omap: add support for pmic_power_en v4 Here's an updated patch, which adds a comment on why there is a delay after activating the alarm. Note that I also increased the delay to include a margin for external latencies. For example, on Beaglebone Black, the PMIC has a 50ms deglitch time on the PWR_EN pin (which is connected to pmic_power_en). Such margins should probably eventually be specified through device tree, for instance, as an attribute to accompany the system-power-controller attribute and be handled by the power-off call chain infrastructure. Signed-off-by: Johan Hovold <johan@xxxxxxxxxx> Cc: Alessandro Zummo <a.zummo@xxxxxxxxxxxx> Cc: Tony Lindgren <tony@xxxxxxxxxxx> Cc: Benot Cousson <bcousson@xxxxxxxxxxxx> Cc: Lokesh Vutla <lokeshvutla@xxxxxx> Cc: Guenter Roeck <linux@xxxxxxxxxxxx> Cc: Sekhar Nori <nsekhar@xxxxxx> Cc: Tero Kristo <t-kristo@xxxxxx> Cc: Keerthy J <j-keerthy@xxxxxx> Tested-by: Felipe Balbi <balbi@xxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- drivers/rtc/rtc-omap.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff -puN drivers/rtc/rtc-omap.c~rtc-omap-add-support-for-pmic_power_en-v4 drivers/rtc/rtc-omap.c --- a/drivers/rtc/rtc-omap.c~rtc-omap-add-support-for-pmic_power_en-v4 +++ a/drivers/rtc/rtc-omap.c @@ -373,7 +373,7 @@ static struct omap_rtc *omap_rtc_power_o * The two-second alarm offset is the shortest offset possible as the alarm * registers must be set before the next timer update and the offset * calculation is too heavy for everything to be done within a single access - * period (~15us). + * period (~15 us). * * Called with local interrupts disabled. */ @@ -417,8 +417,12 @@ static void omap_rtc_power_off(void) rtc_writel(rtc, OMAP_RTC_INTERRUPTS_REG, val | OMAP_RTC_INTERRUPTS_IT_ALARM2); - /* Allow alarm to trigger before returning */ - mdelay(2000); + /* + * Wait for alarm to trigger (within two seconds) and external PMIC to + * power off the system. Add a 500 ms margin for external latencies + * (e.g. debounce circuits). + */ + mdelay(2500); } static struct rtc_class_ops omap_rtc_ops = { _ Patches currently in -mm which might be from johan@xxxxxxxxxx are rtc-omap-fix-clock-source-configuration.patch rtc-omap-fix-missing-wakealarm-attribute.patch rtc-omap-fix-interrupt-disable-at-probe.patch rtc-omap-clean-up-probe-error-handling.patch rtc-omap-fix-class-device-registration.patch rtc-omap-remove-unused-register-base-define.patch rtc-omap-use-dev_info.patch rtc-omap-make-platform-device-id-table-const.patch rtc-omap-add-device-abstraction.patch rtc-omap-remove-driver_name-macro.patch rtc-omap-add-structured-device-type-info.patch rtc-omap-silence-bogus-power-up-reset-message-at-probe.patch rtc-omap-add-helper-to-read-raw-bcd-time.patch rtc-omap-add-helper-to-read-32-bit-registers.patch rtc-omap-add-support-for-pmic_power_en.patch rtc-omap-add-support-for-pmic_power_en-v3.patch rtc-omap-add-support-for-pmic_power_en-v3-fix.patch rtc-omap-add-support-for-pmic_power_en-v4.patch rtc-omap-enable-wake-up-from-power-off.patch rtc-omap-fix-minor-coding-style-issues.patch rtc-omap-add-copyright-entry.patch arm-dts-am33xx-update-rtc-node-compatible-property.patch arm-dts-am335x-boneblack-enable-power-off-and-rtc-wake-up.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