2023-05-26 04:10:56

by Su Hui

[permalink] [raw]
Subject: [PATCH] drm: Remove unnecessary (void*) conversions

Pointer variables of (void*) type do not require type cast.

Signed-off-by: Su Hui <[email protected]>
---
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c | 2 +-
drivers/gpu/drm/amd/pm/amdgpu_pm.c | 2 +-
drivers/gpu/drm/etnaviv/etnaviv_drv.c | 4 ++--
drivers/gpu/drm/nouveau/nouveau_debugfs.c | 2 +-
drivers/gpu/drm/omapdrm/omap_debugfs.c | 6 +++---
drivers/gpu/drm/pl111/pl111_debugfs.c | 2 +-
drivers/gpu/drm/qxl/qxl_debugfs.c | 4 ++--
drivers/gpu/drm/tiny/arcpgu.c | 2 +-
drivers/gpu/drm/ttm/ttm_resource.c | 3 +--
drivers/gpu/drm/virtio/virtgpu_debugfs.c | 6 +++---
drivers/gpu/drm/vmwgfx/ttm_object.c | 5 ++---
drivers/gpu/drm/vmwgfx/vmwgfx_gem.c | 2 +-
12 files changed, 19 insertions(+), 21 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
index 827fcb4fb3b3..8a2c39927167 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
@@ -3312,7 +3312,7 @@ static ssize_t dtn_log_write(

static int mst_topo_show(struct seq_file *m, void *unused)
{
- struct amdgpu_device *adev = (struct amdgpu_device *)m->private;
+ struct amdgpu_device *adev = m->private;
struct drm_device *dev = adev_to_drm(adev);
struct drm_connector *connector;
struct drm_connector_list_iter conn_iter;
diff --git a/drivers/gpu/drm/amd/pm/amdgpu_pm.c b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
index 58c2246918fd..e6c870bd307b 100644
--- a/drivers/gpu/drm/amd/pm/amdgpu_pm.c
+++ b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
@@ -3671,7 +3671,7 @@ static void amdgpu_parse_cg_state(struct seq_file *m, u64 flags)

static int amdgpu_debugfs_pm_info_show(struct seq_file *m, void *unused)
{
- struct amdgpu_device *adev = (struct amdgpu_device *)m->private;
+ struct amdgpu_device *adev = m->private;
struct drm_device *dev = adev_to_drm(adev);
u64 flags = 0;
int r;
diff --git a/drivers/gpu/drm/etnaviv/etnaviv_drv.c b/drivers/gpu/drm/etnaviv/etnaviv_drv.c
index 31a7f59ccb49..dd57f7164e9a 100644
--- a/drivers/gpu/drm/etnaviv/etnaviv_drv.c
+++ b/drivers/gpu/drm/etnaviv/etnaviv_drv.c
@@ -198,7 +198,7 @@ static int etnaviv_ring_show(struct etnaviv_gpu *gpu, struct seq_file *m)

static int show_unlocked(struct seq_file *m, void *arg)
{
- struct drm_info_node *node = (struct drm_info_node *) m->private;
+ struct drm_info_node *node = m->private;
struct drm_device *dev = node->minor->dev;
int (*show)(struct drm_device *dev, struct seq_file *m) =
node->info_ent->data;
@@ -208,7 +208,7 @@ static int show_unlocked(struct seq_file *m, void *arg)

static int show_each_gpu(struct seq_file *m, void *arg)
{
- struct drm_info_node *node = (struct drm_info_node *) m->private;
+ struct drm_info_node *node = m->private;
struct drm_device *dev = node->minor->dev;
struct etnaviv_drm_private *priv = dev->dev_private;
struct etnaviv_gpu *gpu;
diff --git a/drivers/gpu/drm/nouveau/nouveau_debugfs.c b/drivers/gpu/drm/nouveau/nouveau_debugfs.c
index 2a36d1ca8fda..96b59d5d68ed 100644
--- a/drivers/gpu/drm/nouveau/nouveau_debugfs.c
+++ b/drivers/gpu/drm/nouveau/nouveau_debugfs.c
@@ -37,7 +37,7 @@
static int
nouveau_debugfs_vbios_image(struct seq_file *m, void *data)
{
- struct drm_info_node *node = (struct drm_info_node *) m->private;
+ struct drm_info_node *node = m->private;
struct nouveau_drm *drm = nouveau_drm(node->minor->dev);
int i;

diff --git a/drivers/gpu/drm/omapdrm/omap_debugfs.c b/drivers/gpu/drm/omapdrm/omap_debugfs.c
index a3d470468e5b..a94ce502e152 100644
--- a/drivers/gpu/drm/omapdrm/omap_debugfs.c
+++ b/drivers/gpu/drm/omapdrm/omap_debugfs.c
@@ -19,7 +19,7 @@

static int gem_show(struct seq_file *m, void *arg)
{
- struct drm_info_node *node = (struct drm_info_node *) m->private;
+ struct drm_info_node *node = m->private;
struct drm_device *dev = node->minor->dev;
struct omap_drm_private *priv = dev->dev_private;

@@ -33,7 +33,7 @@ static int gem_show(struct seq_file *m, void *arg)

static int mm_show(struct seq_file *m, void *arg)
{
- struct drm_info_node *node = (struct drm_info_node *) m->private;
+ struct drm_info_node *node = m->private;
struct drm_device *dev = node->minor->dev;
struct drm_printer p = drm_seq_file_printer(m);

@@ -45,7 +45,7 @@ static int mm_show(struct seq_file *m, void *arg)
#ifdef CONFIG_DRM_FBDEV_EMULATION
static int fb_show(struct seq_file *m, void *arg)
{
- struct drm_info_node *node = (struct drm_info_node *) m->private;
+ struct drm_info_node *node = m->private;
struct drm_device *dev = node->minor->dev;
struct drm_fb_helper *helper = dev->fb_helper;
struct drm_framebuffer *fb;
diff --git a/drivers/gpu/drm/pl111/pl111_debugfs.c b/drivers/gpu/drm/pl111/pl111_debugfs.c
index 6744fa16f464..4df03ec5d368 100644
--- a/drivers/gpu/drm/pl111/pl111_debugfs.c
+++ b/drivers/gpu/drm/pl111/pl111_debugfs.c
@@ -32,7 +32,7 @@ static const struct {

static int pl111_debugfs_regs(struct seq_file *m, void *unused)
{
- struct drm_info_node *node = (struct drm_info_node *)m->private;
+ struct drm_info_node *node = m->private;
struct drm_device *dev = node->minor->dev;
struct pl111_drm_dev_private *priv = dev->dev_private;
int i;
diff --git a/drivers/gpu/drm/qxl/qxl_debugfs.c b/drivers/gpu/drm/qxl/qxl_debugfs.c
index 2d9ed3b94574..5b4fe3049529 100644
--- a/drivers/gpu/drm/qxl/qxl_debugfs.c
+++ b/drivers/gpu/drm/qxl/qxl_debugfs.c
@@ -38,7 +38,7 @@
static int
qxl_debugfs_irq_received(struct seq_file *m, void *data)
{
- struct drm_info_node *node = (struct drm_info_node *) m->private;
+ struct drm_info_node *node = m->private;
struct qxl_device *qdev = to_qxl(node->minor->dev);

seq_printf(m, "%d\n", atomic_read(&qdev->irq_received));
@@ -52,7 +52,7 @@ qxl_debugfs_irq_received(struct seq_file *m, void *data)
static int
qxl_debugfs_buffers_info(struct seq_file *m, void *data)
{
- struct drm_info_node *node = (struct drm_info_node *) m->private;
+ struct drm_info_node *node = m->private;
struct qxl_device *qdev = to_qxl(node->minor->dev);
struct qxl_bo *bo;

diff --git a/drivers/gpu/drm/tiny/arcpgu.c b/drivers/gpu/drm/tiny/arcpgu.c
index e5b10e41554a..09f728355aba 100644
--- a/drivers/gpu/drm/tiny/arcpgu.c
+++ b/drivers/gpu/drm/tiny/arcpgu.c
@@ -338,7 +338,7 @@ static int arcpgu_unload(struct drm_device *drm)
#ifdef CONFIG_DEBUG_FS
static int arcpgu_show_pxlclock(struct seq_file *m, void *arg)
{
- struct drm_info_node *node = (struct drm_info_node *)m->private;
+ struct drm_info_node *node = m->private;
struct drm_device *drm = node->minor->dev;
struct arcpgu_drm_private *arcpgu = dev_to_arcpgu(drm);
unsigned long clkrate = clk_get_rate(arcpgu->clk);
diff --git a/drivers/gpu/drm/ttm/ttm_resource.c b/drivers/gpu/drm/ttm/ttm_resource.c
index 7333f7a87a2f..540faabcf8a4 100644
--- a/drivers/gpu/drm/ttm/ttm_resource.c
+++ b/drivers/gpu/drm/ttm/ttm_resource.c
@@ -727,9 +727,8 @@ ttm_kmap_iter_linear_io_fini(struct ttm_kmap_iter_linear_io *iter_io,

static int ttm_resource_manager_show(struct seq_file *m, void *unused)
{
- struct ttm_resource_manager *man =
- (struct ttm_resource_manager *)m->private;
struct drm_printer p = drm_seq_file_printer(m);
+ struct ttm_resource_manager *man = m->private;
ttm_resource_manager_debug(man, &p);
return 0;
}
diff --git a/drivers/gpu/drm/virtio/virtgpu_debugfs.c b/drivers/gpu/drm/virtio/virtgpu_debugfs.c
index 853dd9aa397e..577691af9707 100644
--- a/drivers/gpu/drm/virtio/virtgpu_debugfs.c
+++ b/drivers/gpu/drm/virtio/virtgpu_debugfs.c
@@ -43,7 +43,7 @@ static void virtio_gpu_add_int(struct seq_file *m, const char *name, int value)

static int virtio_gpu_features(struct seq_file *m, void *data)
{
- struct drm_info_node *node = (struct drm_info_node *)m->private;
+ struct drm_info_node *node = m->private;
struct virtio_gpu_device *vgdev = node->minor->dev->dev_private;

virtio_gpu_add_bool(m, "virgl", vgdev->has_virgl_3d);
@@ -68,7 +68,7 @@ static int virtio_gpu_features(struct seq_file *m, void *data)
static int
virtio_gpu_debugfs_irq_info(struct seq_file *m, void *data)
{
- struct drm_info_node *node = (struct drm_info_node *) m->private;
+ struct drm_info_node *node = m->private;
struct virtio_gpu_device *vgdev = node->minor->dev->dev_private;

seq_printf(m, "fence %llu %lld\n",
@@ -80,7 +80,7 @@ virtio_gpu_debugfs_irq_info(struct seq_file *m, void *data)
static int
virtio_gpu_debugfs_host_visible_mm(struct seq_file *m, void *data)
{
- struct drm_info_node *node = (struct drm_info_node *)m->private;
+ struct drm_info_node *node = m->private;
struct virtio_gpu_device *vgdev = node->minor->dev->dev_private;
struct drm_printer p;

diff --git a/drivers/gpu/drm/vmwgfx/ttm_object.c b/drivers/gpu/drm/vmwgfx/ttm_object.c
index ddf8373c1d77..e9e3cc8f5b49 100644
--- a/drivers/gpu/drm/vmwgfx/ttm_object.c
+++ b/drivers/gpu/drm/vmwgfx/ttm_object.c
@@ -513,8 +513,7 @@ static void ttm_prime_refcount_release(struct ttm_base_object **p_base)
*/
static void ttm_prime_dmabuf_release(struct dma_buf *dma_buf)
{
- struct ttm_prime_object *prime =
- (struct ttm_prime_object *) dma_buf->priv;
+ struct ttm_prime_object *prime = dma_buf->priv;
struct ttm_base_object *base = &prime->base;
struct ttm_object_device *tdev = base->tfile->tdev;

@@ -554,7 +553,7 @@ int ttm_prime_fd_to_handle(struct ttm_object_file *tfile,
if (dma_buf->ops != &tdev->ops)
return -ENOSYS;

- prime = (struct ttm_prime_object *) dma_buf->priv;
+ prime = dma_buf->priv;
base = &prime->base;
*handle = base->handle;
ret = ttm_ref_object_add(tfile, base, NULL, false);
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_gem.c b/drivers/gpu/drm/vmwgfx/vmwgfx_gem.c
index c0da89e16e6f..3267a4e61382 100644
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_gem.c
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_gem.c
@@ -220,7 +220,7 @@ static void vmw_bo_print_info(int id, struct vmw_bo *bo, struct seq_file *m)

static int vmw_debugfs_gem_info_show(struct seq_file *m, void *unused)
{
- struct vmw_private *vdev = (struct vmw_private *)m->private;
+ struct vmw_private *vdev = m->private;
struct drm_device *dev = &vdev->drm;
struct drm_file *file;
int r;
--
2.30.2



2023-05-26 07:34:58

by Christian König

[permalink] [raw]
Subject: Re: [PATCH] drm: Remove unnecessary (void*) conversions

Am 26.05.23 um 05:32 schrieb Su Hui:
> Pointer variables of (void*) type do not require type cast.

Please split that up by subsystem/driver. Taking it through the misc
tree might just cause merge conflicts.

Christian.

>
> Signed-off-by: Su Hui <[email protected]>
> ---
> drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c | 2 +-
> drivers/gpu/drm/amd/pm/amdgpu_pm.c | 2 +-
> drivers/gpu/drm/etnaviv/etnaviv_drv.c | 4 ++--
> drivers/gpu/drm/nouveau/nouveau_debugfs.c | 2 +-
> drivers/gpu/drm/omapdrm/omap_debugfs.c | 6 +++---
> drivers/gpu/drm/pl111/pl111_debugfs.c | 2 +-
> drivers/gpu/drm/qxl/qxl_debugfs.c | 4 ++--
> drivers/gpu/drm/tiny/arcpgu.c | 2 +-
> drivers/gpu/drm/ttm/ttm_resource.c | 3 +--
> drivers/gpu/drm/virtio/virtgpu_debugfs.c | 6 +++---
> drivers/gpu/drm/vmwgfx/ttm_object.c | 5 ++---
> drivers/gpu/drm/vmwgfx/vmwgfx_gem.c | 2 +-
> 12 files changed, 19 insertions(+), 21 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
> index 827fcb4fb3b3..8a2c39927167 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
> @@ -3312,7 +3312,7 @@ static ssize_t dtn_log_write(
>
> static int mst_topo_show(struct seq_file *m, void *unused)
> {
> - struct amdgpu_device *adev = (struct amdgpu_device *)m->private;
> + struct amdgpu_device *adev = m->private;
> struct drm_device *dev = adev_to_drm(adev);
> struct drm_connector *connector;
> struct drm_connector_list_iter conn_iter;
> diff --git a/drivers/gpu/drm/amd/pm/amdgpu_pm.c b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> index 58c2246918fd..e6c870bd307b 100644
> --- a/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> +++ b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
> @@ -3671,7 +3671,7 @@ static void amdgpu_parse_cg_state(struct seq_file *m, u64 flags)
>
> static int amdgpu_debugfs_pm_info_show(struct seq_file *m, void *unused)
> {
> - struct amdgpu_device *adev = (struct amdgpu_device *)m->private;
> + struct amdgpu_device *adev = m->private;
> struct drm_device *dev = adev_to_drm(adev);
> u64 flags = 0;
> int r;
> diff --git a/drivers/gpu/drm/etnaviv/etnaviv_drv.c b/drivers/gpu/drm/etnaviv/etnaviv_drv.c
> index 31a7f59ccb49..dd57f7164e9a 100644
> --- a/drivers/gpu/drm/etnaviv/etnaviv_drv.c
> +++ b/drivers/gpu/drm/etnaviv/etnaviv_drv.c
> @@ -198,7 +198,7 @@ static int etnaviv_ring_show(struct etnaviv_gpu *gpu, struct seq_file *m)
>
> static int show_unlocked(struct seq_file *m, void *arg)
> {
> - struct drm_info_node *node = (struct drm_info_node *) m->private;
> + struct drm_info_node *node = m->private;
> struct drm_device *dev = node->minor->dev;
> int (*show)(struct drm_device *dev, struct seq_file *m) =
> node->info_ent->data;
> @@ -208,7 +208,7 @@ static int show_unlocked(struct seq_file *m, void *arg)
>
> static int show_each_gpu(struct seq_file *m, void *arg)
> {
> - struct drm_info_node *node = (struct drm_info_node *) m->private;
> + struct drm_info_node *node = m->private;
> struct drm_device *dev = node->minor->dev;
> struct etnaviv_drm_private *priv = dev->dev_private;
> struct etnaviv_gpu *gpu;
> diff --git a/drivers/gpu/drm/nouveau/nouveau_debugfs.c b/drivers/gpu/drm/nouveau/nouveau_debugfs.c
> index 2a36d1ca8fda..96b59d5d68ed 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_debugfs.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_debugfs.c
> @@ -37,7 +37,7 @@
> static int
> nouveau_debugfs_vbios_image(struct seq_file *m, void *data)
> {
> - struct drm_info_node *node = (struct drm_info_node *) m->private;
> + struct drm_info_node *node = m->private;
> struct nouveau_drm *drm = nouveau_drm(node->minor->dev);
> int i;
>
> diff --git a/drivers/gpu/drm/omapdrm/omap_debugfs.c b/drivers/gpu/drm/omapdrm/omap_debugfs.c
> index a3d470468e5b..a94ce502e152 100644
> --- a/drivers/gpu/drm/omapdrm/omap_debugfs.c
> +++ b/drivers/gpu/drm/omapdrm/omap_debugfs.c
> @@ -19,7 +19,7 @@
>
> static int gem_show(struct seq_file *m, void *arg)
> {
> - struct drm_info_node *node = (struct drm_info_node *) m->private;
> + struct drm_info_node *node = m->private;
> struct drm_device *dev = node->minor->dev;
> struct omap_drm_private *priv = dev->dev_private;
>
> @@ -33,7 +33,7 @@ static int gem_show(struct seq_file *m, void *arg)
>
> static int mm_show(struct seq_file *m, void *arg)
> {
> - struct drm_info_node *node = (struct drm_info_node *) m->private;
> + struct drm_info_node *node = m->private;
> struct drm_device *dev = node->minor->dev;
> struct drm_printer p = drm_seq_file_printer(m);
>
> @@ -45,7 +45,7 @@ static int mm_show(struct seq_file *m, void *arg)
> #ifdef CONFIG_DRM_FBDEV_EMULATION
> static int fb_show(struct seq_file *m, void *arg)
> {
> - struct drm_info_node *node = (struct drm_info_node *) m->private;
> + struct drm_info_node *node = m->private;
> struct drm_device *dev = node->minor->dev;
> struct drm_fb_helper *helper = dev->fb_helper;
> struct drm_framebuffer *fb;
> diff --git a/drivers/gpu/drm/pl111/pl111_debugfs.c b/drivers/gpu/drm/pl111/pl111_debugfs.c
> index 6744fa16f464..4df03ec5d368 100644
> --- a/drivers/gpu/drm/pl111/pl111_debugfs.c
> +++ b/drivers/gpu/drm/pl111/pl111_debugfs.c
> @@ -32,7 +32,7 @@ static const struct {
>
> static int pl111_debugfs_regs(struct seq_file *m, void *unused)
> {
> - struct drm_info_node *node = (struct drm_info_node *)m->private;
> + struct drm_info_node *node = m->private;
> struct drm_device *dev = node->minor->dev;
> struct pl111_drm_dev_private *priv = dev->dev_private;
> int i;
> diff --git a/drivers/gpu/drm/qxl/qxl_debugfs.c b/drivers/gpu/drm/qxl/qxl_debugfs.c
> index 2d9ed3b94574..5b4fe3049529 100644
> --- a/drivers/gpu/drm/qxl/qxl_debugfs.c
> +++ b/drivers/gpu/drm/qxl/qxl_debugfs.c
> @@ -38,7 +38,7 @@
> static int
> qxl_debugfs_irq_received(struct seq_file *m, void *data)
> {
> - struct drm_info_node *node = (struct drm_info_node *) m->private;
> + struct drm_info_node *node = m->private;
> struct qxl_device *qdev = to_qxl(node->minor->dev);
>
> seq_printf(m, "%d\n", atomic_read(&qdev->irq_received));
> @@ -52,7 +52,7 @@ qxl_debugfs_irq_received(struct seq_file *m, void *data)
> static int
> qxl_debugfs_buffers_info(struct seq_file *m, void *data)
> {
> - struct drm_info_node *node = (struct drm_info_node *) m->private;
> + struct drm_info_node *node = m->private;
> struct qxl_device *qdev = to_qxl(node->minor->dev);
> struct qxl_bo *bo;
>
> diff --git a/drivers/gpu/drm/tiny/arcpgu.c b/drivers/gpu/drm/tiny/arcpgu.c
> index e5b10e41554a..09f728355aba 100644
> --- a/drivers/gpu/drm/tiny/arcpgu.c
> +++ b/drivers/gpu/drm/tiny/arcpgu.c
> @@ -338,7 +338,7 @@ static int arcpgu_unload(struct drm_device *drm)
> #ifdef CONFIG_DEBUG_FS
> static int arcpgu_show_pxlclock(struct seq_file *m, void *arg)
> {
> - struct drm_info_node *node = (struct drm_info_node *)m->private;
> + struct drm_info_node *node = m->private;
> struct drm_device *drm = node->minor->dev;
> struct arcpgu_drm_private *arcpgu = dev_to_arcpgu(drm);
> unsigned long clkrate = clk_get_rate(arcpgu->clk);
> diff --git a/drivers/gpu/drm/ttm/ttm_resource.c b/drivers/gpu/drm/ttm/ttm_resource.c
> index 7333f7a87a2f..540faabcf8a4 100644
> --- a/drivers/gpu/drm/ttm/ttm_resource.c
> +++ b/drivers/gpu/drm/ttm/ttm_resource.c
> @@ -727,9 +727,8 @@ ttm_kmap_iter_linear_io_fini(struct ttm_kmap_iter_linear_io *iter_io,
>
> static int ttm_resource_manager_show(struct seq_file *m, void *unused)
> {
> - struct ttm_resource_manager *man =
> - (struct ttm_resource_manager *)m->private;
> struct drm_printer p = drm_seq_file_printer(m);
> + struct ttm_resource_manager *man = m->private;
> ttm_resource_manager_debug(man, &p);
> return 0;
> }
> diff --git a/drivers/gpu/drm/virtio/virtgpu_debugfs.c b/drivers/gpu/drm/virtio/virtgpu_debugfs.c
> index 853dd9aa397e..577691af9707 100644
> --- a/drivers/gpu/drm/virtio/virtgpu_debugfs.c
> +++ b/drivers/gpu/drm/virtio/virtgpu_debugfs.c
> @@ -43,7 +43,7 @@ static void virtio_gpu_add_int(struct seq_file *m, const char *name, int value)
>
> static int virtio_gpu_features(struct seq_file *m, void *data)
> {
> - struct drm_info_node *node = (struct drm_info_node *)m->private;
> + struct drm_info_node *node = m->private;
> struct virtio_gpu_device *vgdev = node->minor->dev->dev_private;
>
> virtio_gpu_add_bool(m, "virgl", vgdev->has_virgl_3d);
> @@ -68,7 +68,7 @@ static int virtio_gpu_features(struct seq_file *m, void *data)
> static int
> virtio_gpu_debugfs_irq_info(struct seq_file *m, void *data)
> {
> - struct drm_info_node *node = (struct drm_info_node *) m->private;
> + struct drm_info_node *node = m->private;
> struct virtio_gpu_device *vgdev = node->minor->dev->dev_private;
>
> seq_printf(m, "fence %llu %lld\n",
> @@ -80,7 +80,7 @@ virtio_gpu_debugfs_irq_info(struct seq_file *m, void *data)
> static int
> virtio_gpu_debugfs_host_visible_mm(struct seq_file *m, void *data)
> {
> - struct drm_info_node *node = (struct drm_info_node *)m->private;
> + struct drm_info_node *node = m->private;
> struct virtio_gpu_device *vgdev = node->minor->dev->dev_private;
> struct drm_printer p;
>
> diff --git a/drivers/gpu/drm/vmwgfx/ttm_object.c b/drivers/gpu/drm/vmwgfx/ttm_object.c
> index ddf8373c1d77..e9e3cc8f5b49 100644
> --- a/drivers/gpu/drm/vmwgfx/ttm_object.c
> +++ b/drivers/gpu/drm/vmwgfx/ttm_object.c
> @@ -513,8 +513,7 @@ static void ttm_prime_refcount_release(struct ttm_base_object **p_base)
> */
> static void ttm_prime_dmabuf_release(struct dma_buf *dma_buf)
> {
> - struct ttm_prime_object *prime =
> - (struct ttm_prime_object *) dma_buf->priv;
> + struct ttm_prime_object *prime = dma_buf->priv;
> struct ttm_base_object *base = &prime->base;
> struct ttm_object_device *tdev = base->tfile->tdev;
>
> @@ -554,7 +553,7 @@ int ttm_prime_fd_to_handle(struct ttm_object_file *tfile,
> if (dma_buf->ops != &tdev->ops)
> return -ENOSYS;
>
> - prime = (struct ttm_prime_object *) dma_buf->priv;
> + prime = dma_buf->priv;
> base = &prime->base;
> *handle = base->handle;
> ret = ttm_ref_object_add(tfile, base, NULL, false);
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_gem.c b/drivers/gpu/drm/vmwgfx/vmwgfx_gem.c
> index c0da89e16e6f..3267a4e61382 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_gem.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_gem.c
> @@ -220,7 +220,7 @@ static void vmw_bo_print_info(int id, struct vmw_bo *bo, struct seq_file *m)
>
> static int vmw_debugfs_gem_info_show(struct seq_file *m, void *unused)
> {
> - struct vmw_private *vdev = (struct vmw_private *)m->private;
> + struct vmw_private *vdev = m->private;
> struct drm_device *dev = &vdev->drm;
> struct drm_file *file;
> int r;


2023-05-26 10:31:31

by Su Hui

[permalink] [raw]
Subject: Re: [PATCH] drm: Remove unnecessary (void*) conversions

On 2023/5/26 15:27, Christian König wrote:
> Am 26.05.23 um 05:32 schrieb Su Hui:
>> Pointer variables of (void*) type do not require type cast.
>
> Please split that up by subsystem/driver. Taking it through the misc
> tree might just cause merge conflicts.
>
Sorry for that, I will split it and send again.
Thanks for your reply!

Su Hui

> Christian.
>
>>
>> Signed-off-by: Su Hui <[email protected]>
>> ---
>>   drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c | 2 +-
>>   drivers/gpu/drm/amd/pm/amdgpu_pm.c                        | 2 +-
>>   drivers/gpu/drm/etnaviv/etnaviv_drv.c                     | 4 ++--
>>   drivers/gpu/drm/nouveau/nouveau_debugfs.c                 | 2 +-
>>   drivers/gpu/drm/omapdrm/omap_debugfs.c                    | 6 +++---
>>   drivers/gpu/drm/pl111/pl111_debugfs.c                     | 2 +-
>>   drivers/gpu/drm/qxl/qxl_debugfs.c                         | 4 ++--
>>   drivers/gpu/drm/tiny/arcpgu.c                             | 2 +-
>>   drivers/gpu/drm/ttm/ttm_resource.c                        | 3 +--
>>   drivers/gpu/drm/virtio/virtgpu_debugfs.c                  | 6 +++---
>>   drivers/gpu/drm/vmwgfx/ttm_object.c                       | 5 ++---
>>   drivers/gpu/drm/vmwgfx/vmwgfx_gem.c                       | 2 +-
>>   12 files changed, 19 insertions(+), 21 deletions(-)
>>
>> diff --git
>> a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
>> b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
>> index 827fcb4fb3b3..8a2c39927167 100644
>> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
>> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
>> @@ -3312,7 +3312,7 @@ static ssize_t dtn_log_write(
>>     static int mst_topo_show(struct seq_file *m, void *unused)
>>   {
>> -    struct amdgpu_device *adev = (struct amdgpu_device *)m->private;
>> +    struct amdgpu_device *adev = m->private;
>>       struct drm_device *dev = adev_to_drm(adev);
>>       struct drm_connector *connector;
>>       struct drm_connector_list_iter conn_iter;
>> diff --git a/drivers/gpu/drm/amd/pm/amdgpu_pm.c
>> b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
>> index 58c2246918fd..e6c870bd307b 100644
>> --- a/drivers/gpu/drm/amd/pm/amdgpu_pm.c
>> +++ b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
>> @@ -3671,7 +3671,7 @@ static void amdgpu_parse_cg_state(struct
>> seq_file *m, u64 flags)
>>     static int amdgpu_debugfs_pm_info_show(struct seq_file *m, void
>> *unused)
>>   {
>> -    struct amdgpu_device *adev = (struct amdgpu_device *)m->private;
>> +    struct amdgpu_device *adev = m->private;
>>       struct drm_device *dev = adev_to_drm(adev);
>>       u64 flags = 0;
>>       int r;
>> diff --git a/drivers/gpu/drm/etnaviv/etnaviv_drv.c
>> b/drivers/gpu/drm/etnaviv/etnaviv_drv.c
>> index 31a7f59ccb49..dd57f7164e9a 100644
>> --- a/drivers/gpu/drm/etnaviv/etnaviv_drv.c
>> +++ b/drivers/gpu/drm/etnaviv/etnaviv_drv.c
>> @@ -198,7 +198,7 @@ static int etnaviv_ring_show(struct etnaviv_gpu
>> *gpu, struct seq_file *m)
>>     static int show_unlocked(struct seq_file *m, void *arg)
>>   {
>> -    struct drm_info_node *node = (struct drm_info_node *) m->private;
>> +    struct drm_info_node *node = m->private;
>>       struct drm_device *dev = node->minor->dev;
>>       int (*show)(struct drm_device *dev, struct seq_file *m) =
>>               node->info_ent->data;
>> @@ -208,7 +208,7 @@ static int show_unlocked(struct seq_file *m, void
>> *arg)
>>     static int show_each_gpu(struct seq_file *m, void *arg)
>>   {
>> -    struct drm_info_node *node = (struct drm_info_node *) m->private;
>> +    struct drm_info_node *node = m->private;
>>       struct drm_device *dev = node->minor->dev;
>>       struct etnaviv_drm_private *priv = dev->dev_private;
>>       struct etnaviv_gpu *gpu;
>> diff --git a/drivers/gpu/drm/nouveau/nouveau_debugfs.c
>> b/drivers/gpu/drm/nouveau/nouveau_debugfs.c
>> index 2a36d1ca8fda..96b59d5d68ed 100644
>> --- a/drivers/gpu/drm/nouveau/nouveau_debugfs.c
>> +++ b/drivers/gpu/drm/nouveau/nouveau_debugfs.c
>> @@ -37,7 +37,7 @@
>>   static int
>>   nouveau_debugfs_vbios_image(struct seq_file *m, void *data)
>>   {
>> -    struct drm_info_node *node = (struct drm_info_node *) m->private;
>> +    struct drm_info_node *node = m->private;
>>       struct nouveau_drm *drm = nouveau_drm(node->minor->dev);
>>       int i;
>>   diff --git a/drivers/gpu/drm/omapdrm/omap_debugfs.c
>> b/drivers/gpu/drm/omapdrm/omap_debugfs.c
>> index a3d470468e5b..a94ce502e152 100644
>> --- a/drivers/gpu/drm/omapdrm/omap_debugfs.c
>> +++ b/drivers/gpu/drm/omapdrm/omap_debugfs.c
>> @@ -19,7 +19,7 @@
>>     static int gem_show(struct seq_file *m, void *arg)
>>   {
>> -    struct drm_info_node *node = (struct drm_info_node *) m->private;
>> +    struct drm_info_node *node = m->private;
>>       struct drm_device *dev = node->minor->dev;
>>       struct omap_drm_private *priv = dev->dev_private;
>>   @@ -33,7 +33,7 @@ static int gem_show(struct seq_file *m, void *arg)
>>     static int mm_show(struct seq_file *m, void *arg)
>>   {
>> -    struct drm_info_node *node = (struct drm_info_node *) m->private;
>> +    struct drm_info_node *node = m->private;
>>       struct drm_device *dev = node->minor->dev;
>>       struct drm_printer p = drm_seq_file_printer(m);
>>   @@ -45,7 +45,7 @@ static int mm_show(struct seq_file *m, void *arg)
>>   #ifdef CONFIG_DRM_FBDEV_EMULATION
>>   static int fb_show(struct seq_file *m, void *arg)
>>   {
>> -    struct drm_info_node *node = (struct drm_info_node *) m->private;
>> +    struct drm_info_node *node = m->private;
>>       struct drm_device *dev = node->minor->dev;
>>       struct drm_fb_helper *helper = dev->fb_helper;
>>       struct drm_framebuffer *fb;
>> diff --git a/drivers/gpu/drm/pl111/pl111_debugfs.c
>> b/drivers/gpu/drm/pl111/pl111_debugfs.c
>> index 6744fa16f464..4df03ec5d368 100644
>> --- a/drivers/gpu/drm/pl111/pl111_debugfs.c
>> +++ b/drivers/gpu/drm/pl111/pl111_debugfs.c
>> @@ -32,7 +32,7 @@ static const struct {
>>     static int pl111_debugfs_regs(struct seq_file *m, void *unused)
>>   {
>> -    struct drm_info_node *node = (struct drm_info_node *)m->private;
>> +    struct drm_info_node *node = m->private;
>>       struct drm_device *dev = node->minor->dev;
>>       struct pl111_drm_dev_private *priv = dev->dev_private;
>>       int i;
>> diff --git a/drivers/gpu/drm/qxl/qxl_debugfs.c
>> b/drivers/gpu/drm/qxl/qxl_debugfs.c
>> index 2d9ed3b94574..5b4fe3049529 100644
>> --- a/drivers/gpu/drm/qxl/qxl_debugfs.c
>> +++ b/drivers/gpu/drm/qxl/qxl_debugfs.c
>> @@ -38,7 +38,7 @@
>>   static int
>>   qxl_debugfs_irq_received(struct seq_file *m, void *data)
>>   {
>> -    struct drm_info_node *node = (struct drm_info_node *) m->private;
>> +    struct drm_info_node *node = m->private;
>>       struct qxl_device *qdev = to_qxl(node->minor->dev);
>>         seq_printf(m, "%d\n", atomic_read(&qdev->irq_received));
>> @@ -52,7 +52,7 @@ qxl_debugfs_irq_received(struct seq_file *m, void
>> *data)
>>   static int
>>   qxl_debugfs_buffers_info(struct seq_file *m, void *data)
>>   {
>> -    struct drm_info_node *node = (struct drm_info_node *) m->private;
>> +    struct drm_info_node *node = m->private;
>>       struct qxl_device *qdev = to_qxl(node->minor->dev);
>>       struct qxl_bo *bo;
>>   diff --git a/drivers/gpu/drm/tiny/arcpgu.c
>> b/drivers/gpu/drm/tiny/arcpgu.c
>> index e5b10e41554a..09f728355aba 100644
>> --- a/drivers/gpu/drm/tiny/arcpgu.c
>> +++ b/drivers/gpu/drm/tiny/arcpgu.c
>> @@ -338,7 +338,7 @@ static int arcpgu_unload(struct drm_device *drm)
>>   #ifdef CONFIG_DEBUG_FS
>>   static int arcpgu_show_pxlclock(struct seq_file *m, void *arg)
>>   {
>> -    struct drm_info_node *node = (struct drm_info_node *)m->private;
>> +    struct drm_info_node *node = m->private;
>>       struct drm_device *drm = node->minor->dev;
>>       struct arcpgu_drm_private *arcpgu = dev_to_arcpgu(drm);
>>       unsigned long clkrate = clk_get_rate(arcpgu->clk);
>> diff --git a/drivers/gpu/drm/ttm/ttm_resource.c
>> b/drivers/gpu/drm/ttm/ttm_resource.c
>> index 7333f7a87a2f..540faabcf8a4 100644
>> --- a/drivers/gpu/drm/ttm/ttm_resource.c
>> +++ b/drivers/gpu/drm/ttm/ttm_resource.c
>> @@ -727,9 +727,8 @@ ttm_kmap_iter_linear_io_fini(struct
>> ttm_kmap_iter_linear_io *iter_io,
>>     static int ttm_resource_manager_show(struct seq_file *m, void
>> *unused)
>>   {
>> -    struct ttm_resource_manager *man =
>> -        (struct ttm_resource_manager *)m->private;
>>       struct drm_printer p = drm_seq_file_printer(m);
>> +    struct ttm_resource_manager *man = m->private;
>>       ttm_resource_manager_debug(man, &p);
>>       return 0;
>>   }
>> diff --git a/drivers/gpu/drm/virtio/virtgpu_debugfs.c
>> b/drivers/gpu/drm/virtio/virtgpu_debugfs.c
>> index 853dd9aa397e..577691af9707 100644
>> --- a/drivers/gpu/drm/virtio/virtgpu_debugfs.c
>> +++ b/drivers/gpu/drm/virtio/virtgpu_debugfs.c
>> @@ -43,7 +43,7 @@ static void virtio_gpu_add_int(struct seq_file *m,
>> const char *name, int value)
>>     static int virtio_gpu_features(struct seq_file *m, void *data)
>>   {
>> -    struct drm_info_node *node = (struct drm_info_node *)m->private;
>> +    struct drm_info_node *node = m->private;
>>       struct virtio_gpu_device *vgdev = node->minor->dev->dev_private;
>>         virtio_gpu_add_bool(m, "virgl", vgdev->has_virgl_3d);
>> @@ -68,7 +68,7 @@ static int virtio_gpu_features(struct seq_file *m,
>> void *data)
>>   static int
>>   virtio_gpu_debugfs_irq_info(struct seq_file *m, void *data)
>>   {
>> -    struct drm_info_node *node = (struct drm_info_node *) m->private;
>> +    struct drm_info_node *node = m->private;
>>       struct virtio_gpu_device *vgdev = node->minor->dev->dev_private;
>>         seq_printf(m, "fence %llu %lld\n",
>> @@ -80,7 +80,7 @@ virtio_gpu_debugfs_irq_info(struct seq_file *m,
>> void *data)
>>   static int
>>   virtio_gpu_debugfs_host_visible_mm(struct seq_file *m, void *data)
>>   {
>> -    struct drm_info_node *node = (struct drm_info_node *)m->private;
>> +    struct drm_info_node *node = m->private;
>>       struct virtio_gpu_device *vgdev = node->minor->dev->dev_private;
>>       struct drm_printer p;
>>   diff --git a/drivers/gpu/drm/vmwgfx/ttm_object.c
>> b/drivers/gpu/drm/vmwgfx/ttm_object.c
>> index ddf8373c1d77..e9e3cc8f5b49 100644
>> --- a/drivers/gpu/drm/vmwgfx/ttm_object.c
>> +++ b/drivers/gpu/drm/vmwgfx/ttm_object.c
>> @@ -513,8 +513,7 @@ static void ttm_prime_refcount_release(struct
>> ttm_base_object **p_base)
>>    */
>>   static void ttm_prime_dmabuf_release(struct dma_buf *dma_buf)
>>   {
>> -    struct ttm_prime_object *prime =
>> -        (struct ttm_prime_object *) dma_buf->priv;
>> +    struct ttm_prime_object *prime = dma_buf->priv;
>>       struct ttm_base_object *base = &prime->base;
>>       struct ttm_object_device *tdev = base->tfile->tdev;
>>   @@ -554,7 +553,7 @@ int ttm_prime_fd_to_handle(struct
>> ttm_object_file *tfile,
>>       if (dma_buf->ops != &tdev->ops)
>>           return -ENOSYS;
>>   -    prime = (struct ttm_prime_object *) dma_buf->priv;
>> +    prime = dma_buf->priv;
>>       base = &prime->base;
>>       *handle = base->handle;
>>       ret = ttm_ref_object_add(tfile, base, NULL, false);
>> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_gem.c
>> b/drivers/gpu/drm/vmwgfx/vmwgfx_gem.c
>> index c0da89e16e6f..3267a4e61382 100644
>> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_gem.c
>> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_gem.c
>> @@ -220,7 +220,7 @@ static void vmw_bo_print_info(int id, struct
>> vmw_bo *bo, struct seq_file *m)
>>     static int vmw_debugfs_gem_info_show(struct seq_file *m, void
>> *unused)
>>   {
>> -    struct vmw_private *vdev = (struct vmw_private *)m->private;
>> +    struct vmw_private *vdev = m->private;
>>       struct drm_device *dev = &vdev->drm;
>>       struct drm_file *file;
>>       int r;
>