Re: [PATCH 04/71] drm: dropped obsolete/unused intel_bo_get_image()

On Mon, Apr 17, 2017 at 06:56:43PM +0200, Enrico Weigelt, metux IT consult wrote:
> Signed-off-by: Enrico Weigelt, metux IT consult <[email protected]>

Reviewed-by: Bryce Harrington <[email protected]>

To ssh://git.freedesktop.org/git/cairo
   182104d..72c600a  master -> master

(It would save me some reviewer time if you could include more
information in your commit messages.)

> ---
>  src/drm/cairo-drm-intel-private.h |  5 -----
>  src/drm/cairo-drm-intel.c         | 44 ---------------------------------------
>  2 files changed, 49 deletions(-)
> 
> diff --git a/src/drm/cairo-drm-intel-private.h b/src/drm/cairo-drm-intel-private.h
> index 48b2355d8..0cfded1bd 100644
> --- a/src/drm/cairo-drm-intel-private.h
> +++ b/src/drm/cairo-drm-intel-private.h
> @@ -314,11 +314,6 @@ intel_bo_init_for_name (const intel_device_t *dev,
>  			uint32_t size,
>  			uint32_t name);
>  
> -cairo_private cairo_surface_t *
> -intel_bo_get_image (const intel_device_t *device,
> -		    intel_bo_t *bo,
> -		    const cairo_drm_surface_t *surface);
> -
>  cairo_private cairo_status_t
>  intel_bo_put_image (intel_device_t *dev,
>  		    intel_bo_t *bo,
> diff --git a/src/drm/cairo-drm-intel.c b/src/drm/cairo-drm-intel.c
> index aabd016fd..8bc4ad8be 100644
> --- a/src/drm/cairo-drm-intel.c
> +++ b/src/drm/cairo-drm-intel.c
> @@ -530,50 +530,6 @@ intel_bo_set_tiling (const intel_device_t *device,
>      bo->_stride = bo->stride;
>  }
>  
> -cairo_surface_t *
> -intel_bo_get_image (const intel_device_t *device,
> -		    intel_bo_t *bo,
> -		    const cairo_drm_surface_t *surface)
> -{
> -    cairo_image_surface_t *image;
> -    uint8_t *dst;
> -    int size, row;
> -
> -    image = (cairo_image_surface_t *)
> -	cairo_image_surface_create (surface->format,
> -				    surface->width,
> -				    surface->height);
> -    if (unlikely (image->base.status))
> -	return &image->base;
> -
> -    intel_bo_set_tiling (device, bo);
> -
> -    if (bo->tiling == I915_TILING_NONE && image->stride == surface->stride) {
> -	size = surface->stride * surface->height;
> -	intel_bo_read (device, bo, 0, size, image->data);
> -    } else {
> -	const uint8_t *src;
> -
> -	src = intel_bo_map (device, bo);
> -	if (unlikely (src == NULL))
> -	    return _cairo_surface_create_in_error (_cairo_error (CAIRO_STATUS_NO_MEMORY));
> -
> -	size = surface->width;
> -	if (surface->format != CAIRO_FORMAT_A8)
> -	    size *= 4;
> -
> -	row = surface->height;
> -	dst = image->data;
> -	while (row--) {
> -	    memcpy (dst, src, size);
> -	    dst += image->stride;
> -	    src += surface->stride;
> -	}
> -    }
> -
> -    return &image->base;
> -}
> -
>  static cairo_status_t
>  _intel_bo_put_a1_image (intel_device_t *device,
>  			intel_bo_t *bo,
> -- 
> 2.11.0.rc0.7.gbe5a750
> 
> -- 
> cairo mailing list
> [email protected]
> https://lists.cairographics.org/mailman/listinfo/cairo
-- 
cairo mailing list
[email protected]
https://lists.cairographics.org/mailman/listinfo/cairo