Re: [PATCH V2] rtc: add RTC driver for TPS6586x

From: Andrew Morton
Date: Wed Jan 02 2013 - 18:50:46 EST


On Thu, 27 Dec 2012 20:29:14 +0530
Laxman Dewangan <ldewangan@xxxxxxxxxx> wrote:

> On Thursday 27 December 2012 08:26 PM, Marc Dietrich wrote:
> > Hi Laxman,
> >
> > On Friday 21 December 2012 20:42:28 you wrote:
> >
> >> +
> >> + /* Set epoch start as 00:00:00:01:01:2000 */
> >> + rtc->epoch_start = mktime(2000, 1, 1, 0, 0, 0);
> > any reason why you hard coded it to 2000? All boards I know use 2009, so with
> > this patch, everyone needs to set his clock again.
> >
>
> No specific reason. If 2009 should be default then I can push patch to
> set it as 2009.

I'd like to get this into Linus soon so let's get it finished. I
assume it's merely this?

--- a/drivers/rtc/rtc-tps6586x.c~rtc-add-rtc-driver-for-tps6586x-fix2
+++ a/drivers/rtc/rtc-tps6586x.c
@@ -260,8 +260,8 @@ static int tps6586x_rtc_probe(struct pla
rtc->dev = &pdev->dev;
rtc->irq = platform_get_irq(pdev, 0);

- /* Set epoch start as 00:00:00:01:01:2000 */
- rtc->epoch_start = mktime(2000, 1, 1, 0, 0, 0);
+ /* Set epoch start as 00:00:00:01:01:2009 */
+ rtc->epoch_start = mktime(2009, 1, 1, 0, 0, 0);

/* 1 kHz tick mode, enable tick counting */
ret = tps6586x_update(tps_dev, RTC_CTRL,
_

--
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/