On Apr 1, 2009, at 3:11 PM, Kim Phillips wrote:

> On Wed,  1 Apr 2009 11:06:27 -0500
> Kumar Gala <ga...@kernel.crashing.org> wrote:
>
>> -#if (CONFIG_NUM_CPUS > 1)
>> -void cpu_mp_lmb_reserve(struct lmb *lmb)
>> +int cpu_reset(int nr)
>> +{
>> +    volatile immap_t *immr = (immap_t *)CONFIG_SYS_IMMR;
>> +    volatile ccsr_pic_t *pic = &immr->im_pic;
>
> blank line here
>
>> +    out_be32(&pic->pir, 1 << nr);
>> +    (void)in_be32(&pic->pir);
>> +    out_be32(&pic->pir, 0x0);
>
> can be two setbits32

this is now a noop.

>> +u32 determine_mp_bootpg(void)
>> +{
>>      /* if we have 4G or more of memory, put the boot page at 4Gb-1M */
>>      if ((u64)gd->ram_size > 0xfffff000)
>> -            bootpg = 0xfff00000;
>> -    else
>> -            bootpg = gd->ram_size - (1024 * 1024);
>> +            return (0xfff00000);
>> +
>> +    return (gd->ram_size - (1024 * 1024));
>
> ? this line not reached

the diff is funny here, look again its:

if (foo)
        return A

return B;

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

Reply via email to