[PATCH v4 3/6] ARM: U8500 core machine support

[Russell King - ARM Linux]

... going back to this patch...

On Fri, Nov 27, 2009 at 05:25:56AM +0530, srinidhi kasagar wrote:
> +/*
> + * U8500 specific entry point for secondary CPUs.
> + */
> +ENTRY(u8500_secondary_startup)
> +	mrc	p15, 0, r0, c0, c0, 5
> +	and	r0, r0, #15
> +	adr	r4, 1f
> +	ldmia	r4, {r5, r6}
> +	sub	r4, r4, r5
> +	add	r6, r6, r4
> +	dsb

Can you please explain why you added a dsb here, whereas most other
platforms don't require it?

> +pen:	ldr	r7, [r6]
> +	cmp	r7, r0
> +	bne	pen
> +
> +	/*
> +	 * we've been released from the holding pen: secondary_stack
> +	 * should now contain the SVC stack for this core
> +	 */
> +	b	secondary_startup
> +
> +1:	.long	.
> +	.long	pen_release