On Tue, Jul 21, 2009 at 04:17:43PM +0530, Poonam Aggrwal wrote:
> +int cpu_numcores() {
> +     struct cpu_type *cpu;
> +     cpu = gd->cpu;
> +     return cpu->num_cores;
> +}

Shouldn't this check CONFIG_MP and return 1 if not set?  Otherwise you
get a weird mix of MP and UP.

The checks to warn the user of a UP u-boot on an MP chip would have to
reference cpu->num_cores directly, or we could add a hard_cpu_numcores(),
or similar.

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

Reply via email to