[U-Boot] [PATCH 4/4] arm/kirkwood: Add Keymile SUEN3 support
Stefan Roese
sr at denx.de
Thu Jul 9 09:02:29 CEST 2009
Hi Wolfgang,
On Wednesday 08 July 2009 20:53:58 Wolfgang Denk wrote:
> > diff --git a/MAKEALL b/MAKEALL
> > index 41f1445..6c80694 100755
> > --- a/MAKEALL
> > +++ b/MAKEALL
> > @@ -524,6 +524,7 @@ LIST_ARM9=" \
> > scb9328 \
> > smdk2400 \
> > smdk2410 \
> > + suen3 \
> > trab \
> > VCMA9 \
> > versatile \
>
> This is the ARM9 list. Are you sure the board belongs there?
It's using the U-Boot ARM926EJS infrastructure (ARMv5TE). Which the board in
this list seem to be using as well. So yes, I guess the board belongs here.
> > diff --git a/Makefile b/Makefile
> > index 2a06440..3afc4ba 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -2925,6 +2925,9 @@ smdk2400_config : unconfig
> > smdk2410_config : unconfig
> > @$(MKCONFIG) $(@:_config=) arm arm920t smdk2410 samsung s3c24x0
> >
> > +suen3_config: unconfig
> > + @$(MKCONFIG) $(@:_config=) arm arm926ejs $(@:_config=) keymile kirkwood
> > +
>
> This is the ARM9xxx section. Are you sure the board belongs there?
Yes.
> > diff --git a/board/keymile/suen3/suen3.c b/board/keymile/suen3/suen3.c
> > new file mode 100644
> > index 0000000..231745f
> > --- /dev/null
> > +++ b/board/keymile/suen3/suen3.c
> > @@ -0,0 +1,127 @@
>
> ..
>
> > +int dram_init(void)
> > +{
> > + int i;
> > +
> > + for (i = 0; i < CONFIG_NR_DRAM_BANKS; i++) {
> > + gd->bd->bi_dram[i].start = kw_sdram_bar(i);
> > + gd->bd->bi_dram[i].size = kw_sdram_bs(i);
> > + }
> > + return 0;
> > +}
>
> Maybe we can have get_ram_size() support with auto-sizing and testing
> here?
OK, I'll will change it.
> > diff --git a/include/configs/suen3.h b/include/configs/suen3.h
> > new file mode 100644
> > index 0000000..1b971e0
> > --- /dev/null
> > +++ b/include/configs/suen3.h
>
> ..
>
> > +#define CONFIG_EXTRA_ENV_SETTINGS \
> > + "load=tftp 400000 suen3/u-boot.bin\0" \
> > + "update=nand erase 0 60000;nand write 400000 0 60000\0" \
> > + "upd=run load update\0" \
> > + "mtdparts=orion_nand:512k(uboot),512k at 512k(env)," \
> > + "2m at 1m(kernel),13m at 3m(rootfs)\0" \
>
> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> ..
>
> > +#define MTDPARTS_DEFAULT "mtdparts=orion_nand:512k(u-boot)," \
> > + "256k(env)," \
> > + "2m(kernel)," \
> > + "16m(rootfs)," \
> > + "32m(fs)," \
> > + "-(ubifs)"
>
> This seems inconsistent to me. How about defining MTDPARTS_DEFAULT
> first, and then just inserting the already defined value, so you can
> be sure that this doesn;t happen again?
Yes, good catch. Will change. Thanks for spotting.
Best regards,
Stefan
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
More information about the U-Boot
mailing list