From: Daniel Vetter <daniel.vetter@ffwll.ch> To: DRI Development <dri-devel@lists.freedesktop.org> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>, Intel Graphics Development <intel-gfx@lists.freedesktop.org>, Thomas Zimmermann <tzimmermann@suse.de>, Daniel Vetter <daniel.vetter@intel.com>, Sam Ravnborg <sam@ravnborg.org> Subject: [PATCH 27/52] drm: Manage drm_mode_config_init with drmm_ Date: Wed, 19 Feb 2020 11:20:57 +0100 [thread overview] Message-ID: <20200219102122.1607365-28-daniel.vetter@ffwll.ch> (raw) In-Reply-To: <20200219102122.1607365-1-daniel.vetter@ffwll.ch> drm_mode_config_cleanup is idempotent, so no harm in calling this twice. This allows us to gradually switch drivers over by removing explicit drm_mode_config_cleanup calls. With this step it's not also possible that (at least for simple drivers) automatic resource cleanup can be done correctly without a drm_driver->release hook. Therefore allow this now in devm_drm_dev_init(). Also with drmm_ explicit drm_driver->release hooks are kinda not the best option, so deprecate that hook to discourage future users. v2: Fixup the example in the kerneldoc too. Cc: "Noralf Trønnes" <noralf@tronnes.org> Cc: Sam Ravnborg <sam@ravnborg.org> Cc: Thomas Zimmermann <tzimmermann@suse.de> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> --- drivers/gpu/drm/drm_drv.c | 21 +++++---------------- drivers/gpu/drm/drm_mode_config.c | 12 +++++++++++- include/drm/drm_mode_config.h | 2 +- 3 files changed, 17 insertions(+), 18 deletions(-) diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c index 3cf40864d4a6..428c569aaaf1 100644 --- a/drivers/gpu/drm/drm_drv.c +++ b/drivers/gpu/drm/drm_drv.c @@ -267,8 +267,7 @@ void drm_minor_release(struct drm_minor *minor) * * The following example shows a typical structure of a DRM display driver. * The example focus on the probe() function and the other functions that is - * almost always present and serves as a demonstration of devm_drm_dev_init() - * usage with its accompanying drm_driver->release callback. + * almost always present and serves as a demonstration of devm_drm_dev_init(). * * .. code-block:: c * @@ -278,16 +277,8 @@ void drm_minor_release(struct drm_minor *minor) * struct clk *pclk; * }; * - * static void driver_drm_release(struct drm_device *drm) - * { - * struct driver_device *priv = container_of(...); - * - * drm_mode_config_cleanup(drm); - * } - * * static struct drm_driver driver_drm_driver = { * [...] - * .release = driver_drm_release, * }; * * static int driver_probe(struct platform_device *pdev) @@ -312,7 +303,9 @@ void drm_minor_release(struct drm_minor *minor) * } * drmm_add_final_kfree(drm, priv); * - * drm_mode_config_init(drm); + * ret = drm_mode_config_init(drm); + * if (ret) + * return ret; * * priv->userspace_facing = drmm_kzalloc(..., GFP_KERNEL); * if (!priv->userspace_facing) @@ -710,8 +703,7 @@ static void devm_drm_dev_init_release(void *data) * @driver: DRM driver * * Managed drm_dev_init(). The DRM device initialized with this function is - * automatically put on driver detach using drm_dev_put(). You must supply a - * &drm_driver.release callback to control the finalization explicitly. + * automatically put on driver detach using drm_dev_put(). * * RETURNS: * 0 on success, or error code on failure. @@ -722,9 +714,6 @@ int devm_drm_dev_init(struct device *parent, { int ret; - if (WARN_ON(!driver->release)) - return -EINVAL; - ret = drm_dev_init(dev, driver, parent); if (ret) return ret; diff --git a/drivers/gpu/drm/drm_mode_config.c b/drivers/gpu/drm/drm_mode_config.c index 08e6eff6a179..957db1edba0c 100644 --- a/drivers/gpu/drm/drm_mode_config.c +++ b/drivers/gpu/drm/drm_mode_config.c @@ -25,6 +25,7 @@ #include <drm/drm_drv.h> #include <drm/drm_encoder.h> #include <drm/drm_file.h> +#include <drm/drm_managed.h> #include <drm/drm_mode_config.h> #include <drm/drm_print.h> #include <linux/dma-resv.h> @@ -373,6 +374,11 @@ static int drm_mode_create_standard_properties(struct drm_device *dev) return 0; } +static void drm_mode_config_init_release(struct drm_device *dev, void *ptr) +{ + drm_mode_config_cleanup(dev); +} + /** * drm_mode_config_init - initialize DRM mode_configuration structure * @dev: DRM device @@ -384,8 +390,10 @@ static int drm_mode_create_standard_properties(struct drm_device *dev) * problem, since this should happen single threaded at init time. It is the * driver's problem to ensure this guarantee. * + * Cleanup is automatically handled through registering drm_mode_config_cleanup + * with drmm_add_action(). */ -void drm_mode_config_init(struct drm_device *dev) +int drm_mode_config_init(struct drm_device *dev) { mutex_init(&dev->mode_config.mutex); drm_modeset_lock_init(&dev->mode_config.connection_mutex); @@ -443,6 +451,8 @@ void drm_mode_config_init(struct drm_device *dev) drm_modeset_acquire_fini(&modeset_ctx); dma_resv_fini(&resv); } + + return drmm_add_action(dev, drm_mode_config_init_release, NULL); } EXPORT_SYMBOL(drm_mode_config_init); diff --git a/include/drm/drm_mode_config.h b/include/drm/drm_mode_config.h index 3bcbe30339f0..160a3e4b51c3 100644 --- a/include/drm/drm_mode_config.h +++ b/include/drm/drm_mode_config.h @@ -929,7 +929,7 @@ struct drm_mode_config { const struct drm_mode_config_helper_funcs *helper_private; }; -void drm_mode_config_init(struct drm_device *dev); +int drm_mode_config_init(struct drm_device *dev); void drm_mode_config_reset(struct drm_device *dev); void drm_mode_config_cleanup(struct drm_device *dev); -- 2.24.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
WARNING: multiple messages have this Message-ID (diff)
From: Daniel Vetter <daniel.vetter@ffwll.ch> To: DRI Development <dri-devel@lists.freedesktop.org> Cc: "Daniel Vetter" <daniel.vetter@ffwll.ch>, "Intel Graphics Development" <intel-gfx@lists.freedesktop.org>, "Noralf Trønnes" <noralf@tronnes.org>, "Thomas Zimmermann" <tzimmermann@suse.de>, "Daniel Vetter" <daniel.vetter@intel.com>, "Sam Ravnborg" <sam@ravnborg.org> Subject: [Intel-gfx] [PATCH 27/52] drm: Manage drm_mode_config_init with drmm_ Date: Wed, 19 Feb 2020 11:20:57 +0100 [thread overview] Message-ID: <20200219102122.1607365-28-daniel.vetter@ffwll.ch> (raw) In-Reply-To: <20200219102122.1607365-1-daniel.vetter@ffwll.ch> drm_mode_config_cleanup is idempotent, so no harm in calling this twice. This allows us to gradually switch drivers over by removing explicit drm_mode_config_cleanup calls. With this step it's not also possible that (at least for simple drivers) automatic resource cleanup can be done correctly without a drm_driver->release hook. Therefore allow this now in devm_drm_dev_init(). Also with drmm_ explicit drm_driver->release hooks are kinda not the best option, so deprecate that hook to discourage future users. v2: Fixup the example in the kerneldoc too. Cc: "Noralf Trønnes" <noralf@tronnes.org> Cc: Sam Ravnborg <sam@ravnborg.org> Cc: Thomas Zimmermann <tzimmermann@suse.de> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> --- drivers/gpu/drm/drm_drv.c | 21 +++++---------------- drivers/gpu/drm/drm_mode_config.c | 12 +++++++++++- include/drm/drm_mode_config.h | 2 +- 3 files changed, 17 insertions(+), 18 deletions(-) diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c index 3cf40864d4a6..428c569aaaf1 100644 --- a/drivers/gpu/drm/drm_drv.c +++ b/drivers/gpu/drm/drm_drv.c @@ -267,8 +267,7 @@ void drm_minor_release(struct drm_minor *minor) * * The following example shows a typical structure of a DRM display driver. * The example focus on the probe() function and the other functions that is - * almost always present and serves as a demonstration of devm_drm_dev_init() - * usage with its accompanying drm_driver->release callback. + * almost always present and serves as a demonstration of devm_drm_dev_init(). * * .. code-block:: c * @@ -278,16 +277,8 @@ void drm_minor_release(struct drm_minor *minor) * struct clk *pclk; * }; * - * static void driver_drm_release(struct drm_device *drm) - * { - * struct driver_device *priv = container_of(...); - * - * drm_mode_config_cleanup(drm); - * } - * * static struct drm_driver driver_drm_driver = { * [...] - * .release = driver_drm_release, * }; * * static int driver_probe(struct platform_device *pdev) @@ -312,7 +303,9 @@ void drm_minor_release(struct drm_minor *minor) * } * drmm_add_final_kfree(drm, priv); * - * drm_mode_config_init(drm); + * ret = drm_mode_config_init(drm); + * if (ret) + * return ret; * * priv->userspace_facing = drmm_kzalloc(..., GFP_KERNEL); * if (!priv->userspace_facing) @@ -710,8 +703,7 @@ static void devm_drm_dev_init_release(void *data) * @driver: DRM driver * * Managed drm_dev_init(). The DRM device initialized with this function is - * automatically put on driver detach using drm_dev_put(). You must supply a - * &drm_driver.release callback to control the finalization explicitly. + * automatically put on driver detach using drm_dev_put(). * * RETURNS: * 0 on success, or error code on failure. @@ -722,9 +714,6 @@ int devm_drm_dev_init(struct device *parent, { int ret; - if (WARN_ON(!driver->release)) - return -EINVAL; - ret = drm_dev_init(dev, driver, parent); if (ret) return ret; diff --git a/drivers/gpu/drm/drm_mode_config.c b/drivers/gpu/drm/drm_mode_config.c index 08e6eff6a179..957db1edba0c 100644 --- a/drivers/gpu/drm/drm_mode_config.c +++ b/drivers/gpu/drm/drm_mode_config.c @@ -25,6 +25,7 @@ #include <drm/drm_drv.h> #include <drm/drm_encoder.h> #include <drm/drm_file.h> +#include <drm/drm_managed.h> #include <drm/drm_mode_config.h> #include <drm/drm_print.h> #include <linux/dma-resv.h> @@ -373,6 +374,11 @@ static int drm_mode_create_standard_properties(struct drm_device *dev) return 0; } +static void drm_mode_config_init_release(struct drm_device *dev, void *ptr) +{ + drm_mode_config_cleanup(dev); +} + /** * drm_mode_config_init - initialize DRM mode_configuration structure * @dev: DRM device @@ -384,8 +390,10 @@ static int drm_mode_create_standard_properties(struct drm_device *dev) * problem, since this should happen single threaded at init time. It is the * driver's problem to ensure this guarantee. * + * Cleanup is automatically handled through registering drm_mode_config_cleanup + * with drmm_add_action(). */ -void drm_mode_config_init(struct drm_device *dev) +int drm_mode_config_init(struct drm_device *dev) { mutex_init(&dev->mode_config.mutex); drm_modeset_lock_init(&dev->mode_config.connection_mutex); @@ -443,6 +451,8 @@ void drm_mode_config_init(struct drm_device *dev) drm_modeset_acquire_fini(&modeset_ctx); dma_resv_fini(&resv); } + + return drmm_add_action(dev, drm_mode_config_init_release, NULL); } EXPORT_SYMBOL(drm_mode_config_init); diff --git a/include/drm/drm_mode_config.h b/include/drm/drm_mode_config.h index 3bcbe30339f0..160a3e4b51c3 100644 --- a/include/drm/drm_mode_config.h +++ b/include/drm/drm_mode_config.h @@ -929,7 +929,7 @@ struct drm_mode_config { const struct drm_mode_config_helper_funcs *helper_private; }; -void drm_mode_config_init(struct drm_device *dev); +int drm_mode_config_init(struct drm_device *dev); void drm_mode_config_reset(struct drm_device *dev); void drm_mode_config_cleanup(struct drm_device *dev); -- 2.24.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2020-02-19 10:22 UTC|newest] Thread overview: 310+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-02-19 10:20 [PATCH 00/52] drm_device managed resources Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 01/52] mm/sl[uo]b: export __kmalloc_track(_node)_caller Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` Daniel Vetter 2020-02-19 19:42 ` Andrew Morton 2020-02-19 19:42 ` [Intel-gfx] " Andrew Morton 2020-02-19 19:42 ` Andrew Morton 2020-02-22 3:42 ` Christopher Lameter 2020-02-22 3:42 ` [Intel-gfx] " Christopher Lameter 2020-02-22 3:42 ` Christopher Lameter 2020-02-19 10:20 ` [PATCH 02/52] drm/i915: Don't clear drvdata in ->release Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 03/52] drm: add managed resources tied to drm_device Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 12:31 ` Neil Armstrong 2020-02-19 12:31 ` [Intel-gfx] " Neil Armstrong 2020-02-19 13:24 ` Daniel Vetter 2020-02-19 13:24 ` [Intel-gfx] " Daniel Vetter 2020-02-19 13:28 ` Laurent Pinchart 2020-02-19 13:28 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 13:33 ` Greg Kroah-Hartman 2020-02-19 13:33 ` [Intel-gfx] " Greg Kroah-Hartman 2020-02-19 14:22 ` Daniel Vetter 2020-02-19 14:22 ` [Intel-gfx] " Daniel Vetter 2020-02-19 16:09 ` Emil Velikov 2020-02-19 16:09 ` Emil Velikov 2020-02-19 16:22 ` Daniel Vetter 2020-02-19 16:22 ` Daniel Vetter 2020-02-19 16:41 ` Emil Velikov 2020-02-19 16:41 ` Emil Velikov 2020-02-19 16:46 ` Laurent Pinchart 2020-02-19 16:46 ` Laurent Pinchart 2020-02-19 16:53 ` Daniel Vetter 2020-02-19 16:53 ` Daniel Vetter 2020-02-19 17:02 ` Laurent Pinchart 2020-02-19 17:02 ` Laurent Pinchart 2020-02-19 17:06 ` Daniel Vetter 2020-02-19 17:06 ` Daniel Vetter 2020-02-19 17:00 ` Greg Kroah-Hartman 2020-02-19 17:00 ` [Intel-gfx] " Greg Kroah-Hartman 2020-02-19 17:36 ` Laurent Pinchart 2020-02-19 17:36 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 18:19 ` Greg Kroah-Hartman 2020-02-19 18:19 ` [Intel-gfx] " Greg Kroah-Hartman 2020-02-19 19:57 ` Daniel Vetter 2020-02-19 19:57 ` [Intel-gfx] " Daniel Vetter 2020-02-20 14:58 ` Laurent Pinchart 2020-02-20 14:58 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 13:57 ` Daniel Vetter 2020-02-19 13:57 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 04/52] drm: Set final_kfree in drm_dev_alloc Daniel Vetter 2020-02-19 10:20 ` [Xen-devel] " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 12:03 ` Oleksandr Andrushchenko 2020-02-19 12:03 ` [Xen-devel] " Oleksandr Andrushchenko 2020-02-19 12:03 ` [Intel-gfx] " Oleksandr Andrushchenko 2020-02-19 13:39 ` Laurent Pinchart 2020-02-19 13:39 ` [Xen-devel] " Laurent Pinchart 2020-02-19 13:39 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 14:41 ` Daniel Vetter 2020-02-19 14:41 ` [Xen-devel] " Daniel Vetter 2020-02-19 14:41 ` [Intel-gfx] " Daniel Vetter 2020-02-21 19:07 ` Daniel Vetter 2020-02-21 19:07 ` [Xen-devel] " Daniel Vetter 2020-02-21 19:07 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 05/52] drm/mipi_dbi: Use drmm_add_final_kfree in all drivers Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 11:47 ` Thomas Zimmermann 2020-02-19 11:47 ` [Intel-gfx] " Thomas Zimmermann 2020-02-19 12:45 ` Thomas Zimmermann 2020-02-19 12:45 ` [Intel-gfx] " Thomas Zimmermann 2020-02-19 13:23 ` Daniel Vetter 2020-02-19 13:23 ` [Intel-gfx] " Daniel Vetter 2020-02-19 13:29 ` Thomas Zimmermann 2020-02-19 13:29 ` [Intel-gfx] " Thomas Zimmermann 2020-02-19 14:32 ` Daniel Vetter 2020-02-19 14:32 ` [Intel-gfx] " Daniel Vetter 2020-02-20 16:18 ` Noralf Trønnes 2020-02-20 16:18 ` [Intel-gfx] " Noralf Trønnes 2020-02-19 10:20 ` [PATCH 06/52] drm/udl: Use drmm_add_final_kfree Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 07/52] " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 13:42 ` Laurent Pinchart 2020-02-19 13:42 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 14:43 ` Daniel Vetter 2020-02-19 14:43 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 08/52] drm/qxl: " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` Daniel Vetter 2020-02-19 10:20 ` [PATCH 09/52] drm/i915: " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 10/52] drm/cirrus: " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` Daniel Vetter 2020-02-24 8:13 ` Gerd Hoffmann 2020-02-24 8:13 ` [Intel-gfx] " Gerd Hoffmann 2020-02-24 8:13 ` Gerd Hoffmann 2020-02-19 10:20 ` [PATCH 11/52] drm/v3d: " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 18:43 ` Eric Anholt 2020-02-19 18:43 ` [Intel-gfx] " Eric Anholt 2020-02-19 10:20 ` [PATCH 12/52] drm/tidss: " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 13/52] drm/mcde: " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 16:12 ` Linus Walleij 2020-02-19 16:12 ` [Intel-gfx] " Linus Walleij 2020-02-19 10:20 ` [PATCH 14/52] drm/vgem: " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 15/52] drm/vkms: " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 16/52] drm/repaper: " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-20 16:18 ` Noralf Trønnes 2020-02-20 16:18 ` [Intel-gfx] " Noralf Trønnes 2020-02-19 10:20 ` [PATCH 17/52] drm/inigenic: " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 18/52] drm/gm12u320: " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 19/52] drm/<drivers>: " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 14:11 ` Laurent Pinchart 2020-02-19 14:11 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 14:30 ` Daniel Vetter 2020-02-19 14:30 ` [Intel-gfx] " Daniel Vetter 2020-02-19 14:39 ` Laurent Pinchart 2020-02-19 14:39 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 15:29 ` Daniel Vetter 2020-02-19 15:29 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 20/52] drm: Cleanups after drmm_add_final_kfree rollout Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 21/52] drm: Handle dev->unique with drmm_ Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 14:28 ` Laurent Pinchart 2020-02-19 14:28 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 15:24 ` Daniel Vetter 2020-02-19 15:24 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 22/52] drm: Use drmm_ for drm_dev_init cleanup Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 14:35 ` Laurent Pinchart 2020-02-19 14:35 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 15:27 ` Daniel Vetter 2020-02-19 15:27 ` [Intel-gfx] " Daniel Vetter 2020-02-19 15:37 ` Laurent Pinchart 2020-02-19 15:37 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 15:44 ` Daniel Vetter 2020-02-19 15:44 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 23/52] drm: manage drm_minor cleanup with drmm_ Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 14:47 ` Laurent Pinchart 2020-02-19 14:47 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 15:34 ` Daniel Vetter 2020-02-19 15:34 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 24/52] drm: Manage drm_gem_init " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 14:22 ` Laurent Pinchart 2020-02-19 14:22 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 14:37 ` Daniel Vetter 2020-02-19 14:37 ` [Intel-gfx] " Daniel Vetter 2020-02-19 14:52 ` Laurent Pinchart 2020-02-19 14:52 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 14:56 ` Daniel Vetter 2020-02-19 14:56 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 25/52] drm: Manage drm_vblank_cleanup " Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 26/52] drm: Garbage collect drm_dev_fini Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` Daniel Vetter [this message] 2020-02-19 10:20 ` [Intel-gfx] [PATCH 27/52] drm: Manage drm_mode_config_init with drmm_ Daniel Vetter 2020-02-19 13:49 ` Laurent Pinchart 2020-02-19 13:49 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 15:47 ` Daniel Vetter 2020-02-19 15:47 ` [Intel-gfx] " Daniel Vetter 2020-02-19 16:07 ` Laurent Pinchart 2020-02-19 16:07 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 16:23 ` Daniel Vetter 2020-02-19 16:23 ` [Intel-gfx] " Daniel Vetter 2020-02-19 17:30 ` Noralf Trønnes 2020-02-19 17:30 ` [Intel-gfx] " Noralf Trønnes 2020-02-19 18:12 ` Daniel Vetter 2020-02-19 18:12 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` [PATCH 28/52] drm/bochs: Remove leftover drm_atomic_helper_shutdown Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-24 8:14 ` Gerd Hoffmann 2020-02-24 8:14 ` [Intel-gfx] " Gerd Hoffmann 2020-02-19 10:20 ` [PATCH 29/52] drm/bochs: Drop explicit drm_mode_config_cleanup Daniel Vetter 2020-02-19 10:20 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:20 ` Daniel Vetter 2020-02-24 8:15 ` Gerd Hoffmann 2020-02-24 8:15 ` [Intel-gfx] " Gerd Hoffmann 2020-02-24 8:15 ` Gerd Hoffmann 2020-02-19 10:21 ` [PATCH 30/52] drm/cirrus: Drop explicit drm_mode_config_cleanup call Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` Daniel Vetter 2020-02-24 8:16 ` Gerd Hoffmann 2020-02-24 8:16 ` [Intel-gfx] " Gerd Hoffmann 2020-02-24 8:16 ` Gerd Hoffmann 2020-02-19 10:21 ` [PATCH 31/52] drm/cirrus: Fully embrace devm_ Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` Daniel Vetter 2020-02-24 8:18 ` Gerd Hoffmann 2020-02-24 8:18 ` [Intel-gfx] " Gerd Hoffmann 2020-02-24 8:18 ` Gerd Hoffmann 2020-02-19 10:21 ` [PATCH 32/52] drm/ingenic: Drop explicit drm_mode_config_cleanup call Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` [PATCH 33/52] drm/mcde: " Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 16:12 ` Linus Walleij 2020-02-19 16:12 ` [Intel-gfx] " Linus Walleij 2020-02-19 10:21 ` [PATCH 34/52] drm/mcde: More devm_drm_dev_init Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 16:13 ` Linus Walleij 2020-02-19 16:13 ` [Intel-gfx] " Linus Walleij 2020-02-19 10:21 ` [PATCH 35/52] drm/meson: Drop explicit drm_mode_config_cleanup call Daniel Vetter 2020-02-19 10:21 ` Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` Daniel Vetter 2020-02-19 10:39 ` Neil Armstrong 2020-02-19 10:39 ` Neil Armstrong 2020-02-19 10:39 ` [Intel-gfx] " Neil Armstrong 2020-02-19 10:39 ` Neil Armstrong 2020-02-19 10:21 ` [PATCH 36/52] drm/pl111: " Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` [PATCH 37/52] drm/rcar-du: " Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` Daniel Vetter 2020-02-19 10:30 ` Geert Uytterhoeven 2020-02-19 10:30 ` [Intel-gfx] " Geert Uytterhoeven 2020-02-19 10:30 ` Geert Uytterhoeven 2020-02-19 10:56 ` Daniel Vetter 2020-02-19 10:56 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:56 ` Daniel Vetter 2020-02-19 11:10 ` Geert Uytterhoeven 2020-02-19 11:10 ` [Intel-gfx] " Geert Uytterhoeven 2020-02-19 11:10 ` Geert Uytterhoeven 2020-02-19 12:17 ` Laurent Pinchart 2020-02-19 12:17 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 12:17 ` Laurent Pinchart 2020-02-19 12:40 ` Daniel Vetter 2020-02-19 12:40 ` [Intel-gfx] " Daniel Vetter 2020-02-19 12:40 ` Daniel Vetter 2020-02-19 13:53 ` Laurent Pinchart 2020-02-19 13:53 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 13:53 ` Laurent Pinchart 2020-02-19 14:29 ` Daniel Vetter 2020-02-19 14:29 ` [Intel-gfx] " Daniel Vetter 2020-02-19 14:29 ` Daniel Vetter [not found] ` <20200219102122.1607365-1-daniel.vetter-/w4YWyX8dFk@public.gmane.org> 2020-02-19 10:21 ` [PATCH 38/52] drm/rockchip: " Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` Daniel Vetter 2020-02-19 10:21 ` Daniel Vetter 2020-02-19 10:21 ` [PATCH 39/52] drm/stm: " Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` Daniel Vetter 2020-02-20 14:18 ` Philippe CORNU 2020-02-20 14:18 ` [Intel-gfx] " Philippe CORNU 2020-02-20 14:18 ` Philippe CORNU 2020-02-20 16:12 ` Daniel Vetter 2020-02-20 16:12 ` [Intel-gfx] " Daniel Vetter 2020-02-20 16:12 ` Daniel Vetter 2020-02-19 10:21 ` [PATCH 40/52] drm/shmob: " Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` Daniel Vetter 2020-02-19 13:57 ` Laurent Pinchart 2020-02-19 13:57 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 13:57 ` Laurent Pinchart 2020-02-19 10:21 ` [PATCH 41/52] drm/mtk: " Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` [PATCH 42/52] drm/tidss: " Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` [PATCH 43/52] drm/gm12u320: More drmm_ Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` [PATCH 44/52] drm/gm12u320: Use devm_drm_dev_init Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` [PATCH 45/52] drm/gm12u320: Use helpers for shutdown/suspend/resume Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` [PATCH 46/52] drm/gm12u320: Simplify upload work Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` [PATCH 47/52] drm/repaper: Drop explicit drm_mode_config_cleanup call Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-20 16:21 ` Noralf Trønnes 2020-02-20 16:21 ` [Intel-gfx] " Noralf Trønnes 2020-02-19 10:21 ` [PATCH 48/52] drm/mipi-dbi: Move drm_mode_config_init into mipi library Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-20 16:22 ` Noralf Trønnes 2020-02-20 16:22 ` [Intel-gfx] " Noralf Trønnes 2020-02-19 10:21 ` [PATCH 49/52] drm/mipi-dbi: Drop explicit drm_mode_config_cleanup call Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-20 16:22 ` Noralf Trønnes 2020-02-20 16:22 ` [Intel-gfx] " Noralf Trønnes 2020-02-19 10:21 ` [PATCH 50/52] drm/udl: " Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` [PATCH 51/52] drm/udl: drop drm_driver.release hook Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 10:21 ` [PATCH 52/52] drm: Add docs for managed resources Daniel Vetter 2020-02-19 10:21 ` [Intel-gfx] " Daniel Vetter 2020-02-19 15:08 ` Laurent Pinchart 2020-02-19 15:08 ` [Intel-gfx] " Laurent Pinchart 2020-02-19 15:40 ` Daniel Vetter 2020-02-19 15:40 ` [Intel-gfx] " Daniel Vetter 2020-02-21 20:23 ` Sam Ravnborg 2020-02-21 20:23 ` [Intel-gfx] " Sam Ravnborg 2020-02-21 21:13 ` Sam Ravnborg 2020-02-21 21:13 ` [Intel-gfx] " Sam Ravnborg 2020-02-19 11:30 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm_device " Patchwork 2020-02-19 11:32 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork 2020-02-19 11:54 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2020-02-21 1:37 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20200219102122.1607365-28-daniel.vetter@ffwll.ch \ --to=daniel.vetter@ffwll.ch \ --cc=daniel.vetter@intel.com \ --cc=dri-devel@lists.freedesktop.org \ --cc=intel-gfx@lists.freedesktop.org \ --cc=sam@ravnborg.org \ --cc=tzimmermann@suse.de \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.