[PATCH 3/5] drm: Move client-device functions in to drm_client_dev.c
Jocelyn Falempe
jfalempe at redhat.com
Mon Sep 30 11:49:12 UTC 2024
On 27/09/2024 16:37, Thomas Zimmermann wrote:
> A number of DRM-client functions serve as entry points from device
> operations to client code. Move them info a separate file, so that
> the other client functions can be moved into a different module.
Thanks, I'm not sure I can review this, but it looks good to me.
Reviewed-by: Jocelyn Falempe <jfalempe at redhat.com>
--
Jocelyn
>
> Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de>
> ---
> Documentation/gpu/drm-client.rst | 3 +
> drivers/gpu/drm/Makefile | 1 +
> drivers/gpu/drm/drm_client.c | 121 ---------------------------
> drivers/gpu/drm/drm_client_dev.c | 138 +++++++++++++++++++++++++++++++
> 4 files changed, 142 insertions(+), 121 deletions(-)
> create mode 100644 drivers/gpu/drm/drm_client_dev.c
>
> diff --git a/Documentation/gpu/drm-client.rst b/Documentation/gpu/drm-client.rst
> index 58b5a1d1219d..6d8142f159a1 100644
> --- a/Documentation/gpu/drm-client.rst
> +++ b/Documentation/gpu/drm-client.rst
> @@ -13,3 +13,6 @@ Kernel clients
>
> .. kernel-doc:: drivers/gpu/drm/drm_client_modeset.c
> :export:
> +
> +.. kernel-doc:: drivers/gpu/drm/drm_client_dev.c
> + :export:
> diff --git a/drivers/gpu/drm/Makefile b/drivers/gpu/drm/Makefile
> index 3894f43f6d47..c50443756457 100644
> --- a/drivers/gpu/drm/Makefile
> +++ b/drivers/gpu/drm/Makefile
> @@ -42,6 +42,7 @@ drm-y := \
> drm_bridge.o \
> drm_cache.o \
> drm_client.o \
> + drm_client_dev.o \
> drm_client_modeset.o \
> drm_color_mgmt.o \
> drm_connector.o \
> diff --git a/drivers/gpu/drm/drm_client.c b/drivers/gpu/drm/drm_client.c
> index bfedcbf516db..549b28a5918c 100644
> --- a/drivers/gpu/drm/drm_client.c
> +++ b/drivers/gpu/drm/drm_client.c
> @@ -10,7 +10,6 @@
> #include <linux/slab.h>
>
> #include <drm/drm_client.h>
> -#include <drm/drm_debugfs.h>
> #include <drm/drm_device.h>
> #include <drm/drm_drv.h>
> #include <drm/drm_file.h>
> @@ -172,99 +171,6 @@ void drm_client_release(struct drm_client_dev *client)
> }
> EXPORT_SYMBOL(drm_client_release);
>
> -/**
> - * drm_client_dev_unregister - Unregister clients
> - * @dev: DRM device
> - *
> - * This function releases all clients by calling each client's
> - * &drm_client_funcs.unregister callback. The callback function
> - * is responsibe for releaseing all resources including the client
> - * itself.
> - *
> - * The helper drm_dev_unregister() calls this function. Drivers
> - * that use it don't need to call this function themselves.
> - */
> -void drm_client_dev_unregister(struct drm_device *dev)
> -{
> - struct drm_client_dev *client, *tmp;
> -
> - if (!drm_core_check_feature(dev, DRIVER_MODESET))
> - return;
> -
> - mutex_lock(&dev->clientlist_mutex);
> - list_for_each_entry_safe(client, tmp, &dev->clientlist, list) {
> - list_del(&client->list);
> - if (client->funcs && client->funcs->unregister) {
> - client->funcs->unregister(client);
> - } else {
> - drm_client_release(client);
> - kfree(client);
> - }
> - }
> - mutex_unlock(&dev->clientlist_mutex);
> -}
> -EXPORT_SYMBOL(drm_client_dev_unregister);
> -
> -/**
> - * drm_client_dev_hotplug - Send hotplug event to clients
> - * @dev: DRM device
> - *
> - * This function calls the &drm_client_funcs.hotplug callback on the attached clients.
> - *
> - * drm_kms_helper_hotplug_event() calls this function, so drivers that use it
> - * don't need to call this function themselves.
> - */
> -void drm_client_dev_hotplug(struct drm_device *dev)
> -{
> - struct drm_client_dev *client;
> - int ret;
> -
> - if (!drm_core_check_feature(dev, DRIVER_MODESET))
> - return;
> -
> - if (!dev->mode_config.num_connector) {
> - drm_dbg_kms(dev, "No connectors found, will not send hotplug events!\n");
> - return;
> - }
> -
> - mutex_lock(&dev->clientlist_mutex);
> - list_for_each_entry(client, &dev->clientlist, list) {
> - if (!client->funcs || !client->funcs->hotplug)
> - continue;
> -
> - if (client->hotplug_failed)
> - continue;
> -
> - ret = client->funcs->hotplug(client);
> - drm_dbg_kms(dev, "%s: ret=%d\n", client->name, ret);
> - if (ret)
> - client->hotplug_failed = true;
> - }
> - mutex_unlock(&dev->clientlist_mutex);
> -}
> -EXPORT_SYMBOL(drm_client_dev_hotplug);
> -
> -void drm_client_dev_restore(struct drm_device *dev)
> -{
> - struct drm_client_dev *client;
> - int ret;
> -
> - if (!drm_core_check_feature(dev, DRIVER_MODESET))
> - return;
> -
> - mutex_lock(&dev->clientlist_mutex);
> - list_for_each_entry(client, &dev->clientlist, list) {
> - if (!client->funcs || !client->funcs->restore)
> - continue;
> -
> - ret = client->funcs->restore(client);
> - drm_dbg_kms(dev, "%s: ret=%d\n", client->name, ret);
> - if (!ret) /* The first one to return zero gets the privilege to restore */
> - break;
> - }
> - mutex_unlock(&dev->clientlist_mutex);
> -}
> -
> static void drm_client_buffer_delete(struct drm_client_buffer *buffer)
> {
> if (buffer->gem) {
> @@ -584,30 +490,3 @@ int drm_client_framebuffer_flush(struct drm_client_buffer *buffer, struct drm_re
> 0, 0, NULL, 0);
> }
> EXPORT_SYMBOL(drm_client_framebuffer_flush);
> -
> -#ifdef CONFIG_DEBUG_FS
> -static int drm_client_debugfs_internal_clients(struct seq_file *m, void *data)
> -{
> - struct drm_debugfs_entry *entry = m->private;
> - struct drm_device *dev = entry->dev;
> - struct drm_printer p = drm_seq_file_printer(m);
> - struct drm_client_dev *client;
> -
> - mutex_lock(&dev->clientlist_mutex);
> - list_for_each_entry(client, &dev->clientlist, list)
> - drm_printf(&p, "%s\n", client->name);
> - mutex_unlock(&dev->clientlist_mutex);
> -
> - return 0;
> -}
> -
> -static const struct drm_debugfs_info drm_client_debugfs_list[] = {
> - { "internal_clients", drm_client_debugfs_internal_clients, 0 },
> -};
> -
> -void drm_client_debugfs_init(struct drm_device *dev)
> -{
> - drm_debugfs_add_files(dev, drm_client_debugfs_list,
> - ARRAY_SIZE(drm_client_debugfs_list));
> -}
> -#endif
> diff --git a/drivers/gpu/drm/drm_client_dev.c b/drivers/gpu/drm/drm_client_dev.c
> new file mode 100644
> index 000000000000..3e41fd1f0771
> --- /dev/null
> +++ b/drivers/gpu/drm/drm_client_dev.c
> @@ -0,0 +1,138 @@
> +// SPDX-License-Identifier: GPL-2.0 or MIT
> +/*
> + * Copyright 2018 Noralf Trønnes
> + */
> +
> +#include <linux/list.h>
> +#include <linux/mutex.h>
> +#include <linux/seq_file.h>
> +
> +#include <drm/drm_client.h>
> +#include <drm/drm_debugfs.h>
> +#include <drm/drm_device.h>
> +#include <drm/drm_drv.h>
> +#include <drm/drm_print.h>
> +
> +/**
> + * DOC: overview
> + *
> + * This library provides support for clients running in the kernel like fbdev and bootsplash.
> + *
> + * GEM drivers which provide a GEM based dumb buffer with a virtual address are supported.
> + */
> +
> +/**
> + * drm_client_dev_unregister - Unregister clients
> + * @dev: DRM device
> + *
> + * This function releases all clients by calling each client's
> + * &drm_client_funcs.unregister callback. The callback function
> + * is responsibe for releaseing all resources including the client
> + * itself.
> + *
> + * The helper drm_dev_unregister() calls this function. Drivers
> + * that use it don't need to call this function themselves.
> + */
> +void drm_client_dev_unregister(struct drm_device *dev)
> +{
> + struct drm_client_dev *client, *tmp;
> +
> + if (!drm_core_check_feature(dev, DRIVER_MODESET))
> + return;
> +
> + mutex_lock(&dev->clientlist_mutex);
> + list_for_each_entry_safe(client, tmp, &dev->clientlist, list) {
> + list_del(&client->list);
> + if (client->funcs && !drm_WARN_ON(dev, !client->funcs->unregister))
> + client->funcs->unregister(client);
> + }
> + mutex_unlock(&dev->clientlist_mutex);
> +}
> +EXPORT_SYMBOL(drm_client_dev_unregister);
> +
> +/**
> + * drm_client_dev_hotplug - Send hotplug event to clients
> + * @dev: DRM device
> + *
> + * This function calls the &drm_client_funcs.hotplug callback on the attached clients.
> + *
> + * drm_kms_helper_hotplug_event() calls this function, so drivers that use it
> + * don't need to call this function themselves.
> + */
> +void drm_client_dev_hotplug(struct drm_device *dev)
> +{
> + struct drm_client_dev *client;
> + int ret;
> +
> + if (!drm_core_check_feature(dev, DRIVER_MODESET))
> + return;
> +
> + if (!dev->mode_config.num_connector) {
> + drm_dbg_kms(dev, "No connectors found, will not send hotplug events!\n");
> + return;
> + }
> +
> + mutex_lock(&dev->clientlist_mutex);
> + list_for_each_entry(client, &dev->clientlist, list) {
> + if (!client->funcs || !client->funcs->hotplug)
> + continue;
> +
> + if (client->hotplug_failed)
> + continue;
> +
> + ret = client->funcs->hotplug(client);
> + drm_dbg_kms(dev, "%s: ret=%d\n", client->name, ret);
> + if (ret)
> + client->hotplug_failed = true;
> + }
> + mutex_unlock(&dev->clientlist_mutex);
> +}
> +EXPORT_SYMBOL(drm_client_dev_hotplug);
> +
> +void drm_client_dev_restore(struct drm_device *dev)
> +{
> + struct drm_client_dev *client;
> + int ret;
> +
> + if (!drm_core_check_feature(dev, DRIVER_MODESET))
> + return;
> +
> + mutex_lock(&dev->clientlist_mutex);
> + list_for_each_entry(client, &dev->clientlist, list) {
> + if (!client->funcs || !client->funcs->restore)
> + continue;
> +
> + ret = client->funcs->restore(client);
> + drm_dbg_kms(dev, "%s: ret=%d\n", client->name, ret);
> + if (!ret) /* The first one to return zero gets the privilege to restore */
> + break;
> + }
> + mutex_unlock(&dev->clientlist_mutex);
> +}
> +
> +#ifdef CONFIG_DEBUG_FS
> +static int drm_client_debugfs_internal_clients(struct seq_file *m, void *data)
> +{
> + struct drm_debugfs_entry *entry = m->private;
> + struct drm_device *dev = entry->dev;
> + struct drm_printer p = drm_seq_file_printer(m);
> + struct drm_client_dev *client;
> +
> + mutex_lock(&dev->clientlist_mutex);
> + list_for_each_entry(client, &dev->clientlist, list)
> + drm_printf(&p, "%s\n", client->name);
> + mutex_unlock(&dev->clientlist_mutex);
> +
> + return 0;
> +}
> +
> +static const struct drm_debugfs_info drm_client_debugfs_list[] = {
> + { "internal_clients", drm_client_debugfs_internal_clients, 0 },
> +};
> +
> +void drm_client_debugfs_init(struct drm_device *dev)
> +{
> + drm_debugfs_add_files(dev, drm_client_debugfs_list,
> + ARRAY_SIZE(drm_client_debugfs_list));
> +}
> +#endif
More information about the amd-gfx
mailing list