From: valentin.longchamp@epfl.ch (Valentin Longchamp)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] [RTC] Add Freescale MC13783 RTC driver
Date: Wed, 04 Nov 2009 19:12:51 +0100 [thread overview]
Message-ID: <4AF1C423.3010407@epfl.ch> (raw)
In-Reply-To: <20091101203443.GB18116@pengutronix.de>
Hi Uwe,
Uwe Kleine-K?nig wrote:
> Hello,
>
> On Sat, Oct 24, 2009 at 10:35:42AM +0200, Uwe Kleine-K?nig wrote:
>> This driver provides support for the RTC part integrated into the
>> Freescale MC13783 PMIC and bases on patch created earlier by Sascha
>> Hauer.
>>
>> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
>> Signed-off-by: Uwe Kleine-K?nig <u.kleine-k?nig@pengutronix.de>
>> Cc: Valentin Longchamp <valentin.longchamp@epfl.ch>
>> Cc: Paul Gortmaker <p_gortmaker@yahoo.com>
>> Cc: Alessandro Zummo <a.zummo@towertech.it>
>> Cc: rtc-linux at googlegroups.com
>> ---
>> Hello,
>>
>> this patch depends on
>>
>> mfd/mc13783: near complete rewrite
>>
>> sent earlier on lkml[1]. Compared to the earlier version of rtc support
>> on mc13783 as sent by Sascha, this driver got reset detection and
>> therefore depends on the patch above.
>>
>> A tree runnable on Phytec's PCM038 is available in my git tree
>>
>> git://git.pengutronix.de/git/ukl/linux-2.6.git mc13783
>>
>> . (Maybe I will rewrite these commits, so please expect it might change
>> in a non-fast-forward manner.)
>
> Valentin, could you already test this? Any comments by the others?
>
I have tested your patches (taken your mc13783 branch today, and merged
it into my patches rebased on 2.6.32-rc6).
Your mfd/mc13783 rewrite seems to work for me (but since there is no
real usage for now, I don't use ADC yet and regulator don't do a lot).
But it runs fine on my hardware.
However, I get the hctosys: unable to read the hardware clock error
message at boot (from drivers/rtc/hctosys.c:62). Is it normal ?
Furthermore, the date and time are saved during system off, but the time
is not updated: if I shut down the system during 10 minutes, my time
will get a 10 minute delay. Is this a normal behavior with you current
implementation or is there something we have wrong in our design/code
(we have battery for the mc13783) ? I will have a further look at this
later, didn't have time now.
Val
--
Valentin Longchamp, PhD Student, EPFL-STI-LSRO1
valentin.longchamp at epfl.ch, Phone: +41216937827
http://people.epfl.ch/valentin.longchamp
MEA3485, Station 9, CH-1015 Lausanne
WARNING: multiple messages have this Message-ID (diff)
From: Valentin Longchamp <valentin.longchamp@epfl.ch>
To: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
Sascha Hauer <s.hauer@pengutronix.de>,
Paul Gortmaker <p_gortmaker@yahoo.com>,
Alessandro Zummo <a.zummo@towertech.it>,
"rtc-linux@googlegroups.com" <rtc-linux@googlegroups.com>
Subject: Re: [PATCH] [RTC] Add Freescale MC13783 RTC driver
Date: Wed, 04 Nov 2009 19:12:51 +0100 [thread overview]
Message-ID: <4AF1C423.3010407@epfl.ch> (raw)
In-Reply-To: <20091101203443.GB18116@pengutronix.de>
Hi Uwe,
Uwe Kleine-König wrote:
> Hello,
>
> On Sat, Oct 24, 2009 at 10:35:42AM +0200, Uwe Kleine-König wrote:
>> This driver provides support for the RTC part integrated into the
>> Freescale MC13783 PMIC and bases on patch created earlier by Sascha
>> Hauer.
>>
>> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
>> Signed-off-by: Uwe Kleine-König <u.kleine-könig@pengutronix.de>
>> Cc: Valentin Longchamp <valentin.longchamp@epfl.ch>
>> Cc: Paul Gortmaker <p_gortmaker@yahoo.com>
>> Cc: Alessandro Zummo <a.zummo@towertech.it>
>> Cc: rtc-linux@googlegroups.com
>> ---
>> Hello,
>>
>> this patch depends on
>>
>> mfd/mc13783: near complete rewrite
>>
>> sent earlier on lkml[1]. Compared to the earlier version of rtc support
>> on mc13783 as sent by Sascha, this driver got reset detection and
>> therefore depends on the patch above.
>>
>> A tree runnable on Phytec's PCM038 is available in my git tree
>>
>> git://git.pengutronix.de/git/ukl/linux-2.6.git mc13783
>>
>> . (Maybe I will rewrite these commits, so please expect it might change
>> in a non-fast-forward manner.)
>
> Valentin, could you already test this? Any comments by the others?
>
I have tested your patches (taken your mc13783 branch today, and merged
it into my patches rebased on 2.6.32-rc6).
Your mfd/mc13783 rewrite seems to work for me (but since there is no
real usage for now, I don't use ADC yet and regulator don't do a lot).
But it runs fine on my hardware.
However, I get the hctosys: unable to read the hardware clock error
message at boot (from drivers/rtc/hctosys.c:62). Is it normal ?
Furthermore, the date and time are saved during system off, but the time
is not updated: if I shut down the system during 10 minutes, my time
will get a 10 minute delay. Is this a normal behavior with you current
implementation or is there something we have wrong in our design/code
(we have battery for the mc13783) ? I will have a further look at this
later, didn't have time now.
Val
--
Valentin Longchamp, PhD Student, EPFL-STI-LSRO1
valentin.longchamp@epfl.ch, Phone: +41216937827
http://people.epfl.ch/valentin.longchamp
MEA3485, Station 9, CH-1015 Lausanne
next prev parent reply other threads:[~2009-11-04 18:12 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-10-23 20:38 [PATCH] mfd/mc13783: near complete rewrite Uwe Kleine-König
2009-10-24 8:35 ` [PATCH] [RTC] Add Freescale MC13783 RTC driver Uwe Kleine-König
2009-10-24 8:35 ` Uwe Kleine-König
2009-11-01 20:34 ` Uwe Kleine-König
2009-11-01 20:34 ` Uwe Kleine-König
2009-11-04 18:12 ` Valentin Longchamp [this message]
2009-11-04 18:12 ` Valentin Longchamp
2009-11-05 15:06 ` Valentin Longchamp
2009-11-05 15:06 ` Valentin Longchamp
2009-11-10 8:32 ` [PATCH RESENT] " Uwe Kleine-König
2009-11-10 9:30 ` Alessandro Zummo
2009-11-10 11:10 ` Uwe Kleine-König
2009-11-22 21:49 ` Uwe Kleine-König
2009-11-03 19:20 ` [PATCH] " Uwe Kleine-König
2009-11-03 19:20 ` Uwe Kleine-König
2009-11-01 21:06 ` [PATCH] mfd/mc13783: near complete rewrite Uwe Kleine-König
2009-11-02 11:51 ` Mark Brown
2009-11-02 13:58 ` Uwe Kleine-König
2009-11-02 14:09 ` Mark Brown
2009-11-02 14:32 ` Uwe Kleine-König
2009-11-02 15:12 ` Mark Brown
2009-11-02 20:56 ` [PATCH] mfd/mc13783: change type of irq handlers to irq_handler_t Uwe Kleine-König
2009-11-03 19:31 ` [PATCH] mfd/mc13783: near complete rewrite Uwe Kleine-König
2009-11-04 18:35 ` Samuel Ortiz
2009-11-04 22:28 ` Uwe Kleine-König
2009-11-05 22:31 ` Samuel Ortiz
2009-11-05 23:53 ` Uwe Kleine-König
2009-11-05 23:56 ` Uwe Kleine-König
2009-11-06 0:28 ` Samuel Ortiz
2009-11-10 8:18 ` [PATCH 1/2] regulator/mc13783: rename source file to match other drivers Uwe Kleine-König
2009-11-10 8:18 ` [PATCH 2/2] regulator/mc13783: various cleanups Uwe Kleine-König
2009-11-10 13:08 ` Mark Brown
2009-11-11 14:11 ` Liam Girdwood
2009-11-11 14:16 ` Uwe Kleine-König
2009-11-11 14:30 ` Liam Girdwood
2009-11-10 11:44 ` [PATCH 1/2] regulator/mc13783: rename source file to match other drivers Mark Brown
2009-11-11 14:09 ` Liam Girdwood
2009-11-24 21:44 ` [PATCH] mfd/mc13783: near complete rewrite Uwe Kleine-König
2009-11-24 23:26 ` Samuel Ortiz
2009-12-02 18:54 ` Uwe Kleine-König
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4AF1C423.3010407@epfl.ch \
--to=valentin.longchamp@epfl.ch \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.