Hi Tom,
On Wed, 30 Oct 2024 at 01:36, Tom Rini <tr...@konsulko.com> wrote:
>
> When reading/writing to memory we cannot assume that a base address of
> 0x0 is correct and functional. So use CONFIG_SYS_LOAD_ADDR as the base
> from which we add a bit more padding and being our tests.
>
> Signed-off-by: Tom Rini <tr...@konsulko.com>
> ---
> test/cmd/mem_copy.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/cmd/mem_copy.c b/test/cmd/mem_copy.c
> index 1ba0cebbbe04..67eca328777a 100644
> --- a/test/cmd/mem_copy.c
> +++ b/test/cmd/mem_copy.c
> @@ -21,7 +21,7 @@ struct param {
> static int do_test(struct unit_test_state *uts,
> const char *suffix, int d, int s, int count)
> {
> - const long addr = 0x1000;
> + const long addr = CONFIG_SYS_LOAD_ADDR + 0x1000;
CFG_SYS_SDRAM_BASE would be better here, since it is the start of
memory (0 on sandbox and most boards).
> u8 shadow[BUF_SIZE];
> u8 *buf;
> int i, w, bytes;
> --
> 2.43.0
>
Regards,
Simon
