linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: rob.lee@linaro.org (Robert Lee)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v5 0/9] Consolidate cpuidle functionality
Date: Sun, 26 Feb 2012 22:47:34 -0600	[thread overview]
Message-ID: <1330318063-25460-1-git-send-email-rob.lee@linaro.org> (raw)

This patch series moves vaious functionality duplicated in platform 
cpuidle drivers to the core cpuidle driver. Also, the platform irq
disabling was removed as it appears that all calls into 
cpuidle_call_idle will have already called local_irq_disable().


Based on 3.3-rc5 plus recent exynos cpuidle patch:
http://www.spinics.net/lists/linux-samsung-soc/msg09467.html

v4 submission can be found here:
http://lists.infradead.org/pipermail/linux-arm-kernel/2012-January/082742.html

Changes since v4:
* Added common cpu_do_idle function to core cpuidle
* Added time keep irq en wrapper to core cpuidle
* Removed pre/post enter
* Re-added platforms that can use new common code.

v3 submission can be found here:
http://www.spinics.net/lists/arm-kernel/msg156751.html
Changes since v3:
* Removed drivers/cpuidle/common.c
** Removed the initialization helper functions
** Removed the wrapper used to consolidate time keeping and irq enable/disable
* Add time keeping and local_irq_disable handling in cpuidle_call_idle().
* Made necessary modifications to a few platforms that required the most changes
** Note on omap3: changed structure of omap3_idle_drvdata and added
  per_next_state and per_saved_state vars to accomodate new framework.

v2 submission can be found here:
http://comments.gmane.org/gmane.linux.ports.arm.kernel/144199

Changes since v2:
* Made various code organization and style changes as suggested in v1 review.
* Removed at91 use of common code.  A separate effort is underway to clean
at91 code and the author has offered to convert to common interface as part
of those changes (if this common interface is accepted in time).
* Made platform cpuidle_driver objects __initdata and dynamically added one
persistent instance of this object in common code.
* Removed imx5 pm usage of gpc_dvfs clock as it is no longer needed after
being enabled during clock initialization.
* Re-organized patches.

v1 submission can be found here:
http://comments.gmane.org/gmane.linux.ports.arm.kernel/142791

Changes since v1:
* Common interface moved to drivers/cpuidle and made non arch-specific.
* Made various fixes and suggested additions to the common cpuidle
code from v1 review.
* Added callback for filling in driver_data field as needed.
* Modified the various platforms with these changes.

Robert Lee (9):
  cpuidle: Add commonly used functionality for consolidation
  SH: shmobile: cpuidle consolidation
  ARM: omap: Consolidate OMAP3 cpuidle time keeping and irq enable
  ARM: omap: Consolidate OMAP4 cpuidle time keeping and irq enable
  ARM: shmobile: Consolidate cpuidle functionality
  ARM: davinci: Consolidate cpuidle functionality
  ARM: exynos: Consolidate cpuidle functionality
  ARM: kirkwood: Consolidate cpuidle functionality
  ARM: at91: Consolidate cpuidle functionality

 arch/arm/mach-at91/cpuidle.c          |   64 +++++++++------------------
 arch/arm/mach-davinci/cpuidle.c       |   77 +++++++++++++--------------------
 arch/arm/mach-exynos/cpuidle.c        |   52 ++--------------------
 arch/arm/mach-kirkwood/cpuidle.c      |   71 +++++++++----------------------
 arch/arm/mach-omap2/cpuidle34xx.c     |   43 ++++++++-----------
 arch/arm/mach-omap2/cpuidle44xx.c     |   21 +--------
 arch/arm/mach-shmobile/cpuidle.c      |   22 +---------
 arch/sh/kernel/cpu/shmobile/cpuidle.c |   10 +---
 drivers/cpuidle/cpuidle.c             |   37 +++++++++------
 include/linux/cpuidle.h               |   55 +++++++++++++++++++++++
 10 files changed, 180 insertions(+), 272 deletions(-)

             reply	other threads:[~2012-02-27  4:47 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-27  4:47 Robert Lee [this message]
2012-02-27  4:47 ` [PATCH v5 1/9] cpuidle: Add commonly used functionality for consolidation Robert Lee
2012-02-27 16:19   ` Jean Pihet
2012-02-27 19:23     ` Rob Lee
2012-02-28  0:06   ` Turquette, Mike
2012-02-28 15:45     ` Rob Lee
2012-02-28 15:58       ` Rob Herring
2012-02-28 20:49         ` Rob Lee
2012-02-28  0:49   ` Turquette, Mike
2012-02-28 15:50     ` Rob Lee
2012-02-28 21:42       ` Turquette, Mike
2012-02-28 23:33         ` Rob Lee
2012-02-28 23:47           ` Turquette, Mike
2012-02-29  0:04             ` Rob Lee
2012-02-27  4:47 ` [PATCH v5 2/9] SH: shmobile: cpuidle consolidation Robert Lee
2012-02-27 15:13   ` Rob Lee
2012-02-27  4:47 ` [PATCH v5 3/9] ARM: omap: Consolidate OMAP3 cpuidle time keeping and irq enable Robert Lee
2012-02-27 16:26   ` Jean Pihet
2012-02-27 19:27     ` Rob Lee
2012-02-27  4:47 ` [PATCH v5 4/9] ARM: omap: Consolidate OMAP4 " Robert Lee
2012-02-27  4:47 ` [PATCH v5 5/9] ARM: shmobile: Consolidate cpuidle functionality Robert Lee
2012-02-27  4:47 ` [PATCH v5 6/9] ARM: davinci: " Robert Lee
2012-02-27  4:47 ` [PATCH v5 7/9] ARM: exynos: " Robert Lee
2012-02-27  4:47 ` [PATCH v5 8/9] ARM: kirkwood: " Robert Lee
2012-02-27  4:47 ` [PATCH v5 9/9] ARM: at91: " Robert Lee
2012-02-27 16:32 ` [PATCH v5 0/9] " Amit Kucheria

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=1330318063-25460-1-git-send-email-rob.lee@linaro.org \
    --to=rob.lee@linaro.org \
    --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 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).