Yue, Thanks, Reviewed-by: Thomas Hellstrom <thellstrom@xxxxxxxxxx> Will include in the next -next pull. /Thomas On Wed, 2019-01-16 at 01:55 +0000, YueHaibing wrote: > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/gpu/drm/vmwgfx/vmwgfx_surface.c: In function > 'vmw_hw_surface_destroy': > drivers/gpu/drm/vmwgfx/vmwgfx_surface.c:335:22: warning: > variable 'srf' set but not used [-Wunused-but-set-variable] > > It never used since commit 543831cfc976 ï¼^"drm/vmwgfx: Break out > surface and > context management to separate files") > > Signed-off-by: YueHaibing <yuehaibing@xxxxxxxxxx> > --- > drivers/gpu/drm/vmwgfx/vmwgfx_surface.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_surface.c > b/drivers/gpu/drm/vmwgfx/vmwgfx_surface.c > index 80a01cd..3b6976e 100644 > --- a/drivers/gpu/drm/vmwgfx/vmwgfx_surface.c > +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_surface.c > @@ -332,7 +332,6 @@ static void vmw_hw_surface_destroy(struct > vmw_resource *res) > { > > struct vmw_private *dev_priv = res->dev_priv; > - struct vmw_surface *srf; > void *cmd; > > if (res->func->destroy == vmw_gb_surface_destroy) { > @@ -359,7 +358,6 @@ static void vmw_hw_surface_destroy(struct > vmw_resource *res) > */ > > mutex_lock(&dev_priv->cmdbuf_mutex); > - srf = vmw_res_to_srf(res); > dev_priv->used_memory_size -= res->backup_size; > mutex_unlock(&dev_priv->cmdbuf_mutex); > } > > > _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel