public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] Rename RESET_VECTOR_ADDRESS ?
@ 2010-11-28 19:52 Wolfgang Denk
  2010-11-29 14:53 ` Kumar Gala
  0 siblings, 1 reply; 2+ messages in thread
From: Wolfgang Denk @ 2010-11-28 19:52 UTC (permalink / raw)
  To: u-boot

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: wd at denx.de
Never call a man a fool.  Borrow from him.

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [U-Boot] Rename RESET_VECTOR_ADDRESS ?
  2010-11-28 19:52 [U-Boot] Rename RESET_VECTOR_ADDRESS ? Wolfgang Denk
@ 2010-11-29 14:53 ` Kumar Gala
  0 siblings, 0 replies; 2+ messages in thread
From: Kumar Gala @ 2010-11-29 14:53 UTC (permalink / raw)
  To: u-boot


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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-11-29 14:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-11-28 19:52 [U-Boot] Rename RESET_VECTOR_ADDRESS ? Wolfgang Denk
2010-11-29 14:53 ` Kumar Gala

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox