Thread (7 messages) 7 messages, 3 authors, 2016-07-08

[rtc-linux] Re: [PATCH] rtc: ds1307: Fix relying on reset value for weekday

From: Keerthy <hidden>
Date: 2016-06-15 04:15:57
Also in: lkml


On Tuesday 14 June 2016 08:09 PM, Alexandre Belloni wrote:
On 07/06/2016 at 14:59:05 +0530, Keerthy wrote :
quoted
Hi Alexandre,

On Wednesday 01 June 2016 06:06 PM, Keerthy wrote:
quoted

On Wednesday 01 June 2016 05:48 PM, Alexandre Belloni wrote:
quoted
Hi,

On 01/06/2016 at 16:19:07 +0530, Keerthy wrote :
quoted
The reset value of weekday is 0x1. This is wrong since
the reset values of the day/month/year make up to Jan 1 2001.
When computed weekday comes out to be Monday. On a scale
of 1-7(Sunday - Saturday) it should be 0x2. So we should not
be relying on the reset value.
Hum, what are the chances that the reset value is actually the correct
date/time?
Won't that be corrected after the first call to set_time? Until then,
the date is not correct so, do we care anyway?
Yes if an alarm is programmed without set_time.

ex: rtcwake -d /dev/rtc0 -s 5

Even the basic rtctest under tools/testing/selftests/timers/rtctest.c fails
as the wday is wrong and no alarm fires.

Instead of relying on some one to call set_time before programming alarm
its better to fix the wday to reflect the current day right?

quoted
Maybe I'm missing something here.
Let me know if you feel this is a valid fix.
Well, this is probably a valid fix and I'll take it as is if I don't
find the time to make something more generic.
Okay Thanks.
-- 
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
--- 
You received this message because you are subscribed to the Google Groups "rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help