Hi All, I'm looking at removing the following, last remaining, line from config.mk of the eNET board:
LDPPFLAGS += -DFLASH_SIZE=0x40000 This is needed to allow the calculation of the location of the reset vector so it resided in the last 16 bytes of the boot flash. Looking at how the compilation of u-boot.lds works, there are three locations where defines are exposed to the compilation of u-boot.lds 1) LDPPFLAGS or CPPFLAGS/PLATFORM_CPPFLAGS 2) /include/u-boot/u-boot.lds.h 3) config.mk detects a specific set of defined define and if present, adds them to CPPFLAGS - For example config.mk lines 169 through 175: ifneq ($(CONFIG_SYS_TEXT_BASE),) CPPFLAGS += -DCONFIG_SYS_TEXT_BASE=$(CONFIG_SYS_TEXT_BASE) endif ifneq ($(RESET_VECTOR_ADDRESS),) CPPFLAGS += -DRESET_VECTOR_ADDRESS=$(RESET_VECTOR_ADDRESS) endif LDPPFLAGS forces the use of a board specific config.mk, so that is out of the question. Patching /include/u-boot/u-boot.lds.h seems a bit messy Which leaves option #3 and adding the following to config.mk ifneq ($(CONFIG_SYS_FLASH_SIZE),) CPPFLAGS += -DCONFIG_SYS_FLASH_SIZE=$(CONFIG_SYS_FLASH_SIZE) endif and defining CONFIG_SYS_FLASH_SIZE in /include/configs/eNET.h But why not just include config.h when building u-boot.lds? For example, change Makefile lines 282 through 288 to: # Special flags for CPP when processing the linker script. # Pass the version down so we can handle backwards compatibility # on the fly. LDPPFLAGS += \ -include $(TOPDIR)/include/u-boot/u-boot.lds.h \ -include $(TOPDIR)/include/config.h \ $(shell $(LD) --version | \ sed -ne 's/GNU ld version \([0-9][0-9]*\)\.\([0-9][0-9]*\).*/-DLD_MAJOR=\1 -DLD_MINOR=\2/p') Thoughts? Regards, Graeme _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot