On Fri, Nov 14, 2025 at 05:31:40AM -0700, Simon Glass wrote: > Hi Heinrich, > > On Sun, 9 Nov 2025 at 03:10, Heinrich Schuchardt > <[email protected]> wrote: > > > > The location of memory depends on the board. Do not assume memory at fixed > > memory locations. Use memalign() instead to allocate a buffer. > > > > Signed-off-by: Heinrich Schuchardt <[email protected]> [snip] > > /* bytes */ > > print_hex_dump_bytes("", DUMP_PREFIX_ADDRESS, buf, 0x12); > > ut_assert_nextline("%0*lx: 00 11 22 33 44 55 66 77 88 99 aa bb cc > > dd ee ff ..\"3DUfw........", > > - IS_ENABLED(CONFIG_PHYS_64BIT) ? 16 : 8, > > - (uintptr_t)buf); > > + IS_ENABLED(CONFIG_PHYS_64BIT) ? 16 : 8, addr); [snip] > This is adding memory allocations to a test for hex dumping. > > It would be better and simpler to use a fixed address and make this a > sandbox-only test. I struggle to see the value of running these sorts > of tests under QEMU?
Removing context to highlight value of running tests on multiple platforms. -- Tom
signature.asc
Description: PGP signature
