On 23/08/13 11:25, Lokesh Vutla wrote:
> Hi Mark,
> 
> On Friday 23 August 2013 02:58 PM, Mark Jackson wrote:
>> On 30/07/13 06:18, Lokesh Vutla wrote:
>>> From: Heiko Schocher <h...@denx.de>
>>>
>>> s_init has the same outline for all the AM33xx based
>>> board. So making it generic.
>>> This also helps in addition of new Soc with minimal changes.
>>>
>>> Signed-off-by: Lokesh Vutla <lokeshvu...@ti.com>
>>> Signed-off-by: Heiko Schocher <h...@denx.de>
>>> Signed-off-by: Tom Rini <tr...@ti.com>
>>

<snip>

>> But this fails to boot. However, if I use the old ASM code:-
>>
>> void enable_norboot_pin_mux(void)
>> {
>>      asm("stmfd      sp!, {r2 - r4}");
>>      asm("movw       r4, #0x8A4");
>>      asm("movw       r3, #0x44E1");
>>      asm("orr        r4, r4, r3, lsl #16");
>>      asm("mov        r2, #9");
>>      asm("mov        r3, #8");
>>      asm("gpmc_mux:  str     r2, [r4], #4");
>>      asm("subs       r3, r3, #1");
>>      asm("bne        gpmc_mux");
>>      asm("ldmfd      sp!, {r2 - r4}");
>> }
> This code writes 0x9 into 8 continuous registers starting from
> 0x44e108a4, this is what done in module_pin_mux norboot_pin_mux
> except that it has 9 registers(i guess 9th register was added by mistake..:( )
> Correct me if I am wrong.

Not sure about the code, but it was introduced here:-

http://git.denx.de/?p=u-boot/u-boot-ti.git;a=commit;h=c5c7a7c32d552592ac49749e5c184c89bd50c098

> So you are telling this is wrong but boots properly ?

Basically ... yes !!
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to