Hello Jason and all, i started to work to rewrite the patch. Starting from mcf532x/cpu.c, i am not sure this is the right place to add MCF5307.
1) On "checkcpu", mcf5307 don't have ccm->cir where to read mask/ver. 2) About "do_reset", here also mcf5307 don't have MMAP_RCM, memory map and reset procedure are different. 3) In general, as memory map, mcf5307 is much more similar to some mcf52x2 cpu. This is the reason why i thought to treat it as a separate family. Looking here: http://www.freescale.com/webapp/sps/site/taxonomy.jsp?code=68KCFV3 seems Freescale also treat it as the only cpu of the MCF530X family. I can still go ahead, but cpu.c should have a great #ifdef CONFIG_MCF5307 to handle separately this cpu. How do you think i should proceed ? regards angelo On 22/08/2011 05:49, Jin Zhengxiong-R64188 wrote: >> -----Original Message----- >> From: Angelo Dureghello [mailto:angel...@gmail.com] >> Sent: Friday, August 12, 2011 5:20 AM >> To: U-Boot@lists.denx.de >> Cc: Jin Zhengxiong-R64188 >> Subject: [PATCH 1/1 V2] add support for mcf5307 cpu >> >> Add cpu related files and memory map includes. Apply the related >> minor changes to general coldfire architecture include files. >> >> Signed-off-by: Angelo Dureghello <sysa...@gmail.com> >> --- >> arch/m68k/cpu/mcf530x/Makefile | 48 +++++++ >> arch/m68k/cpu/mcf530x/config.mk | 25 ++++ >> arch/m68k/cpu/mcf530x/cpu.c | 48 +++++++ >> arch/m68k/cpu/mcf530x/cpu.h | 33 +++++ >> arch/m68k/cpu/mcf530x/cpu_init.c | 164 +++++++++++++++++++++++ >> arch/m68k/cpu/mcf530x/interrupts.c | 42 ++++++ >> arch/m68k/cpu/mcf530x/speed.c | 36 +++++ >> arch/m68k/cpu/mcf530x/start.S | 253 >> ++++++++++++++++++++++++++++++++++++ >> arch/m68k/include/asm/immap_5307.h | 78 +++++++++++ >> arch/m68k/include/asm/m5307.h | 119 +++++++++++++++++ >> 10 files changed, 846 insertions(+), 0 deletions(-) >> create mode 100644 arch/m68k/cpu/mcf530x/Makefile >> create mode 100644 arch/m68k/cpu/mcf530x/config.mk >> create mode 100644 arch/m68k/cpu/mcf530x/cpu.c >> create mode 100644 arch/m68k/cpu/mcf530x/cpu.h >> create mode 100644 arch/m68k/cpu/mcf530x/cpu_init.c >> create mode 100644 arch/m68k/cpu/mcf530x/interrupts.c >> create mode 100644 arch/m68k/cpu/mcf530x/speed.c >> create mode 100644 arch/m68k/cpu/mcf530x/start.S >> create mode 100644 arch/m68k/include/asm/immap_5307.h >> create mode 100644 arch/m68k/include/asm/m5307.h > > [Jin Zhengxiong-R64188] Could you please try to merge the 5307 cpu to mcf532x > as we > already have two ColdFire V3 boards here(We can try to rename the mcf532x to > such as > mcf53xx later for less confuse). You can keep the immap_5307.h and m5307.h > for the 5307 cpu. All the other files are similar with the files for the two > V3 boards > except the cpu_init.c file. > > Thanks. > > Jason > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot