[PATCH] cmd: mem: Remove CONFIG_SYS_MEMTEST_SCRATCH mapping
Heiko Schocher
hs at denx.de
Thu May 7 05:33:29 CEST 2020
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 <michal.simek at 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(-)
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 <hs at denx.de>
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: hs at denx.de
More information about the U-Boot
mailing list