drm/rockchip: vop: Use cleanup helper directly as destroy funcs
authorJonas Karlman <jonas@kwiboo.se>
Wed, 21 Jun 2023 22:33:18 +0000 (22:33 +0000)
committerHeiko Stuebner <heiko@sntech.de>
Sat, 12 Aug 2023 21:53:34 +0000 (23:53 +0200)
vop_plane_destroy and vop_crtc_destroy are plain wrappers around
drm_plane_cleanup and drm_crtc_cleanup. Use them directly as plane and
crtc funcs to closer match VOP2 driver.

Signed-off-by: Jonas Karlman <jonas@kwiboo.se>
Reviewed-by: Sascha Hauer <s.hauer@pengutronix.de>
Signed-off-by: Heiko Stuebner <heiko@sntech.de>
Link: https://patchwork.freedesktop.org/patch/msgid/20230621223311.2239547-3-jonas@kwiboo.se
drivers/gpu/drm/rockchip/rockchip_drm_vop.c

index 3c68a239d88a426d983bf3cfa8f80eda37155b5a..8e89d78342e939a2d9d5fba2c930e95a4a4bd4e2 100644 (file)
@@ -765,11 +765,6 @@ out:
        }
 }
 
-static void vop_plane_destroy(struct drm_plane *plane)
-{
-       drm_plane_cleanup(plane);
-}
-
 static inline bool rockchip_afbc(u64 modifier)
 {
        return modifier == ROCKCHIP_AFBC_MOD;
@@ -1130,7 +1125,7 @@ static const struct drm_plane_helper_funcs plane_helper_funcs = {
 static const struct drm_plane_funcs vop_plane_funcs = {
        .update_plane   = drm_atomic_helper_update_plane,
        .disable_plane  = drm_atomic_helper_disable_plane,
-       .destroy = vop_plane_destroy,
+       .destroy = drm_plane_cleanup,
        .reset = drm_atomic_helper_plane_reset,
        .atomic_duplicate_state = drm_atomic_helper_plane_duplicate_state,
        .atomic_destroy_state = drm_atomic_helper_plane_destroy_state,
@@ -1601,11 +1596,6 @@ static const struct drm_crtc_helper_funcs vop_crtc_helper_funcs = {
        .atomic_disable = vop_crtc_atomic_disable,
 };
 
-static void vop_crtc_destroy(struct drm_crtc *crtc)
-{
-       drm_crtc_cleanup(crtc);
-}
-
 static struct drm_crtc_state *vop_crtc_duplicate_state(struct drm_crtc *crtc)
 {
        struct rockchip_crtc_state *rockchip_state;
@@ -1710,7 +1700,7 @@ vop_crtc_verify_crc_source(struct drm_crtc *crtc, const char *source_name,
 static const struct drm_crtc_funcs vop_crtc_funcs = {
        .set_config = drm_atomic_helper_set_config,
        .page_flip = drm_atomic_helper_page_flip,
-       .destroy = vop_crtc_destroy,
+       .destroy = drm_crtc_cleanup,
        .reset = vop_crtc_reset,
        .atomic_duplicate_state = vop_crtc_duplicate_state,
        .atomic_destroy_state = vop_crtc_destroy_state,
@@ -1961,7 +1951,7 @@ static void vop_destroy_crtc(struct vop *vop)
         */
        list_for_each_entry_safe(plane, tmp, &drm_dev->mode_config.plane_list,
                                 head)
-               vop_plane_destroy(plane);
+               drm_plane_cleanup(plane);
 
        /*
         * Destroy CRTC after vop_plane_destroy() since vop_disable_plane()