Hi, On 06/05/2014 21:06, Bryan Evenson wrote: > Boris, > >> -----Original Message----- >> From: Boris BREZILLON [mailto:boris.brezil...@free-electrons.com] >> Sent: Tuesday, May 06, 2014 10:28 AM >> To: Bryan Evenson >> Cc: Andrew Victor; Nicolas Ferre; Jean-Christophe Plagniol-Villard; >> linux-arm- >> ker...@lists.infradead.org; Alessandro Zummo; rtc- >> li...@googlegroups.com; linux-kernel@vger.kernel.org; Boris BREZILLON >> Subject: [PATCH] rtc: rtc-at91rm9200: fix infinite wait for ACKUPD irq >> >> The rtc user must wait at least 1 sec between each time/calandar update >> (see atmel's datasheet chapter "Updating Time/Calendar"). >> >> Use the 1Hz interrupt to update the at91_rtc_upd_rdy flag and wait for >> the at91_rtc_wait_upd_rdy event if the rtc is not ready. >> >> Signed-off-by: Boris BREZILLON <boris.brezil...@free-electrons.com> >> Reported-by: Bryan Evenson <beven...@melinkcorp.com> >> --- >> Hello Bryan, >> >> I reproduced your bug (using your script) and this patch seems to fix the >> problem. >> >> Could you try it and let me know if it works for you ? > Looks good to me. I modified the test script as follows: > > ---------- > #!/bin/sh > i=0 > while [ 1 ]; do > hwclock -w -u > /dev/null 2>&1 > echo $$ $i $? > : $((i++)) > done > ---------- > > This version then attempts to write to the RTC as often as possible (script > change was due to a suggestion on the Busybox mailing list). I ran two > instances of the script, which each looped through about 60,000 times over a > 30 minute run. At no point has access to the RTC been permanently locked out > on my system. I'd call this fixed.
Great! > I'd assume this patch would be backported to the longterm releases? I'd like to wait for Nicolas' ack before asking for a backport to stable releases. Best Regards, Boris -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/