On Jul 22, 2009, at 5:14 AM, Aggrwal Poonam-B10812 wrote: >> >> ... >>> --- a/cpu/mpc86xx/cpu.c >>> +++ b/cpu/mpc86xx/cpu.c >> ... >>> +int probecpu (void) >>> +{ >>> + uint svr; >>> + uint ver; >>> + >>> + svr = get_svr(); >>> + ver = SVR_SOC_VER(svr); >>> + >>> + gd->cpu = identify_cpu(ver); >>> + >>> + return 0; >>> +} >>> + >>> +int cpu_numcores() { >>> + struct cpu_type *cpu; >>> + cpu = gd->cpu; >>> + return cpu->num_cores; >>> +} >>> + >> >> This seems to be identically repeated code. Please factor out >> into common code. > Actually the files cpu/mpc86xx/cpu.c cpu/mpc85xx/cpu.c are quite > identical. > Kumar, Please suggest how should this be taken care. > Probably a single file to take care of 85xx and 86xx don't know?
We can move the code into cpu/mpc8xxx/cpu.c - k _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot