[PATCH V3 07/63] ST SPEAr13XX: Adding machine specific src files

viresh kumar viresh.kumar at st.com
Fri Jan 14 05:09:18 EST 2011


Stanley,

Thanks for reviewing!!

On 01/14/2011 03:38 PM, stanley.miao wrote:
> Viresh Kumar wrote:
>> <snip>
>> +
>> +void __cpuinit platform_secondary_init(unsigned int cpu)
>> +{
>> <snip>
>> +
>> +	/*
>> +	 * let the primary processor know we're out of the
>> +	 * pen, then head off into the C entry point
>> +	 */
>> +	pen_release = -1;
>> +	smp_wmb();
>>   
> 
> Russell has fixed this line. See commit
> 3705ff6da538aff6dba535e2e9cbcbb9456d0d53 in the latest kernel.
> 

Ya. I have already seen that in latest kernel and have updated it in
V4.

>> +
>> +	/*
>> +	 * Synchronise with the boot thread.
>> +	 */
>> +	spin_lock(&boot_lock);
>> +	spin_unlock(&boot_lock);
>> +}
>> +
>> <snip>
>> +void __init smp_prepare_cpus(unsigned int max_cpus)
>> +{
>> +	unsigned int ncores = get_core_count();
>> +	unsigned int cpu = smp_processor_id();
>> +	int i;
>> +
>> +	/* sanity check */
>> +	if (ncores == 0) {
>> +		pr_err("Realview: strange CM count of 0? Default to 1\n");
>>   
> 
> RealView ?

Oops!! Anyway this routine is changed now and this error is
also removed.

-- 
viresh



More information about the linux-arm-kernel mailing list