commit 32b17d3b0aae9195aea57d502adc4cd11ca2c077
author Andrei Emeltchenko <andrei.emeltchenko@intel.com> Tue Apr 06 18:19:13 2021 +0300
committer Anas Nashif <anas.nashif@intel.com> Wed Jun 30 18:29:21 2021 -0400
tree 222676fe96f5a17e4d5e9c357c1140949dd4b942
parent 0eb48e30c6973b7c21fd0806be68eaddca634352

edac: tests: Use new EDAC API in tests

Use new EDAC API in tests.

Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@intel.com>

tests/subsys/edac/ibecc/src/ibecc.c

diff --git a/tests/subsys/edac/ibecc/src/ibecc.c b/tests/subsys/edac/ibecc/src/ibecc.c
index deee093..34f8329 100644
--- a/tests/subsys/edac/ibecc/src/ibecc.c
+++ b/tests/subsys/edac/ibecc/src/ibecc.c
@@ -52,19 +52,22 @@
 
 static void test_ibecc_api(void)
 {
-	uint64_t ret;
+	uint64_t value;
+	int ret;
 
 	/* Error log API */
 
-	ret = edac_ecc_error_log_get(dev);
-	zassert_equal(ret, 0, "Error log not zero");
+	ret = edac_ecc_error_log_get(dev, &value);
+	zassert_equal(ret, 0, "edac_ecc_error_log_get failed");
 
-	edac_ecc_error_log_clear(dev);
+	ret = edac_ecc_error_log_clear(dev);
+	zassert_equal(ret, 0, "edac_ecc_error_log_clear failed");
 
-	ret = edac_parity_error_log_get(dev);
-	zassert_equal(ret, 0, "Error log not zero");
+	ret = edac_parity_error_log_get(dev, &value);
+	zassert_equal(ret, 0, "edac_parity_error_log_get failed");
 
-	edac_parity_error_log_clear(dev);
+	ret = edac_parity_error_log_clear(dev);
+	zassert_equal(ret, 0, "edac_parity_error_log_clear failed");
 
 	/* Error stat API */
 
@@ -94,7 +97,8 @@
 	ret = edac_inject_set_param1(dev, UINT64_MAX);
 	zassert_not_equal(ret, 0, "Error setting invalid param1");
 
-	val = edac_inject_get_param1(dev);
+	ret = edac_inject_get_param1(dev, &val);
+	zassert_equal(ret, 0, "Error getting param1");
 	zassert_equal(val, TEST_ADDRESS1, "Read back value differs");
 
 	/* Set correct value of param2 */
@@ -105,7 +109,8 @@
 	ret = edac_inject_set_param2(dev, UINT64_MAX);
 	zassert_not_equal(ret, 0, "Error setting invalid param1");
 
-	val = edac_inject_get_param2(dev);
+	ret = edac_inject_get_param2(dev, &val);
+	zassert_equal(ret, 0, "Error getting param2");
 	zassert_equal(val, TEST_ADDRESS_MASK, "Read back value differs");
 
 	/* Clearing parameters */
@@ -113,13 +118,15 @@
 	ret = edac_inject_set_param1(dev, 0);
 	zassert_equal(ret, 0, "Error setting inject address");
 
-	val = edac_inject_get_param1(dev);
+	ret = edac_inject_get_param1(dev, &val);
+	zassert_equal(ret, 0, "Error getting param1");
 	zassert_equal(val, 0, "Read back value differs");
 
 	ret = edac_inject_set_param2(dev, 0);
 	zassert_equal(ret, 0, "Error setting inject address mask");
 
-	val = edac_inject_get_param2(dev);
+	ret = edac_inject_get_param2(dev, &val);
+	zassert_equal(ret, 0, "Error getting param2");
 	zassert_equal(val, 0, "Read back value differs");
 }
 #else
@@ -149,9 +156,9 @@
 	ret = edac_inject_set_error_type(dev, type);
 	zassert_equal(ret, 0, "Error setting inject error type");
 
-	test_value = edac_inject_get_error_type(dev);
-	zassert_equal(test_value, type,
-		      "Read back value differs");
+	ret = edac_inject_get_error_type(dev, &test_value);
+	zassert_equal(ret, 0, "Error getting error_type");
+	zassert_equal(test_value, type, "Read back value differs");
 
 	ret = edac_inject_error_trigger(dev);
 	zassert_equal(ret, 0, "Error setting ctrl");
@@ -182,7 +189,6 @@
 		      num_int);
 
 	TC_PRINT("Interrupt %d\n", num_int);
-	TC_PRINT("ECC Error Log 0x%llx\n", edac_ecc_error_log_get(dev));
 	TC_PRINT("Error: type %u, address 0x%llx, syndrome %u\n",
 		 error_type, error_address, error_syndrome);
 }