]> code.ossystems Code Review - bsp/u-boot.git/commit
ARM: Fix overflow in MMU setup
authorMarek Vasut <marex@denx.de>
Sun, 3 Aug 2014 23:45:46 +0000 (01:45 +0200)
committerOtavio Salvador <otavio@ossystems.com.br>
Tue, 9 Sep 2014 15:21:48 +0000 (12:21 -0300)
commit9bba571f901494b660d69e98948f1af54221bbbc
tree12684961f220d66e6385b68acef6fb042dc74b51
parent7be9add8cfd41afc603008167ea4412a4e45a381
ARM: Fix overflow in MMU setup

The patch fixes a corner case where adding size to DRAM start resulted
in a value (1 << 32), which in turn overflew the u32 computation, which
resulted in 0 and it therefore prevented correct setup of the MMU tables.

The addition of DRAM bank start and it's size can end up right at the end
of the address space in the special case of a machine with enough memory.
To prevent this overflow, shift the start and size separately and add them
only after they were shifted.

Hopefully, we only have systems in tree which have DRAM size aligned to
1MiB boundary. If not, this patch would break such systems. On the other
hand, such system would be broken by design anyway.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Albert ARIBAUD <albert.u.boot@aribaud.net>
arch/arm/lib/cache-cp15.c