u-boot/board/xilinx/zynq
Nathan Rossi de9bf1b591 ARM: zynq: Replace board specific with generic memory bank decoding
The dram_init and dram_init_banksize functions were using a board
specific implementation for decoding the memory banks from the fdt. This
board specific implementation uses a static variable 'tmp' which makes
these functions unsafe for execution from within the board_init_f
context.

This unsafe use of a static variable was causing a specific bug when
using the zynq_zybo configuration, U-Boot would generate the following
error during image load. This was caused due to dram_init overwriting
the relocations for the 'image' variable within the do_bootm function.
Out of coincidence the un-initialized memory has a compression type
which is the same as the value for the relocation type R_ARM_RELATIVE.

   Uncompressing Invalid Image ... Unimplemented compression type 23

It should be noted that this is just one way the issue could surface,
other cases my not be observed in normal boot flow. Depending on the
size of various sections, and location of relocations within __rel_dyn
and the compiler/linker the outcome of this bug can differ greatly.

This change makes the dram_init* functions use a generic implementation
of decoding and populating memory bank and size data.

Signed-off-by: Nathan Rossi <nathan@nathanrossi.com>
Fixes: 758f29d0f8 ("ARM: zynq: Support systems with more memory banks")
Cc: Michal Simek <monstr@monstr.eu>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
2016-12-20 09:15:28 +01:00
..
zynq-microzed ARM: zynq: Simplify zynq configuration 2016-05-24 13:22:50 +02:00
zynq-zc702 ARM: zynq: Simplify zynq configuration 2016-05-24 13:22:50 +02:00
zynq-zc706 ARM: zynq: Simplify zynq configuration 2016-05-24 13:22:50 +02:00
zynq-zed ARM: zynq: Simplify zynq configuration 2016-05-24 13:22:50 +02:00
zynq-zybo ARM: zynq: Simplify zynq configuration 2016-05-24 13:22:50 +02:00
.gitignore ARM: zynq: add default ps7_init_gpl.c/h for Zed, MicroZed, ZC70x 2015-05-25 10:52:36 +02:00
board.c ARM: zynq: Replace board specific with generic memory bank decoding 2016-12-20 09:15:28 +01:00
MAINTAINERS ARM: zynq: Remove Jagan from list of maintainers 2015-04-08 18:43:51 -04:00
Makefile ARM: zynq: Simplify zynq configuration 2016-05-24 13:22:50 +02:00
xil_io.h zynq: Use GPLed files for SPL 2014-11-11 10:51:43 +01:00