diff mbox series

drm/vmwgfx: Remove busy_places

Message ID 20250108201355.2521070-1-ian.forbes@broadcom.com (mailing list archive)
State New
Headers show
Series drm/vmwgfx: Remove busy_places | expand

Commit Message

Ian Forbes Jan. 8, 2025, 8:13 p.m. UTC
Unused since commit a78a8da51b36
("drm/ttm: replace busy placement with flags v6")

Signed-off-by: Ian Forbes <ian.forbes@broadcom.com>
---
 drivers/gpu/drm/vmwgfx/vmwgfx_bo.c | 1 -
 drivers/gpu/drm/vmwgfx/vmwgfx_bo.h | 1 -
 2 files changed, 2 deletions(-)

Comments

Martin Krastev Jan. 9, 2025, 6:20 p.m. UTC | #1
On Wed, Jan 8, 2025 at 10:14 PM Ian Forbes <ian.forbes@broadcom.com> wrote:
>
> Unused since commit a78a8da51b36
> ("drm/ttm: replace busy placement with flags v6")
>
> Signed-off-by: Ian Forbes <ian.forbes@broadcom.com>
> ---
>  drivers/gpu/drm/vmwgfx/vmwgfx_bo.c | 1 -
>  drivers/gpu/drm/vmwgfx/vmwgfx_bo.h | 1 -
>  2 files changed, 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_bo.c b/drivers/gpu/drm/vmwgfx/vmwgfx_bo.c
> index a0e433fbcba67..5f13285f83e0b 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_bo.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_bo.c
> @@ -228,7 +228,6 @@ int vmw_bo_pin_in_start_of_vram(struct vmw_private *dev_priv,
>                              VMW_BO_DOMAIN_VRAM,
>                              VMW_BO_DOMAIN_VRAM);
>         buf->places[0].lpfn = PFN_UP(bo->resource->size);
> -       buf->busy_places[0].lpfn = PFN_UP(bo->resource->size);
>         ret = ttm_bo_validate(bo, &buf->placement, &ctx);
>
>         /* For some reason we didn't end up at the start of vram */
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_bo.h b/drivers/gpu/drm/vmwgfx/vmwgfx_bo.h
> index 43b5439ec9f76..07749f0a5f294 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_bo.h
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_bo.h
> @@ -83,7 +83,6 @@ struct vmw_bo {
>
>         struct ttm_placement placement;
>         struct ttm_place places[5];
> -       struct ttm_place busy_places[5];
>
>         /* Protected by reservation */
>         struct ttm_bo_kmap_obj map;
> --
> 2.34.1
>

LGTM

Reviewed-by: Martin Krastev <martin.krastev@broadcom.com>

Regards,
Martin
diff mbox series

Patch

diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_bo.c b/drivers/gpu/drm/vmwgfx/vmwgfx_bo.c
index a0e433fbcba67..5f13285f83e0b 100644
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_bo.c
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_bo.c
@@ -228,7 +228,6 @@  int vmw_bo_pin_in_start_of_vram(struct vmw_private *dev_priv,
 			     VMW_BO_DOMAIN_VRAM,
 			     VMW_BO_DOMAIN_VRAM);
 	buf->places[0].lpfn = PFN_UP(bo->resource->size);
-	buf->busy_places[0].lpfn = PFN_UP(bo->resource->size);
 	ret = ttm_bo_validate(bo, &buf->placement, &ctx);
 
 	/* For some reason we didn't end up at the start of vram */
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_bo.h b/drivers/gpu/drm/vmwgfx/vmwgfx_bo.h
index 43b5439ec9f76..07749f0a5f294 100644
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_bo.h
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_bo.h
@@ -83,7 +83,6 @@  struct vmw_bo {
 
 	struct ttm_placement placement;
 	struct ttm_place places[5];
-	struct ttm_place busy_places[5];
 
 	/* Protected by reservation */
 	struct ttm_bo_kmap_obj map;