[PATCH] Revert "common/memsize.c: Fix get_effective_memsize() to check for overflow"

Fabio Estevam festevam at gmail.com
Fri Jan 6 21:29:45 CET 2023


On Fri, Jan 6, 2023 at 12:51 PM Tom Rini <trini at konsulko.com> wrote:
>
> This reverts commit 777aaaa706bcfe08c284aed06886db7d482af3f8.
>
> 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.
>
> Reported-by: Marek Vasut <marex at denx.de>
> Cc: Patrick Delaunay <patrick.delaunay at foss.st.com>
> Cc: Pali Rohár <pali at kernel.org>
> Signed-off-by: Tom Rini <trini at konsulko.com>

Peng also reported an imx issue caused by 777aaaa706bc
(""common/memsize.c: Fix get_effective_memsize() to check for
overflow""):

https://patchwork.ozlabs.org/project/uboot/patch/20221107080016.1304-2-peng.fan@oss.nxp.com/


More information about the U-Boot mailing list