[U-Boot] [PATCH 1/1 V2] add support for mcf5307 cpu

angelo angelo70 at gmail.com
Fri Sep 2 13:41:22 CEST 2011


Hi Jason and all,

i am really interested to bring mcf5307 inside u-boot. But still waiting 
for your feedback/help on my comments below.

Let me know your comments.

Many thanks,

Regards,
angelo

>  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:angelo70 at gmail.com]
>  >  Sent: Friday, August 12, 2011 5:20 AM
>  >  To: U-Boot at 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<sysamfw at 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





More information about the U-Boot mailing list