Hello Michal,

Am 04.05.2020 um 13:54 schrieb Michal Simek:
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(-)

Hmm.. I think this is from old powerpc times, but I do not see a
powerpc board in the list, so this makes sense.

Reviewed-by: Heiko Schocher <[email protected]>

bye,
Heiko
--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: [email protected]

Reply via email to