[PATCH 6/6] drm: provide generic drm_vma_node_unmap() helper

2013-07-01 Thread Daniel Vetter
On Mon, Jul 01, 2013 at 08:33:03PM +0200, David Herrmann wrote: > Instead of unmapping the nodes in TTM and GEM users manually, we provide > a generic wrapper which does the correct thing for all vma-nodes. > > Signed-off-by: David Herrmann Nice. One nitpick below, otherwise: Reviewed-by:

[PATCH 6/6] drm: provide generic drm_vma_node_unmap() helper

2013-07-01 Thread David Herrmann
Instead of unmapping the nodes in TTM and GEM users manually, we provide a generic wrapper which does the correct thing for all vma-nodes. Signed-off-by: David Herrmann --- drivers/gpu/drm/i915/i915_gem.c | 6 +- drivers/gpu/drm/ttm/ttm_bo.c| 8 +--- include/drm/drm_vma_manager.h

[PATCH 6/6] drm: provide generic drm_vma_node_unmap() helper

2013-07-01 Thread David Herrmann
Instead of unmapping the nodes in TTM and GEM users manually, we provide a generic wrapper which does the correct thing for all vma-nodes. Signed-off-by: David Herrmann dh.herrm...@gmail.com --- drivers/gpu/drm/i915/i915_gem.c | 6 +- drivers/gpu/drm/ttm/ttm_bo.c| 8 +---

Re: [PATCH 6/6] drm: provide generic drm_vma_node_unmap() helper

2013-07-01 Thread Daniel Vetter
On Mon, Jul 01, 2013 at 08:33:03PM +0200, David Herrmann wrote: Instead of unmapping the nodes in TTM and GEM users manually, we provide a generic wrapper which does the correct thing for all vma-nodes. Signed-off-by: David Herrmann dh.herrm...@gmail.com Nice. One nitpick below, otherwise: