Re: [U-Boot] [PATCH] Revert "common/memsize.c: Simplify RAM size detection"

2016-02-10 Thread Tom Rini
On Tue, Feb 09, 2016 at 10:38:31PM +0100, Hans de Goede wrote: > This commit breaks bootup on sunxi boards, the get stuck > when running the main u-boot binary at: > > CPU: Allwinner H3 (SUN8I) > I2C: ready > DRAM: > > This reverts commit 8e7cba048baae68ee0916a8f52b4304277328d5e. > >

Re: [U-Boot] [PATCH] Revert "common/memsize.c: Simplify RAM size detection"

2016-02-10 Thread Wolfgang Denk
Dear Hans, In message <56baf2b1.3040...@redhat.com> you wrote: > > It seems this code was as it was for a reason and that a revert > of your simplification is probably the best solution. Indeed. Over the past 15 years I have seen several attempts to improve or simplify the mesize code. Most

Re: [U-Boot] [PATCH] Revert "common/memsize.c: Simplify RAM size detection"

2016-02-10 Thread Hans de Goede
Hi, On 10-02-16 07:11, Eddy Petrișor wrote: Hi Hans, I sent a fixup patch yesterday (I managed somehow to send an outdated patch), Ah I did not notice that one. did you try the fixup patch? I just did try it, same results as Hannes the hang is gone, but now the RAM size is misdedetected

Re: [U-Boot] [PATCH] Revert "common/memsize.c: Simplify RAM size detection"

2016-02-10 Thread Eddy Petrișor
2016-02-10 10:20 GMT+02:00 Hans de Goede : > Hi, > > On 10-02-16 07:11, Eddy Petrișor wrote: >> >> Hi Hans, >> >> I sent a fixup patch yesterday (I managed somehow to send an outdated >> patch), > > > Ah I did not notice that one. > >> did you try the fixup patch? > > > I just

[U-Boot] [PATCH] Revert "common/memsize.c: Simplify RAM size detection"

2016-02-09 Thread Hans de Goede
This commit breaks bootup on sunxi boards, the get stuck when running the main u-boot binary at: CPU: Allwinner H3 (SUN8I) I2C: ready DRAM: This reverts commit 8e7cba048baae68ee0916a8f52b4304277328d5e. Signed-off-by: Hans de Goede --- common/memsize.c | 47

Re: [U-Boot] [PATCH] Revert "common/memsize.c: Simplify RAM size detection"

2016-02-09 Thread Eddy Petrișor
Hi Hans, I sent a fixup patch yesterday (I managed somehow to send an outdated patch), did you try the fixup patch? http://lists.denx.de/pipermail/u-boot/2016-February/245080.html Pe 9 feb. 2016 11:38 p.m., "Hans de Goede" a scris: > This commit breaks bootup on sunxi