OSDN Git Service

drm/connector: Check for destroy implementation
authorMaxime Ripard <maxime@cerno.tech>
Mon, 11 Jul 2022 17:38:38 +0000 (19:38 +0200)
committerMaxime Ripard <maxime@cerno.tech>
Wed, 13 Jul 2022 08:46:06 +0000 (10:46 +0200)
Connectors need to be cleaned up with a call to drm_connector_cleanup()
in their drm_connector_funcs.destroy implementation.

Let's check for this and complain if there's no such function.

Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
Link: https://lore.kernel.org/r/20220711173939.1132294-9-maxime@cerno.tech
drivers/gpu/drm/drm_connector.c

index bbdac23..dfd1702 100644 (file)
@@ -348,6 +348,9 @@ int drm_connector_init(struct drm_device *dev,
                       const struct drm_connector_funcs *funcs,
                       int connector_type)
 {
+       if (drm_WARN_ON(dev, !(funcs && funcs->destroy)))
+               return -EINVAL;
+
        return __drm_connector_init(dev, connector, funcs, connector_type, NULL);
 }
 EXPORT_SYMBOL(drm_connector_init);
@@ -378,6 +381,9 @@ int drm_connector_init_with_ddc(struct drm_device *dev,
                                int connector_type,
                                struct i2c_adapter *ddc)
 {
+       if (drm_WARN_ON(dev, !(funcs && funcs->destroy)))
+               return -EINVAL;
+
        return __drm_connector_init(dev, connector, funcs, connector_type, ddc);
 }
 EXPORT_SYMBOL(drm_connector_init_with_ddc);