Santosh Shilimkar <[email protected]> writes: > Current OMAP2PLUS CPUfreq tagret() functions returns when all > the CPU's are not online. This will break DVFS when secondary > CPUs are offlined. > > The intention of that check was just avoid CPU frequency change > during the window when CPU becomes online but it's cpufreq_init is > not done yet. > > Fix the check accordingly. > > Thanks for Nishant Menon <[email protected]> for reporting it. > > Signed-off-by: Santosh Shilimkar <[email protected]> > Reported-by: Nishanth Menon <[email protected]> > Tested-by: Vishwanath BS <[email protected]> > --- > There were some question of making the variable atomic etc > in an internal discussion. After some thinking, I realised > there is no need of that since this is just a counter which > maintains the count for online_cpus = cpufreq_init_cpus.
Since this is init-time only check, the check for every call to ->target() seems excessive. How about leaving the ->target callback empty until all the CPUs are online. Also, how will this handle an SMP kernel booted with maxcpus=1 on the cmdline? Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
