From: Chunyan Zhang <firstname.lastname@example.org> To: Mark Brown <email@example.com>, Rob Herring <firstname.lastname@example.org> Cc: email@example.com, firstname.lastname@example.org, Ulf Hansson <email@example.com>, Arnd Bergmann <firstname.lastname@example.org>, Chunyan Zhang <email@example.com> Subject: [PATCH v3 0/5] Add regulator suspend and resume support Date: Fri, 12 Jan 2018 18:23:49 +0800 [thread overview] Message-ID: <firstname.lastname@example.org> (raw) In-Reply-To: <email@example.com> Some systems need to set regulators to specific states when they enter low power modes, especially around CPUs. Currently the regulator driver, for suspend and resume features, provides two functions which are exported for being called directly by any modules or subsystems when they thought the regulator should be entering into suspend states. This patchset adds hooks to PM suspend core and provides suspend/resume callback functions to regulator device, for those who can be switched off or set low voltage in suspend states only need to implement the callback functions in the driver, and set the right configurations for suspend states via device tree and the APIs which regulator core driver provides. Those drivers who use the old interfaces - i.e. regulator_suspend_prepare() and regulator_suspend_finish() should stop using that, since we leave these two functions empty and plan to remove them one day in the future. Any comments would be greatly appreciated. Thanks, Chunyan Changes from V1: (https://lkml.org/lkml/2017/12/21/9) * Rephrased the commit message of patch 0001; * Declared the property 'regulator-suspend-microvolt' deperated and documented the way recommended; * Revised of_regulator.c, use suspend_state::min_uV as default suspend voltage if the property 'regulator-suspend-microvolt' is not used. Changes from V2: (https://lkml.org/lkml/2018/1/4/75) * Correct the typo; * Removed redundant comments; * Export regulator_suspend_enable/disable(), don't call these two functions in regulator_set_suspend_voltage() instead; * Checking if rstate->min_uV == rstate->max_uV instead of checking the flag when setting the suspend voltage. Chunyan Zhang (5): bindings: regulator: added support for suspend states regulator: make regulator voltage be an array to support more states drivers: regulator: leave one item to record whether regulator is enabled drivers: regulator: empty the old suspend functions regulator: add PM suspend and resume hooks .../devicetree/bindings/regulator/regulator.txt | 15 +- drivers/regulator/core.c | 336 ++++++++++++++------- drivers/regulator/internal.h | 18 +- drivers/regulator/of_regulator.c | 20 +- include/linux/regulator/driver.h | 2 + include/linux/regulator/machine.h | 37 ++- 6 files changed, 302 insertions(+), 126 deletions(-) -- 2.7.4
prev parent reply other threads:[~2018-01-12 10:23 UTC|newest] Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-01-12 10:18 Chunyan Zhang 2018-01-12 10:18 ` [PATCH v3 1/5] bindings: regulator: added support for suspend states Chunyan Zhang 2018-01-19 22:11 ` Rob Herring 2018-01-22 2:16 ` Chunyan Zhang [not found] ` <1515752305-15907-2-git-send-email-zhang.chunyan-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2018-01-22 7:59 ` [PATCH V4 " Chunyan Zhang 2018-01-22 19:50 ` Rob Herring 2018-01-23 11:28 ` Mark Brown [not found] ` <1516607961-379-1-git-send-email-zhang.chunyan-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2018-01-26 15:26 ` Applied "regulator: added support for suspend states" to the regulator tree Mark Brown 2018-01-12 10:18 ` [PATCH v3 2/5] regulator: make regulator voltage be an array to support more states Chunyan Zhang 2018-01-12 10:18 ` [PATCH v3 3/5] drivers: regulator: leave one item to record whether regulator is enabled Chunyan Zhang 2018-01-26 15:26 ` Applied "regulator: leave one item to record whether regulator is enabled" to the regulator tree Mark Brown [not found] ` <1515752305-15907-1-git-send-email-zhang.chunyan-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2018-01-12 10:18 ` [PATCH v3 4/5] drivers: regulator: empty the old suspend functions Chunyan Zhang 2018-01-12 10:18 ` [PATCH v3 5/5] regulator: add PM suspend and resume hooks Chunyan Zhang 2018-01-12 10:23 ` Chunyan Zhang [this message]
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 \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --subject='Re: [PATCH v3 0/5] Add regulator suspend and resume support' \ /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
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).