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@512k(env)," \
"2m@1m(kernel),13m@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@denx.de =====================================================================