From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dirk Behme Date: Tue, 30 Nov 2010 06:19:36 +0100 Subject: [U-Boot] [PATCH] OMAP3: SDP3430: Fix ARM relocation support In-Reply-To: <0554BEF07D437848AF01B9C9B5F0BC5DBD5BED1C@dlee01.ent.ti.com> References: <1291053077-12248-1-git-send-email-dirk.behme@gmail.com> <0554BEF07D437848AF01B9C9B5F0BC5DBD5BED1C@dlee01.ent.ti.com> Message-ID: <4CF48968.9060000@googlemail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 30.11.2010 04:25, Paulraj, Sandeep wrote: > > >> >> Signed-off-by: Dirk Behme >> --- >> >> Compile tested only. >> >> board/ti/sdp3430/config.mk | 2 +- >> include/configs/omap3_sdp3430.h | 6 ++++++ >> 2 files changed, 7 insertions(+), 1 deletion(-) >> >> Index: u-boot.git/board/ti/sdp3430/config.mk >> =================================================================== >> --- u-boot.git.orig/board/ti/sdp3430/config.mk >> +++ u-boot.git/board/ti/sdp3430/config.mk >> @@ -30,4 +30,4 @@ >> # (mem base + reserved) >> >> # For use with external or internal boots. >> -CONFIG_SYS_TEXT_BASE = 0x80e80000 >> +CONFIG_SYS_TEXT_BASE = 0x80008000 >> Index: u-boot.git/include/configs/omap3_sdp3430.h >> =================================================================== >> --- u-boot.git.orig/include/configs/omap3_sdp3430.h >> +++ u-boot.git/include/configs/omap3_sdp3430.h >> @@ -309,6 +309,12 @@ >> #define CONFIG_STACKSIZE_FIQ (4<< 10) /* FIQ stack */ >> #endif >> >> +#define CONFIG_SYS_SDRAM_BASE PHYS_SDRAM_1 >> +#define CONFIG_SYS_INIT_RAM_ADDR 0x4020f800 >> +#define CONFIG_SYS_INIT_RAM_SIZE 0x800 >> +#define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_INIT_RAM_ADDR + \ >> + CONFIG_SYS_INIT_RAM_SIZE - \ >> + GENERATED_GBL_DATA_SIZE) >> /* >> * SDRAM Memory Map > > > Dirk, > > All 3 of your patches have the same issue. Sorry, which issue? What's wrong with _these_ patches? > Wolfgang, sent a patch some days back (it has not yet been applied and when it does it will be in the next branch) that moves the definition of CONFIG_SYS_TEXT_BASE to the config.h header file. Yes. And? I can't see a reason why not fix the breakage of some boards with a correct patch (if it's correct? see above) just because some other not yet applied patches might touch the same parts of a file. Depending on the order of applying the patches, just the merge issues have to be resolved then. Either by automatic git merge or manually by an add on patch. Do I understand correctly that you like to keep these three boards broken? Please note that your recent u-boot-ti/master pull request touches board/logicpd/am3517evm/config.mk in exactly the same way you seem to complain here about (?). Sorry if I misunderstood anything, Dirk