From: Johannes Berg <johannes@sipsolutions.net>
To: Paul Mackerras <paulus@samba.org>
Cc: linuxppc-dev@ozlabs.org
Subject: Re: [PATCH] powerpc timer sysdev: use mktime
Date: Wed, 23 May 2007 17:28:01 +0200 [thread overview]
Message-ID: <1179934081.14317.29.camel@johannes.berg> (raw)
In-Reply-To: <1178115942.13233.61.camel@johannes.berg>
[-- Attachment #1: Type: text/plain, Size: 2072 bytes --]
On Wed, 2007-05-02 at 16:25 +0200, Johannes Berg wrote:
> This patch makes the timer sysdev use mktime instead of rtc_tm_to_time
> by simply copying the function to avoid using rtc-lib.
I think you asked for this, any reason you then didn't take it? Just
missed? Or is something wrong with it?
> Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
>
> --- wireless-dev.orig/arch/powerpc/sysdev/timer.c 2007-05-02 16:16:52.639177080 +0200
> +++ wireless-dev/arch/powerpc/sysdev/timer.c 2007-05-02 16:21:56.429177080 +0200
> @@ -24,7 +24,12 @@ static int timer_resume(struct sys_devic
>
> /* get current RTC time and convert to seconds */
> get_rtc_time(&cur_rtc_tm);
> - rtc_tm_to_time(&cur_rtc_tm, &cur_rtc_time);
> + cur_rtc_time = mktime(cur_rtc_tm.tm_year + 1900,
> + cur_rtc_tm.tm_mon + 1,
> + cur_rtc_tm.tm_mday,
> + cur_rtc_tm.tm_hour,
> + cur_rtc_tm.tm_min,
> + cur_rtc_tm.tm_sec);
>
> diff = cur_rtc_time - suspend_rtc_time;
>
> @@ -44,7 +49,12 @@ static int timer_suspend(struct sys_devi
> WARN_ON(!ppc_md.get_rtc_time);
>
> get_rtc_time(&suspend_rtc_tm);
> - rtc_tm_to_time(&suspend_rtc_tm, &suspend_rtc_time);
> + suspend_rtc_time = mktime(suspend_rtc_tm.tm_year + 1900,
> + suspend_rtc_tm.tm_mon + 1,
> + suspend_rtc_tm.tm_mday,
> + suspend_rtc_tm.tm_hour,
> + suspend_rtc_tm.tm_min,
> + suspend_rtc_tm.tm_sec);
>
> return 0;
> }
> --- wireless-dev.orig/arch/powerpc/Kconfig 2007-05-02 16:22:07.619177080 +0200
> +++ wireless-dev/arch/powerpc/Kconfig 2007-05-02 16:22:12.759177080 +0200
> @@ -11,11 +11,6 @@ config PPC64
> This option selects whether a 32-bit or a 64-bit kernel
> will be built.
>
> -config PPC_PM_NEEDS_RTC_LIB
> - bool
> - select RTC_LIB
> - default y if PM
> -
> config PPC32
> bool
> default y if !PPC64
>
>
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev@ozlabs.org
> https://ozlabs.org/mailman/listinfo/linuxppc-dev
>
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 190 bytes --]
next prev parent reply other threads:[~2007-05-23 15:26 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-12-13 12:38 [PATCH 0/3] some preparations for suspend on G5 powermac Johannes Berg
2006-12-13 12:38 ` [PATCH 1/3] windfarm: dont die on suspend thread signal Johannes Berg
2007-01-24 6:32 ` Benjamin Herrenschmidt
2006-12-13 12:38 ` [PATCH 2/3] powermac: proper time of day after resume Johannes Berg
2007-01-24 6:34 ` Benjamin Herrenschmidt
2007-01-24 8:46 ` Johannes Berg
2007-01-24 8:51 ` Benjamin Herrenschmidt
2007-01-24 8:54 ` Gabriel Paubert
2007-01-24 9:32 ` Benjamin Herrenschmidt
2007-01-24 10:36 ` Gabriel Paubert
2007-01-24 14:23 ` Segher Boessenkool
2007-01-24 21:44 ` Benjamin Herrenschmidt
2007-01-24 22:24 ` Segher Boessenkool
2007-01-24 22:35 ` Benjamin Herrenschmidt
2007-01-24 23:21 ` Segher Boessenkool
2007-01-25 7:39 ` Michel Dänzer
2007-01-25 8:06 ` Benjamin Herrenschmidt
2007-01-25 8:10 ` Michel Dänzer
2007-01-25 11:09 ` Gabriel Paubert
2007-01-25 12:54 ` Johannes Berg
2007-01-26 14:35 ` Johannes Berg
2007-02-01 19:16 ` [PATCH] " Johannes Berg
2007-02-02 12:19 ` Johannes Berg
2007-02-03 2:41 ` Michael Ellerman
2007-02-03 17:37 ` Johannes Berg
2007-05-02 4:45 ` Paul Mackerras
2007-05-02 4:47 ` Paul Mackerras
2007-05-02 8:26 ` Johannes Berg
2007-05-02 14:25 ` [PATCH] powerpc timer sysdev: use mktime Johannes Berg
2007-05-23 15:28 ` Johannes Berg [this message]
2007-05-28 4:33 ` Paul Mackerras
2006-12-13 12:38 ` [PATCH 3/3] powerpc: fix comment in kernel/irq.c Johannes Berg
2006-12-13 13:13 ` Johannes Berg
2006-12-19 5:40 ` Paul Mackerras
2006-12-19 18:29 ` Johannes Berg
2007-01-24 6:35 ` Benjamin Herrenschmidt
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=1179934081.14317.29.camel@johannes.berg \
--to=johannes@sipsolutions.net \
--cc=linuxppc-dev@ozlabs.org \
--cc=paulus@samba.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).