Re: [PATCH 7/14] RTC: Remove RTC UIP synchronization on MIPS Footbridge

From: Matt Mackall
Date: Fri Mar 17 2006 - 19:05:59 EST


On Fri, Mar 17, 2006 at 05:30:36PM -0600, Matt Mackall wrote:
> Remove RTC UIP synchronization on MIPS Footbridge

Urgh, this is obviously ARM. And I noticed my per-patch cc:es got
lost.

> Signed-off-by: Matt Mackall <mpm@xxxxxxxxxxx>
>
> Index: rtc/arch/arm/mach-footbridge/time.c
> ===================================================================
> --- rtc.orig/arch/arm/mach-footbridge/time.c 2005-10-27 19:02:08.000000000 -0500
> +++ rtc/arch/arm/mach-footbridge/time.c 2006-03-12 13:00:51.000000000 -0600
> @@ -34,27 +34,12 @@ static int rtc_base;
> static unsigned long __init get_isa_cmos_time(void)
> {
> unsigned int year, mon, day, hour, min, sec;
> - int i;
>
> // check to see if the RTC makes sense.....
> if ((CMOS_READ(RTC_VALID) & RTC_VRT) == 0)
> return mktime(1970, 1, 1, 0, 0, 0);
>
> - /* The Linux interpretation of the CMOS clock register contents:
> - * When the Update-In-Progress (UIP) flag goes from 1 to 0, the
> - * RTC registers show the second which has precisely just started.
> - * Let's hope other operating systems interpret the RTC the same way.
> - */
> - /* read RTC exactly on falling edge of update flag */
> - for (i = 0 ; i < 1000000 ; i++) /* may take up to 1 second... */
> - if (CMOS_READ(RTC_FREQ_SELECT) & RTC_UIP)
> - break;
> -
> - for (i = 0 ; i < 1000000 ; i++) /* must try at least 2.228 ms */
> - if (!(CMOS_READ(RTC_FREQ_SELECT) & RTC_UIP))
> - break;
> -
> - do { /* Isn't this overkill ? UIP above should guarantee consistency */
> + do {
> sec = CMOS_READ(RTC_SECONDS);
> min = CMOS_READ(RTC_MINUTES);
> hour = CMOS_READ(RTC_HOURS);
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/

--
Mathematics is the supreme nostalgia of our time.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/