On Nov 19, 2010, at 4:23 AM, DUNDA Matthias wrote:
>>>
>>>> Well, it's actually 2 Gig, just like u-boot recognizes above...
>>>
>>> There's something wrong with the parameter you're passing to print_size().
>
>> I'm guessing its -1 or something like that.
>
> I used the board specific code of the MPC8568MDS and adapted it to our
> configuration.
>
> The code region of interest here is in mpc8568mds.c:
>
> phys_size_t
> initdram(int board_type)
> {
> long dram_size = 0;
> ...
> dram_size = fsl_ddr_sdram();
> dram_size = setup_ddr_tlbs(dram_size / 0x100000);
> ...
> }
>
> The dram_size parameter is signed and thus gives a value of -1 when having 2
> Gig installed (0x80000000).
>
> -1 devided by 0x100000 gives 0xfffff800, which is the 4 PiB when interpreted
> as unsigned again in setup_ddr_tlbs(...).
>
> The MPC8568MDS has only 256 MB in default configuration, so it shouldn't
> occur there.
>
> Anyhow, I don't understand, why it worked in u-boot 2009.11.1... :-/
If you were using 2GB, I'm not sure, but there's been lots of hackery on
print_size and some other stuff here that might have made this noticeable.
Anyway, FYI, I am working on a patch to clean this up for all boards that have
this problem; however, I'm doing some cleanup while I'm at it.... it will
probably be early next week before I have this tested enough to publish.
Cheers,
Becky
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
