[igt-dev] [PATCH i-g-t] igt_core: Fix docs for SLOW_QUICK

Petri Latvala petri.latvala at intel.com
Wed Nov 17 10:42:29 UTC 2021


The macro is for choosing between two values depending on whether IGT
has been told it's running in simulation mode. The parameter names and
their documentation was the opposite of what actually happened.

The only user, gem_lut_handle, uses the macro correctly, having the
"use this value in simulation mode" value as the second parameter.

Signed-off-by: Petri Latvala <petri.latvala at intel.com>
Cc: Arkadiusz Hiler <arek at hiler.eu>
Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
---
 lib/igt_core.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/igt_core.h b/lib/igt_core.h
index 6b8dbf34..077e1bf3 100644
--- a/lib/igt_core.h
+++ b/lib/igt_core.h
@@ -1170,7 +1170,7 @@ bool igt_run_in_simulation(void);
  * Simple macro to select between two values (e.g. number of test rounds or test
  * buffer size) depending upon whether i-g-t is run in simulation mode or not.
  */
-#define SLOW_QUICK(slow,quick) (igt_run_in_simulation() ? (quick) : (slow))
+#define SLOW_QUICK(quick,slow) (igt_run_in_simulation() ? (slow) : (quick))
 
 void igt_skip_on_simulation(void);
 
-- 
2.29.2



More information about the igt-dev mailing list