diff mbox

[v2,3/3] rtc: rtc-s3c: Add BCD register initialization codes

Message ID 1280235553-27369-1-git-send-email-kgene.kim@samsung.com
State Superseded
Headers show

Commit Message

Kukjin Kim July 27, 2010, 12:59 p.m. UTC 
From: Taekgyun Ko <taeggyun.ko@samsung.com>

RTC needs to be initialized when BCD registers have invalid value.

Signed-off-by: Taekgyun Ko <taeggyun.ko@samsung.com>
Signed-off-by: Kukjin Kim <kgene.kim@samsung.com>
---
Changes since v1:
- Removed unnecessary parens around > operations as per Sergei Shtylyov

 drivers/rtc/rtc-s3c.c |   14 ++++++++++++--
 1 files changed, 12 insertions(+), 2 deletions(-)

Comments

Alessandro Zummo July 27, 2010, 5:46 p.m. UTC | #1 
On Tue, 27 Jul 2010 21:59:13 +0900
Kukjin Kim <kgene.kim@samsung.com> wrote:

> From: Taekgyun Ko <taeggyun.ko@samsung.com>
> 
> RTC needs to be initialized when BCD registers have invalid value.
> 
> Signed-off-by: Taekgyun Ko <taeggyun.ko@samsung.com>
> Signed-off-by: Kukjin Kim <kgene.kim@samsung.com>


 Acked-by: Alessandro Zummo <a.zummo@towertech.it>
Kukjin Kim July 28, 2010, 4:30 a.m. UTC | #2 
Alessandro Zummo wrote:
> 
> On Tue, 27 Jul 2010 21:59:13 +0900
> Kukjin Kim <kgene.kim@samsung.com> wrote:
> 
> > From: Taekgyun Ko <taeggyun.ko@samsung.com>
> >
> > RTC needs to be initialized when BCD registers have invalid value.
> >
> > Signed-off-by: Taekgyun Ko <taeggyun.ko@samsung.com>
> > Signed-off-by: Kukjin Kim <kgene.kim@samsung.com>
> 
> 
>  Acked-by: Alessandro Zummo <a.zummo@towertech.it>
> 
Thanks for your ack.

So...which subsystem tree is appropriate for this?

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
Wan ZongShun July 28, 2010, 4:48 a.m. UTC | #3 
2010/7/28 Kukjin Kim <kgene.kim@samsung.com>:
> Alessandro Zummo wrote:
>>
>> On Tue, 27 Jul 2010 21:59:13 +0900
>> Kukjin Kim <kgene.kim@samsung.com> wrote:
>>
>> > From: Taekgyun Ko <taeggyun.ko@samsung.com>
>> >
>> > RTC needs to be initialized when BCD registers have invalid value.
>> >
>> > Signed-off-by: Taekgyun Ko <taeggyun.ko@samsung.com>
>> > Signed-off-by: Kukjin Kim <kgene.kim@samsung.com>
>>
>>
>>  Acked-by: Alessandro Zummo <a.zummo@towertech.it>
>>
> Thanks for your ack.
>
> So...which subsystem tree is appropriate for this?

S3c git tree or Andrew's mm git tree.

>
> Thanks.
>
> Best regards,
> Kgene.
> --
> Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
> SW Solution Development Team, Samsung Electronics Co., Ltd.
>
> --
> 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.
diff mbox

Patch

diff --git a/drivers/rtc/rtc-s3c.c b/drivers/rtc/rtc-s3c.c
index 2040017..e96e109 100644
--- a/drivers/rtc/rtc-s3c.c
+++ b/drivers/rtc/rtc-s3c.c
@@ -536,10 +536,20 @@  static int __devinit s3c_rtc_probe(struct platform_device *pdev)
 
 	s3c_rtc_cpu_type = platform_get_device_id(pdev)->driver_data;
 
-	if (s3c_rtc_cpu_type == TYPE_S3C64XX)
+	if (s3c_rtc_cpu_type == TYPE_S3C64XX) {
 		rtc->max_user_freq = 32768;
-	else
+
+		/* Check RTC Time */
+
+		for (i = S3C2410_RTCSEC; i <= S3C2410_RTCYEAR; i += 0x4) {
+			tmp = readb(s3c_rtc_base + i);
+
+			if ((tmp & 0xf) > 0x9 || ((tmp >> 4) & 0xf) > 0x9)
+				writeb(0, s3c_rtc_base + i);
+		}
+	} else {
 		rtc->max_user_freq = 128;
+	}
 
 	platform_set_drvdata(pdev, rtc);