diff mbox series

[2/3] drm/ttm: ttm_bo_mem_placement doesn't need ctx parameter.

Message ID 20201021044031.1752624-3-airlied@gmail.com (mailing list archive)
State New, archived
Headers show
Series ttm: change move notify + revised multi hop patch | expand

Commit Message

Dave Airlie Oct. 21, 2020, 4:40 a.m. UTC
From: Dave Airlie <airlied@redhat.com>

Removed unused parameter.

Signed-off-by: Dave Airlie <airlied@redhat.com>
---
 drivers/gpu/drm/ttm/ttm_bo.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

Comments

Christian König Oct. 21, 2020, 9:59 a.m. UTC | #1
Am 21.10.20 um 06:40 schrieb Dave Airlie:
> From: Dave Airlie <airlied@redhat.com>
>
> Removed unused parameter.
>
> Signed-off-by: Dave Airlie <airlied@redhat.com>

Reviewed-by: Christian König <christian.koenig@amd.com>

> ---
>   drivers/gpu/drm/ttm/ttm_bo.c | 8 +++-----
>   1 file changed, 3 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c
> index e2afab3d97ee..5b411252a857 100644
> --- a/drivers/gpu/drm/ttm/ttm_bo.c
> +++ b/drivers/gpu/drm/ttm/ttm_bo.c
> @@ -830,7 +830,6 @@ static int ttm_bo_mem_force_space(struct ttm_buffer_object *bo,
>    * @bo: BO to find memory for
>    * @place: where to search
>    * @mem: the memory object to fill in
> - * @ctx: operation context
>    *
>    * Check if placement is compatible and fill in mem structure.
>    * Returns -EBUSY if placement won't work or negative error code.
> @@ -838,8 +837,7 @@ static int ttm_bo_mem_force_space(struct ttm_buffer_object *bo,
>    */
>   static int ttm_bo_mem_placement(struct ttm_buffer_object *bo,
>   				const struct ttm_place *place,
> -				struct ttm_resource *mem,
> -				struct ttm_operation_ctx *ctx)
> +				struct ttm_resource *mem)
>   {
>   	struct ttm_bo_device *bdev = bo->bdev;
>   	struct ttm_resource_manager *man;
> @@ -884,7 +882,7 @@ int ttm_bo_mem_space(struct ttm_buffer_object *bo,
>   		const struct ttm_place *place = &placement->placement[i];
>   		struct ttm_resource_manager *man;
>   
> -		ret = ttm_bo_mem_placement(bo, place, mem, ctx);
> +		ret = ttm_bo_mem_placement(bo, place, mem);
>   		if (ret)
>   			continue;
>   
> @@ -910,7 +908,7 @@ int ttm_bo_mem_space(struct ttm_buffer_object *bo,
>   	for (i = 0; i < placement->num_busy_placement; ++i) {
>   		const struct ttm_place *place = &placement->busy_placement[i];
>   
> -		ret = ttm_bo_mem_placement(bo, place, mem, ctx);
> +		ret = ttm_bo_mem_placement(bo, place, mem);
>   		if (ret)
>   			continue;
>
diff mbox series

Patch

diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c
index e2afab3d97ee..5b411252a857 100644
--- a/drivers/gpu/drm/ttm/ttm_bo.c
+++ b/drivers/gpu/drm/ttm/ttm_bo.c
@@ -830,7 +830,6 @@  static int ttm_bo_mem_force_space(struct ttm_buffer_object *bo,
  * @bo: BO to find memory for
  * @place: where to search
  * @mem: the memory object to fill in
- * @ctx: operation context
  *
  * Check if placement is compatible and fill in mem structure.
  * Returns -EBUSY if placement won't work or negative error code.
@@ -838,8 +837,7 @@  static int ttm_bo_mem_force_space(struct ttm_buffer_object *bo,
  */
 static int ttm_bo_mem_placement(struct ttm_buffer_object *bo,
 				const struct ttm_place *place,
-				struct ttm_resource *mem,
-				struct ttm_operation_ctx *ctx)
+				struct ttm_resource *mem)
 {
 	struct ttm_bo_device *bdev = bo->bdev;
 	struct ttm_resource_manager *man;
@@ -884,7 +882,7 @@  int ttm_bo_mem_space(struct ttm_buffer_object *bo,
 		const struct ttm_place *place = &placement->placement[i];
 		struct ttm_resource_manager *man;
 
-		ret = ttm_bo_mem_placement(bo, place, mem, ctx);
+		ret = ttm_bo_mem_placement(bo, place, mem);
 		if (ret)
 			continue;
 
@@ -910,7 +908,7 @@  int ttm_bo_mem_space(struct ttm_buffer_object *bo,
 	for (i = 0; i < placement->num_busy_placement; ++i) {
 		const struct ttm_place *place = &placement->busy_placement[i];
 
-		ret = ttm_bo_mem_placement(bo, place, mem, ctx);
+		ret = ttm_bo_mem_placement(bo, place, mem);
 		if (ret)
 			continue;