From 03e6a672c964ebaf0e61018b0a7a63edb2748990 Mon Sep 17 00:00:00 2001 From: Paul Selkirk Date: Tue, 17 Nov 2015 23:45:17 -0500 Subject: More post-merge cleanup. - Joachim says always check entropy and csprng for 'valid' before reading. - Harmonize RNG status valid bit with other cores. - Clean up compiler warnings about printf formats. --- tests/test-trng.c | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) (limited to 'tests/test-trng.c') diff --git a/tests/test-trng.c b/tests/test-trng.c index 8ca64e0..ebfe701 100644 --- a/tests/test-trng.c +++ b/tests/test-trng.c @@ -46,7 +46,7 @@ #include #ifndef WAIT_FOR_CSPRNG_VALID -#define WAIT_FOR_CSPRNG_VALID 0 +#define WAIT_FOR_CSPRNG_VALID 1 #endif static void show_core(const hal_core_t *core, const char *whinge) @@ -71,12 +71,7 @@ static hal_error_t test_random(const char *name) return HAL_ERROR_CORE_NOT_FOUND; for (i = 0; i < 8; ++i) { - /* The entropy sources are faster than the bus speed, so querying - * the "status_valid" bit always returns true. Also, at the - * moment, the trng cores use bit 0 for valid, while all the other - * cores use bit 1, so we query the "status_ready" bit. - */ - if (WAIT_FOR_CSPRNG_VALID && (err = hal_io_wait_ready(core)) != HAL_OK) { + if (WAIT_FOR_CSPRNG_VALID && (err = hal_io_wait_valid(core)) != HAL_OK) { printf("hal_io_wait_valid: %s\n", hal_error_string(err)); return err; } @@ -89,7 +84,7 @@ static hal_error_t test_random(const char *name) return err; } - printf("%08lx ", rnd); + printf("%08x ", rnd); } printf("\n"); @@ -105,13 +100,13 @@ int main(void) /* Exercise the API function. This gets random data from the CSPRNG, * so we end up hitting that twice. */ - printf("hal_get_random:\n"); + printf("hal_get_random\n"); if ((err = hal_get_random(NULL, (void *) &rnd, sizeof(rnd))) != HAL_OK) { printf("hal_get_random: %s\n", hal_error_string(err)); } else { for (i = 0; i < 8; ++i) { - printf("%08lx ", rnd[i]); + printf("%08x ", rnd[i]); } printf("\n"); } -- cgit v1.2.3