rtc: remove "RTC_ALM_SET mode" bugs

This fixes a common glitch in how RTC drivers handle two "set alarm" modes,
by getting rid of the surprising/hidden one that was rarely implemented
correctly (and which could expose nonportable hardware-specific behavior).

The glitch comes from the /dev/rtcX logic implementing the legacy
RTC_ALM_SET (limited to 24 hours, needing RTC_AIE_ON) ioctl on top of the
RTC driver call providing access to the newer RTC_WKALM_SET (without those
limitations) by initializing the day/month/year fields to be invalid ...
that second mode.

Now, since few RTC drivers check those fields, and most hardware misbehaves
when faced with invalid date fields, many RTC drivers will set bogus alarm
times on those RTC_ALM_SET code paths.  (Several in-tree drivers have that
issue, and I also noticed it with code reviews on several new RTC drivers.)

This patch ensures that RTC drivers never see such invalid alarm fields, by
moving some logic out of rtc-omap into the RTC_ALM_SET code and adding an
explicit check (which will prevent the issue on other code paths).

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Cc: Scott Wood <scottwood@freescale.com>
Cc: Alessandro Zummo <a.zummo@towertech.it>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

1 files changed, 0 insertions, 28 deletions

diff --git a/drivers/rtc/rtc-omap.c b/drivers/rtc/rtc-omap.c
index e6c7b0149f27..60a8a4bb8bd2 100644
--- a/drivers/rtc/rtc-omap.c
+++ b/drivers/rtc/rtc-omap.c
@@ -289,34 +289,6 @@ static int omap_rtc_set_alarm(struct device *dev, struct rtc_wkalrm *alm)
 {
         u8 reg;
 
-        /* Much userspace code uses RTC_ALM_SET, thus "don't care" for
-         * day/month/year specifies alarms up to 24 hours in the future.
-         * So we need to handle that ... but let's ignore the "don't care"
-         * values for hours/minutes/seconds.
-         */
-        if (alm->time.tm_mday <= 0
-                        && alm->time.tm_mon < 0
-                        && alm->time.tm_year < 0) {
-                struct rtc_time tm;
-                unsigned long now, then;
-
-                omap_rtc_read_time(dev, &tm);
-                rtc_tm_to_time(&tm, &now);
-
-                alm->time.tm_mday = tm.tm_mday;
-                alm->time.tm_mon = tm.tm_mon;
-                alm->time.tm_year = tm.tm_year;
-                rtc_tm_to_time(&alm->time, &then);
-
-                /* sometimes the alarm wraps into tomorrow */
-                if (then < now) {
-                        rtc_time_to_tm(now + 24 * 60 * 60, &tm);
-                        alm->time.tm_mday = tm.tm_mday;
-                        alm->time.tm_mon = tm.tm_mon;
-                        alm->time.tm_year = tm.tm_year;
-                }
-        }
-
         if (tm2bcd(&alm->time) < 0)
                 return -EINVAL;