| Message ID | 20230720-kms-kunit-actions-rework-v2-0-175017bd56ab@kernel.org |
|---|---|
| Headers | show |
| Series | drm: kunit: Switch to kunit actions | expand |
Hi Maxime, On 7/20/23 08:15, Maxime Ripard wrote: > Hi, > > Since v6.5-rc1, kunit gained a devm/drmm-like mechanism that makes tests > resources much easier to cleanup. > > This series converts the existing tests to use those new actions where > relevant. > > Let me know what you think, With the problems pointed out by kernel test bot fixed, the whole series is: Reviewed-by: Maíra Canal <mairacanal@riseup.net> Best Regards, - Maíra > Maxime > > Signed-off-by: Maxime Ripard <mripard@kernel.org> > --- > Changes in v2: > - Fix some typos > - Use plaltform_device_del instead of removing the call to > platform_device_put after calling platform_device_add > - Link to v1: https://lore.kernel.org/r/20230710-kms-kunit-actions-rework-v1-0-722c58d72c72@kernel.org > > --- > Maxime Ripard (11): > drm/tests: helpers: Switch to kunit actions > drm/tests: client-modeset: Remove call to drm_kunit_helper_free_device() > drm/tests: modes: Remove call to drm_kunit_helper_free_device() > drm/tests: probe-helper: Remove call to drm_kunit_helper_free_device() > drm/tests: helpers: Create a helper to allocate a locking ctx > drm/tests: helpers: Create a helper to allocate an atomic state > drm/vc4: tests: pv-muxing: Remove call to drm_kunit_helper_free_device() > drm/vc4: tests: mock: Use a kunit action to unregister DRM device > drm/vc4: tests: pv-muxing: Switch to managed locking init > drm/vc4: tests: Switch to atomic state allocation helper > drm/vc4: tests: pv-muxing: Document test scenario > > drivers/gpu/drm/tests/drm_client_modeset_test.c | 8 -- > drivers/gpu/drm/tests/drm_kunit_helpers.c | 108 +++++++++++++++++++++- > drivers/gpu/drm/tests/drm_modes_test.c | 8 -- > drivers/gpu/drm/tests/drm_probe_helper_test.c | 8 -- > drivers/gpu/drm/vc4/tests/vc4_mock.c | 5 ++ > drivers/gpu/drm/vc4/tests/vc4_test_pv_muxing.c | 115 +++++++++--------------- > include/drm/drm_kunit_helpers.h | 7 ++ > 7 files changed, 158 insertions(+), 101 deletions(-) > --- > base-commit: c58c49dd89324b18a812762a2bfa5a0458e4f252 > change-id: 20230710-kms-kunit-actions-rework-5d163762c93b > > Best regards,
Hi, Since v6.5-rc1, kunit gained a devm/drmm-like mechanism that makes tests resources much easier to cleanup. This series converts the existing tests to use those new actions where relevant. Let me know what you think, Maxime Signed-off-by: Maxime Ripard <mripard@kernel.org> --- Changes in v2: - Fix some typos - Use plaltform_device_del instead of removing the call to platform_device_put after calling platform_device_add - Link to v1: https://lore.kernel.org/r/20230710-kms-kunit-actions-rework-v1-0-722c58d72c72@kernel.org --- Maxime Ripard (11): drm/tests: helpers: Switch to kunit actions drm/tests: client-modeset: Remove call to drm_kunit_helper_free_device() drm/tests: modes: Remove call to drm_kunit_helper_free_device() drm/tests: probe-helper: Remove call to drm_kunit_helper_free_device() drm/tests: helpers: Create a helper to allocate a locking ctx drm/tests: helpers: Create a helper to allocate an atomic state drm/vc4: tests: pv-muxing: Remove call to drm_kunit_helper_free_device() drm/vc4: tests: mock: Use a kunit action to unregister DRM device drm/vc4: tests: pv-muxing: Switch to managed locking init drm/vc4: tests: Switch to atomic state allocation helper drm/vc4: tests: pv-muxing: Document test scenario drivers/gpu/drm/tests/drm_client_modeset_test.c | 8 -- drivers/gpu/drm/tests/drm_kunit_helpers.c | 108 +++++++++++++++++++++- drivers/gpu/drm/tests/drm_modes_test.c | 8 -- drivers/gpu/drm/tests/drm_probe_helper_test.c | 8 -- drivers/gpu/drm/vc4/tests/vc4_mock.c | 5 ++ drivers/gpu/drm/vc4/tests/vc4_test_pv_muxing.c | 115 +++++++++--------------- include/drm/drm_kunit_helpers.h | 7 ++ 7 files changed, 158 insertions(+), 101 deletions(-) --- base-commit: c58c49dd89324b18a812762a2bfa5a0458e4f252 change-id: 20230710-kms-kunit-actions-rework-5d163762c93b Best regards,