From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Simek Date: Thu, 7 May 2020 08:39:20 +0200 Subject: [PATCH] cmd: mem: Remove CONFIG_SYS_MEMTEST_SCRATCH mapping In-Reply-To: References: Message-ID: <637580f3-0042-ded2-173b-c9bddd92c588@xilinx.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 07. 05. 20 5:33, Heiko Schocher wrote: > 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 >> --- >> >> ? 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. And was there any reason to setup specific address on PPC for saving test pattern? Thanks, Michal