On 04.05.20 13:54, Michal Simek 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 <michal.si...@xilinx.com>
---

  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: Stefan Roese <s...@denx.de>

Thanks,
Stefan

Reply via email to