Re: [PATCH 3/3] omap2plus: cpufreq: Add SMP support to cater OMAP4430

[Kevin Hilman <khilman@ti.com>]

Santosh Shilimkar <santosh.shilimkar@ti.com> writes:

> On OMAP SMP configuartion, both processors share the voltage
> and clock. So both CPUs needs to be scaled together and hence
> needs software co-ordination.
>
> Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
> Cc: Kevin Hilman <khilman@ti.com>
> cc: Vishwanath BS <vishwanath.bs@ti.com>

[...]

> @@ -74,9 +76,13 @@ static int omap_target(struct cpufreq_policy *policy,
>  		       unsigned int target_freq,
>  		       unsigned int relation)
>  {
> -	int ret = 0;
> +	int i, ret = 0;
>  	struct cpufreq_freqs freqs;
>  
> +	/* Wait untill all CPU's are initialized */
> +	if (is_smp() && (num_online_cpus() < NR_CPUS))
> +		return ret;
> +

Minor: the 'Wait' in the comment is a bit misleading.  It should
rather say something like "Changes not allowed until all CPUs are online"

Kevin