Hi Maxime,
kernel test robot noticed the following build warnings:
[auto build test WARNING on c58c49dd89324b18a812762a2bfa5a0458e4f252]
url: https://github.com/intel-lab-lkp/linux/commits/Maxime-Ripard/drm-tests-helpe... base: c58c49dd89324b18a812762a2bfa5a0458e4f252 patch link: https://lore.kernel.org/r/20230720-kms-kunit-actions-rework-v2-6-175017bd56a... patch subject: [PATCH v2 06/11] drm/tests: helpers: Create a helper to allocate an atomic state config: arm64-randconfig-r022-20230720 (https://download.01.org/0day-ci/archive/20230721/202307210124.Ur3UNuxZ-lkp@i...) compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project.git 4a5ac14ee968ff0ad5d2cc1ffa0299048db4c88a) reproduce: (https://download.01.org/0day-ci/archive/20230721/202307210124.Ur3UNuxZ-lkp@i...)
If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot lkp@intel.com | Closes: https://lore.kernel.org/oe-kbuild-all/202307210124.Ur3UNuxZ-lkp@intel.com/
All warnings (new ones prefixed by >>):
drivers/gpu/drm/tests/drm_kunit_helpers.c:54:6: warning: cast from 'void (*)(struct platform_driver *)' to 'kunit_action_t *' (aka 'void (*)(void *)') converts to incompatible function type [-Wcast-function-type-strict] 54 | (kunit_action_t *)platform_driver_unregister, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/gpu/drm/tests/drm_kunit_helpers.c:62:6: warning: cast from 'void (*)(struct platform_device *)' to 'kunit_action_t *' (aka 'void (*)(void *)') converts to incompatible function type [-Wcast-function-type-strict] 62 | (kunit_action_t *)platform_device_put, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/gpu/drm/tests/drm_kunit_helpers.c:70:6: warning: cast from 'void (*)(struct platform_device *)' to 'kunit_action_t *' (aka 'void (*)(void *)') converts to incompatible function type [-Wcast-function-type-strict] 70 | (kunit_action_t *)platform_device_del, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/gpu/drm/tests/drm_kunit_helpers.c:90:9: warning: cast from 'void (*)(struct platform_device *)' to 'kunit_action_t *' (aka 'void (*)(void *)') converts to incompatible function type [-Wcast-function-type-strict] 90 | (kunit_action_t *)platform_device_unregister, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/gpu/drm/tests/drm_kunit_helpers.c:94:9: warning: cast from 'void (*)(struct platform_driver *)' to 'kunit_action_t *' (aka 'void (*)(void *)') converts to incompatible function type [-Wcast-function-type-strict] 94 | (kunit_action_t *)platform_driver_unregister, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/tests/drm_kunit_helpers.c:192:6: warning: cast from 'void (*)(struct drm_atomic_state *)' to 'kunit_action_t *' (aka 'void (*)(void *)') converts to incompatible function type [-Wcast-function-type-strict]
192 | (kunit_action_t *)drm_atomic_state_put, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 6 warnings generated.
vim +192 drivers/gpu/drm/tests/drm_kunit_helpers.c
164 165 /** 166 * drm_kunit_helper_atomic_state_alloc - Allocates an atomic state 167 * @test: The test context object 168 * @drm: The device to alloc the state for 169 * @ctx: Locking context for that atomic update 170 * 171 * Allocates a empty atomic state. 172 * 173 * The state is tied to the kunit test context, so we must not call 174 * drm_atomic_state_put() on it, it will be done so automatically. 175 * 176 * Returns: 177 * An ERR_PTR on error, a pointer to the newly allocated state otherwise 178 */ 179 struct drm_atomic_state * 180 drm_kunit_helper_atomic_state_alloc(struct kunit *test, 181 struct drm_device *drm, 182 struct drm_modeset_acquire_ctx *ctx) 183 { 184 struct drm_atomic_state *state; 185 int ret; 186 187 state = drm_atomic_state_alloc(drm); 188 if (!state) 189 return ERR_PTR(-ENOMEM); 190 191 ret = kunit_add_action_or_reset(test,
192 (kunit_action_t *)drm_atomic_state_put,
193 state); 194 if (ret) 195 return ERR_PTR(ret); 196 197 state->acquire_ctx = ctx; 198 199 return state; 200 } 201 EXPORT_SYMBOL_GPL(drm_kunit_helper_atomic_state_alloc); 202