Re: [Intel-gfx] [PATCH] drm/i915/gem: Use local pointer ttm for __i915_ttm_move

["Hellstrom, Thomas" <thomas.hellstrom@intel.com>]

On Wed, 2021-12-08 at 16:05 -0800, Jasmine Newsome wrote:
> Using local pointer ttm as argument in  __i915_ttm_move
> instead of bo->ttm, as local pointer was previously assigned
> to bo->ttm in function. This will make code a bit more readable.
> 
> Signed-off-by: Jasmine Newsome <jasmine.newsome@intel.com>

Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>


> ---
>  drivers/gpu/drm/i915/gem/i915_gem_ttm_move.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_ttm_move.c
> b/drivers/gpu/drm/i915/gem/i915_gem_ttm_move.c
> index 80df9f592407..56b6573b5c93 100644
> --- a/drivers/gpu/drm/i915/gem/i915_gem_ttm_move.c
> +++ b/drivers/gpu/drm/i915/gem/i915_gem_ttm_move.c
> @@ -763,7 +763,7 @@ int i915_ttm_move(struct ttm_buffer_object *bo,
> bool evict,
>                         return PTR_ERR(dep);
>                 }
>  
> -               migration_fence = __i915_ttm_move(bo, clear, dst_mem,
> bo->ttm,
> +               migration_fence = __i915_ttm_move(bo, clear, dst_mem,
> ttm,
>                                                   dst_rsgt, true,
> dep);
>                 dma_fence_put(dep);
>         }

----------------------------------------------------------------------
Intel Sweden AB
Registered Office: Isafjordsgatan 30B, 164 40 Kista, Stockholm, Sweden
Registration Number: 556189-6027

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.