[PATCH 01/11] imx: implement get_effective_memsize

Peng Fan peng.fan at oss.nxp.com
Wed Feb 1 01:15:57 CET 2023



On 1/31/2023 7:10 PM, Stefano Babic wrote:
> On 31.01.23 12:02, Fabio Estevam wrote:
>> Hi Peng,
>>
>> On Mon, Nov 7, 2022 at 4:13 AM Peng Fan (OSS) <peng.fan at oss.nxp.com> 
>> wrote:
>>>
>>> From: Peng Fan <peng.fan at nxp.com>
>>>
>>> To i.MX6/7 which has 2GB memory, the upper 4KB cut off, will cause the
>>> top 1MB not mapped as normal memory, because ARMV7-A use section
>>> mapping. So implement i.MX6/7 specific get_effective_memsize to fix the
>>> issue.
>>>
>>> Fixes: 777aaaa706bc("common/memsize.c: Fix get_effective_memsize() to 
>>> check for overflow")
>>
>> This commit has been reverted in master for non-Power PC platforms:
>>
>> commit d1f4b090dd17ee11373859f2c91af94bf254af7a
>> Author: Pali Rohár <pali at kernel.org>
>> Date:   Sat Jan 7 22:55:26 2023 +0100
>>
>>      common/memsize.c: Check for overflow in get_effective_memsize()
>> only for mpc85xx
>>
>>      This reverts commit 777aaaa706bc ("common/memsize.c: Fix
>>      get_effective_memsize() to check for overflow") for non-mpc85xx 
>> platforms.
>>
>>      The changes to this generic function, which is intended to help with
>>      32bit platforms with large amounts of memory has unintended side 
>> effects
>>      (which in turn lead to boot failures) on other platforms which were
>>      previously functional.
>>
>>      For now do overflow check only for powerpc mpc85xx platform. It 
>> is needed
>>      to prevent crashing of P1/P2 boards with 4GB DDR module in 32-bit 
>> mode.
>>
>>      Fixes: 777aaaa706bc ("common/memsize.c: Fix
>> get_effective_memsize() to check for overflow")
>>      Signed-off-by: Pali Rohár <pali at kernel.org>
>>
>> Can this patch be discarded then?
>>
>> I just noticed that Stefano applied it to u-boot-imx master-next branch.
> 
> 
> Yes, it is in test - I could let it out for the moment, waiting for 
> Peng's answer.


Yes, this patch could dropped, since the common code already has fix.

Thanks,
Peng.

> 
> Regards,
> Stefano
> 


More information about the U-Boot mailing list