From mboxrd@z Thu Jan  1 00:00:00 1970
From: Heiko Schocher <hs@denx.de>
Date: Sat, 02 Oct 2010 10:53:36 +0200
Subject: [U-Boot] ARM relocation,
 probably trivial mistake - back to original problem
In-Reply-To: <4CA5F665.9010700@emk-elektronik.de>
References: <4CA49746.2050301@emk-elektronik.de> <4CA4AEFF.3050101@denx.de>
	<20100930174308.070ECD2B48C@gemini.denx.de> <4CA570D3.9040406@denx.de>
	<4CA57468.6090702@free.fr> <4CA57762.3000201@denx.de>
	<4CA5821E.3070108@emk-elektronik.de> <4CA5873B.6040907@free.fr>
	<4CA590E6.6070701@emk-elektronik.de> <4CA59B89.6090207@denx.de>
	<4CA5BB7A.8050304@emk-elektronik.de>
	<20101001105506.A64D41539A0@gemini.denx.de>
	<4CA5BFEF.3090208@emk-elektronik.de>
	<20101001112125.4076E153A7E@gemini.denx.de>
	<4CA5C7DE.6010300@emk-elektronik.de>
	<20101001115908.A34411539A0@gemini.denx.de>
	<4CA5D26D.2090505@emk-elektronik.de>
	<4CA5D857.5010009@emk-elektronik.de>
	<20101001125502.D0B4E1539A0@gemini.denx.de>
	<4CA5F665.9010700@emk-elektronik.de>
Message-ID: <4CA6F310.6010302@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hello Reinhard,

Reinhard Meyer wrote:
> Dear Wolfgang Denk,
>>> The environment issues still persist. I am at a loss
>>> there now.
>>>
>>> Observation: the old style commands "setenv", "printenv", etc.
>>> work, but any "env" command except for "env" alone crashes.
>> OK. If "printenv" works and "env print" fails then it has nothing to
>> do with the environment code itself, as both call the same function.
>>
>> It must have something to do with the implementation of subcommands
>> then. See do_env() in "common/cmd_nvedit.c"; check if the command
>> table address for find_cmd_tbl() is OK.
>>
>> Eventually other commands with subcommands fail as well (i2c ?) ?
> 
> Nope, other commands with subcommands work fine. I did not test ALL
> but mmc, i2c, nand behave normally.

nand and mmc don;t use a command subtable, so they work. i2c do
the needed fixups for subcommandtables ...

And I see in common/cmd_nvedit.c there is a "static cmd_tbl_t cmd_env_sub"
definition, so this must be fixed too (not only for arm, it should
also don;t work on avr32, mips, m68k and sparc, because this
plattforms also need relocation fixups because CONFIG_RELOC_FIXUP_WORKS
is not defined for them ...

> I suspect something to do with a 256k boundary maybe, u-boot.bin is
> short of 256k. I changed the initial bootstrap to load 496k
> yesterday (before that the unrelocated version did fail in some
> commands).
> 
> I'll keep investigating.

Please try my above idea ...

Thanks!

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany