From mboxrd@z Thu Jan  1 00:00:00 1970
From: =?UTF-8?Q?Eddy_Petri=C8=99or?= <eddy.petrisor@gmail.com>
Date: Wed, 10 Feb 2016 11:00:30 +0200
Subject: [U-Boot] [PATCH] Revert "common/memsize.c: Simplify RAM size
	detection"
In-Reply-To: <56BAF2B1.3040507@redhat.com>
References: <1455053911-7530-1-git-send-email-hdegoede@redhat.com>
 <CAK0XTWeXF=h2jiGz6dnP0CMcHw_=qt1bWkmMnpZjBYjgqFK_Yw@mail.gmail.com>
 <56BAF2B1.3040507@redhat.com>
Message-ID: <CAK0XTWdMAapU59CE=yUzwGDstE6-RzjmtXFXCRoR_VOdfQ=Rjg@mail.gmail.com>
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

2016-02-10 10:20 GMT+02:00 Hans de Goede <hdegoede@redhat.com>:
> 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 (it detects 2 GiB on a 1 GiB board).
>
> It seems this code was as it was for a reason and that a revert
> of your simplification is probably the best solution.

OK, agreed, that makes sense. My buggy HW (DDR unstable) doesn't help.

I will probably rework the patch and target the v2016.05 release.