On 07/05/2012 11:52 PM, Andreas Bießmann wrote:
> On 05.07.2012 17:38, Andreas Bießmann wrote:
> 
> <snip>
> 
>>> -clbss_l:str        r2, [r0]                /* clear loop...                
>>>     */
>>> +clbss_l:
>>> +   cmp     r1, r0
>>> +   bls     clbss_end
>>> +   str     r2, [r0]                /* clear loop...                    */
>>>     add     r0, r0, #4
>>> -   cmp     r0, r1
>>> -   bne     clbss_l
>>> +   b       clbss_l
>>> +clbss_end:
>>
>> Looks sane, I maybe had checked before clbss_l, jumped over clear loop
>> if r0 and r1 equal (to emphasize the fact that this is a check for entry
>> correctness) and wouldn't change the loop at all. To be more precise we
>> could also check if end is after start.
> 
> Well, I just realized you do check for less ;)
> 
Yes,

Thanks,
hongbo

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

Reply via email to