[PATCH v2 07/16] test: Avoid random numbers in dm_test_devm_regmap()

Simon Glass sjg at chromium.org
Fri May 14 03:39:23 CEST 2021


There is no good reason to use a sequence from rand() here. We may as well
invent our own sequence.

This should molify Coverity which does not use rand() being used.

Signed-off-by: Simon Glass <sjg at chromium.org>
Reported-by: Coverity (CID: 312949)
---

(no changes since v1)

 test/dm/regmap.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/test/dm/regmap.c b/test/dm/regmap.c
index 372a73ca0c3..04bb1645d1b 100644
--- a/test/dm/regmap.c
+++ b/test/dm/regmap.c
@@ -306,9 +306,8 @@ static int dm_test_devm_regmap(struct unit_test_state *uts)
 					      &dev));
 	priv = dev_get_priv(dev);
 
-	srand(get_ticks() + rand());
 	for (i = 0; i < REGMAP_TEST_BUF_SZ; i++) {
-		pattern[i] = rand();
+		pattern[i] = i * 0x87654321;
 		ut_assertok(regmap_write(priv->cfg_regmap, i, pattern[i]));
 	}
 	for (i = 0; i < REGMAP_TEST_BUF_SZ; i++) {
-- 
2.31.1.751.gd2f1c929bd-goog



More information about the U-Boot mailing list