On 11/21/2012 01:26 PM, Russell King - ARM Linux wrote: > On Wed, Nov 21, 2012 at 12:22:51PM +0000, Will Deacon wrote: >> Sorry, my mistake (I was thinking on my feet) since secondary_startup is >> declared in a C file, right? How about: >> >> ldr lr, =secondary_startup >> b ll_set_cpu_coherent > > Why? Do we really want LR to be the _virtual_ address? No we don't, > because secondary_startup expects to be called with the MMU off. > > So I go back to my original comment. What's wrong with: > > bl ll_set_cpu_coherent > b secondary_startup > > ?
Nothing wrong, it works perfectly. Whereas indeed the code proposed by Will didn't work very well. I will take Russell's suggestion. Thanks, > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com _______________________________________________ devicetree-discuss mailing list [email protected] https://lists.ozlabs.org/listinfo/devicetree-discuss
