From mboxrd@z Thu Jan  1 00:00:00 1970
From: llandre <r&d2@dave-tech.it>
Date: Thu, 31 Aug 2006 13:15:58 +0200
Subject: [U-Boot-Users] simple_strtoul overwrites bd_info field
In-Reply-To: <20060830101300.93EF4352627@atlas.denx.de>
References: <20060830101300.93EF4352627@atlas.denx.de>
Message-ID: <44F6C4EE.1010605@dave-tech.it>
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

> In message <44F551D6.7070904@dave-tech.it> you wrote:
>> I tried to move the e pointer from start_armboot to the top of the file 
>> (static char *e;) and everything works fine.
> 
> This cannot be accepted as solution for the problem. 
Yes, I absolutely agree. I have just performed this attempt to verify 
that that problem referred to the allocation of variable e.

Anyway I found the bug. Since we had to add more fields for DRAM 
configuration in the struct bd_info (Ep93xx has complex DRAM mapping), 
its size increased up to 160 bytes and I just forgot to change 
CFG_GBL_DATA_SIZE accordingly.


Thanks and best regards,
llandre

DAVE Electronics System House - R&D Department
web:   http://www.dave-tech.it
email: r&d2 at dave-tech.it