Re: [Intel-gfx] [PATCH 04/10] drm/i915: remove intel_memory_region avail

["Thomas Hellström" <thomas.hellstrom@linux.intel.com>]

On 5/25/22 20:43, Matthew Auld wrote:
> No longer used.
>
> Signed-off-by: Matthew Auld <matthew.auld@intel.com>
> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> Cc: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
> Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
> Cc: Jon Bloomfield <jon.bloomfield@intel.com>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Kenneth Graunke <kenneth@whitecape.org>
> Cc: Akeem G Abodunrin <akeem.g.abodunrin@intel.com>
> ---
>   drivers/gpu/drm/i915/intel_memory_region.c | 4 +---
>   drivers/gpu/drm/i915/intel_memory_region.h | 1 -
>   2 files changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_memory_region.c b/drivers/gpu/drm/i915/intel_memory_region.c
> index 94ee26e99549..9a4a7fb55582 100644
> --- a/drivers/gpu/drm/i915/intel_memory_region.c
> +++ b/drivers/gpu/drm/i915/intel_memory_region.c
> @@ -198,8 +198,7 @@ void intel_memory_region_debug(struct intel_memory_region *mr,
>   	if (mr->region_private)
>   		ttm_resource_manager_debug(mr->region_private, printer);
>   	else
> -		drm_printf(printer, "total:%pa, available:%pa bytes\n",
> -			   &mr->total, &mr->avail);
> +		drm_printf(printer, "total:%pa bytes\n", &mr->total);
>   }
>   
>   static int intel_memory_region_memtest(struct intel_memory_region *mem,
> @@ -242,7 +241,6 @@ intel_memory_region_create(struct drm_i915_private *i915,
>   	mem->min_page_size = min_page_size;
>   	mem->ops = ops;
>   	mem->total = size;
> -	mem->avail = mem->total;
>   	mem->type = type;
>   	mem->instance = instance;
>   
> diff --git a/drivers/gpu/drm/i915/intel_memory_region.h b/drivers/gpu/drm/i915/intel_memory_region.h
> index 2214f251bec3..2953ed5c3248 100644
> --- a/drivers/gpu/drm/i915/intel_memory_region.h
> +++ b/drivers/gpu/drm/i915/intel_memory_region.h
> @@ -75,7 +75,6 @@ struct intel_memory_region {
>   	resource_size_t io_size;
>   	resource_size_t min_page_size;
>   	resource_size_t total;
> -	resource_size_t avail;
>   
>   	u16 type;
>   	u16 instance;

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