Re: [U-Boot] Rename RESET_VECTOR_ADDRESS ?

2010-11-29 Thread Kumar Gala

On Nov 28, 2010, at 1:52 PM, Wolfgang Denk wrote:

 Dear Andy  Kumar,
 
 a number of boards define RESET_VECTOR_ADDRESS in their config.mk
 files:
 
 board/freescale/corenet_ds/config.mk
 board/freescale/mpc8536ds/config.mk
 board/freescale/mpc8572ds/config.mk
 board/freescale/p1022ds/config.mk
 board/freescale/p1_p2_rdb/config.mk
 board/freescale/p2020ds/config.mk
 
 The only place where RESET_VECTOR_ADDRESS ever gets used is in the
 respective u-boot.lds linker script.  Do you see any reason why we
 should not move these RESET_VECTOR_ADDRESS settings into the board
 config files?  The linker script just needs to include config.h, and
 we could remove all the config.mk files above.
 
 What do you think?
 
 Best regards,
 
 Wolfgang Denk

I don't see any issues with this, patch is welcome ;)

- k

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] Rename RESET_VECTOR_ADDRESS ?

2010-11-28 Thread Wolfgang Denk
Dear Andy  Kumar,

a number of boards define RESET_VECTOR_ADDRESS in their config.mk
files:

board/freescale/corenet_ds/config.mk
board/freescale/mpc8536ds/config.mk
board/freescale/mpc8572ds/config.mk
board/freescale/p1022ds/config.mk
board/freescale/p1_p2_rdb/config.mk
board/freescale/p2020ds/config.mk

The only place where RESET_VECTOR_ADDRESS ever gets used is in the
respective u-boot.lds linker script.  Do you see any reason why we
should not move these RESET_VECTOR_ADDRESS settings into the board
config files?  The linker script just needs to include config.h, and
we could remove all the config.mk files above.

What do you think?

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: w...@denx.de
Never call a man a fool.  Borrow from him.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot