* [PATCH 4/7] cpuidle: calxeda: add support to use PSCI calls [not found] <1375048598-15637-1-git-send-email-robherring2@gmail.com> @ 2013-07-28 21:56 ` Rob Herring 2013-07-29 14:14 ` Daniel Lezcano 0 siblings, 1 reply; 4+ messages in thread From: Rob Herring @ 2013-07-28 21:56 UTC (permalink / raw) To: linux-arm-kernel; +Cc: Rob Herring, Rafael J. Wysocki, Daniel Lezcano, linux-pm From: Rob Herring <rob.herring@calxeda.com> This updates the Calxeda cpuidle driver to use PSCI calls to powergate cores. This is needed to enable cpuidle for the ECX-2000. This could possibly become a generic PSCI driver, but there are no other PSCI users in the kernel other than mach-virt. Signed-off-by: Rob Herring <rob.herring@calxeda.com> Cc: "Rafael J. Wysocki" <rjw@sisk.pl> Cc: Daniel Lezcano <daniel.lezcano@linaro.org> Cc: linux-pm@vger.kernel.org --- drivers/cpuidle/cpuidle-calxeda.c | 40 ++++++--------------------------------- 1 file changed, 6 insertions(+), 34 deletions(-) diff --git a/drivers/cpuidle/cpuidle-calxeda.c b/drivers/cpuidle/cpuidle-calxeda.c index 0e6e408..01cfecf3 100644 --- a/drivers/cpuidle/cpuidle-calxeda.c +++ b/drivers/cpuidle/cpuidle-calxeda.c @@ -22,51 +22,23 @@ #include <linux/cpuidle.h> #include <linux/init.h> -#include <linux/io.h> #include <linux/of.h> -#include <linux/time.h> -#include <linux/delay.h> -#include <linux/suspend.h> #include <asm/cpuidle.h> -#include <asm/proc-fns.h> -#include <asm/smp_scu.h> #include <asm/suspend.h> -#include <asm/cacheflush.h> -#include <asm/cp15.h> - -extern void highbank_set_cpu_jump(int cpu, void *jump_addr); -extern void *scu_base_addr; - -static noinline void calxeda_idle_restore(void) -{ - set_cr(get_cr() | CR_C); - set_auxcr(get_auxcr() | 0x40); - scu_power_mode(scu_base_addr, SCU_PM_NORMAL); -} +#include <asm/psci.h> static int calxeda_idle_finish(unsigned long val) { - /* Already flushed cache, but do it again as the outer cache functions - * dirty the cache with spinlocks */ - flush_cache_all(); - - set_auxcr(get_auxcr() & ~0x40); - set_cr(get_cr() & ~CR_C); - - scu_power_mode(scu_base_addr, SCU_PM_DORMANT); - - cpu_do_idle(); - - /* Restore things if we didn't enter power-gating */ - calxeda_idle_restore(); - return 1; + const struct psci_power_state ps = { + .type = PSCI_POWER_STATE_TYPE_POWER_DOWN, + }; + return psci_ops.cpu_suspend(ps, __pa(cpu_resume)); } static int calxeda_pwrdown_idle(struct cpuidle_device *dev, struct cpuidle_driver *drv, int index) { - highbank_set_cpu_jump(smp_processor_id(), cpu_resume); cpu_suspend(0, calxeda_idle_finish); return index; } @@ -90,7 +62,7 @@ static struct cpuidle_driver calxeda_idle_driver = { static int __init calxeda_cpuidle_init(void) { - if (!of_machine_is_compatible("calxeda,highbank")) + if (!of_machine_is_compatible("calxeda,highbank") || !psci_ops.cpu_suspend) return -ENODEV; return cpuidle_register(&calxeda_idle_driver, NULL); -- 1.8.1.2 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 4/7] cpuidle: calxeda: add support to use PSCI calls 2013-07-28 21:56 ` [PATCH 4/7] cpuidle: calxeda: add support to use PSCI calls Rob Herring @ 2013-07-29 14:14 ` Daniel Lezcano 2013-07-29 14:39 ` Rob Herring 0 siblings, 1 reply; 4+ messages in thread From: Daniel Lezcano @ 2013-07-29 14:14 UTC (permalink / raw) To: Rob Herring; +Cc: linux-arm-kernel, Rob Herring, Rafael J. Wysocki, linux-pm On 07/28/2013 11:56 PM, Rob Herring wrote: > From: Rob Herring <rob.herring@calxeda.com> > > This updates the Calxeda cpuidle driver to use PSCI calls to powergate > cores. This is needed to enable cpuidle for the ECX-2000. > > This could possibly become a generic PSCI driver, but there are no other > PSCI users in the kernel other than mach-virt. > > Signed-off-by: Rob Herring <rob.herring@calxeda.com> > Cc: "Rafael J. Wysocki" <rjw@sisk.pl> > Cc: Daniel Lezcano <daniel.lezcano@linaro.org> > Cc: linux-pm@vger.kernel.org > --- Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org> Rob, Shall I take the patch in my tree ? Thanks -- Daniel > drivers/cpuidle/cpuidle-calxeda.c | 40 ++++++--------------------------------- > 1 file changed, 6 insertions(+), 34 deletions(-) > > diff --git a/drivers/cpuidle/cpuidle-calxeda.c b/drivers/cpuidle/cpuidle-calxeda.c > index 0e6e408..01cfecf3 100644 > --- a/drivers/cpuidle/cpuidle-calxeda.c > +++ b/drivers/cpuidle/cpuidle-calxeda.c > @@ -22,51 +22,23 @@ > > #include <linux/cpuidle.h> > #include <linux/init.h> > -#include <linux/io.h> > #include <linux/of.h> > -#include <linux/time.h> > -#include <linux/delay.h> > -#include <linux/suspend.h> > #include <asm/cpuidle.h> > -#include <asm/proc-fns.h> > -#include <asm/smp_scu.h> > #include <asm/suspend.h> > -#include <asm/cacheflush.h> > -#include <asm/cp15.h> > - > -extern void highbank_set_cpu_jump(int cpu, void *jump_addr); > -extern void *scu_base_addr; > - > -static noinline void calxeda_idle_restore(void) > -{ > - set_cr(get_cr() | CR_C); > - set_auxcr(get_auxcr() | 0x40); > - scu_power_mode(scu_base_addr, SCU_PM_NORMAL); > -} > +#include <asm/psci.h> > > static int calxeda_idle_finish(unsigned long val) > { > - /* Already flushed cache, but do it again as the outer cache functions > - * dirty the cache with spinlocks */ > - flush_cache_all(); > - > - set_auxcr(get_auxcr() & ~0x40); > - set_cr(get_cr() & ~CR_C); > - > - scu_power_mode(scu_base_addr, SCU_PM_DORMANT); > - > - cpu_do_idle(); > - > - /* Restore things if we didn't enter power-gating */ > - calxeda_idle_restore(); > - return 1; > + const struct psci_power_state ps = { > + .type = PSCI_POWER_STATE_TYPE_POWER_DOWN, > + }; > + return psci_ops.cpu_suspend(ps, __pa(cpu_resume)); > } > > static int calxeda_pwrdown_idle(struct cpuidle_device *dev, > struct cpuidle_driver *drv, > int index) > { > - highbank_set_cpu_jump(smp_processor_id(), cpu_resume); > cpu_suspend(0, calxeda_idle_finish); > return index; > } > @@ -90,7 +62,7 @@ static struct cpuidle_driver calxeda_idle_driver = { > > static int __init calxeda_cpuidle_init(void) > { > - if (!of_machine_is_compatible("calxeda,highbank")) > + if (!of_machine_is_compatible("calxeda,highbank") || !psci_ops.cpu_suspend) > return -ENODEV; > > return cpuidle_register(&calxeda_idle_driver, NULL); > -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 4/7] cpuidle: calxeda: add support to use PSCI calls 2013-07-29 14:14 ` Daniel Lezcano @ 2013-07-29 14:39 ` Rob Herring 2013-07-29 14:46 ` Daniel Lezcano 0 siblings, 1 reply; 4+ messages in thread From: Rob Herring @ 2013-07-29 14:39 UTC (permalink / raw) To: Daniel Lezcano; +Cc: linux-arm-kernel, Rob Herring, Rafael J. Wysocki, linux-pm On 07/29/2013 09:14 AM, Daniel Lezcano wrote: > On 07/28/2013 11:56 PM, Rob Herring wrote: >> From: Rob Herring <rob.herring@calxeda.com> >> >> This updates the Calxeda cpuidle driver to use PSCI calls to powergate >> cores. This is needed to enable cpuidle for the ECX-2000. >> >> This could possibly become a generic PSCI driver, but there are no other >> PSCI users in the kernel other than mach-virt. >> >> Signed-off-by: Rob Herring <rob.herring@calxeda.com> >> Cc: "Rafael J. Wysocki" <rjw@sisk.pl> >> Cc: Daniel Lezcano <daniel.lezcano@linaro.org> >> Cc: linux-pm@vger.kernel.org >> --- > > Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org> Thanks. > > Rob, > > Shall I take the patch in my tree ? No, there are interdependencies with the rest of the series, so I plan to send to arm-soc. Rob ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 4/7] cpuidle: calxeda: add support to use PSCI calls 2013-07-29 14:39 ` Rob Herring @ 2013-07-29 14:46 ` Daniel Lezcano 0 siblings, 0 replies; 4+ messages in thread From: Daniel Lezcano @ 2013-07-29 14:46 UTC (permalink / raw) To: Rob Herring; +Cc: linux-arm-kernel, Rob Herring, Rafael J. Wysocki, linux-pm On 07/29/2013 04:39 PM, Rob Herring wrote: > On 07/29/2013 09:14 AM, Daniel Lezcano wrote: >> On 07/28/2013 11:56 PM, Rob Herring wrote: >>> From: Rob Herring <rob.herring@calxeda.com> >>> >>> This updates the Calxeda cpuidle driver to use PSCI calls to powergate >>> cores. This is needed to enable cpuidle for the ECX-2000. >>> >>> This could possibly become a generic PSCI driver, but there are no other >>> PSCI users in the kernel other than mach-virt. >>> >>> Signed-off-by: Rob Herring <rob.herring@calxeda.com> >>> Cc: "Rafael J. Wysocki" <rjw@sisk.pl> >>> Cc: Daniel Lezcano <daniel.lezcano@linaro.org> >>> Cc: linux-pm@vger.kernel.org >>> --- >> >> Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org> > > Thanks. > >> >> Rob, >> >> Shall I take the patch in my tree ? > > No, there are interdependencies with the rest of the series, so I plan > to send to arm-soc. Ok, thanks. -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-07-29 14:46 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1375048598-15637-1-git-send-email-robherring2@gmail.com>
2013-07-28 21:56 ` [PATCH 4/7] cpuidle: calxeda: add support to use PSCI calls Rob Herring
2013-07-29 14:14 ` Daniel Lezcano
2013-07-29 14:39 ` Rob Herring
2013-07-29 14:46 ` Daniel Lezcano
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).