From: Kukjin Kim <kgene.kim@samsung.com>
To: 'Daniel Lezcano' <daniel.lezcano@linaro.org>,
ben-linux@fluff.org, broonie@opensource.wolfsonmicro.com
Cc: linux-arm-kernel@lists.infradead.org,
linaro-dev@lists.linaro.org, patches@linaro.org,
linux-samsung-soc@vger.kernel.org, heiko@sntech.de
Subject: RE: [PATCH 0/2] ARM: S3C64xx: cpuidle cleanups
Date: Thu, 17 May 2012 15:57:45 +0900 [thread overview]
Message-ID: <0b2001cd33fa$5cd05490$1670fdb0$%kim@samsung.com> (raw)
In-Reply-To: <1337004377-17732-1-git-send-email-daniel.lezcano@linaro.org>
Daniel Lezcano wrote:
>
> These couple of patches use the new cpuidle core api to refactor
> some part of the code. The first one declares the states directly
> in the driver declaration and the second one use the timekeeping
> flag to let the cpuidle core to compute the idle time.
>
Basically, looks ok to me.
> I don't have this board, I was not able to test these patches.
>
Mark, could you please check this patches on your board?
Now, my smdk6410 has some problem to test this :(
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
> Daniel Lezcano (2):
> ARM: s3c64xx: cpuidle - declare the states with the new api
> ARM: s3c64xx: cpuidle - use timekeeping wrapper
>
> arch/arm/mach-s3c64xx/cpuidle.c | 45
+++++++++++++----------------------
> ----
> 1 files changed, 15 insertions(+), 30 deletions(-)
>
> --
> 1.7.5.4
WARNING: multiple messages have this Message-ID (diff)
From: kgene.kim@samsung.com (Kukjin Kim)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/2] ARM: S3C64xx: cpuidle cleanups
Date: Thu, 17 May 2012 15:57:45 +0900 [thread overview]
Message-ID: <0b2001cd33fa$5cd05490$1670fdb0$%kim@samsung.com> (raw)
In-Reply-To: <1337004377-17732-1-git-send-email-daniel.lezcano@linaro.org>
Daniel Lezcano wrote:
>
> These couple of patches use the new cpuidle core api to refactor
> some part of the code. The first one declares the states directly
> in the driver declaration and the second one use the timekeeping
> flag to let the cpuidle core to compute the idle time.
>
Basically, looks ok to me.
> I don't have this board, I was not able to test these patches.
>
Mark, could you please check this patches on your board?
Now, my smdk6410 has some problem to test this :(
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
> Daniel Lezcano (2):
> ARM: s3c64xx: cpuidle - declare the states with the new api
> ARM: s3c64xx: cpuidle - use timekeeping wrapper
>
> arch/arm/mach-s3c64xx/cpuidle.c | 45
+++++++++++++----------------------
> ----
> 1 files changed, 15 insertions(+), 30 deletions(-)
>
> --
> 1.7.5.4
next prev parent reply other threads:[~2012-05-17 6:57 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-14 14:06 [PATCH 0/2] ARM: S3C64xx: cpuidle cleanups Daniel Lezcano
2012-05-14 14:06 ` Daniel Lezcano
2012-05-14 14:06 ` [PATCH 1/2] ARM: s3c64xx: cpuidle - declare the states with the new api Daniel Lezcano
2012-05-14 14:06 ` Daniel Lezcano
2012-05-17 7:02 ` Kukjin Kim
2012-05-17 7:02 ` Kukjin Kim
2012-05-17 12:28 ` Mark Brown
2012-05-17 12:28 ` Mark Brown
2012-05-19 17:35 ` Kukjin Kim
2012-05-19 17:35 ` Kukjin Kim
[not found] ` <1337004377-17732-1-git-send-email-daniel.lezcano-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-05-14 14:06 ` [PATCH 2/2] ARM: s3c64xx: cpuidle - use timekeeping wrapper Daniel Lezcano
2012-05-14 14:06 ` Daniel Lezcano
2012-05-17 12:28 ` Mark Brown
2012-05-17 12:28 ` Mark Brown
2012-05-17 6:57 ` Kukjin Kim [this message]
2012-05-17 6:57 ` [PATCH 0/2] ARM: S3C64xx: cpuidle cleanups Kukjin Kim
-- strict thread matches above, loose matches on Subject: below --
2012-05-09 14:08 Daniel Lezcano
2012-05-13 23:51 ` Daniel Lezcano
2012-05-14 8:52 ` Heiko Stübner
2012-05-14 8:52 ` Heiko Stübner
2012-05-14 8:57 ` Mark Brown
2012-05-14 8:57 ` Mark Brown
2012-05-14 9:22 ` Daniel Lezcano
2012-05-14 9:22 ` Daniel Lezcano
2012-05-14 13:51 ` Kukjin Kim
2012-05-14 13:51 ` Kukjin Kim
2012-05-14 13:56 ` Daniel Lezcano
2012-05-14 13:56 ` Daniel Lezcano
2012-05-14 13:44 ` Daniel Lezcano
2012-05-14 13:44 ` Daniel Lezcano
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='0b2001cd33fa$5cd05490$1670fdb0$%kim@samsung.com' \
--to=kgene.kim@samsung.com \
--cc=ben-linux@fluff.org \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=daniel.lezcano@linaro.org \
--cc=heiko@sntech.de \
--cc=linaro-dev@lists.linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=patches@linaro.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.