diff options
| author | Zack Rusin <zackr@vmware.com> | 2023-01-30 22:35:36 -0500 |
|---|---|---|
| committer | Zack Rusin <zackr@vmware.com> | 2023-02-13 21:34:13 -0500 |
| commit | 6b2e8aa45126161135fb4a88870c9526fd8319f8 (patch) | |
| tree | 936ed4120e5b1b538044b9b4228b167dfaed240b /drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c | |
| parent | 9da2957f9f81ed29d0046021f131be352cb2199e (diff) | |
drm/vmwgfx: Remove the duplicate bo_free function
Remove the explicit bo_free parameter which was switching between
vmw_bo_bo_free and vmw_gem_destroy which had exactly the same
implementation.
It makes no sense to keep parameter which is always the same, remove it
and all code referencing it. Instead use the vmw_bo_bo_free directly.
Signed-off-by: Zack Rusin <zackr@vmware.com>
Reviewed-by: Martin Krastev <krastevm@vmware.com>
Reviewed-by: Maaz Mombasawala <mombasawalam@vmware.com>
Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
Link: https://patchwork.freedesktop.org/patch/msgid/20230131033542.953249-3-zack@kde.org
Diffstat (limited to 'drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c')
| -rw-r--r-- | drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c b/drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c index e1f36a09c59c..e51a63c05943 100644 --- a/drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c @@ -445,7 +445,7 @@ vmw_sou_primary_plane_prepare_fb(struct drm_plane *plane, vmw_overlay_pause_all(dev_priv); ret = vmw_bo_create(dev_priv, size, &vmw_vram_placement, - false, true, &vmw_bo_bo_free, &vps->bo); + false, true, &vps->bo); vmw_overlay_resume_all(dev_priv); if (ret) { vps->bo = NULL; /* vmw_bo_init frees on error */ |