On Mon, 4 May 2020 at 05:54, Michal Simek <[email protected]> wrote: > > There is no real need to exactly define space for saving patterns for > alternate memory test. It is much easier to allocate space on the stack and > use it instead of trying to find out space where pattern should be saved. > > For example if you want to test the whole DDR memory you can't save patter > to DDR and you need to find it out. On Xilinx devices DDR or OCM addresses > were chosen but that means that OCM needs to be mapped and U-Boot has > access permission there. > > It is easier to remove this limitation and simply save it on stack because > it is very clear that memory test can't rewrite U-Boot and U-Boot has also > full access to memory where runs from. > > Signed-off-by: Michal Simek <[email protected]> > --- > > README | 4 ---- > cmd/mem.c | 9 ++------- > include/configs/MigoR.h | 1 - > include/configs/advantech_dms-ba16.h | 1 - > include/configs/apalis_imx6.h | 1 - > include/configs/aristainetos2.h | 1 - > include/configs/armadillo-800eva.h | 1 - > include/configs/blanche.h | 1 - > include/configs/cgtqmx6eval.h | 1 - > include/configs/colibri_imx6.h | 1 - > include/configs/dh_imx6.h | 1 - > include/configs/el6x_common.h | 1 - > include/configs/embestmx6boards.h | 1 - > include/configs/ge_bx50v3.h | 1 - > include/configs/gw_ventana.h | 1 - > include/configs/imx6_logic.h | 1 - > include/configs/kzm9g.h | 1 - > include/configs/mx6sabre_common.h | 1 - > include/configs/nitrogen6x.h | 1 - > include/configs/presidio_asic.h | 1 - > include/configs/sh7752evb.h | 1 - > include/configs/sh7753evb.h | 1 - > include/configs/sh7757lcr.h | 1 - > include/configs/stm32mp1.h | 1 - > include/configs/tao3530.h | 1 - > include/configs/tricorder.h | 1 - > include/configs/xilinx_versal.h | 1 - > include/configs/xilinx_versal_mini.h | 1 - > include/configs/xilinx_zynqmp.h | 4 ---- > include/configs/xilinx_zynqmp_mini.h | 1 - > include/configs/xilinx_zynqmp_r5.h | 1 - > scripts/config_whitelist.txt | 1 - > 32 files changed, 2 insertions(+), 44 deletions(-)
Reviewed-by: Simon Glass <[email protected]>
