[Mesa-dev] [PATCH v2 12/37] panfrost: Don't check if BO is mmaped before calling panfrost_bo_mmap()

[Alyssa Rosenzweig]

R-b

On Mon, Sep 16, 2019 at 11:36:50AM +0200, Boris Brezillon wrote:
> panfrost_bo_mmap() already takes care of that.
> 
> Signed-off-by: Boris Brezillon <boris.brezillon at collabora.com>
> ---
>  src/gallium/drivers/panfrost/pan_resource.c | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)
> 
> diff --git a/src/gallium/drivers/panfrost/pan_resource.c b/src/gallium/drivers/panfrost/pan_resource.c
> index 97ab2f9d9a4e..7083ee37bae2 100644
> --- a/src/gallium/drivers/panfrost/pan_resource.c
> +++ b/src/gallium/drivers/panfrost/pan_resource.c
> @@ -561,11 +561,7 @@ panfrost_transfer_map(struct pipe_context *pctx,
>          *out_transfer = &transfer->base;
>  
>          /* If we haven't already mmaped, now's the time */
> -
> -        if (!bo->cpu) {
> -                struct panfrost_screen *screen = pan_screen(pctx->screen);
> -                panfrost_bo_mmap(screen, bo);
> -        }
> +        panfrost_bo_mmap(pan_screen(pctx->screen), bo);
>  
>          /* Check if we're bound for rendering and this is a read pixels. If so,
>           * we need to flush */
> -- 
> 2.21.0