drm/i915/plane: drop atomic from intel_atomic_plane_check_clipping()
authorJani Nikula <jani.nikula@intel.com>
Tue, 17 Jun 2025 08:13:37 +0000 (11:13 +0300)
committerJani Nikula <jani.nikula@intel.com>
Thu, 19 Jun 2025 09:14:29 +0000 (12:14 +0300)
Align with intel_plane_check_src_coordinates(). The "atomic" is
superfluous.

Reviewed-by: Gustavo Sousa <gustavo.sousa@intel.com>
Link: https://lore.kernel.org/r/0bebd67e583b6ca56f788bd795ffe77db342e809.1750147992.git.jani.nikula@intel.com
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/display/i9xx_plane.c
drivers/gpu/drm/i915/display/intel_cursor.c
drivers/gpu/drm/i915/display/intel_plane.c
drivers/gpu/drm/i915/display/intel_plane.h
drivers/gpu/drm/i915/display/intel_sprite.c
drivers/gpu/drm/i915/display/skl_universal_plane.c

index eba8eb7..ac84558 100644 (file)
@@ -336,10 +336,10 @@ i9xx_plane_check(struct intel_crtc_state *crtc_state,
        if (ret)
                return ret;
 
-       ret = intel_atomic_plane_check_clipping(plane_state, crtc_state,
-                                               DRM_PLANE_NO_SCALING,
-                                               DRM_PLANE_NO_SCALING,
-                                               i9xx_plane_has_windowing(plane));
+       ret = intel_plane_check_clipping(plane_state, crtc_state,
+                                        DRM_PLANE_NO_SCALING,
+                                        DRM_PLANE_NO_SCALING,
+                                        i9xx_plane_has_windowing(plane));
        if (ret)
                return ret;
 
index 27d8654..198e69e 100644 (file)
@@ -158,10 +158,10 @@ static int intel_check_cursor(struct intel_crtc_state *crtc_state,
                return -EINVAL;
        }
 
-       ret = intel_atomic_plane_check_clipping(plane_state, crtc_state,
-                                               DRM_PLANE_NO_SCALING,
-                                               DRM_PLANE_NO_SCALING,
-                                               true);
+       ret = intel_plane_check_clipping(plane_state, crtc_state,
+                                        DRM_PLANE_NO_SCALING,
+                                        DRM_PLANE_NO_SCALING,
+                                        true);
        if (ret)
                return ret;
 
index 3d6f77a..9c9a522 100644 (file)
@@ -983,10 +983,10 @@ void intel_crtc_planes_update_arm(struct intel_dsb *dsb,
                i9xx_crtc_planes_update_arm(dsb, state, crtc);
 }
 
-int intel_atomic_plane_check_clipping(struct intel_plane_state *plane_state,
-                                     struct intel_crtc_state *crtc_state,
-                                     int min_scale, int max_scale,
-                                     bool can_position)
+int intel_plane_check_clipping(struct intel_plane_state *plane_state,
+                              struct intel_crtc_state *crtc_state,
+                              int min_scale, int max_scale,
+                              bool can_position)
 {
        struct intel_display *display = to_intel_display(plane_state);
        struct intel_plane *plane = to_intel_plane(plane_state->uapi.plane);
index 1dd3258..10a94b9 100644 (file)
@@ -74,10 +74,10 @@ int intel_plane_atomic_check(struct intel_atomic_state *state,
 int intel_plane_calc_min_cdclk(struct intel_atomic_state *state,
                               struct intel_plane *plane,
                               bool *need_cdclk_calc);
-int intel_atomic_plane_check_clipping(struct intel_plane_state *plane_state,
-                                     struct intel_crtc_state *crtc_state,
-                                     int min_scale, int max_scale,
-                                     bool can_position);
+int intel_plane_check_clipping(struct intel_plane_state *plane_state,
+                              struct intel_crtc_state *crtc_state,
+                              int min_scale, int max_scale,
+                              bool can_position);
 int intel_plane_check_src_coordinates(struct intel_plane_state *plane_state);
 void intel_plane_set_invisible(struct intel_crtc_state *crtc_state,
                               struct intel_plane_state *plane_state);
index db0c48d..e6844df 100644 (file)
@@ -1366,8 +1366,8 @@ g4x_sprite_check(struct intel_crtc_state *crtc_state,
                }
        }
 
-       ret = intel_atomic_plane_check_clipping(plane_state, crtc_state,
-                                               min_scale, max_scale, true);
+       ret = intel_plane_check_clipping(plane_state, crtc_state,
+                                        min_scale, max_scale, true);
        if (ret)
                return ret;
 
@@ -1421,10 +1421,10 @@ vlv_sprite_check(struct intel_crtc_state *crtc_state,
        if (ret)
                return ret;
 
-       ret = intel_atomic_plane_check_clipping(plane_state, crtc_state,
-                                               DRM_PLANE_NO_SCALING,
-                                               DRM_PLANE_NO_SCALING,
-                                               true);
+       ret = intel_plane_check_clipping(plane_state, crtc_state,
+                                        DRM_PLANE_NO_SCALING,
+                                        DRM_PLANE_NO_SCALING,
+                                        true);
        if (ret)
                return ret;
 
index 9c8dac9..68f18f1 100644 (file)
@@ -2327,8 +2327,8 @@ static int skl_plane_check(struct intel_crtc_state *crtc_state,
                max_scale = skl_plane_max_scale(display, fb);
        }
 
-       ret = intel_atomic_plane_check_clipping(plane_state, crtc_state,
-                                               min_scale, max_scale, true);
+       ret = intel_plane_check_clipping(plane_state, crtc_state,
+                                        min_scale, max_scale, true);
        if (ret)
                return ret;