[PATCH] cmd: mem: Remove CONFIG_SYS_MEMTEST_SCRATCH mapping

Heiko Schocher hs at denx.de
Thu May 7 11:48:20 CEST 2020


Hello Michal,

Am 07.05.2020 um 08:39 schrieb Michal Simek:
> 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 <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.
> 
> And was there any reason to setup specific address on PPC for saving
> test pattern?

Not that I am aware of anymore...

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