Hardware
/
litex
mirror of https://github.com/enjoy-digital/litex.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

memtest: Fix integer size/type printf errors 

In a couple of places, memtest uses %x to print a pointer which
is illegal (and could be problematic on 64-bit). Use %p instead.

Additionally, use %ld when printing longs

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
This commit is contained in:
Benjamin Herrenschmidt 2020-07-08 13:17:48 +10:00
parent 8af4e05c7f
commit 798b3d7ba4
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
litex/soc/software/libbase/memtest.c
View File

@ -172,7 +172,7 @@ void memspeed(unsigned int *addr, unsigned long size, bool read_only)
__attribute__((unused)) unsigned long data;
const unsigned int sz = sizeof(unsigned long);
printf("Memspeed at 0x%08x...\n", addr);
printf("Memspeed at 0x%p...\n", addr);
/* init timer */
timer0_en_write(0);
@ -221,16 +221,16 @@ int memtest(unsigned int *addr, unsigned long maxsize)
unsigned long addr_size = MEMTEST_ADDR_SIZE < maxsize ? MEMTEST_ADDR_SIZE : maxsize;
unsigned long data_size = MEMTEST_DATA_SIZE < maxsize ? MEMTEST_DATA_SIZE : maxsize;
printf("Memtest at 0x%08x...\n", addr);
printf("Memtest at 0x%p...\n", addr);
bus_errors = memtest_bus(addr, bus_size);
addr_errors = memtest_addr(addr, addr_size, MEMTEST_ADDR_RANDOM);
data_errors = memtest_data(addr, data_size, MEMTEST_DATA_RANDOM);
if(bus_errors + addr_errors + data_errors != 0) {
printf("- bus errors: %d/%d\n", bus_errors, 2*bus_size/4);
printf("- addr errors: %d/%d\n", addr_errors, addr_size/4);
printf("- data errors: %d/%d\n", data_errors, data_size/4);
printf("- bus errors: %d/%ld\n", bus_errors, 2*bus_size/4);
printf("- addr errors: %d/%ld\n", addr_errors, addr_size/4);
printf("- data errors: %d/%ld\n", data_errors, data_size/4);
printf("Memtest KO\n");
return 0;
}