In message <[EMAIL PROTECTED]> you wrote:
>
> Add support for the ixdp465 evaluation board
> 
> Signed-off-by: Michael Trimarchi <[EMAIL PROTECTED]>
> 
> ---
>  Makefile                      |    3 +
>  board/ixdp465/Makefile        |   47 ++++++++
>  board/ixdp465/config.mk       |    3 +
>  board/ixdp465/ixdp465.c       |   62 ++++++++++
>  board/ixdp465/lowlevel_init.S |  129 +++++++++++++++++++++
>  board/ixdp465/u-boot.lds      |   57 +++++++++
>  include/configs/ixdp465.h     |  251 
> +++++++++++++++++++++++++++++++++++++++++
>  7 files changed, 552 insertions(+), 0 deletions(-)
>  create mode 100644 board/ixdp465/Makefile
>  create mode 100644 board/ixdp465/config.mk
>  create mode 100644 board/ixdp465/ixdp465.c
>  create mode 100644 board/ixdp465/lowlevel_init.S
>  create mode 100644 board/ixdp465/u-boot.lds
>  create mode 100644 include/configs/ixdp465.h

Entries in README, MAINTAINERS and MAKEALL missing.

> --- /dev/null
> +++ b/board/ixdp465/Makefile
...
> +include $(TOPDIR)/config.mk
> +
> +LIB  = lib$(BOARD).a
> +
> +OBJS := ixdp465.o
> +SOBJS := lowlevel_init.o
> +
> +$(LIB):      $(OBJS) $(SOBJS)
> +     $(AR) crv $@ $^
> +
> +clean:
> +     rm -f $(SOBJS) $(OBJS)
> +
> +distclean:   clean
> +     rm -f $(LIB) core *.bak .depend

Support for out-of-tree builds is missing completely.

> diff --git a/board/ixdp465/config.mk b/board/ixdp465/config.mk
> new file mode 100644
> index 0000000..dbfec9c
> --- /dev/null
> +++ b/board/ixdp465/config.mk
> @@ -0,0 +1,3 @@
> +TEXT_BASE = 0x01600000
> +# include NPE ethernet driver
> +BOARDLIBS = $(obj)cpu/ixp/npe/libnpe.a
> diff --git a/board/ixdp465/ixdp465.c b/board/ixdp465/ixdp465.c
> new file mode 100644
> index 0000000..c90841c


> diff --git a/include/configs/ixdp465.h b/include/configs/ixdp465.h
> new file mode 100644
> index 0000000..06a4166
> --- /dev/null
> +++ b/include/configs/ixdp465.h
...
> +/*#define CONFIG_ETHADDR     08:00:3e:26:0a:5b*/
> +#define CONFIG_NETMASK               255.255.255.0
> +#define CONFIG_IPADDR                10.30.3.204
> +#define CONFIG_SERVERIP              10.30.3.72

Please omit such network config data from thje config file; they are
almost always wrong and a PITA to the end user.

> +#define CONFIG_EXTRA_ENV_SETTINGS            \
> +     "ethaddr=00:02:b3:01:01:01\0"           \
> +     "eth1addr=00:02:b3:02:02:02\0"

Never do this.

Also, the MAC addresses above belong  to  the  Intel  Corporation;  I
doubt you have permissions to use these on your boards.

> +#define CONFIG_NR_DRAM_BANKS    1          /* we have 2 banks of DRAM */
> +#define PHYS_SDRAM_1            0x00000000 /* SDRAM Bank #1 */
> +#define PHYS_SDRAM_1_SIZE       0x08000000 /* 128 MB */

Please avoid hard-wired RAM sizes. Try to auto-adjust like most other
boards are doing.

> +#define PHYS_FLASH_1            0x50000000 /* Flash Bank #1 */
> +#define PHYS_FLASH_SIZE         0x02000000 /* 32 MB */
> +#define PHYS_FLASH_BANK_SIZE    0x02000000 /* 32 MB Banks */
> +#define PHYS_FLASH_SECT_SIZE    0x00020000 /* 128 KB sectors (x1) */

Ditto here.

> +#define CFG_DRAM_BASE           0x00000000
> +#define CFG_DRAM_SIZE           0x08000000

Is this SDRAM or DRAM ? Seems to be a bit redundant here...


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
Any time things appear to be going better, you have overlooked  some-
thing.

-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
U-Boot-Users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Reply via email to