drm/i915: Correct for_each_old_global_obj_in_state() arguments
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Thu, 8 Feb 2024 15:17:08 +0000 (17:17 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 9 Feb 2024 12:30:51 +0000 (14:30 +0200)
for_each_old_global_obj_in_state() gives us the old state, not the
new state. Correct the name of the macro argument.

Note that while the argument was misnamed the macro did work
correctly regardless.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240208151720.7866-2-ville.syrjala@linux.intel.com
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/display/intel_global_state.h

index 5c8545d7a76a7736a7e3e782187c75adfe606d39..6506a8e32972a8c6efd0280e3ea29dd8041421ef 100644 (file)
@@ -37,11 +37,11 @@ struct intel_global_obj {
             (__i)++) \
                for_each_if(obj)
 
-#define for_each_old_global_obj_in_state(__state, obj, new_obj_state, __i) \
+#define for_each_old_global_obj_in_state(__state, obj, old_obj_state, __i) \
        for ((__i) = 0; \
             (__i) < (__state)->num_global_objs && \
                     ((obj) = (__state)->global_objs[__i].ptr, \
-                     (new_obj_state) = (__state)->global_objs[__i].old_state, 1); \
+                     (old_obj_state) = (__state)->global_objs[__i].old_state, 1); \
             (__i)++) \
                for_each_if(obj)