| Message ID | 20170514183613.160312642@linutronix.de (mailing list archive) |
|---|---|
| State | Not Applicable |
| Headers | show |
On 14-05-17, 20:27, Thomas Gleixner wrote: > To enable smp_processor_id() and might_sleep() debug checks earlier, it's > required to add system states between SYSTEM_BOOTING and SYSTEM_RUNNING. > > Adjust the system_state check in pas_cpufreq_cpu_exit() to handle the extra > states. > > Signed-off-by: Thomas Gleixner <tglx@linutronix.de> > Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net> > Cc: Viresh Kumar <viresh.kumar@linaro.org> > Cc: linuxppc-dev@lists.ozlabs.org > --- > drivers/cpufreq/pasemi-cpufreq.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > --- a/drivers/cpufreq/pasemi-cpufreq.c > +++ b/drivers/cpufreq/pasemi-cpufreq.c > @@ -226,7 +226,7 @@ static int pas_cpufreq_cpu_exit(struct c > * We don't support CPU hotplug. Don't unmap after the system > * has already made it to a running state. > */ > - if (system_state != SYSTEM_BOOTING) > + if (system_state >= SYSTEM_RUNNING) > return 0; > > if (sdcasr_mapbase) Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
On Mon, 15 May 2017 10:54:03 +0530 Viresh Kumar <viresh.kumar@linaro.org> wrote: > On 14-05-17, 20:27, Thomas Gleixner wrote: > > To enable smp_processor_id() and might_sleep() debug checks earlier, it's > > required to add system states between SYSTEM_BOOTING and SYSTEM_RUNNING. > > > > Adjust the system_state check in pas_cpufreq_cpu_exit() to handle the extra > > states. > > > > Signed-off-by: Thomas Gleixner <tglx@linutronix.de> > > Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net> > > Cc: Viresh Kumar <viresh.kumar@linaro.org> > > Cc: linuxppc-dev@lists.ozlabs.org > > --- > > drivers/cpufreq/pasemi-cpufreq.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > --- a/drivers/cpufreq/pasemi-cpufreq.c > > +++ b/drivers/cpufreq/pasemi-cpufreq.c > > @@ -226,7 +226,7 @@ static int pas_cpufreq_cpu_exit(struct c > > * We don't support CPU hotplug. Don't unmap after the system > > * has already made it to a running state. > > */ > > - if (system_state != SYSTEM_BOOTING) > > + if (system_state >= SYSTEM_RUNNING) > > return 0; > > > > if (sdcasr_mapbase) > > Acked-by: Viresh Kumar <viresh.kumar@linaro.org> > This reminds me. My pasemi died. I was trying to see if anyone had an extra one they could send me ;-) -- Steve
--- a/drivers/cpufreq/pasemi-cpufreq.c +++ b/drivers/cpufreq/pasemi-cpufreq.c @@ -226,7 +226,7 @@ static int pas_cpufreq_cpu_exit(struct c * We don't support CPU hotplug. Don't unmap after the system * has already made it to a running state. */ - if (system_state != SYSTEM_BOOTING) + if (system_state >= SYSTEM_RUNNING) return 0; if (sdcasr_mapbase)
To enable smp_processor_id() and might_sleep() debug checks earlier, it's required to add system states between SYSTEM_BOOTING and SYSTEM_RUNNING. Adjust the system_state check in pas_cpufreq_cpu_exit() to handle the extra states. Signed-off-by: Thomas Gleixner <tglx@linutronix.de> Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net> Cc: Viresh Kumar <viresh.kumar@linaro.org> Cc: linuxppc-dev@lists.ozlabs.org --- drivers/cpufreq/pasemi-cpufreq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)