Commit cf1788fb authored by Maxime Ripard's avatar Maxime Ripard

drm/vc4: tests: pv-muxing: Remove call to drm_kunit_helper_free_device()

Calling drm_kunit_helper_free_device() to clean up the resources
allocated by drm_kunit_helper_alloc_device() is now optional and not
needed in most cases.

Remove it.
Reviewed-by: default avatarJavier Martinez Canillas <javierm@redhat.com>
Reviewed-by: default avatarMaíra Canal <mairacanal@riseup.net>
Link: https://lore.kernel.org/r/20230728-kms-kunit-actions-rework-v3-7-952565ccccfe@kernel.orgSigned-off-by: default avatarMaxime Ripard <mripard@kernel.org>
parent 394ba10e
...@@ -762,7 +762,6 @@ static void vc4_pv_muxing_test_exit(struct kunit *test) ...@@ -762,7 +762,6 @@ static void vc4_pv_muxing_test_exit(struct kunit *test)
drm_modeset_drop_locks(&priv->ctx); drm_modeset_drop_locks(&priv->ctx);
drm_modeset_acquire_fini(&priv->ctx); drm_modeset_acquire_fini(&priv->ctx);
drm_dev_unregister(drm); drm_dev_unregister(drm);
drm_kunit_helper_free_device(test, vc4->dev);
} }
static struct kunit_case vc4_pv_muxing_tests[] = { static struct kunit_case vc4_pv_muxing_tests[] = {
...@@ -873,7 +872,6 @@ static void drm_test_vc5_pv_muxing_bugs_subsequent_crtc_enable(struct kunit *tes ...@@ -873,7 +872,6 @@ static void drm_test_vc5_pv_muxing_bugs_subsequent_crtc_enable(struct kunit *tes
drm_modeset_drop_locks(&ctx); drm_modeset_drop_locks(&ctx);
drm_modeset_acquire_fini(&ctx); drm_modeset_acquire_fini(&ctx);
drm_dev_unregister(drm); drm_dev_unregister(drm);
drm_kunit_helper_free_device(test, vc4->dev);
} }
static void drm_test_vc5_pv_muxing_bugs_stable_fifo(struct kunit *test) static void drm_test_vc5_pv_muxing_bugs_stable_fifo(struct kunit *test)
...@@ -963,7 +961,6 @@ static void drm_test_vc5_pv_muxing_bugs_stable_fifo(struct kunit *test) ...@@ -963,7 +961,6 @@ static void drm_test_vc5_pv_muxing_bugs_stable_fifo(struct kunit *test)
drm_modeset_drop_locks(&ctx); drm_modeset_drop_locks(&ctx);
drm_modeset_acquire_fini(&ctx); drm_modeset_acquire_fini(&ctx);
drm_dev_unregister(drm); drm_dev_unregister(drm);
drm_kunit_helper_free_device(test, vc4->dev);
} }
static void static void
...@@ -1017,7 +1014,6 @@ drm_test_vc5_pv_muxing_bugs_subsequent_crtc_enable_too_many_crtc_state(struct ku ...@@ -1017,7 +1014,6 @@ drm_test_vc5_pv_muxing_bugs_subsequent_crtc_enable_too_many_crtc_state(struct ku
drm_modeset_drop_locks(&ctx); drm_modeset_drop_locks(&ctx);
drm_modeset_acquire_fini(&ctx); drm_modeset_acquire_fini(&ctx);
drm_dev_unregister(drm); drm_dev_unregister(drm);
drm_kunit_helper_free_device(test, vc4->dev);
} }
static struct kunit_case vc5_pv_muxing_bugs_tests[] = { static struct kunit_case vc5_pv_muxing_bugs_tests[] = {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment