[U-Boot] [PATCH] arm: Add option to disable code relocation

[Wolfgang Denk]

Dear Simon Glass,

In message <1328424259-12914-1-git-send-email-sjg at chromium.org> you wrote:
> This permits disabling the code relation in U-Boot which is useful when
> debugging with an ICE.
> 
> To use it define CONFIG_SYS_SKIP_ARM_RELOCATION in your board.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>

NAK.

We are trying really hard to unify the code, i. e. to make all
architectures behave the same.  I will not accept any such
architecture-specific implementation.

All the more, there is really ne need for this.  Relocation does not
cause any specific problems to debugging.  We know exaclt when it
happens, and it happens exaclty once during the life time of U-Boot.
And all you  have to do is load the symbol tables to the right
address.  Before relcation this just works, and after relocation you
just need to issue a single GDB command.

This is even documented.

Adding this options also creates the risk that people who don't really
understand the inner workings of U-Boot will try and use this for
other purposes.  Read: mis-use it.


Sorry, but NAK.

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
One does not thank logic.
	-- Sarek, "Journey to Babel", stardate 3842.4